mutant 0.9.7 → 0.9.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9191d87ebe0bde331ae432c838ec2718f41f89d099952598e98fe6d2ee3b9885
-  data.tar.gz: 713134773a289c22d6b43b678c44b483f383aeb2520fd0f95711c47489ad64f0
+  metadata.gz: 71f2d39499ffdc3adced885c60b9b22220e409f360fce5ab8ac0aaeb36bdbda3
+  data.tar.gz: dee6aa26ddce9e06949566dfc75d4daf295ec76ea46903af80b0ac4027f338ac
 SHA512:
-  metadata.gz: 00b83021df08f58b84688d6dad27792e0d5aab81288bf5f7afca9a43b2ebdf05fe09133c18dc19e8169727c20e4815caaafabfab9bbc9c7258100b9201ba967a
-  data.tar.gz: 79659afba997fc24650398f33bc0de60905845429b01a49d999c24e2004f97b0c0ac3bee987e1cbf73010c213906e79a0e8fb856b5de826049cfc169f8154f24
+  metadata.gz: e6a44765e6fec377e0f4d9e3047832468af386b34f5d7a231fb940802a97977bba6000b016ca6739f402314bf03027dd8852776ff1ee687d6e67fb6168d5416a
+  data.tar.gz: 9d696a03e4a86c8838fa73ab147a75dac9f8f14b1f20eef653b30a3758c4044f016f4444f94827738d6bd39d05fdd9cddafdcfbd2a6475c2ebbc8674f005738d

data/lib/mutant.rb

@@ -51,6 +51,7 @@ require 'mutant/ast/types'
 require 'mutant/ast/nodes'
 require 'mutant/ast/named_children'
 require 'mutant/ast/node_predicates'
+require 'mutant/ast/find_metaclass_containing'
 require 'mutant/ast/meta'
 require 'mutant/ast/meta/send'
 require 'mutant/ast/meta/const'
@@ -112,12 +113,14 @@ require 'mutant/mutator/node/send/conditional'
 require 'mutant/mutator/node/send/attribute_assignment'
 require 'mutant/mutator/node/when'
 require 'mutant/mutator/node/class'
+require 'mutant/mutator/node/sclass'
 require 'mutant/mutator/node/define'
 require 'mutant/mutator/node/mlhs'
 require 'mutant/mutator/node/nthref'
 require 'mutant/mutator/node/masgn'
 require 'mutant/mutator/node/return'
 require 'mutant/mutator/node/block'
+require 'mutant/mutator/node/block_pass'
 require 'mutant/mutator/node/if'
 require 'mutant/mutator/node/case'
 require 'mutant/mutator/node/splat'
@@ -134,11 +137,13 @@ require 'mutant/subject'
 require 'mutant/subject/method'
 require 'mutant/subject/method/instance'
 require 'mutant/subject/method/singleton'
+require 'mutant/subject/method/metaclass'
 require 'mutant/matcher'
 require 'mutant/matcher/config'
 require 'mutant/matcher/chain'
 require 'mutant/matcher/method'
 require 'mutant/matcher/method/singleton'
+require 'mutant/matcher/method/metaclass'
 require 'mutant/matcher/method/instance'
 require 'mutant/matcher/methods'
 require 'mutant/matcher/namespace'
@@ -161,8 +166,6 @@ require 'mutant/selector/expression'
 require 'mutant/selector/null'
 require 'mutant/config'
 require 'mutant/cli'
-require 'mutant/color'
-require 'mutant/diff'
 require 'mutant/runner'
 require 'mutant/runner/sink'
 require 'mutant/result'

data/lib/mutant/ast.rb

@@ -27,15 +27,6 @@ module Mutant
       path
     end
 
-    # Walk all ast nodes keeping track of path
-    #
-    # @param [Parser::AST::Node] root
-    # @param [Array<Parser::AST::Node>] stack
-    #
-    # @yield [Parser::AST::Node]
-    #   all nodes visited recursively including root
-    #
-    # @return [undefined]
     def self.walk(node, stack, &block)
       block.call(node, stack)
       node.children.grep(::Parser::AST::Node) do |child|

data/lib/mutant/ast/find_metaclass_containing.rb

@@ -0,0 +1,48 @@
+# frozen_string_literal: true
+
+module Mutant
+  module AST
+    # Given an AST, finds the sclass that directly(-ish) contains the provided
+    # node.
+    # This won't match arbitrarily complex structures - it only searches the
+    # first level deep (no begins-in-begins, for example). This is in
+    # keeping with mutant generally not supporting 'weird' syntax.
+    # Descending into 'begin' nodes is supported because these are generated for
+    # the one-line syntax class << self; def foo; end
+    class FindMetaclassContaining
+      include NodePredicates, Concord.new(:root, :target), Procto.call
+
+      SCLASS_BODY_INDEX = 1
+
+      private_constant(*constants(false))
+
+      # Find metaclass node containing target node
+      #
+      # @return [Parser::AST::Node, nil]
+      #
+      # @api private
+      def call
+        AST.find_last_path(root) do |current|
+          next unless n_sclass?(current)
+
+          metaclass_of?(current)
+        end.last
+      end
+
+    private
+
+      def metaclass_of?(sclass)
+        body = sclass.children.fetch(SCLASS_BODY_INDEX)
+        body.equal?(target) || transparently_contains?(body)
+      end
+
+      def transparently_contains?(body)
+        n_begin?(body) && include_exact?(body.children, target)
+      end
+
+      def include_exact?(haystack, needle)
+        haystack.any? { |elem| elem.equal?(needle) }
+      end
+    end
+  end
+end

data/lib/mutant/ast/meta/send.rb

@@ -56,16 +56,10 @@ module Mutant
 
       private
 
-        # Test if node is `proc { ... }`
-        #
-        # @return [Boolean]
         def naked_proc?
           !receiver && selector.equal?(:proc)
         end
 
-        # Test if node is `Proc.new { ... }`
-        #
-        # @return [Boolean]
         def proc_new?
           receiver                &&
             selector.equal?(:new) &&

data/lib/mutant/bootstrap.rb

@@ -49,9 +49,6 @@ module Mutant
     end
     # rubocop:enable Metrics/MethodLength
 
-    # Infect environment
-    #
-    # @return [undefined]
     def self.infect(env)
       config, world = env.config, env.world
 
@@ -60,12 +57,6 @@ module Mutant
     end
     private_class_method :infect
 
-    # Matchable scopes
-    #
-    # @param [World] world
-    # @param [Config] config
-    #
-    # @return [Array<Scope>]
     def self.matchable_scopes(world, config)
       scopes = world.object_space.each_object(Module).each_with_object([]) do |scope, aggregate|
         expression = expression(config.reporter, config.expression_parser, scope) || next
@@ -76,15 +67,6 @@ module Mutant
     end
     private_class_method :matchable_scopes
 
-    # Scope name from scoping object
-    #
-    # @param [Class, Module] scope
-    #
-    # @return [String]
-    #   if scope has a name and does not raise exceptions obtaining it
-    #
-    # @return [nil]
-    #   otherwise
     def self.scope_name(reporter, scope)
       scope.name
     rescue => exception
@@ -99,20 +81,7 @@ module Mutant
     end
     private_class_method :scope_name
 
-    # Try to turn scope into expression
-    #
-    # @param [Expression::Parser] expression_parser
-    # @param [Class, Module] scope
-    #
-    # @return [Expression]
-    #   if scope can be represented in an expression
-    #
-    # @return [nil]
-    #   otherwise
-    #
     # rubocop:disable Metrics/MethodLength
-    #
-    # ignore :reek:LongParameterList
     def self.expression(reporter, expression_parser, scope)
       name = scope_name(reporter, scope) or return
 
@@ -132,11 +101,6 @@ module Mutant
     private_class_method :expression
     # rubocop:enable Metrics/MethodLength
 
-    # Write a semantics warning
-    #
-    # @return [undefined]
-    #
-    # ignore :reek:LongParameterList
     def self.semantics_warning(reporter, format, options)
       reporter.warn(SEMANTICS_MESSAGE_FORMAT % { message: format % options })
     end

data/lib/mutant/cli.rb

@@ -88,23 +88,12 @@ module Mutant
 
   private
 
-    # Parse matchers
-    #
-    # @param [Array<String>] expressions
-    #
-    # @return [undefined]
     def parse_match_expressions(expressions)
       expressions.each do |expression|
         add_matcher(:match_expressions, config.expression_parser.apply(expression).from_right)
       end
     end
 
-    # Add environmental options
-    #
-    # @param [Object] opts
-    #
-    # @return [undefined]
-    #
     # rubocop:disable Metrics/MethodLength
     def add_environment_options(opts)
       opts.separator('Environment:')
@@ -121,12 +110,8 @@ module Mutant
         with(jobs: Integer(number))
       end
     end
+    # rubocop:enable Metrics/MethodLength
 
-    # Add mutation options
-    #
-    # @param [OptionParser] opts
-    #
-    # @return [undefined]
     def add_mutation_options(opts)
       opts.separator(nil)
       opts.separator('Options:')
@@ -136,11 +121,7 @@ module Mutant
       end
     end
 
-    # Add filter options
-    #
-    # @param [OptionParser] opts
-    #
-    # @return [undefined]
+    # rubocop:disable Metrics/MethodLength
     def add_filter_options(opts)
       opts.on('--ignore-subject EXPRESSION', 'Ignore subjects that match EXPRESSION as prefix') do |pattern|
         add_matcher(:ignore_expressions, config.expression_parser.apply(pattern).from_right)
@@ -154,12 +135,9 @@ module Mutant
         )
       end
     end
+    # rubocop:enable Metrics/MethodLength
 
-    # Add debug options
-    #
-    # @param [OptionParser] opts
-    #
-    # @return [undefined]
+    # rubocop:disable Metrics/MethodLength
     def add_debug_options(opts)
       opts.on('--fail-fast', 'Fail fast') do
         with(fail_fast: true)
@@ -173,38 +151,16 @@ module Mutant
         world.kernel.exit
       end
     end
+    # rubocop:enable Metrics/MethodLength
 
-    # With configuration
-    #
-    # @param [Hash<Symbol, Object>] attributes
-    #
-    # @return [undefined]
     def with(attributes)
       @config = config.with(attributes)
     end
 
-    # Add configuration
-    #
-    # @param [Symbol] attribute
-    #   the attribute to add to
-    #
-    # @param [Object] value
-    #   the value to add
-    #
-    # @return [undefined]
     def add(attribute, value)
       with(attribute => config.public_send(attribute) + [value])
     end
 
-    # Add matcher configuration
-    #
-    # @param [Symbol] attribute
-    #   the attribute to add to
-    #
-    # @param [Object] value
-    #   the value to add
-    #
-    # @return [undefined]
     def add_matcher(attribute, value)
       with(matcher: config.matcher.add(attribute, value))
     end

data/lib/mutant/config.rb

@@ -126,14 +126,6 @@ module Mutant
       end
     end
 
-    # Load contents of file
-    #
-    # @param [Pathname] path
-    #
-    # @return [Config]
-    #
-    # @raise [Either<String, Hash{Symbol => Object}>]
-    #   in case of config file error
     def self.load_contents(path)
       Transform::Named
         .new(path.to_s, TRANSFORM)

data/lib/mutant/context.rb

@@ -81,9 +81,6 @@ module Mutant
 
   private
 
-    # Nesting of names in scope
-    #
-    # @return [Array<String>]
     def name_nesting
       scope.name.split(NAMESPACE_DELIMITER)
     end

data/lib/mutant/env.rb

@@ -126,12 +126,6 @@ module Mutant
 
   private
 
-    # Kill mutation under isolation with integration
-    #
-    # @param [Mutation] mutation
-    # @param [Array<Test>] test
-    #
-    # @return [Result::Isolation]
     def run_mutation_tests(mutation, tests)
       config.isolation.call do
         result = mutation.insert(world.kernel)

data/lib/mutant/expression/method.rb

@@ -14,8 +14,8 @@ module Mutant
       private(*anima.attribute_names)
 
       MATCHERS = IceNine.deep_freeze(
-        '.' => Matcher::Methods::Singleton,
-        '#' => Matcher::Methods::Instance
+        '.' => [Matcher::Methods::Singleton, Matcher::Methods::Metaclass],
+        '#' => [Matcher::Methods::Instance]
       )
 
       METHOD_NAME_PATTERN = Regexp.union(
@@ -39,16 +39,16 @@ module Mutant
       #
       # @return [Matcher]
       def matcher
-        methods_matcher = MATCHERS.fetch(scope_symbol).new(scope)
+        matcher_candidates = MATCHERS.fetch(scope_symbol)
+          .map { |submatcher| submatcher.new(scope) }
+
+        methods_matcher = Matcher::Chain.new(matcher_candidates)
 
         Matcher::Filter.new(methods_matcher, ->(subject) { subject.expression.eql?(self) })
       end
 
     private
 
-      # Scope object
-      #
-      # @return [Class, Method]
       def scope
         Object.const_get(scope_name)
       end

data/lib/mutant/expression/methods.rb

@@ -13,8 +13,8 @@ module Mutant
       private(*anima.attribute_names)
 
       MATCHERS = IceNine.deep_freeze(
-        '.' => Matcher::Methods::Singleton,
-        '#' => Matcher::Methods::Instance
+        '.' => [Matcher::Methods::Singleton, Matcher::Methods::Metaclass],
+        '#' => [Matcher::Methods::Instance]
       )
       private_constant(*constants(false))
 
@@ -32,7 +32,10 @@ module Mutant
       #
       # @return [Matcher::Method]
       def matcher
-        MATCHERS.fetch(scope_symbol).new(scope)
+        matcher_candidates = MATCHERS.fetch(scope_symbol)
+          .map { |submatcher| submatcher.new(scope) }
+
+        Matcher::Chain.new(matcher_candidates)
       end
 
       # Length of match with other expression
@@ -50,9 +53,6 @@ module Mutant
 
     private
 
-      # Scope object
-      #
-      # @return [Class, Method]
       def scope
         Object.const_get(scope_name)
       end

data/lib/mutant/expression/parser.rb

@@ -28,12 +28,6 @@ module Mutant
 
     private
 
-      # Expressions parsed from input
-      #
-      # @param [String] input
-      #
-      # @return [Array<Expression>]
-      #   if expressions can be parsed from input
       def expressions(input)
         types.each_with_object([]) do |type, aggregate|
           expression = type.try_parse(input) and aggregate << expression

data/lib/mutant/integration.rb

@@ -32,14 +32,6 @@ module Mutant
         .fmap { |klass| klass.new(env.config).setup }
     end
 
-    # Attempt to require integration
-    #
-    # @param env [Bootstrap]
-    #
-    # @return [Either<String, undefined>]
-    #
-    # @api private
-    #
     # rubocop:disable Style/MultilineBlockChain
     def self.attempt_require(env)
       integration_name = env.config.integration
@@ -56,13 +48,6 @@ module Mutant
     private_class_method :attempt_require
     # rubocop:enable Style/MultilineBlockChain
 
-    # Attempt const get
-    #
-    # @param env [Boostrap]
-    #
-    # @return [Either<String, Class<Integration>>]
-    #
-    # @api private
     def self.attempt_const_get(env)
       integration_name = env.config.integration
       constant_name    = integration_name.capitalize
@@ -98,9 +83,6 @@ module Mutant
 
   private
 
-    # Expression parser
-    #
-    # @return [Expression::Parser]
     def expression_parser
       config.expression_parser
     end