crude-mutant 0.5.5 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 15e303ad1352ba2818a0f79e470d4991efca505c4714a0f54ea6a0e42a82ddbe
-  data.tar.gz: 241e688e2dc2740eca82d97c787821311b65b2fb12fea7ca748eee8352823dc0
+  metadata.gz: c2de77d0a729f2d9979af69f870ed458ab7bcc0e7b21d06c91a05e34d054a14f
+  data.tar.gz: e9eca3bdb377d9d4c1408208f1de5418a91a60f970f3443c45807e00b2c08b7c
 SHA512:
-  metadata.gz: 7235f34036f9f7588bc5d9e1f13e793dc0c09f767485fff9ae2b3f0985cf23d6086c1186efda0ac84a9039b3950f9952c4d3a3c6c1251610538750e9da8fc08b
-  data.tar.gz: 917d0b1b09195c45f1f823aee7ee9563a00a81a8e931de19824a07e6cefe82f6f47989b764faac1b4724e85427154d2c598beef5f737a80b2278b6dfef06e9d3
+  metadata.gz: 8b1e55aaff7fdfb50d913e5954c2f36efd3ed49f08bf7404b2bfca723de3a0b888d4af45bb030d269b97a78c3d9a6ff653a85d76b64a0f2cbafe47a72d9a6856
+  data.tar.gz: 7e8702f3850647893cda4978c7eeb02a35962a948b44c7254dc734c4fbcd42e3a647b582edd4ca519e30a711b0cab0a9fa85791e98a24cee673f2b6d81706dd6

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    crude-mutant (0.5.5)
+    crude-mutant (0.6.0)
 
 GEM
   remote: https://rubygems.org/

data/exe/cm

@@ -12,7 +12,7 @@ Options = Struct.new(
   :file_to_mutate,
 )
 
-args = Options.new(false, 1, 1)
+args = Options.new(false, 0, 1)
 OptionParser.new do |opts|
   opts.banner = "Usage: crude-mutant [options]"
 

data/exe/crude-mutant

@@ -12,7 +12,7 @@ Options = Struct.new(
   :file_to_mutate,
 )
 
-args = Options.new(false, 1, 1)
+args = Options.new(false, 0, 1)
 OptionParser.new do |opts|
   opts.banner = "Usage: crude-mutant [options]"
 

data/lib/crude_mutant/json_result_printer.rb

@@ -9,8 +9,8 @@ module CrudeMutant
         stream.print(
           JSON.dump({
             result.file_path => {
-              passed_lines: result.run_results.select(&:success?).map(&:line_number),
-              failed_lines: result.run_results.reject(&:success?).map(&:line_number),
+              passed_lines: result.run_results.reject(&:success?).map(&:line_number),
+              failed_lines: result.run_results.select(&:success?).map(&:line_number),
             }
           }),
         )

data/lib/crude_mutant/line_permuter.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+module CrudeMutant
+  class LinePermuter
+    def initialize(file_contents)
+      @file_contents = file_contents
+    end
+
+    def number_of_permutations
+      @number_of_permutations ||= @file_contents.split("\n").size
+    end
+
+    def take(permutation_number)
+      if permutation_number < 0
+        raise ArgumentError, 'permutation_number must be 0 or more'
+      end
+
+      if permutation_number > number_of_permutations - 1
+        raise ArgumentError, 'permutation_number must be less than number_of_permutations - 1'
+      end
+
+      (contents_as_array.slice(0, permutation_number) +
+        contents_as_array.slice(permutation_number + 1, number_of_permutations)).join("\n")
+    end
+
+    def line(line_number)
+      contents_as_array.fetch(line_number)
+    end
+
+    private
+
+    def contents_as_array
+      @contents_as_array ||= @file_contents.split("\n")
+    end
+  end
+end

data/lib/crude_mutant/permutation_selector.rb

@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+module CrudeMutant
+  class PermutationSelector
+    def self.select(number_of_permutations:, number_of_sections:, section_number:)
+      if section_number < 0
+        raise ArgumentError, 'section_number must be 0 or greater'
+      end
+
+      if section_number >= number_of_sections
+        raise ArgumentError, 'section_number must be less than number_of_sections - 1'
+      end
+
+      if number_of_sections <= 0
+        raise ArgumentError, 'number_of_sections must be greater than 0'
+      end
+
+      if number_of_sections > number_of_permutations
+        raise ArgumentError, 'number_of_sections must less than number_of_permutations'
+      end
+
+      section_size = (number_of_permutations.to_f / number_of_sections).ceil
+      starting_number = section_number * section_size
+
+      buckets = Array.new(number_of_sections, 0)
+
+      current_bucket = 0
+      i = 0
+
+      while i < number_of_permutations do
+        buckets[current_bucket] += 1
+        i += 1
+        current_bucket = i % number_of_sections
+      end
+
+      start = buckets.slice(0, section_number).sum
+      stop = start + buckets[section_number]
+
+      buckets[section_number].times.map do |i|
+        i + start
+      end
+    end
+  end
+end

data/lib/crude_mutant/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module CrudeMutant
-  VERSION = "0.5.5"
+  VERSION = "0.6.0"
 end

data/lib/crude_mutant.rb

@@ -3,12 +3,12 @@
 require "benchmark"
 
 require "crude_mutant/executor"
-require "crude_mutant/file_loader"
-require "crude_mutant/file_writer"
+require "crude_mutant/json_result_printer"
+require "crude_mutant/line_permuter"
+require "crude_mutant/permutation_selector"
 require "crude_mutant/progress"
 require "crude_mutant/result"
 require "crude_mutant/result_printer"
-require "crude_mutant/json_result_printer"
 require "crude_mutant/run_result"
 require "crude_mutant/terminal_calculator"
 require "crude_mutant/version"
@@ -22,11 +22,14 @@ module CrudeMutant
       printer_klass = result_printer == :json ? JsonResultPrinter : ResultPrinter
 
       start_time = Time.now.to_f
-      file = FileLoader.load(file_path)
-      num_lines_in_file = file.lines_in_file
-      section_size = (num_lines_in_file.to_f / total_sections).ceil
-      starting_line_number = section * section_size - 1
-      stopping_line_number = starting_line_number + section_size
+      original_file_contents = File.read(file_path)
+      permuter = LinePermuter.new(original_file_contents)
+
+      permutations_to_run = PermutationSelector.select(
+        number_of_permutations: permuter.number_of_permutations,
+        number_of_sections: total_sections,
+        section_number: section,
+      )
 
       initial_success = Executor.call(test_command)
 
@@ -35,24 +38,25 @@ module CrudeMutant
       end
 
       begin
-        line_number = starting_line_number
-        test_runs = file.contents_as_array.reduce([]) do |acc, contents|
-          line_number += 1
-
-          if (starting_line_number..stopping_line_number).cover?(line_number) && contents.strip.size != 0
-            result = [perform_run(
-              file,
-              test_command,
-              line_number
-            )]
-          else
-            result = [NullRunResult.new(line_number)]
-          end
+        test_runs = permutations_to_run.reduce([]) do |acc, permutation|
+          success, bench = perform_run(
+            file_path: file_path,
+            file_contents: permuter.take(permutation),
+            command: test_command,
+          )
+
+          result = [RunResult.new(
+            file_path,
+            permutation,
+            success,
+            permuter.line(permutation),
+            bench,
+          )]
 
           if block_given?
             block.call(
               Progress.new(
-                num_lines_in_file,
+                permutations_to_run.size,
                 acc + result
               )
             )
@@ -61,7 +65,7 @@ module CrudeMutant
           acc + result
         end
       ensure
-        FileWriter.write(file_path, file.contents_as_array)
+        File.write(file_path, original_file_contents)
       end
 
       stop_time = Time.now.to_f
@@ -73,21 +77,15 @@ module CrudeMutant
 
     private
 
-    def perform_run(file_loader, test_command, line_number)
+    def perform_run(file_path:, file_contents:, command:)
       success = false
       bench = Benchmark.measure do
-        FileWriter.write(file_loader.file_path, file_loader.without_line(line_number))
+        File.write(file_path, file_contents)
 
-        success = Executor.call(test_command)
+        success = Executor.call(command)
       end
 
-      RunResult.new(
-        file_loader.file_path,
-        line_number,
-        success,
-        file_loader.contents_as_array[line_number],
-        bench
-      )
+      [success, bench]
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: crude-mutant
 version: !ruby/object:Gem::Version
-  version: 0.5.5
+  version: 0.6.0
 platform: ruby
 authors:
 - Kelly Sutton
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-05-20 00:00:00.000000000 Z
+date: 2019-05-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -81,9 +81,9 @@ files:
 - lib/crude-mutant.rb
 - lib/crude_mutant.rb
 - lib/crude_mutant/executor.rb
-- lib/crude_mutant/file_loader.rb
-- lib/crude_mutant/file_writer.rb
 - lib/crude_mutant/json_result_printer.rb
+- lib/crude_mutant/line_permuter.rb
+- lib/crude_mutant/permutation_selector.rb
 - lib/crude_mutant/progress.rb
 - lib/crude_mutant/result.rb
 - lib/crude_mutant/result_printer.rb

data/lib/crude_mutant/file_loader.rb

@@ -1,35 +0,0 @@
-# frozen_string_literal: true
-
-module CrudeMutant
-  class FileLoader
-    class LoadError < StandardError; end
-
-    def self.load(file_path)
-      if !file_path.is_a?(String) || !File.exist?(file_path)
-        raise LoadError, "Could not load file: #{file_path}"
-      end
-
-      new(file_path, File.readlines(file_path))
-    end
-
-    attr_reader :file_path
-
-    def initialize(file_path, contents_as_array)
-      @file_path = file_path
-      @contents_as_array = contents_as_array
-    end
-
-    def lines_in_file
-      @contents_as_array.size
-    end
-
-    def contents_as_array
-      @contents_as_array
-    end
-
-    def without_line(line_number)
-      (contents_as_array.slice(0, line_number) || []) +
-        (contents_as_array.slice(line_number + 1, lines_in_file) || [])
-    end
-  end
-end

data/lib/crude_mutant/file_writer.rb

@@ -1,15 +0,0 @@
-# frozen_string_literal: true
-
-module CrudeMutant
-  class FileWriter
-    def self.write(file_path, contents_as_array)
-      File.open(file_path, 'w') do |f|
-        contents_as_array.each do |line|
-          f.write(line)
-        end
-
-        f.flush
-      end
-    end
-  end
-end