mutant 0.8.1 → 0.8.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ac3f098a786f3940bb0bdc9b291994aa068fbf3d
-  data.tar.gz: 4acf7c5f8e0d04bbbe87ad2ebfb403650372cf4a
+  metadata.gz: f16485d23238c84b07b2e62df8a1c4af57badb14
+  data.tar.gz: e0b9ddb90dccee908622683295235e177d4d7dcf
 SHA512:
-  metadata.gz: ce34952eb570ea87bb7169785b556431eb9820dea70261f0772c89e3d648cf70a4f819b9be86448f6b4c8ea88b0721c6f3aa6c3f9e3680127f80c805826c6fb2
-  data.tar.gz: 89eb14bfeb66f63899033195bfbec8562c821e19fea0aa51997b8e57fcb3897b7de26e0d5b642d14c7c276c50b6d28f3df0fbc23741a5944523ff6e0c5117c2e
+  metadata.gz: 992b3447fda4a045e9462f292f17da506141b386e9c494d94c13a6807450f9b73826f230e468a883f1177df70db409de9b400d107183d053444061c96ba7193f
+  data.tar.gz: 3c8319ee768ca67e2a0cd9fbf65c5a3652425f73ecc8206bdca143da85f6a16fbf875505f5f14f9038319b5e35615e6a4e8ec0bf5ad2620cca4ed1d8c6e57f0c

data/Changelog.md

@@ -1,3 +1,15 @@
+# v0.8.2 2015-08-11
+
+* Remove invalid mutation `foo or bar` to `!(foo or bar)` see #287
+* Add mutation from `#to_h` to `#to_hash` #218
+<<<<<<< 2914e9c8d906540edc1cc7d4bc6393f68db39b8d
+* Add mutation from `#defined?` to `true` / `false` #334
+* Add mutation from `super` to `super()` #309
+=======
+* Add mutation from `#defined?` to `true` / `false` #399
+* Reduce framed (multiline) progress reporter noise
+>>>>>>> Reduce framed progress reporter noise
+
 # v0.8.1 2015-07-24
 
 * Add --since flag to constrain mutated subjects based on

data/README.md

@@ -118,6 +118,79 @@ Example for a subject like `Foo::Bar#baz` it will run all example groups with de
 `Foo::Bar#baz`, `Foo::Bar` and `Foo`. The order is important, so if mutant finds example groups in the
 current prefix level, these example groups *must* kill the mutation.
 
+Reading Reports
+---------------
+
+Mutation output is grouped by selection groups. Each group contains three sections:
+
+1. An identifier for the current group.
+
+   **Format**:
+
+   ```text
+   [SUBJECT EXPRESSION]:[SOURCE LOCATION]:[LINENO]
+   ```
+
+   **Example**:
+
+   ```text
+   Book#add_page:Book#add_page:/home/dev/mutant-examples/lib/book.rb:18
+   ```
+
+2. A list of specs that mutant ran to try to kill mutations for the current group.
+
+   **Format**:
+
+   ```text
+   - [INTEGRATION]:0:[SPEC LOCATION]:[SPEC DESCRIPTION]
+   - [INTEGRATION]:1:[SPEC LOCATION]:[SPEC DESCRIPTION]
+   ```
+
+   **Example**:
+
+   ```text
+   - rspec:0:./spec/unit/book_spec.rb:9/Book#add_page should return self
+   - rspec:1:./spec/unit/book_spec.rb:13/Book#add_page should add page to book
+   ```
+
+3. A list of unkilled mutations diffed against the original unparsed source
+
+   **Format**:
+
+   ```text
+   [MUTATION TYPE]:[SUBJECT EXPRESSION]:[SOURCE LOCATION]:[SOURCE LINENO]:[IDENTIFIER]
+   [DIFF]
+   -----------------------
+   ```
+
+   - `[MUTATION TYPE]` will be one of the following:
+      - `evil` - a mutation of your source was not killed by your tests
+      - `neutral` your original source was injected and one or more tests failed
+   - `[IDENTIFIER]` - Unique identifier for this mutation
+
+   **Example**:
+
+   ```diff
+   evil:Book#add_page:Book#add_page:/home/dev/mutant-examples/lib/book.rb:18:01f69
+   @@ -1,6 +1,6 @@
+    def add_page(page)
+   -  @pages << page
+   +  @pages
+      @index[page.number] = page
+      self
+    end
+   -----------------------
+   evil:Book#add_page:Book#add_page:/home/dev/mutant-examples/lib/book.rb:18:b1ff2
+   @@ -1,6 +1,6 @@
+    def add_page(page)
+   -  @pages << page
+   +  self
+      @index[page.number] = page
+      self
+    end
+   -----------------------
+   ```
+
 Rails
 -------
 

data/Rakefile

@@ -4,16 +4,21 @@ require 'devtools'
 
 Devtools.init_rake_tasks
 
-# Frequent lookups in MRI make mutation analysis getting stuck on CI
-# See: https://github.com/mbj/mutant/issues/265
-if ENV['CI']
-  Rake.application.load_imports
+Rake.application.load_imports
 
-  task('metrics:mutant').clear
-  namespace :metrics do
-    task :mutant => :coverage do
-      $stderr.puts 'Mutant self test via zombie not active on CI'
-    end
+task('metrics:mutant').clear
+namespace :metrics do
+  task :mutant => :coverage do
+    success = Kernel.system(*%w[
+      bundle exec mutant
+      --zombie
+      --use rspec
+      --include lib
+      --require mutant
+      --since HEAD~1
+      --
+      Mutant*
+    ]) or fail 'Mutant task is not successful'
   end
 end
 

data/config/flay.yml

@@ -1,3 +1,3 @@
 ---
 threshold: 18
-total_score: 1208
+total_score: 1193

data/config/reek.yml

@@ -43,12 +43,12 @@ NestedIterators:
   - Mutant#self.singleton_subclass_instance
   - Mutant::CLI#parse
   - Mutant::Isolation::Fork#self.call
-  - Mutant::Mutator::Util::Array::Element#dispatch
-  - Mutant::Mutator::Node::Resbody#mutate_captures
   - Mutant::Mutator::Node::Arguments#emit_argument_mutations
+  - Mutant::Mutator::Node::Resbody#mutate_captures
+  - Mutant::Mutator::Util::Array::Element#dispatch
+  - Mutant::Parallel::Master#run
   - Mutant::RequireHighjack#self.call
   - Mutant::Selector::Expression#call
-  - Mutant::Parallel::Master#run
   - Parser::Lexer#self.new
   max_allowed_nesting: 1
   ignore_iterators: []
@@ -70,26 +70,26 @@ TooManyMethods:
   enabled: true
   exclude:
   - Mutant::CLI
-  - Mutant::Mutator::Node
   - Mutant::Meta::Example::Verification
+  - Mutant::Mutator::Node
   - Mutant::Parallel::Master
   max_methods: 10
 TooManyStatements:
   enabled: true
   exclude:
+  - Mutant::CLI#add_debug_options
+  - Mutant::CLI#add_environment_options
   - Mutant::Isolation::Fork#self.call
-  - Mutant::Reporter::CLI::Printer::EnvProgress#run
   - Mutant::Reporter::CLI::Printer::Config#run
-  - Mutant::Zombifier::File#self.find
-  - Mutant::CLI#add_environment_options
-  - Mutant::CLI#add_debug_options
+  - Mutant::Reporter::CLI::Printer::EnvProgress#run
   - Mutant::Runner#run_driver
+  - Mutant::Zombifier::File#self.find
   max_statements: 7
 UncommunicativeMethodName:
   enabled: true
   exclude:
-  - Mutant::Mutation#sha1
   - Mutant::AST::Sexp#s
+  - Mutant::Mutation#sha1
   reject:
   - !ruby/regexp /^[a-z]$/
   - !ruby/regexp /[0-9]$/
@@ -124,14 +124,15 @@ UnusedParameters:
 UtilityFunction:
   enabled: true
   exclude:
-  - Mutant::Actor::Env#new_mailbox
   - Mutant::AST::Sexp#s
+  - Mutant::Actor::Env#new_mailbox
   - Mutant::CLI#reporter
   - Mutant::Integration::Null#call
   - Mutant::Integration::Rspec#parse_example
+  - Mutant::Integration::Rspec#parse_expression # intentional, private
   - Mutant::Meta::Example::Verification#format_mutation
-  - Mutant::Repository::Diff#tracks? # intentional, private
   - Mutant::Reporter::CLI::Format::Progressive#new_buffer
   - Mutant::Reporter::CLI::Printer::StatusProgressive#object # False positive calls super
-  - Mutant::Integration::Rspec#parse_expression # intentional, private
+  - Mutant::Repository::Diff#tracks? # intentional, private
+  - Mutant::Repository::Diff#within_working_directory? # intentional, private
   max_helper_calls: 0

data/lib/mutant/isolation.rb

@@ -39,9 +39,10 @@ module Mutant
       # @api private
       def self.call(&block)
         IO.pipe(binmode: true) do |reader, writer|
+          writer.binmode
           begin
             pid = Process.fork do
-              File.open(File::NULL, 'w') do |file|
+              File.open(File::NULL, File::WRONLY) do |file|
                 $stderr.reopen(file)
                 reader.close
                 writer.write(Marshal.dump(block.call))

data/lib/mutant/matcher/method.rb

@@ -77,12 +77,13 @@ module Mutant
 
       # Path to source
       #
-      # @return [String]
+      # @return [Pathname]
       #
       # @api private
       def source_path
-        source_location.first
+        Pathname.new(source_location.first)
       end
+      memoize :source_path
 
       # Source file line
       #

data/lib/mutant/mutator/node/binary.rb

@@ -22,10 +22,9 @@ module Mutant
         # @api private
         def dispatch
           emit_singletons
-          emit(left)
-          emit(right)
-          mutate_operator
-          mutate_operands
+          emit_promotions
+          emit_operator_mutations
+          emit_left_negation
           emit_left_mutations
           emit_right_mutations
         end
@@ -35,18 +34,39 @@ module Mutant
         # @return [undefined]
         #
         # @api private
-        def mutate_operator
+        def emit_operator_mutations
           emit(s(INVERSE.fetch(node.type), left, right))
         end
 
-        # Emit condition mutations
+        # Emit promotions
+        #
+        # @return [undefined]
+        #
+        # @api private
+        #
+        def emit_promotions
+          emit(left)
+          emit(right)
+        end
+
+        # Emit left negation
+        #
+        # We do not emit right negation as the `and` and `or` nodes
+        # in ruby are also used for control flow.
+        #
+        # Irrespectable of their syntax, aka `||` parses internally to `or`.
+        #
+        # `do_a or do_b`. Negating left makes sense, negating right
+        # only when the result is actualy used.
+        #
+        # It *would* be possible to emit the right negation in case the use of the result is proved.
+        # Like parent is an assignment to an {l,i}var. Dunno if we ever get the time to do that.
         #
         # @return [undefined]
         #
         # @api private
-        def mutate_operands
+        def emit_left_negation
           emit(s(node.type, n_not(left), right))
-          emit(n_not(node))
         end
 
       end # Binary

data/lib/mutant/mutator/node/defined.rb

@@ -16,9 +16,10 @@ module Mutant
         #
         # @api private
         def dispatch
-          emit_expression_mutations do |node|
-            !n_self?(node)
-          end
+          emit_singletons
+          emit(N_TRUE)
+
+          emit_expression_mutations { |node| !n_self?(node) }
         end
 
       end # Defined

data/lib/mutant/mutator/node/send.rb

@@ -24,6 +24,7 @@ module Mutant
           to_s:          %i[to_str],
           to_i:          %i[to_int],
           to_a:          %i[to_ary],
+          to_h:          %i[to_hash],
           at:            %i[fetch],
           :[] =>         %i[at fetch],
           :== =>         %i[eql? equal?],

data/lib/mutant/mutator/node/zsuper.rb

@@ -16,6 +16,7 @@ module Mutant
         # @api private
         def dispatch
           emit_singletons
+          emit(N_EMPTY_SUPER)
         end
 
       end # ZSuper

data/lib/mutant/reporter/cli/printer/subject_progress.rb

@@ -28,7 +28,6 @@ module Mutant
             print_mutation_results
             print_progress_bar_finish
             print_stats
-            print_tests
           end
 
         private
@@ -50,17 +49,6 @@ module Mutant
             )
           end
 
-          # Print tests
-          #
-          # @return [undefined]
-          #
-          # @api private
-          def print_tests
-            tests.each do |test|
-              puts "- #{test.identification}"
-            end
-          end
-
           # Print progress bar finish
           #
           # @return [undefined]

data/lib/mutant/repository.rb

@@ -48,7 +48,7 @@ module Mutant
       #
       # @api private
       def touches?(path, line_range)
-        return false unless tracks?(path)
+        return false unless within_working_directory?(path) && tracks?(path)
 
         command = %W[
           git log
@@ -83,6 +83,18 @@ module Mutant
         )
       end
 
+      # Test if the path is within the current working directory
+      #
+      # @param [Pathname] path
+      #
+      # @return [TrueClass, nil]
+      #
+      # @api private
+      def within_working_directory?(path)
+        working_directory = Pathname.pwd
+        path.ascend { |parent| return true if working_directory.eql?(parent) }
+      end
+
     end # Diff
   end # Repository
 end # Mutant

data/lib/mutant/subject.rb

@@ -21,7 +21,7 @@ module Mutant
 
     # Source path
     #
-    # @return [String]
+    # @return [Pathname]
     #
     # @api private
     def source_path

data/lib/mutant/version.rb

@@ -1,4 +1,4 @@
 module Mutant
   # Current mutant version
-  VERSION = '0.8.1'.freeze
+  VERSION = '0.8.2'.freeze
 end # Mutant

data/lib/mutant/zombifier.rb

@@ -114,7 +114,7 @@ module Mutant
 
     # Namespaced root node
     #
-    # @param [Symbol] namespace
+    # @param [Pathname] source_path
     #
     # @return [Parser::AST::Node]
     #

data/meta/and.rb

@@ -10,5 +10,4 @@ Mutant::Meta::Example.add do
   mutation 'false and false'
   mutation 'true and true'
   mutation '!true and false'
-  mutation '!(true and false)'
 end

data/meta/defined.rb

@@ -1,5 +1,7 @@
 Mutant::Meta::Example.add do
   source 'defined?(foo)'
 
+  singleton_mutations
   mutation 'defined?(nil)'
+  mutation 'true'
 end

data/meta/or.rb

@@ -10,5 +10,4 @@ Mutant::Meta::Example.add do
   mutation 'true or true'
   mutation 'true and false'
   mutation '!true or false'
-  mutation '!(true or false)'
 end

data/meta/send.rb

@@ -118,6 +118,15 @@ Mutant::Meta::Example.add do
   mutation 'foo.to_int'
 end
 
+Mutant::Meta::Example.add do
+  source 'foo.to_h'
+
+  singleton_mutations
+  mutation 'foo'
+  mutation 'self.to_h'
+  mutation 'foo.to_hash'
+end
+
 Mutant::Meta::Example.add do
   source 'foo == bar'
 

data/meta/super.rb

@@ -2,6 +2,7 @@ Mutant::Meta::Example.add do
   source 'super'
 
   singleton_mutations
+  mutation 'super()'
 end
 
 Mutant::Meta::Example.add do

data/spec/shared/method_matcher_behavior.rb

@@ -28,6 +28,10 @@ RSpec.shared_examples_for 'a method matcher' do
     expect(context.scope).to eql(scope)
   end
 
+  it 'should have the correct source path in context' do
+    expect(context.source_path).to eql(source_path)
+  end
+
   it 'should have the correct node type' do
     expect(node.type).to be(type)
   end

data/spec/support/corpus.rb

@@ -3,13 +3,14 @@ require 'anima'
 require 'mutant'
 
 module MutantSpec
+  ROOT = Pathname.new(__FILE__).parent.parent.parent
+
   # Namespace module for corpus testing
   #
   # rubocop:disable MethodLength
   module Corpus
     # Project under corpus test
     # rubocop:disable ClassLength
-    ROOT = Pathname.new(__FILE__).parent.parent.parent
     TMP = ROOT.join('tmp').freeze
 
     class Project

data/spec/unit/mutant/isolation_spec.rb

@@ -72,13 +72,15 @@ RSpec.describe Mutant::Isolation::Fork do
         expect(IO).to receive(:pipe).with(binmode: true).ordered do |&block|
           block.call([reader, writer])
         end
+        expect(writer).to receive(:binmode).ordered
       end
 
       it 'when fork succeeds' do
         pid = double('PID')
         expect(Process).to receive(:fork).ordered.and_yield.and_return(pid)
         file = double('file')
-        expect(File).to receive(:open).ordered.with('/dev/null', 'w').and_yield(file)
+        expect(File).to receive(:open).ordered
+          .with(File::NULL, File::WRONLY).and_yield(file)
         expect($stderr).to receive(:reopen).ordered.with(file)
         expect(reader).to receive(:close).ordered
         expect(writer).to receive(:write).ordered.with(Marshal.dump(:foo))

data/spec/unit/mutant/matcher/method/instance_spec.rb

@@ -41,38 +41,43 @@ RSpec.describe Mutant::Matcher::Method::Instance do
     end
 
     context 'when method is defined once' do
-      let(:scope)       { base::DefinedOnce }
-      let(:method_line) { 10                }
+      let(:scope)       { base::DefinedOnce                                 }
+      let(:source_path) { MutantSpec::ROOT.join('test_app/lib/test_app.rb') }
+      let(:method_line) { 10                                                }
 
       it_should_behave_like 'a method matcher'
     end
 
     context 'when method is defined once with a memoizer' do
-      let(:scope)       { base::WithMemoizer }
-      let(:method_line) { 15                 }
+      let(:scope)       { base::WithMemoizer                                }
+      let(:source_path) { MutantSpec::ROOT.join('test_app/lib/test_app.rb') }
+      let(:method_line) { 15                                                }
 
       it_should_behave_like 'a method matcher'
     end
 
     context 'when method is defined multiple times' do
       context 'on different lines' do
-        let(:scope)        { base::DefinedMultipleTimes::DifferentLines }
-        let(:method_line)  { 24                                         }
-        let(:method_arity) { 1                                          }
+        let(:scope)        { base::DefinedMultipleTimes::DifferentLines        }
+        let(:source_path)  { MutantSpec::ROOT.join('test_app/lib/test_app.rb') }
+        let(:method_line)  { 24                                                }
+        let(:method_arity) { 1                                                 }
 
         it_should_behave_like 'a method matcher'
       end
 
       context 'on the same line' do
-        let(:scope)        { base::DefinedMultipleTimes::SameLineSameScope }
-        let(:method_line)  { 29                                            }
-        let(:method_arity) { 1                                             }
+        let(:scope)        { base::DefinedMultipleTimes::SameLineSameScope     }
+        let(:source_path)  { MutantSpec::ROOT.join('test_app/lib/test_app.rb') }
+        let(:method_line)  { 29                                                }
+        let(:method_arity) { 1                                                 }
 
         it_should_behave_like 'a method matcher'
       end
 
       context 'on the same line with different scope' do
         let(:scope)        { base::DefinedMultipleTimes::SameLineDifferentScope }
+        let(:source_path)  { MutantSpec::ROOT.join('test_app/lib/test_app.rb')  }
         let(:method_line)  { 33                                                 }
         let(:method_arity) { 1                                                  }
 

data/spec/unit/mutant/matcher/method/singleton_spec.rb

@@ -21,8 +21,9 @@ RSpec.describe Mutant::Matcher::Method::Singleton, '#each' do
   context 'on singleton methods' do
 
     context 'when also defined on lvar' do
-      let(:scope)       { base::AlsoDefinedOnLvar }
-      let(:method_line) { 66                      }
+      let(:scope)       { base::AlsoDefinedOnLvar                           }
+      let(:source_path) { MutantSpec::ROOT.join('test_app/lib/test_app.rb') }
+      let(:method_line) { 66                                                }
 
       it_should_behave_like 'a method matcher'
 
@@ -35,8 +36,9 @@ RSpec.describe Mutant::Matcher::Method::Singleton, '#each' do
     end
 
     context 'when defined on self' do
-      let(:scope)       { base::DefinedOnSelf }
-      let(:method_line) { 61                  }
+      let(:scope)       { base::DefinedOnSelf                               }
+      let(:source_path) { MutantSpec::ROOT.join('test_app/lib/test_app.rb') }
+      let(:method_line) { 61                                                }
 
       it_should_behave_like 'a method matcher'
     end
@@ -44,15 +46,17 @@ RSpec.describe Mutant::Matcher::Method::Singleton, '#each' do
     context 'when defined on constant' do
 
       context 'inside namespace' do
-        let(:scope)       { base::DefinedOnConstant::InsideNamespace }
-        let(:method_line) { 71                                       }
+        let(:scope)       { base::DefinedOnConstant::InsideNamespace          }
+        let(:source_path) { MutantSpec::ROOT.join('test_app/lib/test_app.rb') }
+        let(:method_line) { 71                                                }
 
         it_should_behave_like 'a method matcher'
       end
 
       context 'outside namespace' do
-        let(:method_line) { 78                                        }
-        let(:scope)       { base::DefinedOnConstant::OutsideNamespace }
+        let(:scope)       { base::DefinedOnConstant::OutsideNamespace         }
+        let(:source_path) { MutantSpec::ROOT.join('test_app/lib/test_app.rb') }
+        let(:method_line) { 78                                                }
 
         it_should_behave_like 'a method matcher'
       end
@@ -61,6 +65,7 @@ RSpec.describe Mutant::Matcher::Method::Singleton, '#each' do
     context 'when defined multiple times in the same line' do
       context 'with method on different scope' do
         let(:scope)        { base::DefinedMultipleTimes::SameLine::DifferentScope }
+        let(:source_path)  { MutantSpec::ROOT.join('test_app/lib/test_app.rb')    }
         let(:method_line)  { 97                                                   }
         let(:method_arity) { 1                                                    }
 

data/spec/unit/mutant/reporter/cli/printer/status_spec.rb

@@ -107,7 +107,6 @@ RSpec.describe Mutant::Reporter::CLI::Printer::Status do
             subject-a mutations: 2
             F.
             (01/02)  50% - killtime: 2.00s runtime: 2.00s overhead: 0.00s
-            - test-a
           REPORT
         end
 
@@ -136,7 +135,6 @@ RSpec.describe Mutant::Reporter::CLI::Printer::Status do
             subject-a mutations: 2
             ..
             (02/02) 100% - killtime: 2.00s runtime: 2.00s overhead: 0.00s
-            - test-a
           REPORT
         end
       end

data/spec/unit/mutant/reporter/cli/printer/subject_progress_spec.rb

@@ -9,7 +9,6 @@ RSpec.describe Mutant::Reporter::CLI::Printer::SubjectProgress do
         subject-a mutations: 2
         ..
         (02/02) 100% - killtime: 2.00s runtime: 2.00s overhead: 0.00s
-        - test-a
       STR
     end
 
@@ -20,7 +19,6 @@ RSpec.describe Mutant::Reporter::CLI::Printer::SubjectProgress do
         subject-a mutations: 2
         F.
         (01/02)  50% - killtime: 2.00s runtime: 2.00s overhead: 0.00s
-        - test-a
       STR
     end
 
@@ -30,7 +28,6 @@ RSpec.describe Mutant::Reporter::CLI::Printer::SubjectProgress do
       it_reports <<-'STR'
         subject-a mutations: 2
         (00/02) 100% - killtime: 0.00s runtime: 0.00s overhead: 0.00s
-        - test-a
       STR
     end
   end

data/spec/unit/mutant/repository/diff_spec.rb

@@ -8,25 +8,39 @@ describe Mutant::Repository::Diff do
   end
 
   describe '#touches?' do
-    let(:object)        { described_class.new('from_rev', 'to_rev') }
-    let(:path)          { Pathname.new('foo.rb')                    }
-    let(:line_range)    { 1..2                                      }
+    let(:object)     { described_class.new('from_rev', 'to_rev') }
+    let(:path)       { Pathname.pwd.join('foo.rb')               }
+    let(:line_range) { 1..2                                      }
 
     subject { object.touches?(path, line_range) }
 
-    before do
-      expect(Kernel).to receive(:system)
-        .ordered
-        .with(
-          *%w[git ls-files --error-unmatch -- foo.rb],
-          out: File::NULL,
-          err: File::NULL
-        ).and_return(git_ls_success?)
+    shared_context 'test if git tracks the file' do
+      before do
+        expect(Kernel).to receive(:system)
+          .ordered
+          .with(
+            *%W[git ls-files --error-unmatch -- #{path}],
+            out: File::NULL,
+            err: File::NULL
+          ).and_return(git_ls_success?)
+      end
+    end
+
+    context 'when file is in a different subdirectory' do
+      let(:path) { Pathname.new('/foo.rb') }
+
+      before do
+        expect(Kernel).to_not receive(:system)
+      end
+
+      it { should be(false) }
     end
 
     context 'when file is NOT tracked in repository' do
       let(:git_ls_success?) { false }
 
+      include_context 'test if git tracks the file'
+
       it { should be(false) }
     end
 
@@ -36,6 +50,8 @@ describe Mutant::Repository::Diff do
       let(:stdout)          { double('Stdout', empty?: stdout_empty?) }
       let(:stdout_empty?)   { false                                   }
 
+      include_context 'test if git tracks the file'
+
       before do
         expect(Open3).to receive(:capture2)
           .ordered
@@ -44,9 +60,7 @@ describe Mutant::Repository::Diff do
       end
 
       let(:expected_git_log_command) do
-        %w[
-          git log from_rev...to_rev -L 1,2:foo.rb
-        ]
+        %W[git log from_rev...to_rev -L 1,2:#{path}]
       end
 
       context 'on failure of git log command' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mutant
 version: !ruby/object:Gem::Version
-  version: 0.8.1
+  version: 0.8.2
 platform: ruby
 authors:
 - Markus Schirp
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-07-24 00:00:00.000000000 Z
+date: 2015-08-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: parser