mutant 0.3.0.beta21 → 0.3.0.beta22

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Matcher
     class Method
@@ -92,7 +94,11 @@ module Mutant
           when :const
             receiver_name?(receiver)
           else
-            $stderr.puts "Can only match self or const, got #{receiver.type}, unable to match receiver of defs node"
+            message = sprintf(
+              'Can only match :defs on :self or :const got %s unable to match',
+              receiver.type.inspect
+            )
+            $stderr.puts(message)
             false
           end
         end

data/lib/mutant/matcher/methods.rb

@@ -1,6 +1,8 @@
+# encoding: utf-8
+
 module Mutant
   class Matcher
-    # Abstract base class for matcher that returns method subjects extracted from scope
+    # Abstract base class for matcher that returns method subjects from scope
     class Methods < self
       include AbstractType, Concord::Public.new(:cache, :scope)
 

data/lib/mutant/matcher/namespace.rb

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Matcher
 
@@ -34,7 +36,7 @@ module Mutant
       # @api private
       #
       def pattern
-        %r(\A#{Regexp.escape(namespace.name)}(?:::)?)
+        /\A#{Regexp.escape(namespace.name)}(?:::)?/
       end
       memoize :pattern
 
@@ -62,7 +64,7 @@ module Mutant
       #
       def emit_scope(scope)
         name = scope.name
-        # FIXME Fix nokogiri to return a string here
+        # FIXME: Fix nokogiri to return a string here
         return unless name.kind_of?(String)
         if pattern =~ name
           yield scope

data/lib/mutant/matcher/scope.rb

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

data/lib/mutant/mutation.rb

@@ -1,7 +1,10 @@
+# encoding: utf-8
+
 module Mutant
   # Represent a mutated node with its subject
   class Mutation
-    include AbstractType, Adamantium::Flat, Concord::Public.new(:subject, :node)
+    include AbstractType, Adamantium::Flat
+    include Concord::Public.new(:subject, :node)
 
     # Return mutated root node
     #
@@ -28,12 +31,23 @@ module Mutant
     #
     abstract_method :success?
 
+    # Indicate if a killer should treat a kill as problematic
+    #
+    # @return [true]
+    #   if killing is unexpected
+    #
+    # @return [false]
+    #   if killing is expected
+    #
+    # @api private
+    #
+    abstract_method :should_survive?
+
     # Insert mutated node
     #
-    # FIXME:
-    #   Cache subject visibility in a better way! Ideally dont mutate it implicitly.
-    #   Also subject.public? should NOT be a public interface it is a detail of method
-    #   mutations.
+    # FIXME: Cache subject visibility in a better way! Ideally dont mutate it
+    #   implicitly. Also subject.public? should NOT be a public interface it
+    #   is a detail of method mutations.
     #
     # @return [self]
     #

data/lib/mutant/mutation/evil.rb

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutation
     # Evul mutation
@@ -30,6 +32,16 @@ module Mutant
         killer.killed?
       end
 
+      # Indicate if a killer should treat a kill as problematic
+      #
+      # @return [false] Killing evil mutants is not problematic
+      #
+      # @api private
+      #
+      def should_survive?
+        false
+      end
+
     end # Evil
   end # Mutation
 end # Mutant

data/lib/mutant/mutation/filter.rb

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutation
     # Abstract filter for mutations

data/lib/mutant/mutation/filter/code.rb

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutation
     class Filter
@@ -21,7 +23,7 @@ module Mutant
           mutation.code.eql?(code)
         end
 
-        PATTERN = %r(\Acode:([a-f0-9]{1,6})\z).freeze
+        PATTERN = /\Acode:([a-f0-9]{1,6})\z/.freeze
 
         # Test if class handles string
         #

data/lib/mutant/mutation/filter/regexp.rb

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

data/lib/mutant/mutation/filter/whitelist.rb

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

data/lib/mutant/mutation/neutral.rb

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutation
     # Neutral mutation
@@ -10,6 +12,19 @@ module Mutant
 
         SYMBOL = 'noop'
 
+        # Indicate if a killer should treat a kill as problematic
+        #
+        # @return [false] Killing noop mutants is a serious problem. Failures
+        #   in noop may indicate a broken test suite, but they can also be an
+        #   indication  mutant has altered the runtime environment in a subtle
+        #   way and tickled an odd bug
+        #
+        # @api private
+        #
+        def should_survive?
+          false
+        end
+
       end
 
       # Return identification
@@ -39,6 +54,16 @@ module Mutant
         !killer.killed?
       end
 
+      # Indicate if a killer should treat a kill as problematic
+      #
+      # @return [true] Neutral mutants must die
+      #
+      # @api private
+      #
+      def should_survive?
+        false
+      end
+
     end # Neutral
   end # Mutation
 end # Mutant

data/lib/mutant/mutator.rb

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   # Generator for mutations
   class Mutator
@@ -12,7 +14,7 @@ module Mutant
     #
     # @api private
     #
-    def self.each(node, parent=nil, &block)
+    def self.each(node, parent = nil, &block)
       return to_enum(__method__, node, parent) unless block_given?
       Registry.lookup(node).new(node, parent, block)
 
@@ -135,17 +137,13 @@ module Mutant
 
     # Call block until it generates a mutation
     #
-    # The primary use of this method is to give the random generated object
-    # a nice interface for retring generation when generation accidentally generated the
-    # input
-    #
     # @yield
     #   Execute block until object is generated where new?(object) returns true
     #
     # @return [self]
     #
     # @raise [RuntimeError]
-    #   raises RuntimeError in case no new ast node can be generated after MAX_TRIES.
+    #   raises RuntimeError when no new node can be generated after MAX_TRIES.
     #
     # @api private
     #

data/lib/mutant/mutator/node.rb

@@ -1,6 +1,10 @@
+# encoding: utf-8
+
 module Mutant
+
   # Generator for mutations
   class Mutator
+
     # Abstract base class for node mutators
     class Node < self
       include AbstractType, NodeHelpers, Unparser::Constants
@@ -154,7 +158,7 @@ module Mutant
       #
       def emit_child_update(index, node)
         new_children = children.dup
-        new_children[index]=node
+        new_children[index] = node
         emit_self(*new_children)
       end
 

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node
@@ -30,7 +32,7 @@ module Mutant
         # @api private
         #
         def emit_when_mutations
-          indices = children.each_index.drop(1).take(children.length-2)
+          indices = children.each_index.drop(1).take(children.length - 2)
           one = indices.one?
           indices.each do |index|
             mutate_child(index)

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node
@@ -11,7 +13,7 @@ module Mutant
             :or  => :and,
           }.freeze
 
-          handle *INVERSE.keys
+          handle(*INVERSE.keys)
 
           children :left, :right
 

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node
@@ -8,10 +10,9 @@ module Mutant
         # These nodes still need a dedicated mutator,
         # your contribution is that close!
         handle(
-          :defined,
-          :next, :break, :match, :ensure,
+          :next, :break, :ensure,
           :dstr, :dsym, :yield, :rescue, :redo, :defined?,
-          :blockarg, :block_pass, :op_asgn, :and_asgn,
+          :blockarg, :op_asgn, :and_asgn,
           :regopt, :restarg, :resbody, :retry, :arg_expr,
           :kwrestarg, :kwoptarg, :kwarg, :undef, :module, :empty,
           :alias, :for, :xstr, :back_ref, :nth_ref, :class,

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node
@@ -28,7 +30,7 @@ module Mutant
           # @api private
           #
           def values
-            [0, 1, -value, value+1, value-1]
+            [0, 1, -value, value + 1, value - 1]
           end
 
           # Return value

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node
@@ -48,7 +50,6 @@ module Mutant
           # @api private
           #
           def emit_end_mutations
-            #emit_self(negative_infinity, finish)
             emit_self(NAN, _end)
           end
 

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

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 module Mutant
   class Mutator
     class Node