mutant 0.5.12 → 0.5.13

data/lib/mutant/matcher/method.rb

@@ -24,17 +24,8 @@ module Mutant
       def each
         return to_enum unless block_given?
 
-        unless skip?
-          if subject
-            yield subject
-          else
-            message = sprintf(
-              'Cannot find definition of: %s in %s',
-              identification,
-              source_location.join(':')
-            )
-            $stderr.puts(message)
-          end
+        if !skip? && subject
+          yield subject
         end
 
         self

data/lib/mutant/mutation.rb

@@ -6,6 +6,9 @@ module Mutant
     include AbstractType, Adamantium::Flat
     include Concord::Public.new(:subject, :node)
 
+    CODE_DELIMITER = "\0".freeze
+    CODE_RANGE     = (0..4).freeze
+
     # Return mutated root node
     #
     # @return [Parser::AST::Node]
@@ -54,8 +57,9 @@ module Mutant
     # @api private
     #
     def identification
-      "#{subject.identification}:#{code}"
+      "#{self.class::SYMBOL}:#{subject.identification}:#{code}"
     end
+    memoize :identification
 
     # Return mutation code
     #
@@ -64,7 +68,7 @@ module Mutant
     # @api private
     #
     def code
-      sha1[0..4]
+      sha1[CODE_RANGE]
     end
     memoize :code
 
@@ -89,6 +93,16 @@ module Mutant
       subject.source
     end
 
+    # Test if test should fail under mutation
+    #
+    # @return [Boolean]
+    #
+    # @api private
+    #
+    def should_fail?
+      self.class::SHOULD_FAIL
+    end
+
   private
 
     # Return sha1 sum of source and subject identification
@@ -98,7 +112,7 @@ module Mutant
     # @api private
     #
     def sha1
-      Digest::SHA1.hexdigest(subject.identification + 0.chr + source)
+      Digest::SHA1.hexdigest(subject.identification + CODE_DELIMITER + source)
     end
     memoize :sha1
 

data/lib/mutant/mutation/evil.rb

@@ -5,16 +5,8 @@ module Mutant
     # Evul mutation
     class Evil < self
 
-      # Return identification
-      #
-      # @return [String]
-      #
-      # @api private
-      #
-      def identification
-        "evil:#{super}"
-      end
-      memoize :identification
+      SHOULD_FAIL = true
+      SYMBOL      = 'evil'.freeze
 
       # Test if killer is successful
       #

data/lib/mutant/mutation/neutral.rb

@@ -5,41 +5,15 @@ module Mutant
     # Neutral mutation
     class Neutral < self
 
-      SYMBOL = 'neutral'
+      SYMBOL      = 'neutral'.freeze
+      SHOULD_FAIL = false
 
       # Noop mutation, special case of neutral
       class Noop < self
 
-        SYMBOL = 'noop'
+        SYMBOL = 'noop'.freeze
 
-      end
-
-      # Return identification
-      #
-      # @return [String]
-      #
-      # @api private
-      #
-      def identification
-        "#{self.class::SYMBOL}:#{super}"
-      end
-      memoize :identification
-
-      # Test if killer is successful
-      #
-      # @param [Killer] killer
-      #
-      # @return [true]
-      #   if killer did NOT killed mutation
-      #
-      # @return [false]
-      #   otherwise
-      #
-      # @api private
-      #
-      def success?(killer)
-        !killer.killed?
-      end
+      end # Noop
 
     end # Neutral
   end # Mutation

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

@@ -20,7 +20,6 @@ module Mutant
           def dispatch
             emit_nil
             emit_values(values)
-            emit_new { new_self(Random.fixnum) }
           end
 
           # Return values to mutate against

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

@@ -21,7 +21,6 @@ module Mutant
             emit_nil
             emit_values(values)
             emit_special_cases
-            emit_new { new_self(Random.float) }
           end
 
           SPECIAL = [

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

@@ -19,7 +19,6 @@ module Mutant
           #
           def dispatch
             emit_nil
-            emit_new { new_self(Random.hex_string) }
           end
 
         end # String

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

@@ -9,7 +9,9 @@ module Mutant
 
           handle(:sym)
 
-          PREFIX = 's'
+          children :value
+
+          PREFIX = '__mutant__'.freeze
 
         private
 
@@ -21,7 +23,9 @@ module Mutant
           #
           def dispatch
             emit_nil
-            emit_new { new_self((PREFIX + Random.hex_string).to_sym) }
+            Mutator::Util::Symbol.each(value, self) do |value|
+              emit_self(value)
+            end
           end
 
         end # Symbol

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

@@ -10,11 +10,15 @@ module Mutant
 
           children :name, :value
 
-          MAP = IceNine.deep_freeze(
+          map = {
             gvasgn: '$',
             cvasgn: '@@',
             ivasgn: '@',
             lvasgn: ''
+          }
+
+          MAP = IceNine.deep_freeze(
+            Hash[map.map { |type, prefix| [type, [prefix, /^#{Regexp.escape(prefix)}/]] }]
           )
 
           handle(*MAP.keys)
@@ -40,8 +44,9 @@ module Mutant
           # @api private
           #
           def mutate_name
-            prefix = MAP.fetch(node.type)
-            Mutator::Util::Symbol.each(name, self) do |name|
+            prefix, regexp = MAP.fetch(node.type)
+            stripped = name.to_s.sub(regexp, EMPTY_STRING)
+            Mutator::Util::Symbol.each(stripped, self) do |name|
               emit_name(:"#{prefix}#{name}")
             end
           end

data/lib/mutant/mutator/util/symbol.rb

@@ -9,6 +9,8 @@ module Mutant
 
         handle(::Symbol)
 
+        POSTFIX = '__mutant__'.freeze
+
       private
 
         # Emit mutations
@@ -18,7 +20,7 @@ module Mutant
         # @api private
         #
         def dispatch
-          emit_new { :"s#{Random.hex_string}" }
+          emit((input.to_s + POSTFIX).to_sym)
         end
 
       end # Symbol

data/lib/mutant/node_helpers.rb

@@ -13,7 +13,7 @@ module Mutant
     # @api private
     #
     def s(type, *children)
-      ::Parser::AST::Node.new(type, children)
+      Parser::AST::Node.new(type, children)
     end
     module_function :s
 
@@ -21,8 +21,6 @@ module Mutant
       s(:send, s(:float,  0.0), :/, s(:float, 0.0))
     INFINITY =
       s(:send, s(:float,  1.0), :/, s(:float, 0.0))
-    NEW_OBJECT =
-      s(:send, s(:const, s(:cbase), :Object), :new)
     NEGATIVE_INFINITY =
       s(:send, s(:float, -1.0), :/, s(:float, 0.0))
 

data/lib/mutant/reporter.rb

@@ -15,5 +15,15 @@ module Mutant
     #
     abstract_method :report
 
+    # Report progress on object
+    #
+    # @param [Object] object
+    #
+    # @return [self]
+    #
+    # @api private
+    #
+    abstract_method :progress
+
   end # Reporter
 end # Mutant

data/lib/mutant/reporter/cli.rb

@@ -4,10 +4,23 @@ module Mutant
   class Reporter
     # Reporter that reports in human readable format
     class CLI < self
-      include Concord::Public.new(:output)
+      include Concord.new(:output)
 
       NL = "\n".freeze
 
+      # Report progress object
+      #
+      # @param [Object] object
+      #
+      # @return [self]
+      #
+      # @api private
+      #
+      def progress(object)
+        Progress.run(output, object)
+        self
+      end
+
       # Report object
       #
       # @param [Object] object
@@ -17,7 +30,7 @@ module Mutant
       # @api private
       #
       def report(object)
-        Printer.visit(object, output)
+        Report.run(output, object)
         self
       end
 

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

@@ -6,120 +6,27 @@ module Mutant
 
       # CLI runner status printer base class
       class Printer
-        include AbstractType, Adamantium::Flat, Concord.new(:object, :output)
+        include AbstractType, Delegator, Adamantium::Flat, Concord.new(:output, :object)
 
-        REGISTRY = {}
-
-        # Create delegators to object
-        #
-        # @return [undefined]
-        #
-        # @api private
-        #
-        def self.delegate(*names)
-          names.each do |name|
-            define_delegator(name)
-          end
-        end
-        private_class_method :delegate
-
-        # Create delegator to object
-        #
-        # @param [Symbol] name
-        #
-        # @return [undefined]
-        #
-        # @api private
-        #
-        def self.define_delegator(name)
-          define_method(name) do
-            object.public_send(name)
-          end
-          private name
-        end
-        private_class_method :define_delegator
-
-        # Registre handler for class
-        #
-        # @param [Class] klass
-        #
-        # @return [undefined]
+        # Run printer on object to output
         #
-        # @api private
-        #
-        def self.handle(klass)
-          REGISTRY[klass] = self
-        end
-
-        # Finalize CLI reporter
-        #
-        # @return [undefined]
-        #
-        # @api private
-        #
-        def self.finalize
-          REGISTRY.freeze
-        end
-
-        # Build printer
-        #
-        # @return [Printer]
-        #
-        # @api private
-        #
-        def self.build(*args)
-          new(*args)
-        end
-
-        # Run printer
+        # @param [IO] output
+        # @param [Object] object
         #
         # @return [self]
         #
-        # @api private
-        #
-        def self.run(*args)
-          build(*args).run
+        def self.run(output, object)
+          handler = lookup(object.class)
+          handler.new(output, object).run
           self
         end
 
-        # Visit object
-        #
-        # @param [Object] object
-        # @param [IO] output
-        #
-        # @return [undefined]
-        #
-        # @api private
-        #
-        def self.visit(object, output)
-          printer = lookup(object.class)
-          printer.run(object, output)
-        end
-
-        # Lookup printer class
-        #
-        # @param [Class] klass
-        #
-        # @return [Class:Printer]
-        #   if found
+        # Run printer
         #
-        # @raise [RuntimeError]
-        #   otherwise
+        # @return [self]
         #
         # @api private
         #
-        def self.lookup(klass)
-          current = klass
-          until current == Object
-            if REGISTRY.key?(current)
-              return REGISTRY.fetch(current)
-            end
-            current = current.superclass
-          end
-          raise "No printer for: #{klass}"
-        end
-        private_class_method :lookup
-
         abstract_method :run
 
       private
@@ -130,7 +37,7 @@ module Mutant
         #
         # @api private
         #
-        def color
+        def status_color
           success? ? Color::GREEN : Color::RED
         end
 
@@ -143,7 +50,7 @@ module Mutant
         # @api private
         #
         def visit(object)
-          self.class.visit(object, output)
+          self.class.run(output, object)
         end
 
         # Print an info line to output
@@ -163,7 +70,7 @@ module Mutant
         # @api private
         #
         def status(string, *arguments)
-          puts(colorize(color, sprintf(string, *arguments)))
+          puts(colorize(status_color, sprintf(string, *arguments)))
         end
 
         # Print a line to output