mutant 0.8.8 → 0.8.9

data/meta/or_asgn.rb

@@ -21,6 +21,12 @@ Mutant::Meta::Example.add do
   mutation '@a ||= 2'
 end
 
+Mutant::Meta::Example.add do
+  source 'Foo ||= nil'
+
+  singleton_mutations
+end
+
 Mutant::Meta::Example.add do
   source '@a ||= self.bar'
 

data/meta/send.rb

@@ -223,6 +223,22 @@ Mutant::Meta::Example.add do
   mutation 'self.gsub(a, b)'
 end
 
+Mutant::Meta::Example.add do
+  source 'foo.values_at(a, b)'
+
+  singleton_mutations
+  mutation 'foo.fetch_values(a, b)'
+  mutation 'foo'
+  mutation 'self.values_at(a, b)'
+  mutation 'foo.values_at(a)'
+  mutation 'foo.values_at(b)'
+  mutation 'foo.values_at(nil, b)'
+  mutation 'foo.values_at(self, b)'
+  mutation 'foo.values_at(a, nil)'
+  mutation 'foo.values_at(a, self)'
+  mutation 'foo.values_at'
+end
+
 Mutant::Meta::Example.add do
   source 'foo.__send__(bar)'
 
@@ -359,6 +375,15 @@ Mutant::Meta::Example.add do
   mutation 'foo(nil)'
 end
 
+Mutant::Meta::Example.add do
+  source 'self.fetch(nil)'
+
+  singleton_mutations
+  mutation 'self.fetch'
+  mutation 'fetch(nil)'
+  mutation 'self.key?(nil)'
+end
+
 Unparser::Constants::KEYWORDS.each do |keyword|
   Mutant::Meta::Example.add do
     source "foo.#{keyword}(nil)"
@@ -405,6 +430,7 @@ Mutant::Meta::Example.add do
   mutation 'foo[]'
   mutation 'foo.at(1)'
   mutation 'foo.fetch(1)'
+  mutation 'foo.key?(1)'
   mutation 'self[1]'
   mutation 'foo[0]'
   mutation 'foo[2]'
@@ -420,6 +446,7 @@ Mutant::Meta::Example.add do
   mutation 'self.foo'
   mutation 'self.foo.at()'
   mutation 'self.foo.fetch()'
+  mutation 'self.foo.key?()'
   mutation 'self[]'
   mutation 'foo[]'
 end
@@ -433,6 +460,7 @@ Mutant::Meta::Example.add do
   mutation 'self[]'
   mutation 'self.at(foo)'
   mutation 'self.fetch(foo)'
+  mutation 'self.key?(foo)'
   mutation 'foo'
 end
 
@@ -444,6 +472,7 @@ Mutant::Meta::Example.add do
   mutation 'foo[]'
   mutation 'foo.at(*bar)'
   mutation 'foo.fetch(*bar)'
+  mutation 'foo.key?(*bar)'
   mutation 'foo[nil]'
   mutation 'foo[self]'
   mutation 'foo[bar]'

data/mutant-rspec.gemspec

@@ -16,7 +16,7 @@ Gem::Specification.new do |gem|
   gem.extra_rdoc_files = %w[TODO LICENSE]
 
   gem.add_runtime_dependency('mutant', "~> #{gem.version}")
-  gem.add_runtime_dependency('rspec-core', '>= 3.2.0', '< 3.5.0')
+  gem.add_runtime_dependency('rspec-core', '>= 3.3.0', '< 3.5.0')
 
   gem.add_development_dependency('bundler', '~> 1.3', '>= 1.3.5')
 end

data/spec/integration/mutant/rspec_spec.rb

@@ -2,7 +2,7 @@ RSpec.describe 'rspec integration', mutant: false do
 
   let(:base_cmd) { 'bundle exec mutant -I lib --require test_app --use rspec' }
 
-  %w[3.2 3.3 3.4].each do |version|
+  %w[3.3 3.4].each do |version|
     context "RSpec #{version}" do
       let(:gemfile) { "Gemfile.rspec#{version}" }
 

data/spec/integrations.yml

@@ -1,18 +1,17 @@
 ---
 - name: rubyspec
   namespace: Rubyspec
-  repo_uri: 'https://github.com/rubyspec/rubyspec.git'
+  repo_uri: 'https://github.com/ruby/rubyspec.git'
   mutation_coverage: false
   mutation_generation: true
   expect_coverage: 0 # not run
   exclude:
     # Binary encoded source subjected to limitations see README of unparser
     - core/array/pack/{b,h,u}_spec.rb
-    - language/versions/*1.8*
+    - language/regexp/escapes_spec.rb
     - core/array/pack/shared/float.rb
     - core/array/pack/shared/integer.rb
     - core/array/pack/{c,m,w}_spec.rb
-    - core/regexp/shared/new.rb
     - core/regexp/shared/quote.rb
     - core/encoding/compatible_spec.rb
     - core/io/readpartial_spec.rb

data/spec/shared/method_matcher_behavior.rb

@@ -11,7 +11,7 @@ RSpec.shared_examples_for 'a method matcher' do
     expect(name).to eql(method_name)
   end
 
-  it 'has epxected line number' do
+  it 'has expected line number' do
     expect(node.location.expression.line).to eql(method_line)
   end
 
@@ -33,13 +33,13 @@ RSpec.shared_examples_for 'a method matcher' do
 end
 
 RSpec.shared_examples_for 'skipped candidate' do
-  it 'does not emit matcher' do
-    expect(subject).to eql([])
+  before do
+    expected_warnings.each do |warning|
+      expect(env).to receive(:warn).with(warning).and_return(env)
+    end
   end
 
-  it 'does warn' do
-    subject
-
-    expect(env.config.reporter.warn_calls).to eql(expected_warnings)
+  it 'does not emit matcher' do
+    expect(subject).to eql([])
   end
 end

data/spec/spec_helper.rb

@@ -33,9 +33,8 @@ $LOAD_PATH << File.join(TestApp.root, 'lib')
 require 'test_app'
 
 module Fixtures
-  TEST_CONFIG = Mutant::Config::DEFAULT.with(reporter: Mutant::Reporter::Trace.new)
-  TEST_CACHE  = Mutant::Cache.new
-  TEST_ENV    = Mutant::Env::Bootstrap.(TEST_CONFIG, TEST_CACHE)
+  TEST_CONFIG = Mutant::Config::DEFAULT.with(reporter: Mutant::Reporter::Null.new)
+  TEST_ENV    = Mutant::Env::Bootstrap.(TEST_CONFIG)
 end # Fixtures
 
 module ParserHelper
@@ -43,12 +42,6 @@ module ParserHelper
     Unparser.unparse(node)
   end
 
-  def test_env
-    Fixtures::TEST_ENV.with(
-      config: Mutant::Config::DEFAULT.with(reporter: Mutant::Reporter::Trace.new)
-    )
-  end
-
   def parse(string)
     Unparser::Preprocessor.run(Parser::CurrentRuby.parse(string))
   end

data/spec/support/corpus.rb

@@ -2,6 +2,7 @@ require 'morpher'
 require 'anima'
 require 'mutant'
 
+# @api private
 module MutantSpec
   ROOT = Pathname.new(__FILE__).parent.parent.parent
 
@@ -9,20 +10,28 @@ module MutantSpec
   #
   # rubocop:disable MethodLength
   module Corpus
+    TMP                 = ROOT.join('tmp').freeze
+    EXCLUDE_GLOB_FORMAT = '{%s}'.freeze
+    RUBY_GLOB_PATTERN   = '**/*.rb'.freeze
+
+    # Not in the docs. Number from chatting with their support.
+    # 2 processors allocated per container, 4 processes works well.
+    CIRCLE_CI_CONTAINER_PROCESSES = 4
+
+    private_constant(*constants(false))
+
     # Project under corpus test
     # rubocop:disable ClassLength
-    TMP = ROOT.join('tmp').freeze
-
     class Project
       MUTEX = Mutex.new
       include Adamantium, Anima.new(
-        :name,
-        :repo_uri,
         :exclude,
+        :expect_coverage,
         :mutation_coverage,
         :mutation_generation,
+        :name,
         :namespace,
-        :expect_coverage
+        :repo_uri
       )
 
       # Verify mutation coverage
@@ -31,8 +40,6 @@ module MutantSpec
       #   if successful
       #
       # @raise [Exception]
-      #
-      # @api private
       def verify_mutation_coverage
         checkout
         Dir.chdir(repo_path) do
@@ -59,20 +66,16 @@ module MutantSpec
       #
       # @raise [Exception]
       #   otherwise
-      #
-      # rubocop:disable AbcSize
-      #
-      # @api private
       def verify_mutation_generation
         checkout
         start = Time.now
-        paths = Pathname.glob(repo_path.join('**/*.rb')).sort_by(&:size).reverse
+
         options = {
           finish:       method(:finish),
           start:        method(:start),
           in_processes: parallel_processes
         }
-        total = Parallel.map(paths, options) do |path|
+        total = Parallel.map(effective_ruby_paths, options) do |path|
           count = 0
           node =
             begin
@@ -100,8 +103,6 @@ module MutantSpec
       # Checkout repository
       #
       # @return [self]
-      #
-      # @api private
       def checkout
         return self if noinstall?
         TMP.mkdir unless TMP.directory?
@@ -126,8 +127,6 @@ module MutantSpec
       # Install mutant
       #
       # @return [undefined]
-      #
-      # @api private
       def install_mutant
         return if noinstall?
         relative = ROOT.relative_path_from(repo_path)
@@ -138,20 +137,34 @@ module MutantSpec
         end
         lockfile = repo_path.join('Gemfile.lock')
         lockfile.delete if lockfile.exist?
-        system('bundle install')
+        system(%w[bundle])
       end
 
-      # Not in the docs. Number from chatting with their support.
-      CIRCLE_CI_CONTAINER_PROCESSES = 2
+      # The effective ruby file paths
+      #
+      # @return [Array<Pathname>]
+      def effective_ruby_paths
+        paths = Pathname
+          .glob(repo_path.join(RUBY_GLOB_PATTERN))
+          .sort_by(&:size)
+          .reverse
+
+        paths - excluded_paths
+      end
+
+      # The excluded file paths
+      #
+      # @return [Array<Pathname>]
+      def excluded_paths
+        Pathname.glob(repo_path.join(EXCLUDE_GLOB_FORMAT % exclude.join(',')))
+      end
 
       # Number of parallel processes to use
       #
       # @return [Fixnum]
-      #
-      # @api private
       def parallel_processes
-        if ENV['CI']
-          Mutant::Config::DEFAULT.jobs
+        if ENV.key?('CI')
+          CIRCLE_CI_CONTAINER_PROCESSES
         else
           Parallel.processor_count
         end
@@ -160,8 +173,6 @@ module MutantSpec
       # Repository path
       #
       # @return [Pathname]
-      #
-      # @api private
       def repo_path
         TMP.join(name)
       end
@@ -169,8 +180,6 @@ module MutantSpec
       # Test if installation should be skipped
       #
       # @return [Boolean]
-      #
-      # @api private
       def noinstall?
         ENV.key?('NOINSTALL')
       end
@@ -205,8 +214,6 @@ module MutantSpec
       # Helper method to execute system commands
       #
       # @param [Array<String>] arguments
-      #
-      # @api private
       def system(arguments)
         return if Kernel.system(*arguments)
         if block_given?

data/spec/support/rb_bug.rb

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

data/spec/support/ruby_vm.rb

@@ -12,7 +12,10 @@ module MutantSpec
 
     # An event being expected, can advance the VM
     class EventExpectation
-      include AbstractType, Anima.new(:expected_payload, :trigger_requires)
+      include AbstractType, Anima.new(
+        :expected_payload,
+        :trigger_requires
+      )
 
       DEFAULTS = IceNine.deep_freeze(trigger_requires: [])
 
@@ -56,8 +59,8 @@ module MutantSpec
       handle_event(
         EventObservation.new(
           EventExpectation::Eval,
-          binding:  binding,
-          source:   source,
+          binding:         binding,
+          source:          source,
           source_location: location
         )
       )

data/spec/support/shared_context.rb

@@ -25,22 +25,22 @@ module SharedContext
   # rubocop:disable MethodLength
   # rubocop:disable AbcSize
   def setup_shared_context
-    let(:env)              { double('env', config: config, subjects: [subject_a], mutations: mutations) }
-    let(:job_a)            { Mutant::Parallel::Job.new(index: 0, payload: mutation_a)                   }
-    let(:job_b)            { Mutant::Parallel::Job.new(index: 1, payload: mutation_b)                   }
-    let(:job_a_result)     { Mutant::Runner::JobResult.new(job: job_a, result: mutation_a_result)       }
-    let(:job_b_result)     { Mutant::Runner::JobResult.new(job: job_b, result: mutation_b_result)       }
-    let(:test_a)           { double('test a', identification: 'test-a')                                 }
-    let(:test_b)           { double('test b', identification: 'test-b')                                 }
-    let(:output)           { StringIO.new                                                               }
-    let(:matchable_scopes) { double('matchable scopes', length: 10)                                     }
-    let(:message_sequence) { FakeActor::MessageSequence.new                                             }
-    let(:mutations)        { [mutation_a, mutation_b]                                                   }
-    let(:mutation_a_node)  { s(:false)                                                                  }
-    let(:mutation_b_node)  { s(:nil)                                                                    }
-    let(:mutation_b)       { Mutant::Mutation::Evil.new(subject_a, mutation_b_node)                     }
-    let(:mutation_a)       { Mutant::Mutation::Evil.new(subject_a, mutation_a_node)                     }
-    let(:subject_a_node)   { s(:true)                                                                   }
+    let(:env)              { instance_double(Mutant::Env, config: config, subjects: [subject_a], mutations: mutations) }
+    let(:job_a)            { Mutant::Parallel::Job.new(index: 0, payload: mutation_a)                                  }
+    let(:job_b)            { Mutant::Parallel::Job.new(index: 1, payload: mutation_b)                                  }
+    let(:job_a_result)     { Mutant::Runner::JobResult.new(job: job_a, result: mutation_a_result)                      }
+    let(:job_b_result)     { Mutant::Runner::JobResult.new(job: job_b, result: mutation_b_result)                      }
+    let(:test_a)           { instance_double(Mutant::Test, identification: 'test-a')                                   }
+    let(:test_b)           { instance_double(Mutant::Test, identification: 'test-b')                                   }
+    let(:output)           { StringIO.new                                                                              }
+    let(:matchable_scopes) { instance_double(Array, length: 10)                                                        }
+    let(:message_sequence) { FakeActor::MessageSequence.new                                                            }
+    let(:mutations)        { [mutation_a, mutation_b]                                                                  }
+    let(:mutation_a_node)  { s(:false)                                                                                 }
+    let(:mutation_b_node)  { s(:nil)                                                                                   }
+    let(:mutation_b)       { Mutant::Mutation::Evil.new(subject_a, mutation_b_node)                                    }
+    let(:mutation_a)       { Mutant::Mutation::Evil.new(subject_a, mutation_a_node)                                    }
+    let(:subject_a_node)   { s(:true)                                                                                  }
 
     let(:status) do
       Mutant::Parallel::Status.new(
@@ -53,16 +53,15 @@ module SharedContext
     let(:config) do
       Mutant::Config::DEFAULT.with(
         jobs:      1,
-        reporter:  Mutant::Reporter::Trace.new
+        reporter:  Mutant::Reporter::Null.new
       )
     end
 
     let(:subject_a) do
-      double(
-        'subject a',
+      instance_double(
+        Mutant::Subject,
         node:           subject_a_node,
         source:         Unparser.unparse(subject_a_node),
-        tests:          [test_a],
         identification: 'subject-a'
       )
     end

data/spec/unit/mutant/actor/binding_spec.rb

@@ -1,10 +1,10 @@
 RSpec.describe Mutant::Actor::Binding do
-  let(:actor_a)    { double('Actor-A', sender: sender_a, receiver: receiver_a) }
-  let(:sender_a)   { double('Sender-A')                                        }
-  let(:sender_b)   { double('Sender-B')                                        }
-  let(:receiver_a) { double('Receiver-A')                                      }
-  let(:payload)    { double('Payload')                                         }
-  let(:type)       { double('Type')                                            }
+  let(:actor_a)    { instance_double(Mutant::Actor::Mailbox, sender: sender_a, receiver: receiver_a) }
+  let(:sender_a)   { instance_double(Mutant::Actor::Sender)                                          }
+  let(:sender_b)   { instance_double(Mutant::Actor::Sender)                                          }
+  let(:receiver_a) { instance_double(Mutant::Actor::Receiver)                                        }
+  let(:payload)    { instance_double(Object)                                                         }
+  let(:type)       { instance_double(Symbol)                                                         }
 
   let(:object) { described_class.new(actor_a, sender_b) }