mutant 0.6.3 → 0.6.4

data/lib/mutant/subject.rb

@@ -114,10 +114,7 @@ module Mutant
     #
     # @api private
     #
-    def match_expressions
-      [expression].concat(context.match_expressions)
-    end
-    memoize :match_expressions
+    abstract_method :match_expressions
 
   private
 

data/lib/mutant/subject/method.rb

@@ -32,6 +32,17 @@ module Mutant
       end
       memoize :expression
 
+      # Return match expressions
+      #
+      # @return [Array<Expression>]
+      #
+      # @api private
+      #
+      def match_expressions
+        [expression].concat(context.match_expressions)
+      end
+      memoize :match_expressions
+
     private
 
       # Return scope

data/lib/mutant/version.rb

@@ -1,4 +1,4 @@
 module Mutant
   # The current mutant version
-  VERSION = '0.6.3'.freeze
+  VERSION = '0.6.4'.freeze
 end # Mutant

data/meta/send.rb

@@ -145,6 +145,19 @@ Mutant::Meta::Example.add do
   mutation 'foo.instance_of?(bar)'
 end
 
+Mutant::Meta::Example.add do
+  source 'foo.is_a?(bar)'
+
+  singleton_mutations
+  mutation 'foo'
+  mutation 'bar'
+  mutation 'foo.is_a?'
+  mutation 'foo.is_a?(nil)'
+  mutation 'foo.is_a?(self)'
+  mutation 'self.is_a?(bar)'
+  mutation 'foo.instance_of?(bar)'
+end
+
 Mutant::Meta::Example.add do
   source 'foo.kind_of?(bar)'
 

data/mutant.gemspec

@@ -30,14 +30,14 @@ Gem::Specification.new do |gem|
   gem.add_runtime_dependency('morpher',       '~> 0.2.3')
   gem.add_runtime_dependency('procto',        '~> 0.0.2')
   gem.add_runtime_dependency('abstract_type', '~> 0.0.7')
-  gem.add_runtime_dependency('unparser',      '~> 0.1.14')
+  gem.add_runtime_dependency('unparser',      '~> 0.1.15')
   gem.add_runtime_dependency('ice_nine',      '~> 0.11.0')
   gem.add_runtime_dependency('adamantium',    '~> 0.2.0')
   gem.add_runtime_dependency('memoizable',    '~> 0.4.2')
   gem.add_runtime_dependency('equalizer',     '~> 0.0.9')
-  gem.add_runtime_dependency('inflecto',      '~> 0.0.2')
   gem.add_runtime_dependency('anima',         '~> 0.2.0')
   gem.add_runtime_dependency('concord',       '~> 0.1.5')
 
   gem.add_development_dependency('bundler', '~> 1.3', '>= 1.3.5')
+  gem.add_development_dependency('ffi',     '~> 1.9.6')
 end

data/spec/spec_helper.rb

@@ -1,10 +1,6 @@
 if ENV['COVERAGE'] == 'true'
   require 'simplecov'
 
-  SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter[
-    SimpleCov::Formatter::HTMLFormatter,
- ]
-
   SimpleCov.start do
     command_name 'spec:unit'
 
@@ -16,10 +12,11 @@ if ENV['COVERAGE'] == 'true'
     add_filter 'lib/mutant/zombifier'
     add_filter 'lib/mutant/zombifier/*'
 
-    minimum_coverage 89.77  # TODO: raise this to 100, then mutation test
+    minimum_coverage 97.64  # TODO: raise this to 100, then mutation test
   end
 end
 
+require 'tempfile'
 require 'concord'
 require 'adamantium'
 require 'devtools/spec_helper'
@@ -52,8 +49,4 @@ RSpec.configure do |config|
   config.include(CompressHelper)
   config.include(ParserHelper)
   config.include(Mutant::AST::Sexp)
-
-  config.expect_with :rspec do |rspec|
-    rspec.syntax = :expect
-  end
 end

data/spec/support/corpus.rb

@@ -1,5 +1,6 @@
 require 'morpher'
 require 'anima'
+require 'mutant'
 
 # Namespace module for corpus testing
 module Corpus
@@ -51,8 +52,8 @@ module Corpus
       start = Time.now
       paths = Pathname.glob(repo_path.join('**/*.rb')).sort_by(&:size).reverse
       options = {
-        finish: method(:finish),
-        start: method(:start),
+        finish:       method(:finish),
+        start:        method(:start),
         in_processes: parallel_processes
       }
       total = Parallel.map(paths, options) do |path|
@@ -135,9 +136,8 @@ module Corpus
     # @api private
     #
     def parallel_processes
-      case
-      when Devtools.circle_ci?
-        CIRCLE_CI_CONTAINER_PROCESSES
+      if ENV['CI']
+        Mutant::Config::DEFAULT.jobs
       else
         Parallel.processor_count
       end

data/spec/support/rb_bug.rb

@@ -0,0 +1,18 @@
+require 'ffi'
+
+module RbBug
+  extend FFI::Library
+  ffi_lib 'ruby'
+  attach_function :rb_bug, [:string, :varargs], :void
+
+  # Call the test bug
+  #
+  # @return [undefined]
+  #
+  # @api private
+  #
+  def self.call
+    rb_bug('%s', :string, 'test bug')
+  end
+
+end # RbBug

data/spec/unit/mutant/cli_spec.rb

@@ -128,7 +128,7 @@ Environment:
         --zombie                     Run mutant zombified
     -I, --include DIRECTORY          Add DIRECTORY to $LOAD_PATH
     -r, --require NAME               Require file with NAME
-    -j, --jobs NUMBER                Number of kill processes. Defaults to number of processors.
+    -j, --jobs NUMBER                Number of kill jobs. Defaults to number of processors.
 
 Options:
         --score COVERAGE             Fail unless COVERAGE is not reached exactly
@@ -178,7 +178,7 @@ Options:
       it_should_behave_like 'a cli parser'
 
       it 'configures expected coverage' do
-        expect(subject.config.processes).to eql(0)
+        expect(subject.config.jobs).to eql(0)
       end
     end
 

data/spec/unit/mutant/expression/methods_spec.rb

@@ -16,7 +16,13 @@ RSpec.describe Mutant::Expression::Methods do
       it { should be(object.syntax.length) }
     end
 
-    context 'when other is an unequivalent expression' do
+    context 'when other is matched' do
+      let(:other) { described_class.parse('TestApp::Literal#foo') }
+
+      it { should be(object.syntax.length) }
+    end
+
+    context 'when other is an not matched expression' do
       let(:other) { described_class.parse('Foo*') }
 
       it { should be(0) }

data/spec/unit/mutant/isolation_spec.rb

@@ -39,9 +39,29 @@ RSpec.describe Mutant::Isolation::Fork do
       expect(object.call { :foo }).to be(:foo)
     end
 
-    it 'wraps Parallel::DeadWorker exceptions' do
-      expect { object.call { fail Parallel::DeadWorker } }.to raise_error(Mutant::Isolation::Error)
+    it 'wraps exceptions' do
+      expect { object.call { fail } }.to raise_error(Mutant::Isolation::Error)
     end
 
+    it 'wraps exceptions caused by crashing ruby' do
+      expect do
+        object.call do
+          fail RbBug.call
+        end
+      end.to raise_error(Mutant::Isolation::Error)
+    end
+
+    it 'redirects $stderr of children to /dev/null' do
+      begin
+        Tempfile.open('mutant-test') do |file|
+          $stderr = file
+          object.call { $stderr.puts('test') }
+          file.rewind
+          expect(file.read).to eql('')
+        end
+      ensure
+        $stderr = STDERR
+      end
+    end
   end
 end

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

@@ -43,14 +43,14 @@ RSpec.describe Mutant::Matcher::Method::Instance do
 
     context 'when method is defined once' do
       let(:scope)       { base::DefinedOnce }
-      let(:method_line) { 7                 }
+      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) { 12                 }
+      let(:method_line) { 15                 }
 
       it_should_behave_like 'a method matcher'
     end
@@ -58,7 +58,7 @@ RSpec.describe Mutant::Matcher::Method::Instance do
     context 'when method is defined multiple times' do
       context 'on different lines' do
         let(:scope)        { base::DefinedMultipleTimes::DifferentLines }
-        let(:method_line)  { 21                                         }
+        let(:method_line)  { 24                                         }
         let(:method_arity) { 1                                          }
 
         it_should_behave_like 'a method matcher'
@@ -66,7 +66,7 @@ RSpec.describe Mutant::Matcher::Method::Instance do
 
       context 'on the same line' do
         let(:scope)        { base::DefinedMultipleTimes::SameLineSameScope }
-        let(:method_line)  { 26                                            }
+        let(:method_line)  { 29                                            }
         let(:method_arity) { 1                                             }
 
         it_should_behave_like 'a method matcher'
@@ -74,7 +74,7 @@ RSpec.describe Mutant::Matcher::Method::Instance do
 
       context 'on the same line with different scope' do
         let(:scope)        { base::DefinedMultipleTimes::SameLineDifferentScope }
-        let(:method_line)  { 30                                                 }
+        let(:method_line)  { 33                                                 }
         let(:method_arity) { 1                                                  }
 
         it_should_behave_like 'a method matcher'

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

@@ -23,7 +23,7 @@ RSpec.describe Mutant::Matcher::Method::Singleton, '#each' do
 
     context 'when also defined on lvar' do
       let(:scope)       { base::AlsoDefinedOnLvar }
-      let(:method_line) { 63                      }
+      let(:method_line) { 66                      }
 
       it_should_behave_like 'a method matcher'
 
@@ -37,7 +37,7 @@ RSpec.describe Mutant::Matcher::Method::Singleton, '#each' do
 
     context 'when defined on self' do
       let(:scope)       { base::DefinedOnSelf }
-      let(:method_line) { 58                  }
+      let(:method_line) { 61                  }
 
       it_should_behave_like 'a method matcher'
     end
@@ -46,13 +46,13 @@ RSpec.describe Mutant::Matcher::Method::Singleton, '#each' do
 
       context 'inside namespace' do
         let(:scope)       { base::DefinedOnConstant::InsideNamespace }
-        let(:method_line) { 68                                       }
+        let(:method_line) { 71                                       }
 
         it_should_behave_like 'a method matcher'
       end
 
       context 'outside namespace' do
-        let(:method_line) { 75                                        }
+        let(:method_line) { 78                                        }
         let(:scope)       { base::DefinedOnConstant::OutsideNamespace }
 
         it_should_behave_like 'a method matcher'
@@ -62,7 +62,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(:method_line)  { 94                                                   }
+        let(:method_line)  { 97                                                   }
         let(:method_arity) { 1                                                    }
 
         it_should_behave_like 'a method matcher'

data/spec/unit/mutant/reporter/cli_spec.rb

@@ -33,7 +33,6 @@ RSpec.describe Mutant::Reporter::CLI do
 
   let(:result) do
     Mutant::Result::Env.new(
-      done:            true,
       env:             env,
       runtime:         1.1,
       subject_results: subject_results
@@ -51,7 +50,7 @@ RSpec.describe Mutant::Reporter::CLI do
     )
   end
 
-  let(:config)           { Mutant::Config::DEFAULT.update(processes: 1) }
+  let(:config)           { Mutant::Config::DEFAULT.update(jobs: 1) }
   let(:mutation_class)   { Mutant::Mutation::Evil                       }
   let(:matchable_scopes) { double('Matchable Scopes', length: 10)       }
 
@@ -199,7 +198,7 @@ RSpec.describe Mutant::Reporter::CLI do
         Matcher:            #<Mutant::Matcher::Config match_expressions=[] subject_ignores=[] subject_selects=[]>
         Integration:        null
         Expect Coverage:    100.00%
-        Processes:          1
+        Jobs:               1
         Includes:           []
         Requires:           []
       REPORT
@@ -257,7 +256,7 @@ RSpec.describe Mutant::Reporter::CLI do
           Matcher:            #<Mutant::Matcher::Config match_expressions=[] subject_ignores=[] subject_selects=[]>
           Integration:        null
           Expect Coverage:    100.00%
-          Processes:          1
+          Jobs:               1
           Includes:           []
           Requires:           []
           Available Subjects: 1
@@ -286,7 +285,7 @@ RSpec.describe Mutant::Reporter::CLI do
             Matcher:            #<Mutant::Matcher::Config match_expressions=[] subject_ignores=[] subject_selects=[]>
             Integration:        null
             Expect Coverage:    100.00%
-            Processes:          1
+            Jobs:               1
             Includes:           []
             Requires:           []
             Available Subjects: 1
@@ -323,7 +322,7 @@ RSpec.describe Mutant::Reporter::CLI do
               Matcher:            #<Mutant::Matcher::Config match_expressions=[] subject_ignores=[] subject_selects=[]>
               Integration:        null
               Expect Coverage:    100.00%
-              Processes:          1
+              Jobs:               1
               Includes:           []
               Requires:           []
               Available Subjects: 1
@@ -350,7 +349,7 @@ RSpec.describe Mutant::Reporter::CLI do
               Matcher:            #<Mutant::Matcher::Config match_expressions=[] subject_ignores=[] subject_selects=[]>
               Integration:        null
               Expect Coverage:    100.00%
-              Processes:          1
+              Jobs:               1
               Includes:           []
               Requires:           []
               Available Subjects: 1
@@ -385,7 +384,7 @@ RSpec.describe Mutant::Reporter::CLI do
           Matcher:            #<Mutant::Matcher::Config match_expressions=[] subject_ignores=[] subject_selects=[]>
           Integration:        null
           Expect Coverage:    100.00%
-          Processes:          1
+          Jobs:               1
           Includes:           []
           Requires:           []
           Available Subjects: 1
@@ -418,7 +417,7 @@ RSpec.describe Mutant::Reporter::CLI do
               Matcher:            #<Mutant::Matcher::Config match_expressions=[] subject_ignores=[] subject_selects=[]>
               Integration:        null
               Expect Coverage:    100.00%
-              Processes:          1
+              Jobs:               1
               Includes:           []
               Requires:           []
               Available Subjects: 1
@@ -451,7 +450,7 @@ RSpec.describe Mutant::Reporter::CLI do
               Matcher:            #<Mutant::Matcher::Config match_expressions=[] subject_ignores=[] subject_selects=[]>
               Integration:        null
               Expect Coverage:    100.00%
-              Processes:          1
+              Jobs:               1
               Includes:           []
               Requires:           []
               Available Subjects: 1
@@ -492,7 +491,7 @@ RSpec.describe Mutant::Reporter::CLI do
             Matcher:            #<Mutant::Matcher::Config match_expressions=[] subject_ignores=[] subject_selects=[]>
             Integration:        null
             Expect Coverage:    100.00%
-            Processes:          1
+            Jobs:               1
             Includes:           []
             Requires:           []
             Available Subjects: 1
@@ -527,7 +526,7 @@ RSpec.describe Mutant::Reporter::CLI do
             Matcher:            #<Mutant::Matcher::Config match_expressions=[] subject_ignores=[] subject_selects=[]>
             Integration:        null
             Expect Coverage:    100.00%
-            Processes:          1
+            Jobs:               1
             Includes:           []
             Requires:           []
             Available Subjects: 1
@@ -548,14 +547,14 @@ RSpec.describe Mutant::Reporter::CLI do
         let(:subjects)        { [] }
         let(:subject_results) { [] }
 
-        let(:config) { Mutant::Config::DEFAULT.update(processes: 1, includes: %w[include-dir], requires: %w[require-name]) }
+        let(:config) { Mutant::Config::DEFAULT.update(jobs: 1, includes: %w[include-dir], requires: %w[require-name]) }
 
         it_reports(<<-REPORT)
           Mutant configuration:
           Matcher:            #<Mutant::Matcher::Config match_expressions=[] subject_ignores=[] subject_selects=[]>
           Integration:        null
           Expect Coverage:    100.00%
-          Processes:          1
+          Jobs:               1
           Includes:           ["include-dir"]
           Requires:           ["require-name"]
           Available Subjects: 0

data/spec/unit/mutant/runner/collector_spec.rb

@@ -0,0 +1,198 @@
+require 'spec_helper'
+
+describe Mutant::Runner::Collector do
+  let(:object) { described_class.new(env) }
+
+  before do
+    allow(Time).to receive(:now).and_return(Time.now)
+  end
+
+  let(:env) do
+    double(
+      'env',
+      subjects: [mutation_a.subject]
+    )
+  end
+
+  let(:mutation_a) do
+    double(
+      'mutation a',
+      subject: double('subject', identification: 'A')
+    )
+  end
+
+  let(:mutation_a_result) do
+    double(
+      'mutation a result',
+      index:    0,
+      runtime:  0.0,
+      mutation: mutation_a
+    )
+  end
+
+  let(:subject_a_result) do
+    Mutant::Result::Subject.new(
+      subject: mutation_a.subject,
+      runtime: 0.0,
+      mutation_results: [mutation_a_result]
+    )
+  end
+
+  let(:active_subject_result) do
+    subject_a_result.update(mutation_results: [])
+  end
+
+  let(:active_subject_results) do
+    [active_subject_result]
+  end
+
+  describe '.new' do
+    it 'initializes instance variables' do
+      expect(object.instance_variables).to include(:@last_mutation_result)
+    end
+  end
+
+  describe '#start' do
+    subject { object.start(mutation_a) }
+
+    it 'tracs the mutation as active' do
+      expect { subject }.to change { object.active_subject_results }.from([]).to(active_subject_results)
+    end
+
+    it_should_behave_like 'a command method'
+  end
+
+  describe '#finish' do
+    subject { object.finish(mutation_a_result) }
+
+    before do
+      object.start(mutation_a)
+    end
+
+    it 'removes the tracking of mutation as active' do
+      expect { subject }.to change { object.active_subject_results }.from(active_subject_results).to([])
+    end
+
+    it 'sets last mutation result' do
+      expect { subject }.to change { object.last_mutation_result }.from(nil).to(mutation_a_result)
+    end
+
+    it 'aggregates results in #result' do
+      subject
+      expect(object.result).to eql(
+        Mutant::Result::Env.new(
+          env: object.env,
+          runtime: 0.0,
+          subject_results: [subject_a_result]
+        )
+      )
+    end
+
+    it_should_behave_like 'a command method'
+  end
+
+  describe '#last_mutation_result' do
+    subject { object.last_mutation_result }
+
+    context 'when empty' do
+      it { should be(nil) }
+    end
+
+    context 'with partial state' do
+      before do
+        object.start(mutation_a)
+      end
+
+      it { should be(nil) }
+    end
+
+    context 'with full state' do
+      before do
+        object.start(mutation_a)
+        object.finish(mutation_a_result)
+      end
+
+      it { should be(mutation_a_result) }
+    end
+  end
+
+  describe '#active_subject_results' do
+    subject { object.active_subject_results }
+
+    context 'when empty' do
+      it { should eql([]) }
+    end
+
+    context 'on partial state' do
+      let(:mutation_b) do
+        double(
+          'mutation b',
+          subject: double(
+            'subject',
+            identification: 'B'
+          )
+        )
+      end
+
+      let(:mutation_b_result) do
+        double(
+          'mutation b result',
+          index:    0,
+          runtime:  0.0,
+          mutation: mutation_b
+        )
+      end
+
+      let(:subject_b_result) do
+        Mutant::Result::Subject.new(
+          subject: mutation_b.subject,
+          runtime: 0.0,
+          mutation_results: [mutation_b_result]
+        )
+      end
+
+      let(:active_subject_results) { [subject_a_result, subject_b_result] }
+
+      before do
+        object.start(mutation_b)
+        object.start(mutation_a)
+      end
+
+      it { should eql(active_subject_results.map { |result| result.update(mutation_results: []) }) }
+    end
+
+    context 'on full state' do
+      before do
+        object.start(mutation_a)
+        object.finish(mutation_a_result)
+      end
+
+      it { should eql([]) }
+    end
+  end
+
+  describe '#result' do
+    subject { object.result }
+
+    context 'when empty' do
+      it { should eql(Mutant::Result::Env.new(env: object.env, runtime: 0.0, subject_results: [active_subject_result])) }
+    end
+
+    context 'on partial state' do
+      before do
+        object.start(mutation_a)
+      end
+
+      it { should eql(Mutant::Result::Env.new(env: object.env, runtime: 0.0, subject_results: [active_subject_result])) }
+    end
+
+    context 'on full state' do
+      before do
+        object.start(mutation_a)
+        object.finish(mutation_a_result)
+      end
+
+      it { should eql(Mutant::Result::Env.new(env: object.env, runtime: 0.0, subject_results: [subject_a_result])) }
+    end
+  end
+end