mutant 0.2.4 → 0.2.5

data/.travis.yml

@@ -1,16 +1,16 @@
 before_install:
-  - gem install bundler
+  - gem install mbj-inflector
 language: ruby
 script: 'bundle exec rake spec'
 rvm:
   - 1.8.7
   - 1.9.2
   - 1.9.3
-  - rbx-18mode
   - rbx-19mode
 matrix:
   allow_failures:
-    - rvm: rbx-19mode
+    # No mutators for 1.8 specifc AST nodes
+    - rvm: rbx-18mode  
     - rvm: 1.8.7
 notifications:
   email:

data/Changelog.md

@@ -1,11 +1,32 @@
+# v0.2.5 xxxx
+
+* [feature] Add --debug flag for showing killer output and mutation
+* [feature] Run noop mutation per subject to guard against initial failing specs
+* [feature] Mutate default into required arguments
+* [feature] Mutate default literals
+* [feature] Mutate unwinding of pattern args |(a, b), c] => |a, b, c|
+* [feature] Mutate define and block arguments
+* [feature] Mutate block arguments, inklusive pattern args
+* [feature] Recurse into block bodies
+* [change] Unvendor inflector use mbj-inflector from rubygems
+* [fixed] Insert mutations at correct constant scope
+* [fixed] Crash on mutating yield, added a noop for now
+* [fixed] Crash on singleton methods defined on other than constants or self
+
+[Compare v0.2.4..v0.2.5](https://github.com/mbj/mutant/compare/v0.2.4...v0.2.5)
+
 # v0.2.4 2012-12-12
 
 * [fixed] Correctly vendor inflector
 
+[Compare v0.2.3..v0.2.4](https://github.com/mbj/mutant/compare/v0.2.3...v0.2.4)
+
 # v0.2.3 2012-12-08
 
 * [fixed] Prepend extra elements to hash and array instead of append. This fixes unkillable mutators in parallel assignments!
 
+[Compare v0.2.2..v0.2.3](https://github.com/mbj/mutant/compare/v0.2.2...v0.2.3)
+
 # v0.2.2 2012-12-07
 
 * [feature] Add a shitload of operator expansions for dm2 strategy

data/Gemfile.devtools

@@ -8,6 +8,7 @@ group :guard do
   gem 'guard',         '~> 1.5.4'
   gem 'guard-bundler', '~> 1.0.0'
   gem 'guard-rspec',   '~> 2.1.1'
+  gem 'rb-inotify', :git => 'https://github.com/mbj/rb-inotify'
 end
 
 group :benchmarks do

data/Guardfile

@@ -4,7 +4,7 @@ guard :bundler do
   watch('Gemfile')
 end
 
-guard :rspec, :all_on_start => false do
+guard :rspec, :all_on_start => false, :all_after_pass => false do
   # run all specs if the spec_helper or supporting files files are modified
   watch('spec/spec_helper.rb')                      { 'spec/unit' }
   watch(%r{\Aspec/(?:lib|support|shared)/.+\.rb\z}) { 'spec/unit' }

data/README.md

@@ -6,9 +6,14 @@ mutant
 [![Code Climate](https://codeclimate.com/badge.png)](https://codeclimate.com/github/mbj/mutant)
 
 Mutant is a mutation testing tool for ruby that aims to be better than existing mutation testers.
-The idea is that if code can be changed and your tests don't notice, either that code isn't being covered or it doesn't do anything.
 
-Mutant does currently only support 1.9 mode under rubinius or mri.
+The idea is that if code can be changed and your tests do not notice, either that code isn't being covered 
+or it doesn't do anything (useful).
+
+Mutant does currently only support 1.9 mode under rubinius or mri. It is a young project but already 
+used in the DataMapper-2.0 project.
+
+See this [ASCII-Cast](http://ascii.io/a/1707) for mutant in action! (v0.2.1)
 
 Installation
 ------------
@@ -19,10 +24,49 @@ Examples
 --------
 
 ```
-cd your_lib
-# Run mutant on virtus (that uses the dm-2 style spec layout)
+cd virtus
+# Run mutant on virtus namespace (that uses the dm-2 style spec layout)
 mutant -I lib -r virtus --rspec-dm2 ::Virtus
+# Run mutant on specific virtus class 
+mutant -I lib -r virtus --rspec-dm2 ::Virtus::Attribute
+# Run mutant on specific virtus class method
+mutant -I lib -r virtus --rspec-dm2 ::Virtus::Attribute.build
+# Run mutant on specific virtus instance method
+mutant -I lib -r virtus --rspec-dm2 ::Virtus::Attribute#name
+```
+
+Strategies
+----------
+
+Mutation testing is slow. To make it fast the selection of the correct set of tests to run is the key.
+Mutant currently supports the following buildin strategies:
+
+### --rspec-dm2
+
+This strategy is the *fastest* but requires discipline in spec file naming.
+
+The following specs are executed to kill a mutation on:
 ```
+Public instance  methods: spec/unit/#{namespace}/#{class_name}/#{method_name}_spec.rb
+Public singleton methods: spec/unit/#{namespace}/#{class_name}/class_methods/#{method_name}_spec.rb
+Public instance  methods: spec/unit/#{namespace}/#{class_name}/
+Public singleton methods: spec/unit/#{namespace}/#{class_name}/class_methods
+```
+
+### --rspec-unit
+
+This strategy executes all specs under ``./spec/unit`` for each mutation.
+
+### --rspec-integration
+
+This strategy executes all specs under ``./spec/integration`` for each mutation.
+
+### --rspec-full
+
+This strategy executes all specs under ``./spec`` for each mutation.
+
+It is also plannned to allow explicit selections on specs to run and to support other test frameworks.
+Custom project specific strategies are also on the roadmap.
 
 Credits
 -------

data/config/flay.yml

@@ -1,3 +1,3 @@
 ---
-threshold: 18
-total_score: 750
+threshold: 34 # Todo bring down to ~20
+total_score: 913

data/config/flog.yml

@@ -1,2 +1,2 @@
 ---
-threshold: 14.4
+threshold: 47.0

data/config/site.reek

@@ -11,6 +11,7 @@ LargeClass:
   max_methods: 10
   exclude: 
   - "Mutant::Matcher::Method" # 13 methods
+  - "Mutant::Reporter::CLI" # 16 methods TODO Reduce!
   enabled: true
   max_instance_variables: 3
 UncommunicativeMethodName:
@@ -40,7 +41,8 @@ IrresponsibleModule:
   exclude: []
   enabled: true
 UncommunicativeModuleName:
-  accept: []
+  accept: 
+  - Mutant::Strategy::Rspec::DM2
   exclude: []
   enabled: true
   reject:

data/lib/mutant.rb

@@ -6,13 +6,24 @@ require 'descendants_tracker'
 require 'securerandom'
 require 'equalizer'
 require 'digest/sha1'
+require 'inflector'
 require 'to_source'
 require 'ice_nine'
-require 'ice_nine/core_ext/object'
 require 'diff/lcs'
 require 'diff/lcs/hunk'
 require 'rspec'
 
+module IceNine
+  class Freezer
+    class Rubinius 
+      class AST < IceNine::Freezer::Object
+        class Node < IceNine::Freezer::Object
+        end
+      end
+    end
+  end
+end
+
 # Library namespace
 module Mutant
 
@@ -73,6 +84,8 @@ require 'mutant/mutator/node/send'
 require 'mutant/mutator/node/arguments'
 require 'mutant/mutator/node/define'
 require 'mutant/mutator/node/return'
+require 'mutant/mutator/node/local_variable_assignment'
+require 'mutant/mutator/node/iter_19'
 require 'mutant/mutator/node/if_statement'
 require 'mutant/mutator/node/receiver_case'
 require 'mutant/loader' 
@@ -102,4 +115,3 @@ require 'mutant/reporter'
 require 'mutant/reporter/stats'
 require 'mutant/reporter/null'
 require 'mutant/reporter/cli'
-require 'mutant/inflector'

data/lib/mutant/cli.rb

@@ -1,7 +1,7 @@
 module Mutant
   # Comandline parser
   class CLI
-    include Adamantium::Flat, Equalizer.new(:matcher, :filter, :killer)
+    include Adamantium::Flat, Equalizer.new(:matcher, :filter, :strategy, :reporter)
 
     # Error raised when CLI argv is inalid
     Error = Class.new(RuntimeError)
@@ -44,6 +44,20 @@ module Mutant
     end
     memoize :matcher
 
+    # Test for running in debug mode
+    #
+    # @return [true]
+    #   if debug mode is active
+    #
+    # @return [false]
+    #   otherwise
+    #
+    # @api private
+    #
+    def debug?
+      !!@debug
+    end
+
     # Return mutation filter
     #
     # @return [Mutant::Matcher]
@@ -67,7 +81,9 @@ module Mutant
     #
     def strategy
       @strategy || raise(Error, 'no strategy was set!')
+      @strategy.new(self)
     end
+    memoize :strategy
 
     # Return reporter
     #
@@ -76,7 +92,7 @@ module Mutant
     # @api private
     #
     def reporter
-      Mutant::Reporter::CLI.new($stdout)
+      Mutant::Reporter::CLI.new(self)
     end
     memoize :reporter
 
@@ -88,7 +104,8 @@ module Mutant
       '--include'            => [:add_load_path                                   ],
       '-r'                   => [:require_library                                 ],
       '--require'            => [:require_library                                 ],
-     #'--killer-fork'        => [:enable_killer_fork                              ],
+      '--debug'              => [:set_debug                                       ],
+      '-d'                   => [:set_debug                                       ],
       '--rspec-unit'         => [:set_strategy,         Strategy::Rspec::Unit     ],
       '--rspec-full'         => [:set_strategy,         Strategy::Rspec::Full     ],
       '--rspec-dm2'          => [:set_strategy,         Strategy::Rspec::DM2      ],
@@ -98,33 +115,6 @@ module Mutant
 
     OPTION_PATTERN = %r(\A-(?:-)?[a-zA-Z0-9\-]+\z).freeze
 
-    # Return selected killer
-    #
-    # @return [Killer]
-    #
-    # @api private
-    #
-    def selected_killer
-      unless @rspec
-        raise Error, "Only rspec is supported currently use --rspec switch"
-      end
-
-      Mutant::Killer::Rspec
-    end
-    memoize :selected_killer
-
-    # Return option for argument with index
-    #
-    # @param [Fixnum] index
-    #
-    # @return [String]
-    #
-    # @api private
-    #
-    def option(index)
-      @arguments.fetch(index+1)
-    end
-
     # Initialize CLI 
     #
     # @param [Array<String>] arguments
@@ -146,6 +136,18 @@ module Mutant
       matcher
     end
 
+    # Return option for argument with index
+    #
+    # @param [Fixnum] index
+    #
+    # @return [String]
+    #
+    # @api private
+    #
+    def option(index)
+      @arguments.fetch(index+1)
+    end
+
     # Return current argument
     #
     # @return [String]
@@ -233,7 +235,7 @@ module Mutant
 
     # Add mutation filter
     #
-    # @param [Class<Mutant::Filter>]
+    # @param [Class<Mutant::Filter>] klass
     #
     # @return [undefined]
     #
@@ -268,22 +270,20 @@ module Mutant
       @rspec = true
     end
 
-    # Enable killer forking
+    # Set debug mode
     #
     # @api private
     #
-    # @return [self]
-    #
-    # @api private
+    # @return [undefined]
     #
-    def enable_killer_fork
+    def set_debug
       consume(1)
-      @forking = true
+      @debug = true
     end
 
     # Set strategy
     #
-    # @param [Strategy]
+    # @param [Strategy] strategy
     #
     # @api private
     #
@@ -304,6 +304,5 @@ module Mutant
       require(current_option_value)
       consume(2)
     end
-
   end
 end

data/lib/mutant/context/scope.rb

@@ -2,54 +2,59 @@ module Mutant
   class Context
     # Scope context for mutation (Class or Module)
     class Scope < self
-      include Adamantium::Flat, AbstractType, Equalizer.new(:scope, :source_path)
+      include Adamantium::Flat, Equalizer.new(:scope, :source_path)
 
-      # Class context for mutation
-      class Class < self
-        SCOPE_CLASS = Rubinius::AST::ClassScope
-        KEYWORD     = 'class'.freeze
-      end
-
-      # Module context for mutation
-      class Module < self
-        SCOPE_CLASS = Rubinius::AST::ModuleScope
-        KEYWORD     = 'module'.freeze
+      # Return AST wrapping mutated node
+      #
+      # @return [Rubinius::AST::Script]
+      #
+      # @api private
+      #
+      def root(node)
+        nesting.reverse.inject(node) do |current, scope|
+          self.class.wrap(scope, current)
+        end
       end
 
-      TABLE = {
-        ::Module => Module,
-        ::Class => Class
-      }.freeze
-
-      # Build scope context from class or module
+      # Wrap node into ast node
       #
-      # @param [String] source_path
+      # @param [Class, Module] scope 
+      # @param [Rubinius::AST::Node] node
       #
-      # @param [::Class|::Module] scope
+      # @return [Rubinius::AST::Class]
+      #   if scope is of kind Class
       #
-      # @return [Context::Scope]
+      # @return [Rubinius::AST::Module]
+      #   if scope is of kind module
       #
       # @api private
       #
-      def self.build(scope, source_path)
-        scope_class = scope.class
-        klass = TABLE.fetch(scope_class) do
-          raise ArgumentError, "Can only build mutation scope from class or module got: #{scope_class.inspect}"
-        end.new(scope, source_path)
+      def self.wrap(scope, node)
+        name = scope.name.split('::').last
+        case scope
+        when ::Class
+          ::Rubinius::AST::Class.new(0, name.to_sym, nil, node)
+        when ::Module
+          ::Rubinius::AST::Module.new(0, name.to_sym, node)
+        else
+          raise "Cannot wrap scope: #{scope.inspect}"
+        end
       end
 
-      # Return AST wrapping mutated node
+      # Return nesting
       #
-      # @return [Rubinius::AST::Script]
+      # @return [Enumerable<Class,Module>]
       #
       # @api private
       #
-      def root(node)
-        root = root_ast
-        block = Rubinius::AST::Block.new(1, [node])
-        root.body = scope_class.new(1, root.name, block)
-        root
+      def nesting
+        const = ::Object
+        name_nesting.each_with_object([]) do |name, nesting|
+          const = const.const_get(name)
+          nesting << const
+        end
       end
+      memoize :nesting
 
       # Return unqualified name of scope
       #