mutant 0.10.20 → 0.10.21

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7a3308efe39e70e35d8c0fc11294280713c3c69f402e3dc59a30e3a3789a5744
-  data.tar.gz: ce590187601f50a475271571b7da3bd744d1b2196fc15f249de3fbec62913de5
+  metadata.gz: ace60aadd53eaa9d10a47ad5bfd16cc0bd825be0588356fdfacc8e00ee767c20
+  data.tar.gz: 89f5a6460423c7ae12a735f55df13aa8e0077e76243a6fb0bf09d31babafc774
 SHA512:
-  metadata.gz: adc7dd97643cd658857712d4ae220fa167777145e9cae93db73a1c43be9db63a98b03fe549c0b7eac5c595b4b8f80892e8ca1c12fcfcf8385ffdc360d4b95f65
-  data.tar.gz: 51e49cfc6d35f60b7b9e5f3bd2c4991ceba3e383a60192a57920aab16860fbebda0f560ec39cf552994c3a1f6319793a0483f9fc3d2b4669ec556b7526df8b8d
+  metadata.gz: 3dd24a1870b332e38dd4e40041d6ce60f1fe23bdf26490ace7bb8f5a285f927880a1898cf6e85d04055723aca9043fa382b19bc86ad29565bafd7a51258b1727
+  data.tar.gz: a855387e41f516caf8759eba4e0c0fe10f144dd4d5e9241bf3f41a768d3fde8bcee0ae70ff3b1ddaeeaf97aa8cd38301832c6b963d0c5eff8d245dc721638594

data/lib/mutant.rb

@@ -43,6 +43,7 @@ end # Mutant
 require 'mutant/bootstrap'
 require 'mutant/version'
 require 'mutant/env'
+require 'mutant/pipe'
 require 'mutant/util'
 require 'mutant/registry'
 require 'mutant/ast'

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

@@ -13,7 +13,6 @@ module Mutant
 
         public :receiver, :selector
 
-        INDEX_ASSIGNMENT_SELECTOR            = :[]=
         ATTRIBUTE_ASSIGNMENT_SELECTOR_SUFFIX = '='
 
         # Arguments of mutated node

data/lib/mutant/ast/types.rb

@@ -6,11 +6,6 @@ module Mutant
     module Types
       ASSIGNABLE_VARIABLES = Set.new(%i[ivasgn lvasgn cvasgn gvasgn]).freeze
 
-      INDEX_ASSIGN_OPERATOR = :[]=
-
-      # Set of nodes that cannot be on the LHS of an assignment
-      NOT_ASSIGNABLE         = Set.new(%i[int float str dstr class module self nil]).freeze
-
       # Set of op-assign types
       OP_ASSIGN              = Set.new(%i[or_asgn and_asgn op_asgn]).freeze
       # Set of node types that are not valid when emitted standalone
@@ -53,10 +48,6 @@ module Mutant
         METHOD_OPERATORS - (INDEX_OPERATORS + UNARY_METHOD_OPERATORS)
       )
 
-      OPERATOR_METHODS = Set.new(
-        METHOD_OPERATORS + INDEX_OPERATORS + UNARY_METHOD_OPERATORS
-      ).freeze
-
       # Nodes that are NOT handled by mutant.
       #
       # not - 1.8 only, mutant does not support 1.8

data/lib/mutant/env.rb

@@ -43,19 +43,21 @@ module Mutant
     end
     # rubocop:enable Metrics/MethodLength
 
-    # Kill mutation
+    # Cover mutation with specific index
     #
-    # @param [Mutation] mutation
+    # @param [Integer] mutation_index
     #
-    # @return [Result::Mutation]
-    def kill(mutation)
+    # @return [Result::MutationIndex]
+    def cover_index(mutation_index)
+      mutation = mutations.fetch(mutation_index)
+
       start = timer.now
 
       tests = selections.fetch(mutation.subject)
 
-      Result::Mutation.new(
+      Result::MutationIndex.new(
         isolation_result: run_mutation_tests(mutation, tests),
-        mutation:         mutation,
+        mutation_index:   mutation_index,
         runtime:          timer.now - start
       )
     end

data/lib/mutant/isolation/fork.rb

@@ -230,7 +230,7 @@ module Mutant
 
       end # Child
 
-      private_constant(*(constants(false) - %i[ChildError ForkError]))
+      private_constant(*constants(false))
 
       # Call block in isolation
       #

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

@@ -8,8 +8,6 @@ module Mutant
 
         handle(:args)
 
-        PROCARG = %i[restarg mlhs].freeze
-
       private
 
         def dispatch

data/lib/mutant/mutator/node/literal/symbol.rb

@@ -11,8 +11,6 @@ module Mutant
 
           children :value
 
-          PREFIX = '__mutant__'
-
         private
 
           def dispatch

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

@@ -14,30 +14,32 @@ module Mutant
         children :receiver, :selector
 
         SELECTOR_REPLACEMENTS = IceNine.deep_freeze(
-          reverse_map:   %i[map each],
-          kind_of?:      %i[instance_of?],
+          :< =>          %i[== eql? equal?],
+          :<= =>         %i[< == eql? equal?],
+          :== =>         %i[eql? equal?],
+          :> =>          %i[== eql? equal?],
+          :>= =>         %i[> == eql? equal?],
+          __send__:      %i[public_send],
+          all?:          %i[any?],
+          any?:          %i[all?],
+          at:            %i[fetch key?],
+          eql?:          %i[equal?],
+          fetch:         %i[key?],
+          flat_map:      %i[map],
+          gsub:          %i[sub],
           is_a?:         %i[instance_of?],
+          kind_of?:      %i[instance_of?],
+          map:           %i[each],
+          method:        %i[public_method],
           reverse_each:  %i[each],
+          reverse_map:   %i[map each],
           reverse_merge: %i[merge],
-          map:           %i[each],
-          flat_map:      %i[map],
           send:          %i[public_send __send__],
-          __send__:      %i[public_send],
-          method:        %i[public_method],
-          gsub:          %i[sub],
-          eql?:          %i[equal?],
-          to_s:          %i[to_str],
-          to_i:          %i[to_int],
           to_a:          %i[to_ary],
           to_h:          %i[to_hash],
-          at:            %i[fetch key?],
-          fetch:         %i[key?],
-          values_at:     %i[fetch_values],
-          :== =>         %i[eql? equal?],
-          :>= =>         %i[> == eql? equal?],
-          :<= =>         %i[< == eql? equal?],
-          :> =>          %i[== eql? equal?],
-          :< =>          %i[== eql? equal?]
+          to_i:          %i[to_int],
+          to_s:          %i[to_str],
+          values_at:     %i[fetch_values]
         )
 
         RECEIVER_SELECTOR_REPLACEMENTS = IceNine.deep_freeze(

data/lib/mutant/parallel.rb

@@ -6,45 +6,61 @@ module Mutant
 
     # Run async computation returning driver
     #
+    # @param [World] world
     # @param [Config] config
     #
     # @return [Driver]
-    def self.async(config)
-      shared = {
-        var_active_jobs: shared(Variable::IVar, config, value: Set.new),
-        var_final:       shared(Variable::IVar, config),
-        var_sink:        shared(Variable::IVar, config, value: config.sink)
-      }
+    def self.async(world, config)
+      shared  = shared_state(world, config)
+      workers = workers(world, config, shared)
 
       Driver.new(
-        threads: threads(config, worker(config, **shared)),
+        workers: workers,
+        threads: threads(world, config, workers),
         **shared
       )
     end
 
-    # The worker
-    #
-    # @param [Config] config
-    #
-    # @return [Worker]
-    def self.worker(config, **shared)
-      Worker.new(
-        processor:   config.processor,
-        var_running: shared(Variable::MVar, config, value: config.jobs),
-        var_source:  shared(Variable::IVar, config, value: config.source),
-        **shared
-      )
+    def self.workers(world, config, shared)
+      Array.new(config.jobs) do |index|
+        Worker.start(
+          block:        config.block,
+          index:        index,
+          process_name: "#{config.process_name}-#{index}",
+          world:        world,
+          **shared
+        )
+      end
     end
+    private_class_method :workers
+
+    def self.shared_state(world, config)
+      {
+        var_active_jobs: shared(Variable::IVar, world, value: Set.new),
+        var_final:       shared(Variable::IVar, world),
+        var_running:     shared(Variable::MVar, world, value: config.jobs),
+        var_sink:        shared(Variable::IVar, world, value: config.sink),
+        var_source:      shared(Variable::IVar, world, value: config.source)
+      }
+    end
+    private_class_method :shared_state
+
+    def self.threads(world, config, workers)
+      thread = world.thread
 
-    def self.threads(config, worker)
-      Array.new(config.jobs) { config.thread.new(&worker.method(:call)) }
+      workers.map do |worker|
+        thread.new do
+          thread.current.name = "#{config.thread_name}-#{worker.index}"
+          worker.call
+        end
+      end
     end
     private_class_method :threads
 
-    def self.shared(klass, config, **attributes)
+    def self.shared(klass, world, **attributes)
       klass.new(
-        condition_variable: config.condition_variable,
-        mutex:              config.mutex,
+        condition_variable: world.condition_variable,
+        mutex:              world.mutex,
         **attributes
       )
     end
@@ -75,13 +91,12 @@ module Mutant
     # Parallel run configuration
     class Config
       include Adamantium::Flat, Anima.new(
-        :condition_variable,
+        :block,
         :jobs,
-        :mutex,
-        :processor,
+        :process_name,
         :sink,
         :source,
-        :thread
+        :thread_name
       )
     end # Config
 

data/lib/mutant/parallel/driver.rb

@@ -8,7 +8,10 @@ module Mutant
         :threads,
         :var_active_jobs,
         :var_final,
-        :var_sink
+        :var_running,
+        :var_sink,
+        :var_source,
+        :workers
       )
 
       private(*anima.attribute_names)
@@ -29,7 +32,10 @@ module Mutant
 
       def finalize(status)
         status.tap do
-          threads.each(&:join) if status.done?
+          if status.done?
+            workers.each(&:join)
+            threads.each(&:join)
+          end
         end
       end
 
@@ -38,7 +44,7 @@ module Mutant
           var_sink.with do |sink|
             Status.new(
               active_jobs: active_jobs.dup.freeze,
-              done:        threads.all? { |thread| !thread.alive? },
+              done:        threads.all? { |worker| !worker.alive? },
               payload:     sink.status
             )
           end

data/lib/mutant/parallel/worker.rb

@@ -4,7 +4,10 @@ module Mutant
   module Parallel
     class Worker
       include Adamantium::Flat, Anima.new(
-        :processor,
+        :connection,
+        :index,
+        :pid,
+        :process,
         :var_active_jobs,
         :var_final,
         :var_running,
@@ -14,6 +17,45 @@ module Mutant
 
       private(*anima.attribute_names)
 
+      public :index
+
+      # rubocop:disable Metrics/MethodLength
+      # rubocop:disable Metrics/ParameterLists
+      def self.start(world:, block:, process_name:, **attributes)
+        io      = world.io
+        process = world.process
+
+        request  = Pipe.from_io(io)
+        response = Pipe.from_io(io)
+
+        pid = process.fork do
+          world.thread.current.name = process_name
+          world.process.setproctitle(process_name)
+
+          Child.new(
+            block:      block,
+            connection: Pipe::Connection.from_pipes(
+              marshal: world.marshal,
+              reader:  request,
+              writer:  response
+            )
+          ).call
+        end
+
+        new(
+          pid:        pid,
+          process:    process,
+          connection: Pipe::Connection.from_pipes(
+            marshal: world.marshal,
+            reader:  response,
+            writer:  request
+          ),
+          **attributes
+        )
+      end
+      # rubocop:enable Metrics/MethodLength
+      # rubocop:enable Metrics/ParameterLists
+
       # Run worker payload
       #
       # @return [self]
@@ -23,7 +65,7 @@ module Mutant
 
           job_start(job)
 
-          result = processor.call(job.payload)
+          result = connection.call(job.payload)
 
           job_done(job)
 
@@ -35,6 +77,12 @@ module Mutant
         self
       end
 
+      def join
+        process.kill('TERM', pid)
+        process.wait(pid)
+        self
+      end
+
     private
 
       def next_job
@@ -66,6 +114,16 @@ module Mutant
         var_final.put(nil) if var_running.modify(&:pred).zero?
       end
 
+      class Child
+        include Anima.new(:block, :connection)
+
+        def call
+          loop do
+            connection.send_value(block.call(connection.receive_value))
+          end
+        end
+      end
+      private_constant :Child
     end # Worker
   end # Parallel
 end # Mutant

data/lib/mutant/pipe.rb

@@ -0,0 +1,94 @@
+# frozen_string_literal: true
+
+module Mutant
+  # Pipe abstraction
+  class Pipe
+    include Adamantium::Flat, Anima.new(:reader, :writer)
+
+    # Run block with pipe in binmode
+    #
+    # @return [undefined]
+    def self.with(io)
+      io.pipe(binmode: true) do |(reader, writer)|
+        yield new(reader: reader, writer: writer)
+      end
+    end
+
+    def self.from_io(io)
+      reader, writer = io.pipe(binmode: true)
+      new(reader: reader, writer: writer)
+    end
+
+    # Writer end of the pipe
+    #
+    # @return [IO]
+    def to_writer
+      reader.close
+      writer
+    end
+
+    # Parent reader end of the pipe
+    #
+    # @return [IO]
+    def to_reader
+      writer.close
+      reader
+    end
+
+    class Connection
+      include Anima.new(:marshal, :reader, :writer)
+
+      Error = Class.new(RuntimeError)
+
+      class Frame
+        include Concord.new(:io)
+
+        HEADER_FORMAT = 'N'
+        MAX_BYTES     = (2**32).pred
+        HEADER_SIZE   = 4
+
+        def receive_value
+          header = read(HEADER_SIZE)
+          read(Util.one(header.unpack(HEADER_FORMAT)))
+        end
+
+        def send_value(body)
+          bytesize = body.bytesize
+
+          fail Error, 'message to big' if bytesize > MAX_BYTES
+
+          io.write([bytesize].pack(HEADER_FORMAT))
+          io.write(body)
+        end
+
+      private
+
+        def read(bytes)
+          io.read(bytes) or fail Error, 'Unexpected EOF'
+        end
+      end
+
+      def call(payload)
+        send_value(payload)
+        receive_value
+      end
+
+      def receive_value
+        marshal.load(reader.receive_value)
+      end
+
+      def send_value(value)
+        writer.send_value(marshal.dump(value))
+        self
+      end
+
+      def self.from_pipes(marshal:, reader:, writer:)
+        new(
+          marshal: marshal,
+          reader:  Frame.new(reader.to_reader),
+          writer:  Frame.new(writer.to_writer)
+        )
+      end
+    end
+  end # Pipe
+end # Mutant

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

@@ -12,11 +12,6 @@ module Mutant
             %s
           MESSAGE
 
-          LOG_MESSAGES = <<~'MESSAGE'
-            Log messages (combined stderr and stdout):
-            %s
-          MESSAGE
-
           EXCEPTION_ERROR_MESSAGE = <<~'MESSAGE'
             Killing the mutation resulted in an integration error.
             This is the case when the tests selected for the current mutation
@@ -36,7 +31,7 @@ module Mutant
             ```
           MESSAGE
 
-          TIMEOUT_ERROR_MESSAGE =<<~'MESSAGE'
+          TIMEOUT_ERROR_MESSAGE = <<~'MESSAGE'
             Mutation analysis ran into the configured timeout of %0.9<timeout>g seconds.
           MESSAGE
 

data/lib/mutant/result.rb

@@ -232,6 +232,14 @@ module Mutant
       end
     end
 
+    class MutationIndex
+      include Anima.new(
+        :isolation_result,
+        :mutation_index,
+        :runtime
+      )
+    end # MutationIndex
+
     # Mutation result
     class Mutation
       include Result, Anima.new(

data/lib/mutant/runner.rb

@@ -17,7 +17,7 @@ module Mutant
 
       run_driver(
         reporter,
-        Parallel.async(mutation_test_config(env))
+        Parallel.async(env.world, mutation_test_config(env))
       ).tap do |result|
         reporter.report(result)
       end
@@ -34,16 +34,13 @@ module Mutant
     private_class_method :run_driver
 
     def self.mutation_test_config(env)
-      world = env.world
-
       Parallel::Config.new(
-        condition_variable: world.condition_variable,
-        jobs:               env.config.jobs,
-        mutex:              world.mutex,
-        processor:          env.method(:kill),
-        sink:               Sink.new(env),
-        source:             Parallel::Source::Array.new(env.mutations),
-        thread:             world.thread
+        block:        env.method(:cover_index),
+        jobs:         env.config.jobs,
+        process_name: 'mutant-worker-process',
+        sink:         Sink.new(env),
+        source:       Parallel::Source::Array.new(env.mutations.each_index.to_a),
+        thread_name:  'mutant-worker-thread'
       )
     end
     private_class_method :mutation_test_config

data/lib/mutant/runner/sink.rb

@@ -34,10 +34,12 @@ module Mutant
 
       # Handle mutation finish
       #
-      # @param [Result::Mutation] mutation_result
+      # @param [Result::MutationIndex] mutation_index_result
       #
       # @return [self]
-      def result(mutation_result)
+      def result(mutation_index_result)
+        mutation_result = mutation_result(mutation_index_result)
+
         subject = mutation_result.mutation.subject
 
         @subject_results[subject] = Result::Subject.new(
@@ -58,6 +60,14 @@ module Mutant
         )
       end
 
+      def mutation_result(mutation_index_result)
+        Result::Mutation.new(
+          isolation_result: mutation_index_result.isolation_result,
+          mutation:         env.mutations.fetch(mutation_index_result.mutation_index),
+          runtime:          mutation_index_result.runtime
+        )
+      end
+
       def previous_coverage_results(subject)
         subject_result = @subject_results.fetch(subject) { return EMPTY_ARRAY }
         subject_result.coverage_results

data/lib/mutant/timer.rb

@@ -54,8 +54,6 @@ module Mutant
       class None < self
         include Concord.new
 
-        STATUS = Status.new(nil)
-
         # The time left
         #
         # @return [Float, nil]

data/lib/mutant/transform.rb

@@ -383,8 +383,6 @@ module Mutant
       #
       # @param [Object]
       #
-      # ignore :reek:NestedIterators
-      #
       # @return [Either<Error, Object>]
       def call(input)
         current = input

data/lib/mutant/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mutant
 version: !ruby/object:Gem::Version
-  version: 0.10.20
+  version: 0.10.21
 platform: ruby
 authors:
 - Markus Schirp
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-12-16 00:00:00.000000000 Z
+date: 2020-12-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: abstract_type
@@ -412,6 +412,7 @@ files:
 - lib/mutant/parallel/source.rb
 - lib/mutant/parallel/worker.rb
 - lib/mutant/parser.rb
+- lib/mutant/pipe.rb
 - lib/mutant/range.rb
 - lib/mutant/registry.rb
 - lib/mutant/reporter.rb