mutant 0.11.0 → 0.11.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dfa3f4386dd2bb68e31b67124aac333cf71d13bb2fec0cf46fea56360475f3f4
-  data.tar.gz: c1c3284ed18f7e2b268c8536a9103d11b7eb8abb0f3fc73a1890ffc8d461e9e5
+  metadata.gz: 3615060fe7453d95369d3a331f37af2ccaf30227b60ed88fe10cbbd647c1aef1
+  data.tar.gz: 26883c9dd809504502b21f95f93853144fd0c80bf4ce09b4709c9c3eb75742fe
 SHA512:
-  metadata.gz: a20b1be07d23f2a0939bab498c500cf1707f75c647af84409d5ec42437bdd26aa6d4f89c9f70e558408e1280812da0cbc7cf7758af1c80c499289e3554b0b92d
-  data.tar.gz: 276e169499aa4b48bc2488178de01b3d3ce1cb4647796bef023491c5d21faaed2d88e1ee61ae012c09d5c86f68ac68cda4a705174cc3cb88dab69e7cc4ea1578
+  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.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/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/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/version.rb

@@ -2,5 +2,5 @@
 
 module Mutant
   # Current mutant version
-  VERSION = '0.11.0'
+  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

@@ -38,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'
@@ -234,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.11.0
+  version: 0.11.1
 platform: ruby
 authors:
 - Markus Schirp
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-10-18 00:00:00.000000000 Z
+date: 2021-11-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: diff-lcs