mutant 0.8.0 → 0.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fa063e8c2f6e2cf17858c47e1725f315e64c4c0b
-  data.tar.gz: b96166a5788f2de7088b36033ed7dc0198295b8f
+  metadata.gz: ac3f098a786f3940bb0bdc9b291994aa068fbf3d
+  data.tar.gz: 4acf7c5f8e0d04bbbe87ad2ebfb403650372cf4a
 SHA512:
-  metadata.gz: a4f4038557d14c1da74122005c6dce8e510d3808cc01efec45f3bbcc6e53ece5430b2b1ce0b65d45f4e7234143881e14b21581a66a4f80c89c093b64c4c16dc9
-  data.tar.gz: f31d6f38605ba70960779c66ff5fa8767c508a9e87b2ed49c0ff3033635ad210a2a79040458bca86188e733d0e6cde77902e45f6f6b995e0ea6239e7f66ff5c3
+  metadata.gz: ce34952eb570ea87bb7169785b556431eb9820dea70261f0772c89e3d648cf70a4f819b9be86448f6b4c8ea88b0721c6f3aa6c3f9e3680127f80c805826c6fb2
+  data.tar.gz: 89eb14bfeb66f63899033195bfbec8562c821e19fea0aa51997b8e57fcb3897b7de26e0d5b642d14c7c276c50b6d28f3df0fbc23741a5944523ff6e0c5117c2e

data/Changelog.md

@@ -1,3 +1,10 @@
+# v0.8.1 2015-07-24
+
+* Add --since flag to constrain mutated subjects based on
+  repository diff from HEAD to other REVISON.
+* Add mutation from #[] to #at / #fetch
+* Some internal improvements
+
 # v0.8.0 2015-06-15
 
 * Drop support for ruby < 2.1

data/config/flay.yml

@@ -1,3 +1,3 @@
 ---
 threshold: 18
-total_score: 1227
+total_score: 1208

data/config/reek.yml

@@ -130,6 +130,7 @@ UtilityFunction:
   - Mutant::Integration::Null#call
   - Mutant::Integration::Rspec#parse_example
   - Mutant::Meta::Example::Verification#format_mutation
+  - Mutant::Repository::Diff#tracks? # intentional, private
   - Mutant::Reporter::CLI::Format::Progressive#new_buffer
   - Mutant::Reporter::CLI::Printer::StatusProgressive#object # False positive calls super
   - Mutant::Integration::Rspec#parse_expression # intentional, private

data/lib/mutant.rb

@@ -33,7 +33,6 @@ module Mutant
   # @return [Boolean]
   #
   # @api private
-  #
   def self.ci?
     ENV.key?('CI')
   end
@@ -143,6 +142,7 @@ require 'mutant/matcher/scope'
 require 'mutant/matcher/filter'
 require 'mutant/matcher/null'
 require 'mutant/expression'
+require 'mutant/expression/parser'
 require 'mutant/expression/method'
 require 'mutant/expression/methods'
 require 'mutant/expression/namespace'
@@ -174,6 +174,7 @@ require 'mutant/reporter/cli/printer/status_progressive'
 require 'mutant/reporter/cli/printer/test_result'
 require 'mutant/reporter/cli/tput'
 require 'mutant/reporter/cli/format'
+require 'mutant/repository'
 require 'mutant/zombifier'
 
 module Mutant
@@ -184,7 +185,7 @@ module Mutant
     DEFAULT = new(
       debug:             false,
       fail_fast:         false,
-      integration:       Integration::Null.new,
+      integration:       Integration::Null,
       matcher:           Matcher::Config::DEFAULT,
       includes:          EMPTY_ARRAY,
       requires:          EMPTY_ARRAY,
@@ -192,7 +193,13 @@ module Mutant
       reporter:          Reporter::CLI.build($stdout),
       zombie:            false,
       jobs:              Mutant.ci? ? CI_DEFAULT_PROCESSOR_COUNT : ::Parallel.processor_count,
-      expected_coverage: Rational(1)
+      expected_coverage: Rational(1),
+      expression_parser: Expression::Parser.new([
+        Expression::Method,
+        Expression::Methods,
+        Expression::Namespace::Exact,
+        Expression::Namespace::Recursive
+      ])
     )
   end # Config
 end # Mutant

data/lib/mutant/actor.rb

@@ -10,12 +10,11 @@ module Mutant
     Undefined = Class.new do
       INSPECT = 'Mutant::Actor::Undefined'.freeze
 
-      # Return object inspection
+      # Object inspection
       #
       # @return [String]
       #
       # @api private
-      #
       def inspect
         INSPECT
       end
@@ -25,7 +24,7 @@ module Mutant
     class Message
       include Concord::Public.new(:type, :payload)
 
-      # Return new message
+      # New message
       #
       # @param [Symbol] type
       # @param [Object] payload
@@ -33,7 +32,6 @@ module Mutant
       # @return [Message]
       #
       # @api private
-      #
       def self.new(_type, _payload = Undefined)
         super
       end
@@ -51,7 +49,6 @@ module Mutant
       # @return [Object]
       #
       # @api private
-      #
       def call(type)
         other.call(Message.new(type, mailbox.sender))
         message = mailbox.receiver.call

data/lib/mutant/actor/env.rb

@@ -9,7 +9,6 @@ module Mutant
       # @return [Actor::Sender]
       #
       # @api private
-      #
       def spawn
         mailbox = new_mailbox
 
@@ -20,12 +19,11 @@ module Mutant
         mailbox.sender
       end
 
-      # Return new unbound mailbox
+      # New unbound mailbox
       #
       # @return [Mailbox]
       #
       # @api private
-      #
       def new_mailbox
         Mailbox.new
       end

data/lib/mutant/actor/mailbox.rb

@@ -4,12 +4,11 @@ module Mutant
     class Mailbox
       include Adamantium::Flat, Concord::Public.new(:receiver, :sender)
 
-      # Return new mailbox
+      # New mailbox
       #
       # @return [Mailbox]
       #
       # @api private
-      #
       def self.new
         mutex              = Mutex.new
         condition_variable = ConditionVariable.new
@@ -21,14 +20,13 @@ module Mutant
         )
       end
 
-      # Return binding for RPC to other actors
+      # Binding for RPC to other actors
       #
       # @param [Actor::Sender] other
       #
       # @return [Binding]
       #
       # @api private
-      #
       def bind(other)
         Binding.new(self, other)
       end

data/lib/mutant/actor/receiver.rb

@@ -9,7 +9,6 @@ module Mutant
       # @return [Object]
       #
       # @api private
-      #
       def call
         2.times do
           message = try_blocking_receive
@@ -29,7 +28,6 @@ module Mutant
       #   if there is a message
       #
       # @api private
-      #
       def try_blocking_receive
         mutex.synchronize do
           if messages.empty?

data/lib/mutant/actor/sender.rb

@@ -12,7 +12,6 @@ module Mutant
       # @return [self]
       #
       # @api private
-      #
       def call(message)
         mutex.synchronize do
           messages << message

data/lib/mutant/ast.rb

@@ -2,32 +2,6 @@ module Mutant
   # AST helpers
   module AST
 
-    # Walk all ast nodes
-    #
-    # @param [Parser::AST::Node] root
-    # @param [Array<Parser::AST::Node>] stack
-    #
-    # @yield [Parser::AST::Node]
-    #   all nodes recursively including root
-    #
-    # @return [self]
-    #
-    # @api private
-    #
-    def self.walk(node, stack, &block)
-      fail ArgumentError, 'block expected' unless block_given?
-
-      block.call(node, stack)
-      node.children.grep(Parser::AST::Node).each do |child|
-        stack.push(child)
-        walk(child, stack, &block)
-        stack.pop
-      end
-
-      self
-    end
-    private_class_method :walk
-
     # Find last node satisfying predicate (as block)
     #
     # @return [Parser::AST::Node]
@@ -42,17 +16,37 @@ module Mutant
     #   otherwise
     #
     # @api private
-    #
     def self.find_last_path(node, &predicate)
       fail ArgumentError, 'block expected' unless block_given?
       path = []
       walk(node, [node]) do |candidate, stack|
-        if predicate.call(candidate, &predicate)
+        if predicate.call(candidate)
           path = stack.dup
         end
       end
       path
     end
 
+    # Walk all ast nodes keeping track of path
+    #
+    # @param [Parser::AST::Node] root
+    # @param [Array<Parser::AST::Node>] stack
+    #
+    # @yield [Parser::AST::Node]
+    #   all nodes visited recursively including root
+    #
+    # @return [undefined]
+    #
+    # @api private
+    def self.walk(node, stack, &block)
+      block.call(node, stack)
+      node.children.grep(Parser::AST::Node) do |child|
+        stack.push(child)
+        walk(child, stack, &block)
+        stack.pop
+      end
+    end
+    private_class_method :walk
+
   end # AST
 end # Mutant

data/lib/mutant/ast/meta.rb

@@ -3,65 +3,27 @@ module Mutant
     # Node meta information mixin
     module Meta
 
-      REGISTRY = {}
-
-      # Return meta for node
-      #
-      # @param [Parser::AST::Node] node
-      #
-      # @return [Meta]
-      #
-      # @api private
-      #
-      def self.for(node)
-        REGISTRY.fetch(node.type, Generic).new(node)
-      end
-
-      # Mixin to define meta nodes
-      class Node < Module
-        include Concord.new(:type)
-
-        CONCORD = Concord.new(:node)
-
-        # Hook called when module gets included
-        #
-        # @param [Class, Module] host
-        #
-        # @return [undefined]
-        #
-        # @api private
-        #
-        def included(host)
-          REGISTRY[type] = host
-          host.class_eval do
-            include CONCORD, NamedChildren
-          end
-        end
-
-      end # Node
-
       # Metadata for resbody nods
       class Resbody
-        include Node.new(:resbody)
+        include NamedChildren, Concord.new(:node)
 
         children :captures, :assignment, :body
       end # Resbody
 
       # Metadata for send nodes
       class Send
-        include Node.new(:send)
+        include NamedChildren, Concord.new(:node)
 
         children :receiver, :selector
 
         INDEX_ASSIGNMENT_SELECTOR            = :[]=
         ATTRIBUTE_ASSIGNMENT_SELECTOR_SUFFIX = '='.freeze
 
-        # Return arguments
+        # Arguments of mutated node
         #
         # @return [Enumerable<Parser::AST::Node>]
         #
         # @api private
-        #
         alias_method :arguments, :remaining_children
 
         # Test if AST node is a valid assignment target
@@ -69,7 +31,6 @@ module Mutant
         # @return [Boolean]
         #
         # @api private
-        #
         def assignment?
           index_assignment? || attribute_assignment?
         end
@@ -79,9 +40,9 @@ module Mutant
         # @return [Boolean]
         #
         # @api private
-        #
         def attribute_assignment?
-          arguments.one? && attribute_assignment_selector?
+          !Types::METHOD_OPERATORS.include?(selector) &&
+          selector.to_s.end_with?(ATTRIBUTE_ASSIGNMENT_SELECTOR_SUFFIX)
         end
 
         # Test if AST node is an index assign
@@ -89,9 +50,8 @@ module Mutant
         # @return [Boolean]
         #
         # @api private
-        #
         def index_assignment?
-          arguments.length.equal?(2) && index_assignment_selector?
+          selector.equal?(INDEX_ASSIGNMENT_SELECTOR)
         end
 
         # Test for binary operator implemented as method
@@ -99,51 +59,11 @@ module Mutant
         # @return [Boolean]
         #
         # @api private
-        #
         def binary_method_operator?
-          arguments.one? && Types::BINARY_METHOD_OPERATORS.include?(selector)
-        end
-
-      private
-
-        # Test for index assignment operator
-        #
-        # @return [Boolean]
-        #
-        # @api private
-        #
-        def index_assignment_selector?
-          selector.equal?(INDEX_ASSIGNMENT_SELECTOR)
-        end
-
-        # Test for attribute assignment selector
-        #
-        # @return [Boolean]
-        #
-        # @api private
-        #
-        def attribute_assignment_selector?
-          !Types::METHOD_OPERATORS.include?(selector) && selector.to_s.end_with?(ATTRIBUTE_ASSIGNMENT_SELECTOR_SUFFIX)
+          Types::BINARY_METHOD_OPERATORS.include?(selector)
         end
 
       end # Send
-
-      # Generic node metadata
-      class Generic
-        include Adamantium, Concord.new(:node)
-
-        # Test if AST node is a valid assign target
-        #
-        # @return [Boolean]
-        #
-        # @api private
-        #
-        def assignment?
-          Types::ASSIGNABLE_VARIABLES.include?(node.type)
-        end
-
-      end # Generic
-
-    end #
+    end # Meta
   end # AST
 end # Mutant

data/lib/mutant/ast/named_children.rb

@@ -11,10 +11,7 @@ module Mutant
       # @return [undefined]
       #
       # @api private
-      #
       def self.included(host)
-        super
-
         host.class_eval do
           include InstanceMethods
           extend ClassMethods
@@ -26,12 +23,11 @@ module Mutant
 
       private
 
-        # Return children
+        # Mutated nodes children
         #
         # @return [Array<Parser::AST::Node]
         #
         # @api private
-        #
         def children
           node.children
         end
@@ -51,7 +47,6 @@ module Mutant
         # @return [undefined]
         #
         # @api private
-        #
         def define_named_child(name, index)
           define_method(name) do
             children.at(index)
@@ -65,7 +60,6 @@ module Mutant
         # @return [undefined]
         #
         # @api private
-        #
         def define_remaining_children(names)
           define_method(:remaining_children_with_index) do
             children.each_with_index.drop(names.length)
@@ -85,7 +79,6 @@ module Mutant
         # @return [undefined]
         #
         # @api private
-        #
         def children(*names)
           names.each_with_index do |name, index|
             define_named_child(name, index)