mutant 0.5.19 → 0.5.20

data/lib/mutant/isolation.rb

@@ -0,0 +1,65 @@
+module Mutant
+  # Module providing isolationg
+  module Isolation
+    Error = Class.new(RuntimeError)
+
+    # Call block in isolation
+    #
+    # This isolation implements the fork strategy.
+    # Future strategies will probably use a process pool that can
+    # handle multiple mutation kills, in-isolation at once.
+    #
+    # @return [Object]
+    #
+    # @raise [Error]
+    #
+    # @api private
+    #
+    def self.call(&block)
+      reader, writer = IO.pipe.each(&:binmode)
+
+      pid = fork
+
+      if pid.nil?
+        begin
+          reader.close
+          writer.write(Marshal.dump(block.call))
+          Kernel.exit!(0)
+        ensure
+          Kernel.exit!(1)
+        end
+      end
+
+      writer.close
+
+      read_result(reader, pid)
+    end
+
+    # Read result from child process
+    #
+    # @param [IO] reader
+    # @param [Fixnum] pid
+    #
+    # @return [Object]
+    #
+    # @raise [Error]
+    #
+    def self.read_result(reader, pid)
+      begin
+        data = Marshal.load(reader.read)
+      rescue ArgumentError, TypeError
+        raise Error, 'Childprocess wrote un-unmarshallable data'
+      end
+
+      status = Process.waitpid2(pid).last
+
+      unless status.exitstatus.zero?
+        raise Error, "Childprocess exited with nonzero exit status: #{status.exitstatus}"
+      end
+
+      data
+    end
+    private_class_method :read_result
+
+  end # Isolator
+end # Mutant

data/lib/mutant/killer.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   # Mutation killer
   class Killer
@@ -31,7 +29,7 @@ module Mutant
     # @api private
     #
     def run
-      test_report = Mutant.isolate do
+      test_report = Isolation.call do
         mutation.insert
         test.run
       end

data/lib/mutant/loader.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   # Base class for code loaders
   class Loader

data/lib/mutant/matcher.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   # Abstract matcher to find subjects to mutate
   class Matcher

data/lib/mutant/matcher/chain.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   class Matcher
     # A chain of matchers

data/lib/mutant/matcher/filter.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   class Matcher
     # Matcher filter

data/lib/mutant/matcher/method.rb

@@ -1,10 +1,9 @@
-# encoding: utf-8
-
 module Mutant
   class Matcher
     # Matcher for subjects that are a specific method
     class Method < self
       include Adamantium::Flat, Concord::Public.new(:cache, :scope, :method)
+      include Equalizer.new(:identification)
 
       # Methods within rbx kernel directory are precompiled and their source
       # cannot be accessed via reading source location

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

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   class Matcher
     class Method

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

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   class Matcher
     class Method
@@ -19,7 +17,7 @@ module Mutant
         #
         def self.build(cache, scope, method)
           name = method.name
-          if scope.ancestors.include?(::Memoizable) and scope.memoized?(name)
+          if scope.ancestors.include?(::Memoizable) && scope.memoized?(name)
             return Memoized.new(cache, scope, method)
           end
           super

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

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   class Matcher
     class Method

data/lib/mutant/matcher/methods.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   class Matcher
     # Abstract base class for matcher that returns method subjects from scope

data/lib/mutant/matcher/namespace.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   class Matcher
 
@@ -36,7 +34,7 @@ module Mutant
       # @api private
       #
       def pattern
-        /\A#{Regexp.escape(namespace)}(?:::)?/
+        /\A#{Regexp.escape(namespace)}(?:\z|::)/
       end
       memoize :pattern
 
@@ -95,9 +93,7 @@ Fix your lib to support normal ruby semantics!
           MESSAGE
           return
         end
-        if pattern =~ name
-          yield scope
-        end
+        yield scope if pattern =~ name
       end
 
     end # Namespace

data/lib/mutant/matcher/null.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   class Matcher
     # A null matcher, that does not match any subjects

data/lib/mutant/matcher/scope.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   class Matcher
     # Matcher for specific namespace

data/lib/mutant/meta/example.rb

@@ -15,20 +15,33 @@ module Mutant
         Verification.new(self, generated)
       end
 
+      # Return source
+      #
+      # @return [String]
+      #
+      # @api private
+      #
+      def source
+        Unparser.unparse(node)
+      end
+      memoize :source
+
       # Return generated mutations
       #
-      # @return [Emumerable<Parser::AST::Node>]
+      # @return [Emumerable<Mutant::Mutation>]
       #
       # @api private
       #
       def generated
-        Mutant::Mutator.each(node).to_a
+        Mutant::Mutator.each(node).map do |node|
+          Mutant::Mutation::Evil.new(self, node)
+        end
       end
       memoize :generated
 
       # Example verification
       class Verification
-        include Adamantium::Flat, Concord.new(:example, :generated)
+        include Adamantium::Flat, Concord.new(:example, :mutations)
 
         # Test if mutation was verified successfully
         #
@@ -37,7 +50,7 @@ module Mutant
         # @api private
         #
         def success?
-          unparser.success? && missing.empty? && unexpected.empty?
+          unparser.success? && missing.empty? && unexpected.empty? && no_diffs.empty?
         end
 
         # Return error report
@@ -62,10 +75,21 @@ module Mutant
         # @api private
         #
         def unexpected
-          generated - example.mutations
+          mutations.map(&:node) - example.mutations
         end
         memoize :unexpected
 
+        # Return mutations with no diff to original
+        #
+        # @return [Enumerable<Mutation>]
+        #
+        # @api private
+        #
+        def no_diffs
+          mutations.select { |mutation| mutation.source.eql?(example.source) }
+        end
+        memoize :no_diffs
+
         # Return mutation report
         #
         # @return [String]
@@ -78,9 +102,10 @@ module Mutant
             'Original-AST:',
             original_node.inspect,
             'Original-Source:',
-            Unparser.unparse(original_node),
+            example.source,
             *missing_report,
-            *unexpected_report
+            *unexpected_report,
+            *no_diff_report
           ].join("\n======\n")
         end
 
@@ -91,9 +116,25 @@ module Mutant
         # @api private
         #
         def missing_report
-          if missing.any?
-            ['Missing mutations:', missing.map(&method(:format_mutation)).join("\n-----\n")]
-          end
+          [
+            'Missing mutations:',
+            missing.map(&method(:format_mutation)).join("\n-----\n")
+          ] if missing.any?
+        end
+
+        # Return no diff report
+        #
+        # @return [Array, nil]
+        #
+        # @api private
+        #
+        def no_diff_report
+          [
+            'No source diffs to original:',
+            no_diffs.map do |mutation|
+              "#{mutation.node.inspect}\n#{mutation.source}"
+            end
+          ] if no_diffs.any?
         end
 
         # Return unexpected report
@@ -103,12 +144,10 @@ module Mutant
         # @api private
         #
         def unexpected_report
-          if unexpected.any?
-            [
-              'Unexpected mutations:',
-              unexpected.map(&method(:format_mutation)).join("\n-----\n")
-            ]
-          end
+          [
+            'Unexpected mutations:',
+            unexpected.map(&method(:format_mutation)).join("\n-----\n")
+          ] if unexpected.any?
         end
 
         # Format mutation
@@ -124,14 +163,14 @@ module Mutant
           ].join("\n")
         end
 
-        # Return missing mutationso
+        # Return missing mutations
         #
         # @return [Array<Parser::AST::Node>]
         #
         # @api private
         #
         def missing
-          example.mutations - generated
+          example.mutations - mutations.map(&:node)
         end
         memoize :missing
 

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

@@ -81,8 +81,8 @@ module Mutant
         # @api private
         #
         def singleton_mutations
-          mutation 'nil'
-          mutation 'self'
+          mutation('nil')
+          mutation('self')
         end
 
         # Helper method to coerce input to node
@@ -97,17 +97,14 @@ module Mutant
         # @api private
         #
         def node(input)
-          node =
-            case input
-            when String
-              Parser::CurrentRuby.parse(input)
-            when Parser::AST::Node
-              input
-            else
-              raise "Cannot coerce to node: #{source.inspect}"
-            end
-
-          Unparser::Preprocessor.run(node)
+          case input
+          when String
+            Unparser::Preprocessor.run(Parser::CurrentRuby.parse(input))
+          when Parser::AST::Node
+            input
+          else
+            raise "Cannot coerce to node: #{source.inspect}"
+          end
         end
 
       end # DSL

data/lib/mutant/mutation.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   # Represent a mutated node with its subject
   class Mutation

data/lib/mutant/mutation/evil.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   class Mutation
     # Evul mutation

data/lib/mutant/mutation/neutral.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   class Mutation
     # Neutral mutation

data/lib/mutant/mutator.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   # Generator for mutations
   class Mutator
@@ -7,7 +5,10 @@ module Mutant
 
     # Run mutator on input
     #
-    # @param [Parser::AST::Node] node
+    # @param [Object] input
+    #   the input to mutate
+    #
+    # @param [Mutator] parent
     #
     # @return [self]
     #
@@ -119,34 +120,6 @@ module Mutant
       emit!(object)
     end
 
-    # Maximum amount of tries to generate a new object
-    MAX_TRIES = 3
-
-    # Call block until it generates a mutation
-    #
-    # @yield
-    #   Execute block until object is generated where new?(object) returns true
-    #
-    # @return [self]
-    #
-    # @raise [RuntimeError]
-    #   raises RuntimeError when no new node can be generated after MAX_TRIES.
-    #
-    # @api private
-    #
-    def emit_new
-      MAX_TRIES.times do
-        object = yield
-
-        if new?(object)
-          emit!(object)
-          return
-        end
-      end
-
-      raise "New AST could not be generated after #{MAX_TRIES} attempts"
-    end
-
     # Call block with node
     #
     # @param [Parser::AST::Node] node