mutant 0.10.33 → 0.11.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2c20fb3fdc41f7d6e0f8164916206353716c037ec596fccccd06db10c05e810c
-  data.tar.gz: 276612decdc11bf06de38e704fed0d6dc3b9b0b590d9e2f333be7ae383396ca0
+  metadata.gz: 3615060fe7453d95369d3a331f37af2ccaf30227b60ed88fe10cbbd647c1aef1
+  data.tar.gz: 26883c9dd809504502b21f95f93853144fd0c80bf4ce09b4709c9c3eb75742fe
 SHA512:
-  metadata.gz: 652ac4959565f27514568c25c43a0ab64d3af74f9f9de50712bf6e6e825dcf44d27e55e40ff76f0e7c91e27c4199c4f67a0d2205f2268eefae5acc338e2aecb5
-  data.tar.gz: a7e9b023ffefb0d3da5a13d7ec83fbb6696a10efe0f8840b867e8f6e788a55e7d2eabd077173957a697e9f0689ce109fec5d8668078946e317f7c9c4b7216614
+  metadata.gz: e99c8649c373b2f001c9e03440947ab456c1b776c0d8841316a9edcc4105ce22229b5227e72f6b1273f11879ca013b7df909286c42b3f9b8fc186827cda47938
+  data.tar.gz: 3a699d84bacfc16d70912534b2e68a8645d4f737bc977e0495c98b84390328f7912379bf623c0ca6a68df8884faf766e256fb3a983c0b4ea944e23c09ba5ae98

data/lib/mutant/bootstrap.rb

@@ -71,6 +71,10 @@ module Mutant
 
       hooks.run(:env_infection_pre, env)
 
+      config.environment_variables.each do |key, value|
+        world.environment_variables[key] = value
+      end
+
       config.includes.each(&world.load_path.public_method(:<<))
       config.requires.each(&world.kernel.public_method(:require))
 

data/lib/mutant/cli/command/environment/irb.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module Mutant
+  module CLI
+    class Command
+      class Environment
+        class IRB < self
+          NAME              = 'irb'
+          SHORT_DESCRIPTION = 'Run irb with mutant environment loaded'
+          SUBCOMMANDS       = EMPTY_ARRAY
+
+        private
+
+          def action
+            bootstrap.fmap { TOPLEVEL_BINDING.irb }
+          end
+        end # IRB
+      end # Environment
+    end # Command
+  end # CLI
+end # Mutant

data/lib/mutant/cli/command/environment/run.rb

@@ -7,7 +7,7 @@ module Mutant
         class Run < self
           NAME              = 'run'
           SHORT_DESCRIPTION = 'Run code analysis'
-          SLEEP             = 40
+          SLEEP             = 60
           SUBCOMMANDS       = EMPTY_ARRAY
 
           UNLICENSED = <<~MESSAGE.lines.freeze
@@ -16,6 +16,20 @@ module Mutant
             See https://github.com/mbj/mutant#licensing
           MESSAGE
 
+          NO_TESTS_MESSAGE = <<~'MESSAGE'
+            ===============
+            Mutant found no tests. Mutation testing cannot be started.
+
+            This can have various reasons:
+
+            * You did not setup an integration, see:
+              https://github.com/mbj/mutant/blob/main/docs/configuration.md#integration
+            * You set environment variables like RSPEC_OPTS that filter out all tests.
+            * You set configuration optiosn like `config.filter_run :focus` which do
+              make rspec to not report any test.
+            ===============
+          MESSAGE
+
           # Test if command needs to be executed in zombie environment
           #
           # @return [Bool]
@@ -28,10 +42,19 @@ module Mutant
           def action
             soft_fail(License.call(world))
               .bind { bootstrap }
+              .bind(&method(:validate_tests))
               .bind(&Runner.public_method(:call))
               .bind(&method(:from_result))
           end
 
+          def validate_tests(environment)
+            if environment.integration.all_tests.length.zero?
+              Either::Left.new(NO_TESTS_MESSAGE)
+            else
+              Either::Right.new(environment)
+            end
+          end
+
           def from_result(result)
             if result.success?
               Either::Right.new(nil)

data/lib/mutant/cli/command/environment.rb

@@ -62,6 +62,7 @@ module Mutant
           set(matcher: @config.matcher.add(attribute, value))
         end
 
+        # rubocop:disable Metrics/MethodLength
         def add_environment_options(parser)
           parser.separator('Environment:')
           parser.on('--zombie', 'Run mutant zombified') do
@@ -73,7 +74,14 @@ module Mutant
           parser.on('-r', '--require NAME', 'Require file with NAME') do |name|
             add(:requires, name)
           end
+          parser.on('--env KEY=VALUE', 'Set environment variable') do |value|
+            match = ENV_VARIABLE_KEY_VALUE_REGEXP.match(value) || fail("Invalid env variable: #{value.inspect}")
+            set(
+              environment_variables: @config.environment_variables.merge(match[:key] => match[:value])
+            )
+          end
         end
+        # rubocop:enable Metrics/MethodLength
 
         def add_integration_options(parser)
           parser.separator('Integration:')

data/lib/mutant/cli/command/root.rb

@@ -4,7 +4,7 @@ module Mutant
   module CLI
     class Command
       class Environment < self
-        SUBCOMMANDS = [Environment::Subject, Environment::Show, Environment::Test].freeze
+        SUBCOMMANDS = [Environment::Subject, Environment::Show, Environment::IRB, Environment::Test].freeze
       end # Environment
 
       class Root < self

data/lib/mutant/cli/command.rb

@@ -130,6 +130,8 @@ module Mutant
       end
 
       def add_global_options(parser)
+        parser.separator("mutant version: #{VERSION}")
+        parser.separator(nil)
         parser.separator('Global Options:')
         parser.separator(nil)
 

data/lib/mutant/config.rb

@@ -5,9 +5,12 @@ module Mutant
   #
   # Does not reference any "external" volatile state. The configuration applied
   # to current environment is being represented by the Mutant::Env object.
+  #
+  # rubocop:disable Metrics/ClassLength
   class Config
     include Adamantium, Anima.new(
       :coverage_criteria,
+      :environment_variables,
       :expression_parser,
       :fail_fast,
       :hooks,
@@ -48,16 +51,17 @@ module Mutant
     # rubocop:disable Metrics/MethodLength
     def merge(other)
       other.with(
-        coverage_criteria: coverage_criteria.merge(other.coverage_criteria),
-        fail_fast:         fail_fast || other.fail_fast,
-        hooks:             hooks + other.hooks,
-        includes:          includes + other.includes,
-        integration:       other.integration || integration,
-        jobs:              other.jobs || jobs,
-        matcher:           matcher.merge(other.matcher),
-        mutation_timeout:  other.mutation_timeout || mutation_timeout,
-        requires:          requires + other.requires,
-        zombie:            zombie || other.zombie
+        coverage_criteria:     coverage_criteria.merge(other.coverage_criteria),
+        environment_variables: environment_variables.merge(other.environment_variables),
+        fail_fast:             fail_fast || other.fail_fast,
+        hooks:                 hooks + other.hooks,
+        includes:              includes + other.includes,
+        integration:           other.integration || integration,
+        jobs:                  other.jobs || jobs,
+        matcher:               matcher.merge(other.matcher),
+        mutation_timeout:      other.mutation_timeout || mutation_timeout,
+        requires:              requires + other.requires,
+        zombie:                zombie || other.zombie
       )
     end
     # rubocop:enable Metrics/AbcSize
@@ -117,6 +121,24 @@ module Mutant
       )
     )
 
+    # Parse a hash of environment variables
+    #
+    # @param [Hash<Object,Object>]
+    #
+    # @return [Either<String,Hash<String,String>]
+    #
+    def self.parse_environment_variables(hash)
+      invalid = hash.keys.reject { |key| key.instance_of?(String) }
+      return Either::Left.new("Non string keys: #{invalid}") if invalid.any?
+
+      invalid = hash.keys.grep_v(ENV_VARIABLE_KEY_REGEXP)
+      return Either::Left.new("Invalid keys: #{invalid}") if invalid.any?
+
+      invalid = hash.values.reject { |value| value.instance_of?(String) }
+      return Either::Left.new("Non string values: #{invalid}") if invalid.any?
+
+      Either::Right.new(hash)
+    end
     TRANSFORM = Transform::Sequence.new(
       [
         Transform::Exception.new(SystemCallError, :read.to_proc),
@@ -124,14 +146,23 @@ module Mutant
         Transform::Hash.new(
           optional: [
             Transform::Hash::Key.new('coverage_criteria', ->(value) { CoverageCriteria::TRANSFORM.call(value) }),
-            Transform::Hash::Key.new('fail_fast',         Transform::BOOLEAN),
-            Transform::Hash::Key.new('hooks',             PATHNAME_ARRAY),
-            Transform::Hash::Key.new('includes',          Transform::STRING_ARRAY),
-            Transform::Hash::Key.new('integration',       Transform::STRING),
-            Transform::Hash::Key.new('jobs',              Transform::INTEGER),
-            Transform::Hash::Key.new('matcher',           Matcher::Config::LOADER),
-            Transform::Hash::Key.new('mutation_timeout',  Transform::FLOAT),
-            Transform::Hash::Key.new('requires',          Transform::STRING_ARRAY)
+            Transform::Hash::Key.new(
+              'environment_variables',
+              Transform::Sequence.new(
+                [
+                  Transform::Primitive.new(Hash),
+                  Transform::Block.capture(:environment_variables, &method(:parse_environment_variables))
+                ]
+              )
+            ),
+            Transform::Hash::Key.new('fail_fast',             Transform::BOOLEAN),
+            Transform::Hash::Key.new('hooks',                 PATHNAME_ARRAY),
+            Transform::Hash::Key.new('includes',              Transform::STRING_ARRAY),
+            Transform::Hash::Key.new('integration',           Transform::STRING),
+            Transform::Hash::Key.new('jobs',                  Transform::INTEGER),
+            Transform::Hash::Key.new('matcher',               Matcher::Config::LOADER),
+            Transform::Hash::Key.new('mutation_timeout',      Transform::FLOAT),
+            Transform::Hash::Key.new('requires',              Transform::STRING_ARRAY)
           ],
           required: []
         ),
@@ -141,4 +172,5 @@ module Mutant
 
     private_constant(:TRANSFORM)
   end # Config
+  # rubocop:enable Metrics/ClassLength
 end # Mutant

data/lib/mutant/matcher/method.rb

@@ -82,7 +82,17 @@ module Mutant
         end
 
         def source_location
-          target_method.source_location
+          signature = sorbet_signature
+
+          if signature
+            signature.method.source_location
+          else
+            target_method.source_location
+          end
+        end
+
+        def sorbet_signature
+          T::Private::Methods.signature_for_method(target_method)
         end
 
         def subject

data/lib/mutant/matcher/methods.rb

@@ -21,7 +21,7 @@ module Mutant
       # @return [Enumerable<Subject>]
       def call(env)
         Chain.new(
-          methods.map { |method| matcher.new(scope, method) }
+          methods(env).map { |method| matcher.new(scope, method) }
         ).call(env)
       end
 
@@ -31,17 +31,16 @@ module Mutant
         self.class::MATCHER
       end
 
-      def methods
+      def methods(env)
         candidate_names.each_with_object([]) do |name, methods|
-          method = access(name)
-          methods << method if method.owner.equal?(candidate_scope)
+          method = access(env, name)
+          methods << method if method&.owner.equal?(candidate_scope)
         end
       end
-      memoize :methods
 
       def candidate_names
         CANDIDATE_NAMES
-          .map(&candidate_scope.public_method(:public_send))
+          .map { |name| candidate_scope.public_send(name, false) }
           .reduce(:+)
           .sort
       end
@@ -55,7 +54,7 @@ module Mutant
 
       private
 
-        def access(method_name)
+        def access(_env, method_name)
           scope.method(method_name)
         end
 
@@ -71,7 +70,7 @@ module Mutant
 
       private
 
-        def access(method_name)
+        def access(_env, method_name)
           scope.method(method_name)
         end
 
@@ -84,11 +83,33 @@ module Mutant
       class Instance < self
         MATCHER = Matcher::Method::Instance
 
+        MESSAGE = <<~'MESSAGE'
+          Caught an exception while accessing a method with
+          #instance_method that is part of #{public,privat,protected}_instance_methods.
+
+          This is a bug in your ruby implementation its stdlib, libaries our your code.
+
+          Mutant will ignore this method:
+
+          Object:    %<scope>s
+          Method:    %<method_name>s
+          Exception: %<exception>s
+
+          See: https://github.com/mbj/mutant/issues/1273
+        MESSAGE
+
       private
 
-        def access(method_name)
+        # rubocop:disable Lint/RescueException
+        def access(env, method_name)
           scope.instance_method(method_name)
+        rescue Exception => exception
+          env.warn(
+            MESSAGE % { scope: scope, method_name: method_name, exception: exception }
+          )
+          nil
         end
+        # rubocop:enable Lint/RescueException
 
         def candidate_scope
           scope

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

@@ -21,8 +21,11 @@ module Mutant
           emit_singletons
           emit_promotions
           emit_operator_mutations
-          emit_left_negation
-          emit_left_mutations
+
+          emit_left_mutations do |mutation|
+            !(n_irange?(mutation) || n_erange?(mutation)) || !mutation.children.fetch(1).nil?
+          end
+
           emit_right_mutations
         end
 
@@ -35,10 +38,6 @@ module Mutant
           emit(right)
         end
 
-        def emit_left_negation
-          emit(s(node.type, n_not(left), right))
-        end
-
       end # Binary
     end # Node
   end # Mutator

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

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module Mutant
+  class Mutator
+    class Node
+      class Numblock < self
+
+        handle(:numblock)
+
+        children :receiver, :count, :block
+
+      private
+
+        def dispatch
+          emit_nil
+          emit_receiver_mutations(&method(:n_send?))
+        end
+      end # Numblock
+    end # Node
+  end # Mutator
+end # Mutant

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

@@ -8,7 +8,7 @@ module Mutant
         class Mutation < self
           NO_DIFF_MESSAGE = <<~'MESSAGE'
             --- Internal failure ---
-            BUG: A generted mutation did not result in exactly one diff hunk!
+            BUG: A generated mutation did not result in exactly one diff hunk!
             This is an invariant violation by the mutation generation engine.
             Please report a reproduction to https://github.com/mbj/mutant
             Original unparsed source:

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

@@ -27,7 +27,7 @@ module Mutant
 
           NO_DIFF_MESSAGE = <<~'MESSAGE'
             --- Internal failure ---
-            BUG: A generted mutation did not result in exactly one diff hunk!
+            BUG: A generated mutation did not result in exactly one diff hunk!
             This is an invariant violation by the mutation generation engine.
             Please report a reproduction to https://github.com/mbj/mutant
             Original unparsed source:

data/lib/mutant/version.rb

@@ -2,5 +2,5 @@
 
 module Mutant
   # Current mutant version
-  VERSION = '0.10.33'
+  VERSION = '0.11.1'
 end # Mutant

data/lib/mutant/world.rb

@@ -5,6 +5,7 @@ module Mutant
   class World
     include Adamantium, Anima.new(
       :condition_variable,
+      :environment_variables,
       :gem,
       :gem_method,
       :io,

data/lib/mutant.rb

@@ -4,6 +4,7 @@ require 'diff/lcs'
 require 'diff/lcs/hunk'
 require 'digest/sha1'
 require 'etc'
+require 'irb'
 require 'json'
 require 'open3'
 require 'optparse'
@@ -13,6 +14,7 @@ require 'pathname'
 require 'regexp_parser'
 require 'set'
 require 'singleton'
+require 'sorbet-runtime'
 require 'stringio'
 require 'unparser'
 require 'yaml'
@@ -36,6 +38,11 @@ module Mutant
   EMPTY_ARRAY    = [].freeze
   EMPTY_HASH     = {}.freeze
   SCOPE_OPERATOR = '::'
+
+  env_key = /[a-zA-Z_\d]+/
+
+  ENV_VARIABLE_KEY_VALUE_REGEXP = /\A(?<key>#{env_key}+)=(?<value>.*)\z/.freeze
+  ENV_VARIABLE_KEY_REGEXP       = /\A#{env_key}\z/.freeze
 end # Mutant
 
 require 'mutant/procto'
@@ -152,6 +159,7 @@ require 'mutant/mutator/node/match_current_line'
 require 'mutant/mutator/node/index'
 require 'mutant/mutator/node/procarg_zero'
 require 'mutant/mutator/node/kwargs'
+require 'mutant/mutator/node/numblock'
 require 'mutant/loader'
 require 'mutant/context'
 require 'mutant/scope'
@@ -193,6 +201,7 @@ require 'mutant/cli'
 require 'mutant/cli/command'
 require 'mutant/cli/command/subscription'
 require 'mutant/cli/command/environment'
+require 'mutant/cli/command/environment/irb'
 require 'mutant/cli/command/environment/run'
 require 'mutant/cli/command/environment/show'
 require 'mutant/cli/command/environment/subject'
@@ -230,46 +239,48 @@ require 'mutant/license/subscription/commercial'
 
 module Mutant
   WORLD = World.new(
-    condition_variable: ConditionVariable,
-    gem:                Gem,
-    gem_method:         method(:gem),
-    io:                 IO,
-    json:               JSON,
-    kernel:             Kernel,
-    load_path:          $LOAD_PATH,
-    marshal:            Marshal,
-    mutex:              Mutex,
-    object_space:       ObjectSpace,
-    open3:              Open3,
-    pathname:           Pathname,
-    process:            Process,
-    stderr:             $stderr,
-    stdout:             $stdout,
-    thread:             Thread,
-    timer:              Timer.new(Process)
+    condition_variable:    ConditionVariable,
+    environment_variables: ENV,
+    gem:                   Gem,
+    gem_method:            method(:gem),
+    io:                    IO,
+    json:                  JSON,
+    kernel:                Kernel,
+    load_path:             $LOAD_PATH,
+    marshal:               Marshal,
+    mutex:                 Mutex,
+    object_space:          ObjectSpace,
+    open3:                 Open3,
+    pathname:              Pathname,
+    process:               Process,
+    stderr:                $stderr,
+    stdout:                $stdout,
+    thread:                Thread,
+    timer:                 Timer.new(Process)
   )
 
   # Reopen class to initialize constant to avoid dep circle
   class Config
     DEFAULT = new(
-      coverage_criteria: Config::CoverageCriteria::EMPTY,
-      expression_parser: Expression::Parser.new([
+      coverage_criteria:     Config::CoverageCriteria::EMPTY,
+      expression_parser:     Expression::Parser.new([
         Expression::Method,
         Expression::Methods,
         Expression::Namespace::Exact,
         Expression::Namespace::Recursive
       ]),
-      fail_fast:         false,
-      hooks:             EMPTY_ARRAY,
-      includes:          EMPTY_ARRAY,
-      integration:       nil,
-      isolation:         Mutant::Isolation::Fork.new(WORLD),
-      jobs:              nil,
-      matcher:           Matcher::Config::DEFAULT,
-      mutation_timeout:  nil,
-      reporter:          Reporter::CLI.build(WORLD.stdout),
-      requires:          EMPTY_ARRAY,
-      zombie:            false
+      fail_fast:             false,
+      environment_variables: EMPTY_HASH,
+      hooks:                 EMPTY_ARRAY,
+      includes:              EMPTY_ARRAY,
+      integration:           nil,
+      isolation:             Mutant::Isolation::Fork.new(WORLD),
+      jobs:                  nil,
+      matcher:               Matcher::Config::DEFAULT,
+      mutation_timeout:      nil,
+      reporter:              Reporter::CLI.build(WORLD.stdout),
+      requires:              EMPTY_ARRAY,
+      zombie:                false
     )
   end # Config
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mutant
 version: !ruby/object:Gem::Version
-  version: 0.10.33
+  version: 0.11.1
 platform: ruby
 authors:
 - Markus Schirp
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-08-25 00:00:00.000000000 Z
+date: 2021-11-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: diff-lcs
@@ -58,6 +58,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 2.0.3
+- !ruby/object:Gem::Dependency
+  name: sorbet-runtime
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.5.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.5.0
 - !ruby/object:Gem::Dependency
   name: unparser
   requirement: !ruby/object:Gem::Requirement
@@ -180,6 +194,7 @@ files:
 - lib/mutant/cli.rb
 - lib/mutant/cli/command.rb
 - lib/mutant/cli/command/environment.rb
+- lib/mutant/cli/command/environment/irb.rb
 - lib/mutant/cli/command/environment/run.rb
 - lib/mutant/cli/command/environment/show.rb
 - lib/mutant/cli/command/environment/subject.rb
@@ -269,6 +284,7 @@ files:
 - lib/mutant/mutator/node/next.rb
 - lib/mutant/mutator/node/noop.rb
 - lib/mutant/mutator/node/nthref.rb
+- lib/mutant/mutator/node/numblock.rb
 - lib/mutant/mutator/node/op_asgn.rb
 - lib/mutant/mutator/node/or_asgn.rb
 - lib/mutant/mutator/node/procarg_zero.rb
@@ -359,7 +375,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.5'
+      version: '2.6'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="