mutant 0.12.1 → 0.12.3

data/lib/mutant/matcher/method.rb

@@ -25,7 +25,7 @@ module Mutant
       #
       # @return [Enumerable<Subject>]
       def call(env)
-        evaluator.call(scope: scope, target_method: target_method, env: env)
+        evaluator.call(scope:, target_method:, env:)
       end
 
       # Abstract method match evaluator
@@ -76,9 +76,9 @@ module Mutant
         def subject
           self.class::SUBJECT_CLASS.new(
             config:     subject_config(matched_view.node),
-            context:    context,
+            context:,
             node:       matched_view.node,
-            visibility: visibility
+            visibility:
           )
         end
 
@@ -87,7 +87,7 @@ module Mutant
         end
 
         def context
-          Context.new(constant_scope: constant_scope, scope: scope, source_path: source_path)
+          Context.new(constant_scope:, scope:, source_path:)
         end
 
         # rubocop:disable Metrics/MethodLength
@@ -101,7 +101,7 @@ module Mutant
               if klass
                 klass.new(
                   const:      node.children.fetch(0),
-                  descendant: descendant
+                  descendant:
                 )
               else
                 descendant
@@ -163,20 +163,6 @@ module Mutant
         end
 
         def visibility
-          # This can be cleaned up once we are on >ruby-3.0
-          # Method#{public,private,protected}? exists there.
-          #
-          # On Ruby 3.1 this can just be:
-          #
-          # if target_method.private?
-          #   :private
-          # elsif target_method.protected?
-          #   :protected
-          # else
-          #   :public
-          # end
-          #
-          # Change to this once 3.0 is EOL.
           if scope.raw.private_methods.include?(method_name)
             :private
           elsif scope.raw.protected_methods.include?(method_name)

data/lib/mutant/matcher/methods.rb

@@ -22,7 +22,7 @@ module Mutant
       def call(env)
         Chain.new(
           matchers: methods(env).map do |target_method|
-            matcher.new(scope: scope, target_method: target_method)
+            matcher.new(scope:, target_method:)
           end
         ).call(env)
       end
@@ -110,7 +110,7 @@ module Mutant
           candidate_scope.instance_method(method_name)
         rescue Exception => exception
           env.warn(
-            MESSAGE % { scope: scope, method_name: method_name, exception: exception.inspect }
+            MESSAGE % { scope:, method_name:, exception: exception.inspect }
           )
           nil
         end

data/lib/mutant/matcher/namespace.rb

@@ -13,7 +13,7 @@ module Mutant
       # @return [Enumerable<Subject>]
       def call(env)
         Chain.new(
-          matchers: matched_scopes(env).map { |scope| Scope.new(scope: scope) }
+          matchers: matched_scopes(env).map { |scope| Scope.new(scope:) }
         ).call(env)
       end
 

data/lib/mutant/matcher/scope.rb

@@ -31,7 +31,7 @@ module Mutant
     private
 
       def effective_matchers
-        MATCHERS.map { |matcher| matcher.new(scope: scope) }
+        MATCHERS.map { |matcher| matcher.new(scope:) }
       end
 
     end # Scope

data/lib/mutant/matcher.rb

@@ -22,7 +22,7 @@ module Mutant
       matcher_config = env.config.matcher
 
       Filter.new(
-        matcher:   Chain.new(matchers: matcher_config.subjects.map { |subject| subject.matcher(env: env) }),
+        matcher:   Chain.new(matchers: matcher_config.subjects.map { |subject| subject.matcher(env:) }),
         predicate: method(:allowed_subject?).curry.call(matcher_config)
       )
     end

data/lib/mutant/meta/example/verification.rb

@@ -94,7 +94,7 @@ module Mutant
 
         def missing
           (example.expected.map(&:node) - mutations.map(&:node)).map do |node|
-            Mutation::Evil.new(subject: example, node: node)
+            Mutation::Evil.new(subject: example, node:)
           end
         end
         memoize :missing

data/lib/mutant/meta/example.rb

@@ -40,7 +40,7 @@ module Mutant
       def context
         Context.new(
           constant_scope: Context::ConstantScope::None.new,
-          scope:          scope,
+          scope:,
           source_path:    location.path
         )
       end
@@ -58,10 +58,10 @@ module Mutant
       # @return [Enumerable<Mutant::Mutation>]
       def generated
         Mutator::Node.mutate(
-          config: Mutation::Config::DEFAULT.with(operators: operators),
-          node:   node
+          config: Mutation::Config::DEFAULT.with(operators:),
+          node:
         ).map do |node|
-          Mutation::Evil.new(subject: self, node: node)
+          Mutation::Evil.new(subject: self, node:)
         end
       end
       memoize :generated

data/lib/mutant/meta.rb

@@ -18,7 +18,7 @@ module Mutant
       # @return [undefined]
       def self.add(*types, operators: :full, &block)
         ALL << DSL.call(
-          block:     block,
+          block:,
           location:  caller_locations(1).first,
           operators: Mutation::Operators.parse(operators.to_s).from_right,
           types:     Set.new(types)

data/lib/mutant/mutation/runner/sink.rb

@@ -22,7 +22,7 @@ module Mutant
         # @return [Result::Env]
         def status
           Result::Env.new(
-            env:             env,
+            env:,
             runtime:         env.world.timer.now - @start,
             subject_results: @subject_results.values
           )
@@ -48,7 +48,7 @@ module Mutant
           subject = mutation_result.mutation.subject
 
           @subject_results[subject] = Result::Subject.new(
-            subject:          subject,
+            subject:,
             coverage_results: previous_coverage_results(subject).dup << coverage_result(mutation_result),
             tests:            env.selections.fetch(subject)
           )
@@ -60,7 +60,7 @@ module Mutant
 
         def coverage_result(mutation_result)
           Result::Coverage.new(
-            mutation_result: mutation_result,
+            mutation_result:,
             criteria_result: mutation_result.criteria_result(env.config.coverage_criteria)
           )
         end

data/lib/mutant/mutation/runner.rb

@@ -45,7 +45,7 @@ module Mutant
           jobs:             env.config.jobs,
           on_process_start: env.method(:emit_mutation_worker_process_start),
           process_name:     'mutant-worker-process',
-          sink:             Sink.new(env: env),
+          sink:             Sink.new(env:),
           source:           Parallel::Source::Array.new(jobs: env.mutations.each_index.to_a),
           timeout:          nil,
           thread_name:      'mutant-worker-thread'

data/lib/mutant/mutation.rb

@@ -66,9 +66,9 @@ module Mutant
       subject.prepare
       Loader.call(
         binding: TOPLEVEL_BINDING,
-        kernel:  kernel,
+        kernel:,
         source:  monkeypatch,
-        subject: subject
+        subject:
       ).fmap do
         subject.post_insert
         nil

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

@@ -56,7 +56,7 @@ module Mutant
         def valid_send_mutation?(node)
           return false unless n_send?(node)
 
-          last = AST::Meta::Send.new(node: node).arguments.last
+          last = AST::Meta::Send.new(node:).arguments.last
 
           !last&.type.equal?(:block_pass)
         end

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

@@ -12,7 +12,7 @@ module Mutant
       private
 
         def dispatch
-          super()
+          super
           emit_singletons
           children.each_index(&method(:delete_child))
         end

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

@@ -76,7 +76,7 @@ module Mutant
         private
 
           def dispatch
-            super()
+            super
 
             return if left_op_assignment?
 

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

@@ -12,7 +12,7 @@ module Mutant
       private
 
         def dispatch
-          super()
+          super
           emit_singletons
         end
 

data/lib/mutant/mutator/node/named_value/access.rb

@@ -28,7 +28,7 @@ module Mutant
 
             def dispatch
               emit_attribute_read
-              super()
+              super
             end
 
             def emit_attribute_read

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

@@ -12,7 +12,7 @@ module Mutant
       private
 
         def dispatch
-          super()
+          super
           emit_singletons
           children.each_index(&method(:delete_child))
           emit(s(:break, *children))

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

@@ -11,7 +11,7 @@ module Mutant
         private
 
           def dispatch
-            super()
+            super
             emit(s(:send, *children))
           end
 

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

@@ -48,7 +48,7 @@ module Mutant
         end
 
         def meta
-          AST::Meta::Send.new(node: node)
+          AST::Meta::Send.new(node:)
         end
         memoize :meta
 

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

@@ -13,7 +13,6 @@ module Mutant
 
         def dispatch
           emit_singletons
-          emit(N_EMPTY_SUPER)
           children.each_index do |index|
             mutate_child(index)
             delete_child(index)

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

@@ -12,7 +12,7 @@ module Mutant
       private
 
         def dispatch
-          super()
+          super
           emit_singletons
           children.each_index(&method(:delete_child))
         end

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

@@ -13,7 +13,6 @@ module Mutant
 
         def dispatch
           emit_singletons
-          emit(N_EMPTY_SUPER)
         end
 
       end # ZSuper

data/lib/mutant/mutator/node.rb

@@ -28,9 +28,9 @@ module Mutant
         end
 
         self::REGISTRY.lookup(node.type).call(
-          config: config,
+          config:,
           input:  node,
-          parent: parent
+          parent:
         )
       end
 
@@ -67,7 +67,7 @@ module Mutant
       alias_method :dup_node, :dup_input
 
       def mutate(node:, parent: nil)
-        self.class.mutate(config: config, node: node, parent: parent)
+        self.class.mutate(config:, node:, parent:)
       end
 
       def mutate_child(index, &block)
@@ -131,8 +131,8 @@ module Mutant
 
       def run(mutator)
         mutator.call(
-          config: config,
-          input:  input,
+          config:,
+          input:,
           parent: nil
         ).each(&method(:emit))
       end

data/lib/mutant/mutator/regexp.rb

@@ -59,7 +59,7 @@ module Mutant
 
       def emit_expression(klass:, text:)
         emit(
-          klass.construct(text: text).tap do |new|
+          klass.construct(text:).tap do |new|
             subexpressions.each do |expression|
               new << Marshal.load(Marshal.dump(expression))
             end
@@ -140,7 +140,7 @@ module Mutant
         def dispatch
           klass, text = MAP.fetch(input.class)
 
-          emit(klass.construct(text: text).tap { |new| new.quantifier = input.quantifier })
+          emit(klass.construct(text:).tap { |new| new.quantifier = input.quantifier })
         end
       end
 

data/lib/mutant/parallel/connection.rb

@@ -46,7 +46,7 @@ module Mutant
 
           Response.new(
             error:  reader.error,
-            job:    job,
+            job:,
             log:    reader.log,
             result: reader.result
           )
@@ -168,7 +168,7 @@ module Mutant
 
       def self.from_pipes(marshal:, reader:, writer:)
         new(
-          marshal: marshal,
+          marshal:,
           reader:  Frame.new(io: reader.to_reader),
           writer:  Frame.new(io: writer.to_writer)
         )

data/lib/mutant/parallel/pipe.rb

@@ -10,13 +10,13 @@ module Mutant
       # @return [undefined]
       def self.with(io)
         io.pipe(binmode: true) do |(reader, writer)|
-          yield new(reader: reader, writer: writer)
+          yield new(reader:, writer:)
         end
       end
 
       def self.from_io(io)
         reader, writer = io.pipe(binmode: true)
-        new(reader: reader, writer: writer)
+        new(reader:, writer:)
       end
 
       # Writer end of the pipe

data/lib/mutant/parallel/worker.rb

@@ -41,20 +41,20 @@ module Mutant
           world.stdout.reopen(log_writer)
 
           run_child(
-            config:     config,
-            connection: Connection.from_pipes(marshal: marshal, reader: request, writer: response),
-            log_writer: log_writer
+            config:,
+            connection: Connection.from_pipes(marshal:, reader: request, writer: response),
+            log_writer:
           )
         end
 
-        connection = Connection.from_pipes(marshal: marshal, reader: response, writer: request)
+        connection = Connection.from_pipes(marshal:, reader: response, writer: request)
 
         new(
-          config:          config,
-          connection:      connection,
+          config:,
+          connection:,
           log_reader:      log.to_reader,
           response_reader: connection.reader.io,
-          pid:             pid
+          pid:
         )
       end
       private_class_method :start_config
@@ -98,10 +98,10 @@ module Mutant
           response = Connection::Reader.read_response(
             deadline:        config.world.deadline(config.timeout),
             io:              config.world.io,
-            job:             job,
-            log_reader:      log_reader,
+            job:,
+            log_reader:,
             marshal:         config.world.marshal,
-            response_reader: response_reader
+            response_reader:
           )
 
           job_done(job)

data/lib/mutant/parallel.rb

@@ -17,7 +17,7 @@ module Mutant
       workers = workers(world, config, shared)
 
       Driver.new(
-        workers: workers,
+        workers:,
         threads: threads(world, config, workers),
         **shared
       )
@@ -28,11 +28,11 @@ module Mutant
       Array.new(config.jobs) do |index|
         Worker.start(
           block:            config.block,
-          index:            index,
+          index:,
           on_process_start: config.on_process_start,
           process_name:     "#{config.process_name}-#{index}",
           timeout:          config.timeout,
-          world:            world,
+          world:,
           **shared
         )
       end

data/lib/mutant/parser.rb

@@ -27,7 +27,7 @@ module Mutant
       node, comments = Unparser.parse_with_comments(source)
 
       AST.new(
-        node:                 node,
+        node:,
         comment_associations: ::Parser::Source::Comment.associate_by_identity(node, comments)
       )
     end

data/lib/mutant/registry.rb

@@ -9,7 +9,7 @@ module Mutant
     #
     # @return [undefined]
     def initialize(default)
-      super(contents: {}, default: default)
+      super(contents: {}, default:)
     end
 
     # Raised when the type is an invalid type

data/lib/mutant/reporter/cli/format.rb

@@ -54,7 +54,7 @@ module Mutant
 
         def format(printer, object)
           buffer = new_buffer
-          printer.call(output: Output.new(tty: tty, buffer: buffer), object: object)
+          printer.call(output: Output.new(tty:, buffer:), object:)
           buffer.rewind
           buffer.read
         end

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

@@ -58,7 +58,7 @@ module Mutant
             puts('Log messages (combined stderr and stdout):')
 
             log.each_line do |line|
-              puts('[killfork] %<line>s' % { line: line })
+              puts('[killfork] %<line>s' % { line: })
             end
           end
 
@@ -74,7 +74,7 @@ module Mutant
 
           def print_timeout
             timeout = object.timeout or return
-            puts(TIMEOUT_ERROR_MESSAGE % { timeout: timeout })
+            puts(TIMEOUT_ERROR_MESSAGE % { timeout: })
           end
 
           def print_exception

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

@@ -35,7 +35,7 @@ module Mutant
         private
 
           def object
-            super().payload
+            super.payload
           end
 
           def mutations_per_second

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

@@ -145,7 +145,7 @@ module Mutant
           private
 
             def object
-              super().payload
+              super.payload
             end
 
             def tests_per_second

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

@@ -61,7 +61,7 @@ module Mutant
         end
 
         def visit(printer, object)
-          printer.call(output: output, object: object)
+          printer.call(output:, object:)
         end
 
         def info(string, *arguments)

data/lib/mutant/reporter/cli.rb

@@ -15,7 +15,7 @@ module Mutant
         new(
           format:         Format::Progressive.new(tty: output.respond_to?(:tty?) && output.tty?),
           print_warnings: false,
-          output:         output
+          output:
         )
       end
 
@@ -80,7 +80,7 @@ module Mutant
       #
       # @return [self]
       def report(env)
-        Printer::EnvResult.call(output: output, object: env)
+        Printer::EnvResult.call(output:, object: env)
         self
       end
 
@@ -90,7 +90,7 @@ module Mutant
       #
       # @return [self]
       def test_report(env)
-        Printer::Test::EnvResult.call(output: output, object: env)
+        Printer::Test::EnvResult.call(output:, object: env)
         self
       end
 

data/lib/mutant/repository/diff.rb

@@ -40,8 +40,8 @@ module Mutant
           .fmap { |status| world.pathname.new(status.stdout.chomp) }
       end
 
-      def touched_path(path, &block)
-        touched_paths.from_right { |message| fail Error, message }.fetch(path, &block)
+      def touched_path(path, &)
+        touched_paths.from_right { |message| fail Error, message }.fetch(path, &)
       end
 
       def touched_paths
@@ -62,15 +62,16 @@ module Mutant
       end
 
       # rubocop:disable Metrics/MethodLength
+      # mutant:disable (3.2 specific mutation)
       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
+              path:  root.join(Util.one(match.captures)),
+              to:,
+              world:
             )
           )
         else

data/lib/mutant/segment/recorder.rb

@@ -23,9 +23,9 @@ module Mutant
 
         yield.tap do
           segments << Segment.new(
-            id:              id,
-            name:            name,
-            parent_id:       parent_id,
+            id:,
+            name:,
+            parent_id:,
             timestamp_end:   timer.now,
             timestamp_start: start
           )
@@ -58,7 +58,7 @@ module Mutant
 
         build_node(
           value:        id_index.fetch(root_id),
-          parent_index: parent_index
+          parent_index:
         )
       end
 
@@ -69,17 +69,17 @@ module Mutant
           if segment.timestamp_end
             segment
           else
-            segment.with(timestamp_end: timestamp_end)
+            segment.with(timestamp_end:)
           end
         end
       end
 
       def build_node(value:, parent_index:)
         Node.new(
-          value:    value,
+          value:,
           children: build_children(
             parent_id:    value.id,
-            parent_index: parent_index
+            parent_index:
           )
         )
       end
@@ -87,7 +87,7 @@ module Mutant
       def build_children(parent_id:, parent_index:)
         parent_index
           .fetch(parent_id, EMPTY_ARRAY)
-          .map { |value| build_node(value: value, parent_index: parent_index) }
+          .map { |value| build_node(value:, parent_index:) }
       end
 
       def print_node(io, node, indent)

data/lib/mutant/subject/config.rb

@@ -13,7 +13,7 @@ module Mutant
       def self.parse(comments:, mutation:)
         new(
           inline_disable: comments.any? { |comment| DISABLE_REGEXP.match?(comment_body(comment)) },
-          mutation:       mutation
+          mutation:
         )
       end