mutant 0.3.0.beta22 → 0.3.0.rc1

data/lib/mutant/matcher/method.rb

@@ -54,16 +54,16 @@ module Mutant
       #
       def skip?
         location = source_location
-        if location.nil? or BLACKLIST.match(location.first)
+        if location.nil? || BLACKLIST.match(location.first)
           message = sprintf(
             '%s does not have valid source location unable to emit matcher',
             method.inspect
           )
           $stderr.puts(message)
-          return true
+          true
+        else
+          false
         end
-
-        false
       end
 
       # Return method name

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

@@ -15,7 +15,7 @@ module Mutant
           :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,
+          :alias, :for, :xstr, :back_ref, :class,
           :sclass, :match_with_lvasgn, :match_current_line, :or_asgn, :kwbegin
         )
 

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

@@ -15,6 +15,8 @@ module Mutant
           #
           # @return [undefined]
           #
+          # @api private
+          #
           def dispatch
             emit_nil
             emit_values(values)

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

@@ -0,0 +1,29 @@
+# encoding: utf-8
+
+module Mutant
+  class Mutator
+    class Node
+      # Mutator for nth-ref nodes
+      class NthRef < self
+
+        handle :nth_ref
+
+        children :number
+
+      private
+
+        # Perform dispatch
+        #
+        # @return [undefined]
+        #
+        # @api private
+        #
+        def dispatch
+          emit_number(number - 1)
+          emit_number(number + 1)
+        end
+
+      end # NthRef
+    end # Node
+  end # Mutator
+end # Mutant

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

@@ -11,6 +11,11 @@ module Mutant
 
         children :receiver, :selector
 
+        SELECTOR_REPLACEMENTS = {
+          :send => :public_send,
+          :gsub => :sub
+        }.freeze
+
         INDEX_REFERENCE = :[]
         INDEX_ASSIGN    = :[]=
         ASSIGN_SUFFIX   = :'='
@@ -98,11 +103,23 @@ module Mutant
         #
         def normal_dispatch
           emit_naked_receiver
+          emit_selector_mutations
           mutate_receiver
           emit_argument_propagation
           mutate_arguments
         end
 
+        # Emit selector mutations
+        #
+        # @return [undefined]
+        #
+        # @api private
+        #
+        def emit_selector_mutations
+          replacement = SELECTOR_REPLACEMENTS.fetch(selector) { return }
+          emit_selector(replacement)
+        end
+
         # Emit naked receiver mutation
         #
         # @return [undefined]

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

@@ -10,6 +10,9 @@ module Mutant
 
           handle(Mutant::Killer::Forked)
 
+          SUCCESS = '.'.freeze
+          FAILURE = 'F'.freeze
+
           # Run printer
           #
           # @return [undefined]
@@ -18,12 +21,14 @@ module Mutant
           #
           def run
             if success?
-              char('.', Color::GREEN)
-              return
+              char(SUCCESS, Color::GREEN)
+            else
+              char(FAILURE, Color::RED)
             end
-            char('F', Color::RED)
           end
 
+        private
+
           # Write colorized char
           #
           # @param [String] char
@@ -37,6 +42,7 @@ module Mutant
             output.write(colorize(color, char))
             output.flush
           end
+
         end # Killer
       end # Printer
     end # CLI

data/lib/mutant/strategy.rb

@@ -6,24 +6,22 @@ module Mutant
   class Strategy
     include AbstractType, Adamantium::Flat
 
-    # Perform setup
+    # Perform strategy setup
     #
     # @return [self]
     #
     # @api private
     #
-    def self.setup
-      self
+    def setup
     end
 
-    # Perform teardown
+    # Perform strategy teardown
     #
     # @return [self]
     #
     # @api private
     #
-    def self.teardown
-      self
+    def teardown
     end
 
     # Kill mutation
@@ -34,18 +32,20 @@ module Mutant
     #
     # @api private
     #
-    def self.kill(mutation)
+    def kill(mutation)
       killer.new(self, mutation)
     end
 
+  private
+
     # Return killer
     #
     # @return [Class:Killer]
     #
     # @api private
     #
-    def self.killer
-      self::KILLER
+    def killer
+      self.class::KILLER
     end
 
   end # Strategy

data/lib/mutant/strategy/rspec.rb

@@ -2,9 +2,9 @@
 
 module Mutant
   class Strategy
-
-    # Rspec strategy base class
+    # Rspec killer strategy
     class Rspec < self
+      include Equalizer.new
 
       KILLER = Killer::Forking.new(Killer::Rspec)
 
@@ -14,52 +14,62 @@ module Mutant
       #
       # @api private
       #
-      def self.setup
-        require('./spec/spec_helper.rb')
+      def setup
+        output = StringIO.new
+        configuration.error_stream = output
+        configuration.output_stream = output
+        options.configure(configuration)
+        configuration.load_spec_files
         self
       end
+      memoize :setup
 
-      # Run all unit specs per mutation
-      class Unit < self
-
-        # Return file name pattern for mutation
-        #
-        # @return [Enumerable<String>]
-        #
-        # @api private
-        #
-        def self.spec_files(_mutation)
-          Dir['spec/unit/**/*_spec.rb']
-        end
-      end # Unit
+      # Return configuration
+      #
+      # @return [RSpec::Core::Configuration]
+      #
+      # @api private
+      #
+      def configuration
+        RSpec::Core::Configuration.new
+      end
+      memoize :configuration, :freezer => :noop
 
-      # Run all integration specs per mutation
-      class Integration < self
+      # Return example groups
+      #
+      # @return [Enumerable<RSpec::Core::ExampleGroup>]
+      #
+      # @api private
+      #
+      def example_groups
+        world.example_groups
+      end
 
-        # Return file name pattern for mutation
-        #
-        # @return [Mutation]
-        #
-        # @api private
-        #
-        def self.spec_files(_mutation)
-          Dir['spec/integration/**/*_spec.rb']
-        end
-      end # Integration
+    private
 
-      # Run all specs per mutation
-      class Full < self
+      # Return world
+      #
+      # @return [RSpec::Core::World]
+      #
+      # @api private
+      #
+      def world
+        RSpec.world
+      end
+      memoize :world, :freezer => :noop
 
-        # Return spec files
-        #
-        # @return [Enumerable<String>]
-        #
-        # @api private
-        #
-        def self.spec_files(_mutation)
-          Dir['spec/**/*_spec.rb']
-        end
-      end # Full
+      # Return options
+      #
+      # @return [RSpec::Core::ConfigurationOptions]
+      #
+      # @api private
+      #
+      def options
+        options = RSpec::Core::ConfigurationOptions.new(%w(--fail-fast spec))
+        options.parse_options
+        options
+      end
+      memoize :options, :freezer => :noop
 
     end # Rspec
   end # Strategy

data/lib/mutant/subject.rb

@@ -46,7 +46,7 @@ module Mutant
     # @api private
     #
     def identification
-      "#{subtype}:#{source_path}:#{source_line}"
+      "#{match_expression}:#{source_path}:#{source_line}"
     end
     memoize :identification
 
@@ -84,16 +84,26 @@ module Mutant
     end
     memoize :original_root
 
-  private
-
-    # Return subtype identifier
+    # Return match expression
     #
     # @return [String]
     #
     # @api private
     #
-    abstract_method :subtype
-    private :subtype
+    abstract_method :match_expression
+
+    # Return match prefixes
+    #
+    # @return [Enumerable<String>]
+    #
+    # @api private
+    #
+    def match_prefixes
+      [match_expression].concat(context.match_prefixes)
+    end
+    memoize :match_prefixes
+
+  private
 
     # Return neutral mutation
     #

data/lib/mutant/subject/method.rb

@@ -27,6 +27,16 @@ module Mutant
         node.children[self.class::NAME_INDEX]
       end
 
+      # Return match expression
+      #
+      # @return [String]
+      #
+      # @api private
+      #
+      def match_expression
+        "#{context.identification}#{self.class::SYMBOL}#{name}"
+      end
+
     private
 
       # Return mutations
@@ -54,16 +64,6 @@ module Mutant
         context.scope
       end
 
-      # Return subtype identifier
-      #
-      # @return [String]
-      #
-      # @api private
-      #
-      def subtype
-        "#{context.identification}#{self.class::SYMBOL}#{name}"
-      end
-
     end # Method
   end # Subject
 end # Mutant

data/lib/mutant/version.rb

@@ -0,0 +1,7 @@
+# encoding: utf-8
+
+module Mutant
+  # The current mutant version
+  VERSION = '0.3.0.rc1'.freeze
+end # Mutant
+

data/lib/mutant/zombifier.rb

@@ -151,13 +151,12 @@ module Mutant
 
         $LOAD_PATH.each do |path|
           path = Pathname.new(path).join(file_name)
-          if path.exist?
-            $stderr.puts "Loading #{path}"
+          if path.file?
             return new(path)
           end
         end
 
-        $stderr.puts "Cannot find #{file_name} in $LOAD_PATH"
+        $stderr.puts "Cannot find file #{file_name} in $LOAD_PATH"
         nil
       end
 

data/mutant.gemspec

@@ -1,8 +1,10 @@
 # encoding: utf-8
+#
+require File.expand_path('../lib/mutant/version', __FILE__)
 
 Gem::Specification.new do |gem|
   gem.name        = 'mutant'
-  gem.version     = '0.3.0.beta22'
+  gem.version     = Mutant::VERSION.dup
   gem.authors     = ['Markus Schirp']
   gem.email       = ['mbj@schirp-dso.com']
   gem.description = 'Mutation testing for ruby'
@@ -16,15 +18,15 @@ Gem::Specification.new do |gem|
   gem.extra_rdoc_files = %w[TODO LICENSE]
   gem.executables      = %w[mutant]
 
-  gem.add_runtime_dependency('parser',              '~> 2.0.0.pre3')
-  gem.add_runtime_dependency('unparser',            '~> 0.0.13')
+  gem.add_runtime_dependency('parser',              '~> 2.0.0.pre6')
+  gem.add_runtime_dependency('unparser',            '~> 0.0.14')
   gem.add_runtime_dependency('ice_nine',            '~> 0.8.0')
   gem.add_runtime_dependency('descendants_tracker', '~> 0.0.1')
   gem.add_runtime_dependency('adamantium',          '~> 0.0.10')
   gem.add_runtime_dependency('equalizer',           '~> 0.0.5')
   gem.add_runtime_dependency('inflecto',            '~> 0.0.2')
   gem.add_runtime_dependency('anima',               '~> 0.0.6')
-  gem.add_runtime_dependency('concord',             '~> 0.1.1')
+  gem.add_runtime_dependency('concord',             '~> 0.1.3')
   gem.add_runtime_dependency('rspec',               '~> 2.14.1')
 
   gem.add_development_dependency('bundler', '~> 1.3', '>= 1.3.5')

data/spec/integration/mutant/rspec_spec.rb

@@ -0,0 +1,26 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Mutant, 'rspec integration' do
+
+  around do |example|
+    Dir.chdir(TestApp.root) do
+      example.run
+    end
+  end
+
+  specify 'it allows to kill mutations' do
+    Kernel.system('bundle exec mutant --rspec ::TestApp::Literal#string').should be(true)
+  end
+
+  pending 'fails to kill mutations when they are not covered' do
+    cli = 'bundle exec mutant --rspec ::TestApp::Literal#uncovered_string'
+    Kernel.system(cli).should be(false)
+  end
+
+  pending 'fails when some mutations when are not covered' do
+    cli = 'bundle exec mutant --rspec ::TestApp::Literal'
+    Kernel.system(cli).should be(false)
+  end
+end