mutant 0.12.4 → 0.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1b880a9125609cec0e9516844abe1e3b1f158c1c7d0773ce65600cb9e2738757
-  data.tar.gz: b451ee917961bd1e09bbc98a7a23db061a92db9b8d8d557bce63216452498c5a
+  metadata.gz: 82b061b05d127f9e8c610effc797e56f102504f0494b908c3e0e6e8bfbb468aa
+  data.tar.gz: ff8e61e682146e74e9853f2580faed2cac0750333e2bb26a9fffe98ace2fec1f
 SHA512:
-  metadata.gz: ad36b6cede53c724b3ac0d9670188ce3df09f817d1bf4137ff779f9152e79dd27a6af59dc9cd934ada0b7bb52baf136e389cd34facb2d6fa2f22996a9df2dad6
-  data.tar.gz: 825cc31fda1e4b48b8819b0748bdff1d9ab4cc532d5aec550a8c446c03b97aeea0307708738c50f786798add258873a85678de533d5cbbfdc106499f44f56ea8
+  metadata.gz: 8a07d644733195d80aea841e88b111f3d7223a22305ab996f5187a362d90db65fab77049bc10b371222e03b7d6c21cf9ea88ba4fe6ac6268ec75726314e74606
+  data.tar.gz: 9a5bcdccede3f23359e35129a436d6eab770f146651024a5a4198dd2df9a97087b22b9c010b5f2a2f3d3c53db5e1844854b7005668cc1969d138c23f77545b5a

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

@@ -20,8 +20,11 @@ module Mutant
               end
             end
 
-            @targets.each(&method(:print_mutations))
-            Either::Right.new(nil)
+            if @targets.map(&method(:print_mutations)).all?
+              Either::Right.new(nil)
+            else
+              Either::Left.new('Invalid mutation detected!')
+            end
           end
 
         private
@@ -70,18 +73,28 @@ module Mutant
             end
           end
 
+          # rubocop:disable Metrics/MethodLength
           def print_mutations(target)
             world.stdout.puts(target.identification)
 
+            success = true
+
             Mutator::Node.mutate(
               config: Mutant::Mutation::Config::DEFAULT.with(ignore_patterns: @ignore_patterns),
               node:   target.node
             ).each do |mutation|
-              Reporter::CLI::Printer::Mutation.call(
-                object: Mutant::Mutation::Evil.new(subject: target, node: mutation),
-                output: world.stdout
+              Mutant::Mutation::Evil.from_node(subject: target, node: mutation).either(
+                ->(violation) { world.stdout.puts(violation.report); success = false },
+                lambda { |object|
+                  Reporter::CLI::Printer::Mutation.call(
+                    object:,
+                    output: world.stdout
+                  )
+                }
               )
             end
+
+            success
           end
 
           def parse_remaining_arguments(arguments)

data/lib/mutant/loader.rb

@@ -15,8 +15,8 @@ module Mutant
     # Call loader
     #
     # @return [Result]
-    def self.call(*arguments)
-      new(*arguments).call
+    def self.call(*)
+      new(*).call
     end
 
     # Call loader

data/lib/mutant/matcher/method/instance.rb

@@ -30,8 +30,12 @@ module Mutant
         end
         # rubocop:enable Metrics/MethodLength
 
+        TARGET_MEMOIZER = ::Mutant::Adamantium
+
+        private_constant(*constants(false))
+
         def self.memoized_method?(scope, method_name)
-          scope.raw < Adamantium && scope.raw.memoized?(method_name)
+          scope.raw < TARGET_MEMOIZER && scope.raw.memoized?(method_name)
         end
         private_class_method :memoized_method?
 

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

@@ -2,20 +2,31 @@
 
 module Mutant
   module Meta
+    # rubocop:disable Metrics/ClassLength
     class Example
       # Example verification
       class Verification
-        include Adamantium, Anima.new(:example, :mutations)
+        include Adamantium, Anima.new(:example, :invalid, :valid)
+
+        def self.from_mutations(example:, mutations:)
+          valid, invalid = [], []
+
+          mutations.each do |mutation|
+            mutation.either(invalid.public_method(:<<), valid.public_method(:<<))
+          end
+
+          new(example:, invalid:, valid:)
+        end
 
         # Test if mutation was verified successfully
         #
         # @return [Boolean]
         def success?
           [
-            original_verification,
-            invalid,
+            invalid_report,
             missing,
             no_diffs,
+            original_verification_report,
             unexpected
           ].all?(&:empty?)
         end
@@ -29,12 +40,12 @@ module Mutant
 
         def reports
           reports = [example.location]
-          reports.concat(original)
-          reports.concat(original_verification)
+          reports.concat(original_report)
+          reports.concat(original_verification_report)
           reports.concat(make_report('Missing mutations:', missing))
           reports.concat(make_report('Unexpected mutations:', unexpected))
           reports.concat(make_report('No-Diff mutations:', no_diffs))
-          reports.concat(invalid)
+          reports.concat(invalid_report)
         end
 
         def make_report(label, mutations)
@@ -52,7 +63,7 @@ module Mutant
           ]
         end
 
-        def original
+        def original_report
           [
             "Original: (operators: #{example.operators.class.operators_name})",
             example.node,
@@ -60,7 +71,7 @@ module Mutant
           ]
         end
 
-        def original_verification
+        def original_verification_report
           validation = Unparser::Validation.from_string(example.original_source)
           if validation.success?
             []
@@ -77,30 +88,34 @@ module Mutant
           end.join
         end
 
-        def invalid
-          mutations.each_with_object([]) do |mutation, aggregate|
-            validation = Unparser::Validation.from_node(mutation.node)
-            aggregate << prefix('[invalid-mutation]', validation.report) unless validation.success?
+        def invalid_report
+          invalid.map do |validation|
+            prefix('[invalid-mutation]', validation.report)
           end
         end
-        memoize :invalid
+        memoize :invalid_report
 
         def unexpected
-          mutations.reject do |mutation|
+          valid.reject do |mutation|
             example.expected.any? { |expected| expected.node.eql?(mutation.node) }
           end
         end
         memoize :unexpected
 
         def missing
-          (example.expected.map(&:node) - mutations.map(&:node)).map do |node|
-            Mutation::Evil.new(subject: example, node:)
+          example.expected.each_with_object([]) do |expected, aggregate|
+            next if valid.any? { |mutation| expected.node.eql?(mutation.node) }
+            aggregate << Mutation::Evil.new(
+              node:    expected.node,
+              source:  expected.original_source,
+              subject: example
+            )
           end
         end
         memoize :missing
 
         def no_diffs
-          mutations.select { |mutation| mutation.source.eql?(example.original_source_generated) }
+          valid.select { |mutation| mutation.source.eql?(example.source) }
         end
         memoize :no_diffs
 

data/lib/mutant/meta/example.rb

@@ -23,7 +23,7 @@ module Mutant
       #
       # @return [Verification]
       def verification
-        Verification.new(example: self, mutations: generated)
+        Verification.from_mutations(example: self, mutations: generated)
       end
       memoize :verification
 
@@ -45,13 +45,13 @@ module Mutant
         )
       end
 
-      # Original source as generated by unparser
+      # Original source
       #
       # @return [String]
-      def original_source_generated
+      def source
         Unparser.unparse(node)
       end
-      memoize :original_source_generated
+      memoize :source
 
       # Generated mutations on example source
       #
@@ -61,7 +61,7 @@ module Mutant
           config: Mutation::Config::DEFAULT.with(operators:),
           node:
         ).map do |node|
-          Mutation::Evil.new(subject: self, node:)
+          Mutation::Evil.from_node(subject: self, node:)
         end
       end
       memoize :generated

data/lib/mutant/mutation.rb

@@ -4,11 +4,69 @@ module Mutant
   # Represent a mutated node with its subject
   class Mutation
     include AbstractType, Adamantium
-    include Anima.new(:subject, :node)
+    include Anima.new(:subject, :node, :source)
 
     CODE_DELIMITER = "\0"
     CODE_RANGE     = (..4)
 
+    class GenerationError
+      include Anima.new(:subject, :node, :unparser_validation)
+
+      MESSAGE = <<~'MESSAGE'
+        === Mutation-Generation-Error ===
+        This is a mutant internal issue detected by a mutant internal cross check.
+        Please report an issue with the details below.
+
+        Subject: %<subject_identification>s.
+
+        Mutation-Source-Diff:
+        %<mutation_source_diff>s
+
+        Mutation-Node-Diff:
+        %<mutation_node_diff>s
+
+        Unparser-Validation:
+        %<unparser_validation>s
+      MESSAGE
+
+      def report
+        MESSAGE % {
+          mutation_source_diff:,
+          mutation_node_diff:,
+          subject_identification: subject.identification,
+          unparser_validation:    unparser_validation.report
+        }
+      end
+
+    private
+
+      def mutation_source_diff
+        mutation = Evil.new(
+          subject:,
+          node:    nil,
+          source:  unparser_validation.original_source.from_right
+        )
+
+        mutation.diff.colorized_diff
+      end
+
+      def mutation_node_diff
+        Unparser::Diff.new(
+          subject.node.to_s.lines,
+          node.to_s.lines
+        ).colorized_diff
+      end
+    end # GenerationError
+
+    def self.from_node(subject:, node:)
+      ast = Unparser::AST.from_node(node:)
+
+      Unparser
+        .unparse_validate_ast_either(ast:)
+        .lmap { |unparser_validation| GenerationError.new(subject:, node:, unparser_validation:) }
+        .fmap { |source| new(node:, source:, subject:) }
+    end
+
     # Mutation identification code
     #
     # @return [String]
@@ -17,14 +75,6 @@ module Mutant
     end
     memoize :code
 
-    # Normalized mutation source
-    #
-    # @return [String]
-    def source
-      Unparser.unparse(node)
-    end
-    memoize :source
-
     # Identification string
     #
     # @return [String]
@@ -75,10 +125,6 @@ module Mutant
       end
     end
 
-    # Rendered mutation diff
-    #
-    # @return [String, nil]
-    #   the diff, if present
     def diff
       Unparser::Diff.build(original_source, source)
     end

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

@@ -14,10 +14,7 @@ module Mutant
       private
 
         def dispatch
-          emit_singletons
-          emit_left_mutations do |node|
-            !n_self?(node)
-          end
+          emit_left_mutations
           emit_right_mutations
         end
 

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

@@ -8,24 +8,11 @@ module Mutant
       class Argument < self
         handle(:arg, :kwarg)
 
-        UNDERSCORE = '_'
-
         children :name
 
       private
 
-        def dispatch
-          emit_name_mutation
-        end
-
-        def emit_name_mutation
-          return if skip?
-          emit_name(:"#{UNDERSCORE}#{name}")
-        end
-
-        def skip?
-          name.start_with?(UNDERSCORE)
-        end
+        def dispatch; end
 
         # Mutator for optional arguments
         class Optional < self
@@ -42,7 +29,6 @@ module Mutant
         private
 
           def dispatch
-            emit_name_mutation
             emit_required_mutation
             emit_default_mutations
           end

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

@@ -13,36 +13,15 @@ module Mutant
       private
 
         def dispatch
-          emit_argument_presence
           emit_argument_mutations
           emit_mlhs_expansion
           emit_procarg0_removal
         end
 
-        def emit_argument_presence
-          emit_type unless removed_block_arg?(EMPTY_ARRAY) || forward_arg?
-
-          children.each_with_index do |removed, index|
-            new_arguments = children.dup
-            new_arguments.delete_at(index)
-            unless forward_type?(removed) || removed_block_arg?(new_arguments) || only_mlhs?(new_arguments)
-              emit_type(*new_arguments)
-            end
-          end
-        end
-
         def forward_type?(removed)
           n_forward_arg?(removed) || n_restarg?(removed) || n_kwrestarg?(removed)
         end
 
-        def only_mlhs?(new_arguments)
-          new_arguments.one? && n_mlhs?(new_arguments.first)
-        end
-
-        def forward_arg?
-          children.any?(&method(:forward_type?))
-        end
-
         def removed_block_arg?(new_arguments)
           anonymous_block_arg? && new_arguments.none?(&ANONYMOUS_BLOCKARG_PRED)
         end

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

@@ -12,9 +12,44 @@ module Mutant
       private
 
         def dispatch
-          ignore_single = children.any?(&method(:ignore?))
-          mutate_single_child do |child|
-            emit(child) unless ignore_single
+          children.each_index do |index|
+            mutate_child(index)
+            delete_child(index)
+          end
+        end
+
+        # rubocop:disable Lint/EmptyWhen
+        # rubocop:disable Metrics/MethodLength
+        def delete_child(index)
+          dup_children = children.dup
+          child = dup_children.delete_at(index)
+          return if contains_lvar_assignment?(child)
+          return if ignore?(child)
+
+          case dup_children.length
+          when 0
+          when 1
+            one = Mutant::Util.one(dup_children)
+            return if ignore?(one)
+            emit(one)
+          else
+            emit_type(*dup_children)
+          end
+        end
+
+        def contains_lvar_assignment?(node)
+          case node.type
+          when Unparser::AST::ASSIGN_NODES
+            true
+          when *Unparser::AST::RESET_NODES
+            false
+          else
+            node.children.each do |child|
+              if child.instance_of?(::Parser::AST::Node) && contains_lvar_assignment?(child)
+                return true
+              end
+            end
+            false
           end
         end
       end # Begin

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

@@ -18,9 +18,8 @@ module Mutant
       private
 
         def dispatch
-          emit_singletons
+          emit_singletons unless left_lvasgn?
           emit_promotions
-          emit_operator_mutations
 
           emit_left_mutations do |mutation|
             !(n_irange?(mutation) || n_erange?(mutation)) || !mutation.children.fetch(1).nil?
@@ -29,13 +28,13 @@ module Mutant
           emit_right_mutations
         end
 
-        def emit_operator_mutations
-          emit(s(INVERSE.fetch(node.type), left, right))
-        end
-
         def emit_promotions
           emit(left)
-          emit(right)
+          emit(right) unless left_lvasgn?
+        end
+
+        def left_lvasgn?
+          n_lvasgn?(left)
         end
 
       end # Binary

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

@@ -25,7 +25,9 @@ module Mutant
           emit_body(N_RAISE)
 
           return unless body
-          emit(body) unless body_has_control?
+
+          emit_body_promotion
+
           emit_body_mutations do |node|
             !(n_nil?(node) && unconditional_loop?)
           end
@@ -45,8 +47,17 @@ module Mutant
           false
         end
 
+        def emit_body_promotion
+          emit(body) if empty_block_argument? && !body_has_control?
+        end
+
+        def empty_block_argument?
+          arguments.children.empty?
+        end
+
         def mutate_body_receiver
           return if n_lambda?(send) || !n_send?(body)
+          return unless empty_block_argument?
 
           body_meta = AST::Meta::Send.new(node: body)
 

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

@@ -21,10 +21,7 @@ module Mutant
         end
 
         def mutate_condition
-          emit_condition_mutations do |node|
-            !n_self?(node)
-          end
-          emit_type(n_not(condition), if_branch, else_branch) unless n_match_current_line?(condition)
+          emit_condition_mutations
           emit_type(N_TRUE,  if_branch, else_branch)
           emit_type(N_FALSE, if_branch, else_branch)
         end

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

@@ -44,7 +44,6 @@ module Mutant
 
         def mutate_indices
           children_indices(index_range).each do |index|
-            emit_propagation(children.fetch(index)) unless left_op_assignment?
             delete_child(index)
             mutate_child(index)
           end

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

@@ -3,9 +3,8 @@
 module Mutant
   class Mutator
     class Node
-
       # Mutation emitter to handle multiple assignment nodes
-      class MultipleAssignment < self
+      class Masgn < self
 
         handle(:masgn)
 
@@ -14,10 +13,10 @@ module Mutant
       private
 
         def dispatch
-          emit_singletons
+          emit_right_mutations
         end
 
-      end # MultipleAssignment
+      end # Masgn
     end # Node
   end # Mutator
 end # Mutant

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

@@ -10,11 +10,7 @@ module Mutant
 
       private
 
-        def dispatch
-          mutate_single_child do |_child, index|
-            delete_child(index)
-          end
-        end
+        def dispatch; end
 
       end # MLHS
     end # Node

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

@@ -26,19 +26,8 @@ module Mutant
         private
 
           def dispatch
-            emit_singletons
-            mutate_name
             emit_value_mutations if value # op asgn!
           end
-
-          def mutate_name
-            prefix, regexp = MAP.fetch(node.type)
-            stripped = name.to_s.sub(regexp, EMPTY_STRING)
-
-            Util::Symbol.call(input: stripped, parent: nil).each do |name|
-              emit_name(:"#{prefix}#{name}")
-            end
-          end
         end # VariableAssignment
       end # NamedValue
     end # Node

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

@@ -14,8 +14,6 @@ module Mutant
       private
 
         def dispatch
-          emit_singletons
-
           left_mutations
 
           emit_right_mutations
@@ -25,7 +23,6 @@ module Mutant
           emit_left_mutations do |node|
             !n_self?(node)
           end
-
           emit_left_promotion if n_send?(left)
         end
 

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

@@ -13,7 +13,6 @@ module Mutant
       private
 
         def dispatch
-          emit_assignment(nil)
           emit_body_mutations if body
         end
       end # Resbody

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

@@ -25,8 +25,10 @@ module Mutant
         def mutate_rescue_bodies
           children_indices(RESCUE_INDICES).each do |index|
             mutate_child(index)
-            resbody_body = AST::Meta::Resbody.new(node: children.fetch(index)).body
-            emit_concat(resbody_body) if resbody_body
+            resbody = AST::Meta::Resbody.new(node: children.fetch(index))
+            if resbody.body && !resbody.assignment
+              emit_concat(resbody.body)
+            end
           end
         end
 

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

@@ -206,7 +206,8 @@ module Mutant
           emit(receiver) if receiver && !left_op_assignment?
         end
 
-        # rubocop:disable Style/HashEachMethods - its not a hash ;)
+        # rubocop:disable Style/HashEachMethods
+        # - its not a hash ;)
         def mutate_arguments
           emit_type(receiver, selector)
           remaining_children_with_index.each do |_node, index|

data/lib/mutant/parser.rb

@@ -24,11 +24,11 @@ module Mutant
   private
 
     def parse(source)
-      node, comments = Unparser.parse_with_comments(source)
+      ast = Unparser.parse_ast_either(source).from_right
 
       AST.new(
-        node:,
-        comment_associations: ::Parser::Source::Comment.associate_by_identity(node, comments)
+        comment_associations: ::Parser::Source::Comment.associate_by_identity(ast.node, ast.comments),
+        node:                 ast.node
       )
     end
 

data/lib/mutant/procto.rb

@@ -15,8 +15,8 @@ module Mutant
     end
 
     module ClassMethods
-      def call(*arguments)
-        new(*arguments).call
+      def call(*)
+        new(*).call
       end
     end
   end # Procto

data/lib/mutant/subject.rb

@@ -10,13 +10,21 @@ module Mutant
     #
     # @return [Enumerable<Mutation>]
     # @return [undefined]
+    #
+    # mutant:disable
+    # rubocop:disable Metrics/MethodLength
     def mutations
       [neutral_mutation].concat(
         Mutator::Node.mutate(
           config: config.mutation,
           node:
-        ).map do |mutant|
-          Mutation::Evil.new(subject: self, node: wrap_node(mutant))
+        ).each_with_object([]) do |mutant, aggregate|
+          Mutation::Evil
+            .from_node(subject: self, node: wrap_node(mutant))
+            .either(
+              ->(validation) {},
+              aggregate.public_method(:<<)
+            )
         end
       )
     end
@@ -92,7 +100,9 @@ module Mutant
   private
 
     def neutral_mutation
-      Mutation::Neutral.new(subject: self, node: wrap_node(node))
+      Mutation::Neutral
+        .from_node(subject: self, node: wrap_node(node))
+        .from_right
     end
 
     def wrap_node(node)

data/lib/mutant/version.rb

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

data/lib/mutant/zombifier.rb

@@ -32,8 +32,8 @@ module Mutant
     # Call zombifier
     #
     # @return [self]
-    def self.call(*args)
-      new(*args).__send__(:call)
+    def self.call(*)
+      new(*).__send__(:call)
       self
     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mutant
 version: !ruby/object:Gem::Version
-  version: 0.12.4
+  version: 0.13.0
 platform: ruby
 authors:
 - Markus Schirp
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-06-30 00:00:00.000000000 Z
+date: 2025-03-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: diff-lcs
@@ -72,14 +72,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.6.14
+        version: 0.7.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.6.14
+        version: 0.7.0
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -355,14 +355,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '3.1'
+      version: '3.2'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.9
+rubygems_version: 3.5.22
 signing_key:
 specification_version: 4
 summary: ''