mutant 0.3.0.beta4 → 0.3.0.beta5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 266262896da3866b48b440ae556a6510a15b351a
-  data.tar.gz: e646685cea23129d46ead4f476940c4b37f27d36
+  metadata.gz: f3ab83b3fe3623d77e0bc897128c0790b6ebcd13
+  data.tar.gz: aabf8cc62a8349f0891cbcca381c682d18055384
 SHA512:
-  metadata.gz: 11e1b5542cc90aa0333a47908a1dee8fe04a6c2ef49df2da4c140a6a273af7e13c868fb81a9f1a1866133b4308a4fe6bc672619a58fec895d948d95485ec8abb
-  data.tar.gz: 7982b06a5e1276a0f64f04bda3009bc986e0b59dcbe94f032251d6a0944d13115d70a35b198e540f458274d5a13fe968c02b7207ad0783212d36b8f59b888f2c
+  metadata.gz: e816a8fd75024969b7a286d4b0e10b122526a768c5746f3615ac4b341d6538794195be8d2dff53f97fab916632e26111edd9156050787642f4197f131c654a0c
+  data.tar.gz: 393035855ab0e2d9b81203e221ae238edf54e92af5b911a290b04ef086fa7c4b7ba4757621141eb442f184bb80686c9f9ecf6ddce1455d717753584a48b28b21

data/.travis.yml

@@ -10,7 +10,7 @@ rvm:
 matrix:
   allow_failures:
     - rvm: jruby-19mode # No fork(2) support, workaround planned
-    - rvm: ruby-head # Broken at this time
+    - rvm: ruby-head # Broken at this time, bundler issue on travis
     - rvm: rbx-19mode # Broken at this time, yard/yardstick issue
 notifications:
   irc:

data/README.md

@@ -10,8 +10,8 @@ Mutant is a mutation testing tool for ruby. It aims to be better than existing m
 The idea is that if code can be changed and your tests do not notice, either that code isn't being covered
 or it does not have a speced side effect.
 
-Mutant supports MRI and RBX gte 19-mode (1.9.3!), while support for jruby is planned. It should also work under any ruby
-engine that supports POSIX-fork(2) semantics.
+Mutant supports MRI and RBX 1.9 and 2.0, while support for jruby is planned. It should also work under
+any ruby engine that supports POSIX-fork(2) semantics.
 
 Only rspec2 is supported currently. This is subject to change.
 
@@ -27,8 +27,8 @@ The following projects adopted mutant, and aim 100% mutation coverage:
 
 * [axiom](https://github.com/dkubb/axiom)
 * [axiom-types](https://github.com/dkubb/axiom-types)
-* [rom-mapper](https://github.com/rom/rom-mapper)
-* [rom-session](https://github.com/rom/rom-session)
+* [rom-mapper](https://github.com/rom-rb/rom-mapper)
+* [rom-session](https://github.com/rom-rb/rom-session)
 * [event_bus](https://github.com/kevinrutherford/event_bus)
 * [virtus](https://github.com/solnic/virtus)
 * [quacky](https://github.com/benmoss/quacky)

data/TODO

@@ -1,12 +1,7 @@
 Code:
   * Test mutant with dynamically created zombie.
-  * Fix ugly code within default parameter mutations
   * Break up lib/mutant/mutator/node/send.rb in class specific files
   * Log all warnings through reporter, so remove random $stderr.puts calls
-  * Use rational to represent percent values
-
-AST:
-  * Use whitequarcks parser and the to be writte unparser. This fixes all RBX AST issues.
 
 Mutations:
   * Add binary operator specific mutations (YAY, finally reached this point)
@@ -34,9 +29,6 @@ Killers:
   * Add a general master <=> killer IPC interface. So different strategies of isolation
     (fork, vs jruby runtime creation) will work without big impact.
 
-Kill:
-  * Introduce mutant Kill that acts as a DTO between killer and runner. (Simplifies reporting)
-
 Strategy:
   * Aggregate warnings on missing spec files
   * Provide "expicit files to kill with" strategy
@@ -46,5 +38,4 @@ Matcher:
   * Allow matches on define_method with literal name argument?
 
 jruby-support:
-  * Use whitequarck parser
   * Create a runtime per mutation to kill mutations in isolation

data/bin/mutant

@@ -9,8 +9,8 @@ require 'mutant'
 namespace =
   if File.basename($0) == 'zombie'
     $stderr.puts('Detected zombie environment...')
-    require File.expand_path('../../spec/support/zombie.rb', __FILE__)
-    Zombie.setup
+    Mutant::Zombifier.zombify
+    Zombie::Mutant
   else
     Mutant
   end

data/bin/zombie

@@ -9,8 +9,8 @@ require 'mutant'
 namespace =
   if File.basename($0) == 'zombie'
     $stderr.puts('Detected zombie environment...')
-    require File.expand_path('../../spec/support/zombie.rb', __FILE__)
-    Zombie.setup
+    Mutant::Zombifier.zombify
+    Zombie::Mutant
   else
     Mutant
   end

data/config/flay.yml

@@ -1,3 +1,3 @@
 ---
 threshold: 16
-total_score: 604
+total_score: 616

data/config/reek.yml

@@ -58,14 +58,13 @@ RepeatedConditional:
   exclude:
   - Mutant::Mutator
   - Mutant::Reporter::CLI
-  - Mutant::Mutator::Node::Send
-  - Mutant::Mutator::Node::If
   max_ifs: 1
 TooManyInstanceVariables:
   enabled: true
   exclude:
   - Mutant::CLI     # 4 vars
   - Mutant::Killer  # 4 vars
+  - Mutant::Mutator # 4 vars
   max_instance_variables: 3
 TooManyMethods:
   enabled: true

data/lib/mutant.rb

@@ -22,6 +22,7 @@ require 'concord'
 module Mutant
 end
 
+require 'mutant/cache'
 require 'mutant/node_helpers'
 require 'mutant/singleton_methods'
 require 'mutant/constants'
@@ -77,6 +78,7 @@ require 'mutant/subject/method'
 require 'mutant/matcher'
 require 'mutant/matcher/chain'
 require 'mutant/matcher/method'
+require 'mutant/matcher/method/finder'
 require 'mutant/matcher/method/singleton'
 require 'mutant/matcher/method/instance'
 require 'mutant/matcher/methods'
@@ -112,3 +114,4 @@ require 'mutant/reporter/cli/printer/config'
 require 'mutant/reporter/cli/printer/subject'
 require 'mutant/reporter/cli/printer/killer'
 require 'mutant/reporter/cli/printer/mutation'
+require 'mutant/zombifier'

data/lib/mutant/cache.rb

@@ -0,0 +1,30 @@
+module Mutant
+  # An AST cache
+  class Cache
+    # This is explicitly empty! Ask me if you are interested in reasons :D
+    include Equalizer.new
+
+    # Initialize object
+    #
+    # @return [undefined]
+    #
+    # @api private
+    #
+    def initialize
+      @cache = {}
+    end
+
+    # Return node for file
+    #
+    # @return [AST::Node]
+    #
+    # @api private
+    #
+    def parse(path)
+      @cache.fetch(path) do
+        @cache[path] = Parser::CurrentRuby.parse(File.read(path))
+      end
+    end
+
+  end # Cache
+end # Mutant

data/lib/mutant/cli.rb

@@ -41,6 +41,8 @@ module Mutant
     def initialize(arguments=[])
       @filters, @matchers = [], []
 
+      @cache = Mutant::Cache.new
+
       parse(arguments)
       strategy
       matcher
@@ -54,6 +56,7 @@ module Mutant
     #
     def config
       Config.new(
+        :cache    => @cache,
         :debug    => debug?,
         :matcher  => matcher,
         :filter   => filter,
@@ -212,7 +215,7 @@ module Mutant
     #
     def parse_matchers(patterns)
       patterns.each do |pattern|
-        matcher = Classifier.build(pattern)
+        matcher = Classifier.build(@cache, pattern)
         @matchers << matcher if matcher
       end
     end

data/lib/mutant/cli/classifier.rb

@@ -2,10 +2,13 @@ module Mutant
   class CLI
     # A classifier for input strings
     class Classifier < Matcher
-      include AbstractType, Adamantium::Flat, Equalizer.new(:identification)
+      include AbstractType, Adamantium::Flat, Concord.new(:cache, :match)
+
+      include Equalizer.new(:identifier)
 
       SCOPE_NAME_PATTERN  = /[A-Za-z][A-Za-z_0-9]*/.freeze
-      METHOD_NAME_PATTERN = /[_A-Za-z][A-Za-z0-9_]*[!?=]?/.freeze
+      OPERATOR_PATTERN    = Regexp.union(*OPERATOR_METHODS.map(&:to_s)).freeze
+      METHOD_NAME_PATTERN = /([_A-Za-z][A-Za-z0-9_]*[!?=]?|#{OPERATOR_PATTERN})/.freeze
       SCOPE_PATTERN       = /(?:::)?#{SCOPE_NAME_PATTERN}(?:::#{SCOPE_NAME_PATTERN})*/.freeze
 
       SINGLETON_PATTERN   = %r(\A(#{SCOPE_PATTERN})\z).freeze
@@ -32,15 +35,13 @@ module Mutant
       # @api private
       #
       def self.constant_lookup(location)
-        location.gsub(%r(\A::), '').split('::').inject(::Object) do |parent, name|
+        location.gsub(%r(\A::), '').split('::').inject(Object) do |parent, name|
           parent.const_get(name)
         end
       end
 
       # Return matchers for input
       #
-      # @param [String] input
-      #
       # @return [Classifier]
       #   if a classifier handles the input
       #
@@ -49,9 +50,9 @@ module Mutant
       #
       # @api private
       #
-      def self.build(input)
+      def self.build(*arguments)
         classifiers = REGISTRY.map do |descendant|
-          descendant.run(input)
+          descendant.run(*arguments)
         end.compact
 
         raise if classifiers.length > 1
@@ -61,6 +62,10 @@ module Mutant
 
       # Run classifier
       #
+      # @param [Cache] cache
+      #
+      # @param [String] input
+      #
       # @return [Classifier]
       #   if input is handled by classifier
       #
@@ -69,11 +74,11 @@ module Mutant
       #
       # @api private
       #
-      def self.run(input)
+      def self.run(cache, input)
         match = self::REGEXP.match(input)
         return unless match
 
-        new(match)
+        new(cache, match)
       end
 
       # No protected_class_method in ruby :(
@@ -95,38 +100,16 @@ module Mutant
         self
       end
 
-      # Return identification
+      # Return identifier
       #
       # @return [String]
       #
       # @api private
       #
-      def identification
+      def identifier
         match.to_s
       end
-      memoize :identification
-
-    private
-
-      # Initialize object
-      #
-      # @param [MatchData] match
-      #
-      # @return [undefined]
-      #
-      # @api private
-      #
-      def initialize(match)
-        @match = match
-      end
-
-      # Return match
-      #
-      # @return [MatchData]
-      #
-      # @api private
-      #
-      attr_reader :match
+      memoize :identifier
 
       # Return matcher
       #

data/lib/mutant/cli/classifier/method.rb

@@ -24,7 +24,7 @@ module Mutant
         # @api private
         #
         def matcher
-          methods_matcher.matcher.new(scope, method)
+          methods_matcher.matcher.new(cache, scope, method)
         end
         memoize :matcher
 
@@ -90,7 +90,7 @@ module Mutant
         # @api private
         #
         def methods_matcher
-          TABLE.fetch(scope_symbol).new(scope)
+          TABLE.fetch(scope_symbol).new(cache, scope)
         end
         memoize :methods_matcher
 

data/lib/mutant/cli/classifier/namespace.rb

@@ -14,7 +14,7 @@ module Mutant
         # @api private
         #
         def matcher
-          self.class::MATCHER.new(namespace)
+          self.class::MATCHER.new(cache, namespace)
         end
 
         # Return namespace

data/lib/mutant/config.rb

@@ -2,7 +2,7 @@ module Mutant
   # The configuration of a mutator run
   class Config
     include Adamantium::Flat, Anima.new(
-      :debug, :strategy, :matcher, :filter, :reporter
+      :cache, :debug, :strategy, :matcher, :filter, :reporter
     )
 
     # Enumerate subjects

data/lib/mutant/constants.rb

@@ -44,6 +44,8 @@ module Mutant
 
   BINARY_METHOD_OPERATORS = (OPERATOR_EXPANSIONS.keys - (INDEX_OPERATORS + UNARY_METHOD_OPERATORS)).to_set.freeze
 
+  OPERATOR_METHODS = OPERATOR_EXPANSIONS.keys + INDEX_OPERATORS + UNARY_METHOD_OPERATORS
+
   # Hopefully all types parser does generate
   NODE_TYPES = [
     :lvasgn, :ivasgn, :cvasgn, :gvasgn,

data/lib/mutant/differ.rb

@@ -3,6 +3,9 @@ module Mutant
   class Differ
     include Adamantium::Flat
 
+    FORMAT = :unified
+    CONTEXT_LINES = 3
+
     # Return source diff
     #
     # @return [String]
@@ -35,9 +38,6 @@ module Mutant
 
   private
 
-    FORMAT = :unified
-    CONTEXT_LINES = 3
-
     # Initialize differ object
     #
     # @param [String] old

data/lib/mutant/loader.rb

@@ -40,7 +40,7 @@ module Mutant
       # @api private
       #
       def run
-        eval(source, TOPLEVEL_BINDING, @subject.source_path, @subject.source_line)
+        eval(source, TOPLEVEL_BINDING, @subject.source_path.to_s, @subject.source_line)
       end
 
       # Return source

data/lib/mutant/matcher.rb

@@ -15,10 +15,10 @@ module Mutant
     #
     # @api private
     #
-    def self.each(input, &block)
-      return to_enum(__method__, input) unless block_given?
+    def self.each(cache, input, &block)
+      return to_enum(__method__, cache, input) unless block_given?
 
-      new(input).each(&block)
+      new(cache, input).each(&block)
 
       self
     end

data/lib/mutant/matcher/method.rb

@@ -2,7 +2,7 @@ module Mutant
   class Matcher
     # Matcher for subjects that are a specific method
     class Method < self
-      include Adamantium::Flat, Concord::Public.new(:scope, :method)
+      include Adamantium::Flat, Concord::Public.new(:cache, :scope, :method)
 
       # Methods within rbx kernel directory are precompiled and their source
       # cannot be accessed via reading source location
@@ -18,13 +18,12 @@ module Mutant
       #
       # @api private
       #
-      def each(&block)
+      def each
         return to_enum unless block_given?
 
-        return self if skip?
-
-        util = subject
-        yield util if util
+        unless skip?
+          yield subject if subject
+        end
 
         self
       end
@@ -78,7 +77,7 @@ module Mutant
       # @api private
       #
       def ast
-        Parser::CurrentRuby.parse(File.read(source_path))
+        cache.parse(source_path)
       end
 
       # Return path to source
@@ -128,72 +127,6 @@ module Mutant
       end
       memoize :subject
 
-      # Visitor to find last match inside AST
-      class Finder
-
-        # Run finder
-        #
-        # @param [Parser::AST::Node]
-        #
-        # @return [Parser::AST::Node]
-        #   if found
-        #
-        # @return [nil]
-        #   otherwise
-        #
-        # @api private
-        #
-        #
-        def self.run(root, &predicate)
-          new(root, predicate).match
-        end
-
-        private_class_method :new
-
-        # Return match
-        #
-        # @return [Parser::AST::Node]
-        #
-        # @api private
-        #
-        attr_reader :match
-
-      private
-
-        # Initialize object
-        #
-        # @param [Parer::AST::Node]
-        #
-        # @return [undefined]
-        #
-        # @api private
-        #
-        #
-        def initialize(root, predicate)
-          @root, @predicate = root, predicate
-          visit(root)
-        end
-
-        # Visit node
-        #
-        # @param [Parser::AST::Node] node
-        #
-        # @return [undefined]
-        #
-        # @api private
-        #
-        def visit(node)
-          if @predicate.call(node)
-            @match = node
-          end
-
-          node.children.each do |child|
-            visit(child) if child.kind_of?(Parser::AST::Node)
-          end
-        end
-
-      end # Finder
-
       # Return matched node
       #
       # @return [Parser::AST::Node]