mutant 0.9.5 → 0.9.10

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

@@ -116,7 +116,7 @@ RSpec.describe Mutant::Reporter::CLI do
       let(:tty?) { true }
 
       # rubocop:disable Metrics/LineLength
-      it_reports Mutant::Color::GREEN.format('progress: 00/02 alive: 0 runtime: 4.00s killtime: 0.00s mutations/s: 0.00') + "\n"
+      it_reports Unparser::Color::GREEN.format('progress: 00/02 alive: 0 runtime: 4.00s killtime: 0.00s mutations/s: 0.00') + "\n"
       # rubocop:enable Metrics/LineLength
     end
 

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

@@ -62,10 +62,6 @@ RSpec.describe Mutant::Subject::Method::Instance do
   end
 
   describe '#prepare' do
-    let(:context) do
-      Mutant::Context.new(scope, instance_double(Pathname))
-    end
-
     subject { object.prepare }
 
     it 'undefines method on scope' do
@@ -104,7 +100,10 @@ RSpec.describe Mutant::Subject::Method::Instance::Memoized do
     )
   end
 
-  let(:context)  { double('Context')                 }
+  let(:context) do
+    Mutant::Context.new(scope, double('Source Path'))
+  end
+
   let(:warnings) { instance_double(Mutant::Warnings) }
 
   let(:node) do
@@ -115,12 +114,7 @@ RSpec.describe Mutant::Subject::Method::Instance::Memoized do
     allow(warnings).to receive(:call).and_yield
   end
 
-  describe '#prepare' do
-
-    let(:context) do
-      Mutant::Context.new(scope, double('Source Path'))
-    end
-
+  shared_context 'memoizable scope setup' do
     let(:scope) do
       Class.new do
         include Memoizable
@@ -128,6 +122,23 @@ RSpec.describe Mutant::Subject::Method::Instance::Memoized do
         memoize :foo
       end
     end
+  end
+
+  shared_context 'adamantium scope setup' do
+    let(:scope) do
+      memoize_options  = self.memoize_options
+      memoize_provider = self.memoize_provider
+
+      Class.new do
+        include memoize_provider
+        def foo; end
+        memoize :foo, **memoize_options
+      end
+    end
+  end
+
+  describe '#prepare' do
+    include_context 'memoizable scope setup'
 
     subject { object.prepare }
 
@@ -142,40 +153,124 @@ RSpec.describe Mutant::Subject::Method::Instance::Memoized do
     it_should_behave_like 'a command method'
   end
 
-  describe '#mutations', mutant_expression: 'Mutant::Subject#mutations' do
+  describe '#mutations' do
     subject { object.mutations }
 
     let(:expected) do
       [
         Mutant::Mutation::Neutral.new(
           object,
-          s(:begin,
-            s(:def, :foo, s(:args)), s(:send, nil, :memoize, s(:args, s(:sym, :foo))))
+          s(:begin, s(:def, :foo, s(:args)), memoize_node)
         ),
         Mutant::Mutation::Evil.new(
           object,
-          s(:begin,
-            s(:def, :foo, s(:args), s(:send, nil, :raise)), s(:send, nil, :memoize, s(:args, s(:sym, :foo))))
+          s(:begin, s(:def, :foo, s(:args), s(:send, nil, :raise)), memoize_node)
         ),
         Mutant::Mutation::Evil.new(
           object,
-          s(:begin,
-            s(:def, :foo, s(:args), s(:zsuper)), s(:send, nil, :memoize, s(:args, s(:sym, :foo))))
+          s(:begin, s(:def, :foo, s(:args), s(:zsuper)), memoize_node)
         ),
         Mutant::Mutation::Evil.new(
           object,
-          s(:begin,
-            s(:def, :foo, s(:args), nil), s(:send, nil, :memoize, s(:args, s(:sym, :foo))))
+          s(:begin, s(:def, :foo, s(:args), nil), memoize_node)
         )
       ]
     end
 
-    it { should eql(expected) }
+    let(:memoize_node) do
+      s(:send, nil, :memoize, s(:args, s(:sym, :foo), *options_node))
+    end
+
+    let(:options_node) { nil }
+
+    context 'when Memoizable is included in scope' do
+      include_context 'memoizable scope setup'
+
+      it { should eql(expected) }
+    end
+
+    context 'when Adamantium is included in scope' do
+      include_context 'adamantium scope setup'
+
+      {
+        Adamantium       => :deep,
+        Adamantium::Flat => :flat
+      }.each do |memoize_provider, default_freezer_option|
+        context "as include #{memoize_provider}" do
+          let(:memoize_provider)       { memoize_provider       }
+          let(:default_freezer_option) { default_freezer_option }
+
+          let(:options_node) do
+            [s(:hash, s(:pair, s(:sym, :freezer), s(:sym, freezer_option)))]
+          end
+
+          context 'when no memoize options are given' do
+            let(:memoize_options) { Mutant::EMPTY_HASH     }
+            let(:freezer_option)  { default_freezer_option }
+
+            it { should eql(expected) }
+          end
+
+          context 'when memoize options are given' do
+            let(:memoize_options) { { freezer: freezer_option } }
+
+            %i[deep flat noop].each do |option|
+              context "as #{option.inspect}" do
+                let(:freezer_option) { option }
+
+                it { should eql(expected) }
+              end
+            end
+          end
+        end
+      end
+    end
   end
 
   describe '#source' do
     subject { object.source }
 
-    it { should eql("def foo\nend\nmemoize(:foo)") }
+    context 'when Memoizable is included in scope' do
+      include_context 'memoizable scope setup'
+
+      let(:source) { "def foo\nend\nmemoize(:foo)" }
+
+      it { should eql(source) }
+    end
+
+    context 'when Adamantium is included in scope' do
+      include_context 'adamantium scope setup'
+
+      let(:source) do
+        "def foo\nend\nmemoize(:foo, { freezer: #{freezer_option.inspect} })"
+      end
+
+      {
+        Adamantium       => :deep,
+        Adamantium::Flat => :flat
+      }.each do |memoize_provider, default_freezer_option|
+        context "as include #{memoize_provider}" do
+          let(:memoize_provider) { memoize_provider }
+
+          context 'when no memoize options are given' do
+            let(:memoize_options) { Mutant::EMPTY_HASH     }
+            let(:freezer_option)  { default_freezer_option }
+
+            it { should eql(source) }
+          end
+
+          context 'when memoize options are given' do
+            %i[deep flat noop].each do |freezer_option|
+              context "as #{freezer_option.inspect}" do
+                let(:memoize_options) { { freezer: freezer_option } }
+                let(:freezer_option)  { freezer_option              }
+
+                it { should eql(source) }
+              end
+            end
+          end
+        end
+      end
+    end
   end
 end

data/spec/unit/mutant/subject/method/metaclass_spec.rb

@@ -0,0 +1,63 @@
+# frozen_string_literal: true
+
+RSpec.describe Mutant::Subject::Method::Metaclass do
+  let(:object) do
+    described_class.new(
+      context:  context,
+      node:     node,
+      warnings: warnings
+    )
+  end
+
+  let(:node)     { s(:def, :foo, s(:args)) }
+  let(:warnings) { instance_double(Mutant::Warnings) }
+
+  let(:context) do
+    Mutant::Context.new(scope, instance_double(Pathname))
+  end
+
+  let(:scope) do
+    Class.new do
+      class << self
+        def foo; end
+
+        def name
+          'Test'
+        end
+      end
+    end
+  end
+
+  describe '#expression' do
+    subject { object.expression }
+
+    it { should eql(parse_expression('Test.foo')) }
+
+    it_should_behave_like 'an idempotent method'
+  end
+
+  describe '#match_expression' do
+    subject { object.match_expressions }
+
+    it { should eql(%w[Test.foo Test*].map(&method(:parse_expression))) }
+
+    it_should_behave_like 'an idempotent method'
+  end
+
+  describe '#prepare' do
+
+    subject { object.prepare }
+
+    it 'undefines method on scope' do
+      expect { subject }.to change { scope.methods.include?(:foo) }.from(true).to(false)
+    end
+
+    it_should_behave_like 'a command method'
+  end
+
+  describe '#source' do
+    subject { object.source }
+
+    it { should eql("class << self\n  def foo\n  end\nend") }
+  end
+end

data/test_app/Gemfile.minitest

@@ -2,3 +2,5 @@ source 'https://rubygems.org'
 
 gem 'mutant',          path: '../'
 gem 'mutant-minitest', path: '../'
+
+eval_gemfile File.expand_path('../../Gemfile.shared', __FILE__)

data/test_app/lib/test_app.rb

@@ -103,7 +103,12 @@ module TestApp
       end
     end
   end
+
+  def self.root
+    File.expand_path('..', __dir__)
+  end
 end
 
+require 'test_app/metaclasses'
 require 'test_app/literal'
 $VERBOSE = original

data/test_app/lib/test_app/metaclasses.rb

@@ -0,0 +1,108 @@
+# frozen_string_literal: true
+
+module TestApp
+  module MetaclassMethodTests
+    module DefinedOnSelf
+      class << self
+        def foo; end
+      end
+
+      # again this is a weird edge-case that's being checked for consistent
+      # behaviour, not something that people ought to be doing.
+      module InsideMetaclass
+        class << self
+          # some older versions of ruby don't have Object#singleton_class,
+          # this is just an implementation of that so we can grab
+          # InsideMetaclass.metaclass for use as the scope object
+          def metaclass
+            class << self
+              self
+            end
+          end
+
+          # InsideMetaclass.foo
+          def foo; end
+
+          class << self
+            # #<Class:InsideMetaclass>.foo
+            def foo; end
+          end
+        end
+      end
+    end
+
+    module DefinedOnLvar
+      a = self
+      class << a
+        def foo; end
+      end
+    end
+
+    module DefinedOnConstant
+      module InsideNamespace
+        class << InsideNamespace
+          def foo; end
+        end
+      end
+
+      module OutsideNamespace
+      end
+
+      class << OutsideNamespace
+        def foo
+        end
+      end
+    end
+
+    module DefinedMultipleTimes
+      module DifferentLines
+        class << self
+          def foo
+          end
+
+          def foo(_arg)
+          end
+        end
+      end
+
+      module SameLine
+        module SameScope
+          class << self
+            def foo; end; def foo(_arg); end
+          end
+        end
+
+        module DifferentScope
+          class << self; def foo; end; end; class << DifferentScope; def foo(_arg); end; end; class << MetaclassMethodTests; def foo; end; end
+        end
+
+        module DifferentName
+          class << self; def foo; end; def bar(_arg); end; end
+        end
+      end
+    end
+
+    module NotActuallyInAMetaclass
+      class << self
+        # some older versions of ruby don't have Object#singleton_class,
+        # this is just an implementation of that so we can grab
+        # InsideMetaclass.metaclass for use as the scope object
+        def metaclass
+          class << self
+            self
+          end
+        end
+
+        # this is a very weird construction that I do not recommend.
+        # to access SomeClass you have to do
+        # NotActuallyInAMetaclass.singleton_class::SomeClass. Very weird
+        # This is only here so we can test that metaclass_receiver? correctly
+        # returns false
+        class SomeClass
+          def foo
+          end
+        end
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mutant
 version: !ruby/object:Gem::Version
-  version: 0.9.5
+  version: 0.9.10
 platform: ruby
 authors:
 - Markus Schirp
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-02-02 00:00:00.000000000 Z
+date: 2020-08-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: abstract_type
@@ -156,14 +156,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.7.0.2
+        version: 2.7.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.7.0.2
+        version: 2.7.1
 - !ruby/object:Gem::Dependency
   name: procto
   requirement: !ruby/object:Gem::Requirement
@@ -184,28 +184,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.4.6
+        version: 0.4.8
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.4.6
+        version: 0.4.8
 - !ruby/object:Gem::Dependency
-  name: devtools
+  name: variable
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.1.25
-  type: :development
+        version: 0.0.1
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.1.25
+        version: 0.0.1
 - !ruby/object:Gem::Dependency
   name: parallel
   requirement: !ruby/object:Gem::Requirement
@@ -220,6 +220,62 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.3'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.9'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.9'
+- !ruby/object:Gem::Dependency
+  name: rspec-core
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.9'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.9'
+- !ruby/object:Gem::Dependency
+  name: rspec-its
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.2.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.2.0
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.79.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.79.0
 description: Mutation Testing for Ruby.
 email:
 - mbj@schirp-dso.com
@@ -229,7 +285,7 @@ extensions: []
 extra_rdoc_files:
 - LICENSE
 files:
-- ".circleci/config.yml"
+- ".github/workflows/ci.yml"
 - ".gitignore"
 - ".rspec"
 - ".rubocop.yml"
@@ -243,7 +299,6 @@ files:
 - bin/mutant
 - config/devtools.yml
 - config/reek.yml
-- config/rubocop.yml
 - config/yardstick.yml
 - docs/commercial-support.md
 - docs/concurrency.md
@@ -256,6 +311,7 @@ files:
 - docs/reading-reports.md
 - lib/mutant.rb
 - lib/mutant/ast.rb
+- lib/mutant/ast/find_metaclass_containing.rb
 - lib/mutant/ast/meta.rb
 - lib/mutant/ast/meta/const.rb
 - lib/mutant/ast/meta/optarg.rb
@@ -269,10 +325,8 @@ files:
 - lib/mutant/ast/types.rb
 - lib/mutant/bootstrap.rb
 - lib/mutant/cli.rb
-- lib/mutant/color.rb
 - lib/mutant/config.rb
 - lib/mutant/context.rb
-- lib/mutant/diff.rb
 - lib/mutant/env.rb
 - lib/mutant/expression.rb
 - lib/mutant/expression/method.rb
@@ -295,6 +349,7 @@ files:
 - lib/mutant/matcher/filter.rb
 - lib/mutant/matcher/method.rb
 - lib/mutant/matcher/method/instance.rb
+- lib/mutant/matcher/method/metaclass.rb
 - lib/mutant/matcher/method/singleton.rb
 - lib/mutant/matcher/methods.rb
 - lib/mutant/matcher/namespace.rb
@@ -354,6 +409,7 @@ files:
 - lib/mutant/mutator/node/resbody.rb
 - lib/mutant/mutator/node/rescue.rb
 - lib/mutant/mutator/node/return.rb
+- lib/mutant/mutator/node/sclass.rb
 - lib/mutant/mutator/node/send.rb
 - lib/mutant/mutator/node/send/attribute_assignment.rb
 - lib/mutant/mutator/node/send/binary.rb
@@ -404,12 +460,12 @@ files:
 - lib/mutant/subject.rb
 - lib/mutant/subject/method.rb
 - lib/mutant/subject/method/instance.rb
+- lib/mutant/subject/method/metaclass.rb
 - lib/mutant/subject/method/singleton.rb
 - lib/mutant/test.rb
 - lib/mutant/timer.rb
 - lib/mutant/transform.rb
 - lib/mutant/util.rb
-- lib/mutant/variable.rb
 - lib/mutant/version.rb
 - lib/mutant/warnings.rb
 - lib/mutant/zombifier.rb
@@ -468,6 +524,7 @@ files:
 - meta/regopt.rb
 - meta/rescue.rb
 - meta/return.rb
+- meta/sclass.rb
 - meta/self.rb
 - meta/send.rb
 - meta/str.rb
@@ -480,6 +537,7 @@ files:
 - mutant-minitest.gemspec
 - mutant-rspec.gemspec
 - mutant.gemspec
+- mutant.sh
 - mutant.yml
 - spec/integration/mutant/corpus_spec.rb
 - spec/integration/mutant/isolation/fork_spec.rb
@@ -488,7 +546,6 @@ files:
 - spec/integration/mutant/rspec_spec.rb
 - spec/integration/mutant/test_mutator_handles_types_spec.rb
 - spec/integrations.yml
-- spec/shared/base_behavior.rb
 - spec/shared/framework_integration_behavior.rb
 - spec/shared/method_matcher_behavior.rb
 - spec/spec_helper.rb
@@ -497,9 +554,8 @@ files:
 - spec/support/ice_nine_config.rb
 - spec/support/ruby_vm.rb
 - spec/support/shared_context.rb
-- spec/support/test_app.rb
-- spec/support/warnings.yml
 - spec/support/xspec.rb
+- spec/unit/mutant/ast/find_metaclass_containing_spec.rb
 - spec/unit/mutant/ast/meta/optarg_spec.rb
 - spec/unit/mutant/ast/meta/send/proc_predicate_spec.rb
 - spec/unit/mutant/ast/meta/send/receiver_possible_top_level_const_predicate_spec.rb
@@ -512,7 +568,6 @@ files:
 - spec/unit/mutant/clock_monotonic_spec.rb
 - spec/unit/mutant/config_spec.rb
 - spec/unit/mutant/context_spec.rb
-- spec/unit/mutant/diff_spec.rb
 - spec/unit/mutant/env_spec.rb
 - spec/unit/mutant/expression/method_spec.rb
 - spec/unit/mutant/expression/methods_spec.rb
@@ -532,8 +587,10 @@ files:
 - spec/unit/mutant/matcher/config_spec.rb
 - spec/unit/mutant/matcher/filter_spec.rb
 - spec/unit/mutant/matcher/method/instance_spec.rb
+- spec/unit/mutant/matcher/method/metaclass_spec.rb
 - spec/unit/mutant/matcher/method/singleton_spec.rb
 - spec/unit/mutant/matcher/methods/instance_spec.rb
+- spec/unit/mutant/matcher/methods/metaclass_spec.rb
 - spec/unit/mutant/matcher/methods/singleton_spec.rb
 - spec/unit/mutant/matcher/namespace_spec.rb
 - spec/unit/mutant/matcher/null_spec.rb
@@ -582,6 +639,7 @@ files:
 - spec/unit/mutant/selector/expression_spec.rb
 - spec/unit/mutant/selector/null_spec.rb
 - spec/unit/mutant/subject/method/instance_spec.rb
+- spec/unit/mutant/subject/method/metaclass_spec.rb
 - spec/unit/mutant/subject/method/singleton_spec.rb
 - spec/unit/mutant/subject_spec.rb
 - spec/unit/mutant/transform/array_spec.rb
@@ -594,7 +652,6 @@ files:
 - spec/unit/mutant/transform/primitive_spec.rb
 - spec/unit/mutant/transform/sequence_spec.rb
 - spec/unit/mutant/util/one_spec.rb
-- spec/unit/mutant/variable_spec.rb
 - spec/unit/mutant/warnings_spec.rb
 - spec/unit/mutant/world_spec.rb
 - spec/unit/mutant/zombifier_spec.rb
@@ -603,6 +660,7 @@ files:
 - test_app/Gemfile.rspec3.8
 - test_app/lib/test_app.rb
 - test_app/lib/test_app/literal.rb
+- test_app/lib/test_app/metaclasses.rb
 - test_app/spec/spec_helper.rb
 - test_app/spec/unit/test_app/literal_spec.rb
 - test_app/test/unit/test_app/literal_test.rb
@@ -610,7 +668,7 @@ homepage: https://github.com/mbj/mutant
 licenses:
 - Nonstandard
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -626,7 +684,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.0.3
-signing_key: 
+signing_key:
 specification_version: 4
 summary: ''
 test_files:
@@ -636,6 +694,7 @@ test_files:
 - spec/integration/mutant/null_spec.rb
 - spec/integration/mutant/rspec_spec.rb
 - spec/integration/mutant/test_mutator_handles_types_spec.rb
+- spec/unit/mutant/ast/find_metaclass_containing_spec.rb
 - spec/unit/mutant/ast/meta/optarg_spec.rb
 - spec/unit/mutant/ast/meta/send/proc_predicate_spec.rb
 - spec/unit/mutant/ast/meta/send/receiver_possible_top_level_const_predicate_spec.rb
@@ -648,7 +707,6 @@ test_files:
 - spec/unit/mutant/clock_monotonic_spec.rb
 - spec/unit/mutant/config_spec.rb
 - spec/unit/mutant/context_spec.rb
-- spec/unit/mutant/diff_spec.rb
 - spec/unit/mutant/env_spec.rb
 - spec/unit/mutant/expression/method_spec.rb
 - spec/unit/mutant/expression/methods_spec.rb
@@ -668,8 +726,10 @@ test_files:
 - spec/unit/mutant/matcher/config_spec.rb
 - spec/unit/mutant/matcher/filter_spec.rb
 - spec/unit/mutant/matcher/method/instance_spec.rb
+- spec/unit/mutant/matcher/method/metaclass_spec.rb
 - spec/unit/mutant/matcher/method/singleton_spec.rb
 - spec/unit/mutant/matcher/methods/instance_spec.rb
+- spec/unit/mutant/matcher/methods/metaclass_spec.rb
 - spec/unit/mutant/matcher/methods/singleton_spec.rb
 - spec/unit/mutant/matcher/namespace_spec.rb
 - spec/unit/mutant/matcher/null_spec.rb
@@ -718,6 +778,7 @@ test_files:
 - spec/unit/mutant/selector/expression_spec.rb
 - spec/unit/mutant/selector/null_spec.rb
 - spec/unit/mutant/subject/method/instance_spec.rb
+- spec/unit/mutant/subject/method/metaclass_spec.rb
 - spec/unit/mutant/subject/method/singleton_spec.rb
 - spec/unit/mutant/subject_spec.rb
 - spec/unit/mutant/transform/array_spec.rb
@@ -730,7 +791,6 @@ test_files:
 - spec/unit/mutant/transform/primitive_spec.rb
 - spec/unit/mutant/transform/sequence_spec.rb
 - spec/unit/mutant/util/one_spec.rb
-- spec/unit/mutant/variable_spec.rb
 - spec/unit/mutant/warnings_spec.rb
 - spec/unit/mutant/world_spec.rb
 - spec/unit/mutant/zombifier_spec.rb