mutant 0.10.12 → 0.10.17

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4c573735f4e77ea0dd9313412e82bbca47771e8fd379c5d53393e855248b7792
-  data.tar.gz: 7c59c273789c9fb3d85dd858a7ed7d6bec85b8440ad05736f70541325d214947
+  metadata.gz: 977c03a9678ae669eaadb837b2d20402e92344cd1aa1d882c358c72819aea2f8
+  data.tar.gz: b27367b4065c83e34423221ba99d868e0a993d35b133eee8a99b18392aaf6a85
 SHA512:
-  metadata.gz: ff3a349908f6f3f83ace0ce4de9555b981f6a82515954e635a7539b2d8852d9bfa740a2c33afd2b3b49a34e0358920be9d0c22b8c05c2903356d75b9c4a4ccc6
-  data.tar.gz: 2b3cc3f947b50bc407c8c5a3239ad5bbddf2ff2adc773e661f8279d650297accaad05cdcdbce5e64ae04e419dd63740e0de493542f44464bb5d7d7ee60ab37c1
+  metadata.gz: a238aea95f75d4bbdccf7418824b62c24c82b56c6f6d04a1e2298783a2208135bf91a2b48e224ad628c033d3f0bf893f9dd6498f3b355133c59339bf5a26964c
+  data.tar.gz: bf517485919b231bd1994fbedab20741de57029add17a22ab15c2d2058c5c1d8453942583fc0a44d99220b09afe60ce891341d7ac42b4c5921cbdee666558503

data/bin/mutant

@@ -28,12 +28,14 @@ status =
         .call(Kernel),
       root_require:     'mutant',
       includes:         %w[
-        mutant
-        unparser
         adamantium
-        equalizer
         anima
         concord
+        equalizer
+        mprelude
+        mutant
+        unparser
+        variable
       ]
     )
 

data/lib/mutant.rb

@@ -226,7 +226,7 @@ module Mutant
   # Reopen class to initialize constant to avoid dep circle
   class Config
     DEFAULT = new(
-      coverage_criteria: Config::CoverageCriteria::DEFAULT,
+      coverage_criteria: Config::CoverageCriteria::EMPTY,
       expression_parser: Expression::Parser.new([
         Expression::Method,
         Expression::Methods,
@@ -245,4 +245,17 @@ module Mutant
       zombie:            false
     )
   end # Config
+
+  # Traverse values against action
+  #
+  # Specialized to Either. Its *always* traverse.
+  def self.traverse(action, values)
+    Either::Right.new(
+      values.map do |value|
+        action.call(value).from_right do |error|
+          return Either::Left.new(error)
+        end
+      end
+    )
+  end
 end # Mutant

data/lib/mutant/cli/command.rb

@@ -75,6 +75,18 @@ module Mutant
         self.class::SUBCOMMANDS
       end
 
+      def execute
+        action.either(
+          method(:fail_message),
+          ->(_) { true }
+        )
+      end
+
+      def fail_message(message)
+        world.stderr.puts(message)
+        false
+      end
+
       def parser
         OptionParser.new do |parser|
           parser.banner = "usage: #{banner}"

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

@@ -29,27 +29,17 @@ module Mutant
         end
 
         def expand(file_config)
-          @config = Config.env.merge(file_config).merge(@config)
+          @config = Config.env.merge(file_config).merge(@config).expand_defaults
         end
 
         def parse_remaining_arguments(arguments)
-          traverse(@config.expression_parser.public_method(:apply), arguments)
+          Mutant.traverse(@config.expression_parser.public_method(:apply), arguments)
             .fmap do |match_expressions|
               matcher(match_expressions: match_expressions)
               self
             end
         end
 
-        def traverse(action, values)
-          Either::Right.new(
-            values.map do |value|
-              action.call(value).from_right do |error|
-                return Either::Left.new(error)
-              end
-            end
-          )
-        end
-
         def set(**attributes)
           @config = @config.with(attributes)
         end

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

@@ -25,12 +25,19 @@ module Mutant
 
         private
 
-          def execute
+          def action
             soft_fail(License.apply(world))
               .bind { bootstrap }
               .bind(&Runner.public_method(:apply))
-              .from_right { |error| world.stderr.puts(error); return false }
-              .success?
+              .bind(&method(:from_result))
+          end
+
+          def from_result(result)
+            if result.success?
+              Either::Right.new(nil)
+            else
+              Either::Left.new('Uncovered mutations detected, exiting nonzero!')
+            end
           end
 
           def soft_fail(result)

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

@@ -11,12 +11,9 @@ module Mutant
 
         private
 
-          def execute
-            Config.load_config_file(world)
-              .fmap(&method(:expand))
-              .bind { Bootstrap.apply(world, @config) }
+          def action
+            bootstrap
               .fmap(&method(:report_env))
-              .right?
           end
 
           def report_env(env)

data/lib/mutant/config.rb

@@ -40,6 +40,12 @@ module Mutant
     class CoverageCriteria
       include Anima.new(:process_abort, :test_result, :timeout)
 
+      EMPTY = new(
+        process_abort: nil,
+        test_result:   nil,
+        timeout:       nil
+      )
+
       DEFAULT = new(
         process_abort: false,
         test_result:   true,
@@ -61,6 +67,22 @@ module Mutant
             ->(value) { Either::Right.new(DEFAULT.with(**value)) }
           ]
         )
+
+      def merge(other)
+        self.class.new(
+          process_abort: overwrite(other, :process_abort),
+          test_result:   overwrite(other, :test_result),
+          timeout:       overwrite(other, :timeout)
+        )
+      end
+
+    private
+
+      def overwrite(other, attribute_name)
+        other_value = other.public_send(attribute_name)
+
+        other_value.nil? ? public_send(attribute_name) : other_value
+      end
     end # CoverageCriteria
 
     # Merge with other config
@@ -68,18 +90,24 @@ module Mutant
     # @param [Config] other
     #
     # @return [Config]
+    #
+    # rubocop:disable Metrics/AbcSize
+    # rubocop:disable Metrics/MethodLength
     def merge(other)
       other.with(
-        fail_fast:        fail_fast || other.fail_fast,
-        includes:         includes + other.includes,
-        jobs:             other.jobs || jobs,
-        integration:      other.integration || integration,
-        mutation_timeout: other.mutation_timeout || mutation_timeout,
-        matcher:          matcher.merge(other.matcher),
-        requires:         requires + other.requires,
-        zombie:           zombie || other.zombie
+        coverage_criteria: coverage_criteria.merge(other.coverage_criteria),
+        fail_fast:         fail_fast || other.fail_fast,
+        includes:          includes + other.includes,
+        jobs:              other.jobs || jobs,
+        integration:       other.integration || integration,
+        mutation_timeout:  other.mutation_timeout || mutation_timeout,
+        matcher:           matcher.merge(other.matcher),
+        requires:          requires + other.requires,
+        zombie:            zombie || other.zombie
       )
     end
+    # rubocop:enable Metrics/AbcSize
+    # rubocop:enable Metrics/MethodLength
 
     # Load config file
     #
@@ -100,6 +128,16 @@ module Mutant
       end
     end
 
+    # Expand config with defaults
+    #
+    # @return [Config]
+    def expand_defaults
+      with(
+        coverage_criteria: CoverageCriteria::DEFAULT.merge(coverage_criteria),
+        jobs:              jobs || 1
+      )
+    end
+
     def self.load_contents(path)
       Transform::Named
         .new(path.to_s, TRANSFORM)

data/lib/mutant/isolation/fork.rb

@@ -64,6 +64,7 @@ module Mutant
         end
       end # Pipe
 
+      # rubocop:disable Metrics/ClassLength
       class Parent
         include(
           Anima.new(*ATTRIBUTES),
@@ -149,17 +150,26 @@ module Mutant
 
             break unless ready
 
-            ready.each do |fd|
-              if fd.eof?
-                targets.delete(fd)
+            ready.each do |target|
+              if target.eof?
+                targets.delete(target)
               else
-                targets.fetch(fd) << fd.read_nonblock(READ_SIZE)
+                read_fragment(target, targets.fetch(target))
               end
             end
           end
         end
         # rubocop:enable Metrics/MethodLength
 
+        def read_fragment(target, fragments)
+          loop do
+            result = target.read_nonblock(READ_SIZE, exception: false)
+            break unless result.instance_of?(String)
+            fragments << result
+            break if result.bytesize < READ_SIZE
+          end
+        end
+
         # rubocop:disable Metrics/MethodLength
         def terminate_graceful
           status = nil
@@ -199,6 +209,7 @@ module Mutant
           @result = defined?(@result) ? @result.add_error(result) : result
         end
       end # Parent
+      # rubocop:enable Metrics/ClassLength
 
       class Child
         include(

data/lib/mutant/mutation.rb

@@ -9,34 +9,37 @@ module Mutant
     CODE_DELIMITER = "\0"
     CODE_RANGE     = (0..4).freeze
 
-    def initialize(subject, node)
-      super(subject, node)
-
-      @source         = Unparser.unparse(node)
-      @code           = sha1[CODE_RANGE]
-      @identification = "#{self.class::SYMBOL}:#{subject.identification}:#{code}"
-      @monkeypatch    = Unparser.unparse(subject.context.root(node))
-    end
-
     # Mutation identification code
     #
     # @return [String]
-    attr_reader :code
+    def code
+      sha1[CODE_RANGE]
+    end
+    memoize :code
 
     # Normalized mutation source
     #
     # @return [String]
-    attr_reader :source
+    def source
+      Unparser.unparse(node)
+    end
+    memoize :source
 
     # Identification string
     #
     # @return [String]
-    attr_reader :identification
+    def identification
+      "#{self.class::SYMBOL}:#{subject.identification}:#{code}"
+    end
+    memoize :identification
 
     # The monkeypatch to insert the mutation
     #
     # @return [String]
-    attr_reader :monkeypatch
+    def monkeypatch
+      Unparser.unparse(subject.context.root(node))
+    end
+    memoize :monkeypatch
 
     # Normalized original source
     #

data/lib/mutant/repository/diff.rb

@@ -23,39 +23,55 @@ module Mutant
       #   when git command failed
       def touches?(path, line_range)
         touched_paths
+          .from_right { |message| fail Error, message }
           .fetch(path) { return false }
           .touches?(line_range)
       end
 
     private
 
-      # rubocop:disable Metrics/MethodLength
+      def repository_root
+        world
+          .capture_stdout(%w[git rev-parse --show-toplevel])
+          .fmap(&:chomp)
+          .fmap(&world.pathname.public_method(:new))
+      end
+
       def touched_paths
-        pathname = world.pathname
-        work_dir = pathname.pwd
+        repository_root.bind(&method(:diff_index))
+      end
+      memoize :touched_paths
 
+      def diff_index(root)
         world
           .capture_stdout(%W[git diff-index #{to}])
-          .from_right
-          .lines
-          .map do |line|
-            path = parse_line(work_dir, line)
-            [path.path, path]
+          .fmap(&:lines)
+          .bind do |lines|
+            Mutant
+              .traverse(->(line) { parse_line(root, line) }, lines)
+              .fmap do |paths|
+                paths.map { |path| [path.path, path] }.to_h
+              end
           end
-          .to_h
       end
-      memoize :touched_paths
-      # rubocop:enable Metrics/MethodLength
-
-      def parse_line(work_dir, line)
-        match = FORMAT.match(line) or fail Error, "Invalid git diff-index line: #{line}"
 
-        Path.new(
-          path:  work_dir.join(match.captures.first),
-          to:    to,
-          world: world
-        )
+      # rubocop:disable Metrics/MethodLength
+      def parse_line(root, line)
+        match = FORMAT.match(line)
+
+        if match
+          Either::Right.new(
+            Path.new(
+              path:  root.join(match.captures.first),
+              to:    to,
+              world: world
+            )
+          )
+        else
+          Either::Left.new("Invalid git diff-index line: #{line}")
+        end
       end
+      # rubocop:enable Metrics/MethodLength
 
       # Path touched by a diff
       class Path

data/lib/mutant/result.rb

@@ -276,7 +276,7 @@ module Mutant
       def process_abort?
         process_status = isolation_result.process_status or return false
 
-        !timeout? && !process_status.exited?
+        !timeout? && !process_status.success?
       end
 
     private

data/lib/mutant/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mutant
 version: !ruby/object:Gem::Version
-  version: 0.10.12
+  version: 0.10.17
 platform: ruby
 authors:
 - Markus Schirp
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-12-03 00:00:00.000000000 Z
+date: 2020-12-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: abstract_type