mutant 0.2.7 → 0.2.8

data/Changelog.md

@@ -1,3 +1,11 @@
+# v0.2.8 2012-12-29
+
+* [feature] Do not mutate argument or local variable names beginning with an underscore
+* [feature] Mutate unary calls ```coerce(object)``` => ```object```
+* [feature] Mutate method call receivers ```foo.bar``` => ```foo```
+
+[Compare v0.2.7..v0.2.8](https://github.com/mbj/mutant/compare/v0.2.7...v0.2.8)
+
 # v0.2.7 2012-12-21
 
 * [fixed] Use latest adamantium and ice_nine
@@ -16,7 +24,7 @@
 * [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 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

data/Gemfile.devtools

@@ -1,5 +1,5 @@
-group :development do 
-  gem 'rake',    '~> 0.9.2'
+group :development do
+  gem 'rake',    '~> 10.0'
   gem 'rspec',   '~> 2.12.0'
   gem 'yard',    '~> 0.8.3'
 end
@@ -26,22 +26,15 @@ group :metrics do
   gem 'flog',      '~> 2.5.1'
   gem 'reek',      '~> 1.2.8', :git => 'https://github.com/dkubb/reek.git'
   gem 'roodi',     '~> 2.1.0'
-  gem 'yardstick', '~> 0.7.0'
-  gem 'simplecov'
+  gem 'yardstick', '~> 0.8.0'
 
   platforms :ruby_18, :ruby_19 do
     # this indirectly depends on ffi which does not build on ruby-head
     gem 'yard-spellcheck', '~> 0.1.5'
   end
 
-  platforms :mri_18 do
-    gem 'arrayfields', '~> 4.7.4'  # for metric_fu
-    gem 'fattr',       '~> 2.2.0'  # for metric_fu
-    gem 'json',        '~> 1.7.3'  # for metric_fu rake task
-    gem 'map',         '~> 6.0.1'  # for metric_fu
-    gem 'metric_fu',   '~> 2.1.1'
-    gem 'mspec',       '~> 1.5.17'
-    gem 'rcov',        '~> 1.0.0'
+  platforms :mri_19 do
+    gem 'simplecov',   '~> 0.7'
   end
 
   platforms :rbx do

data/LICENSE

@@ -0,0 +1,20 @@
+Copyright (c) 2012 Markus Schirp
+
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the
+"Software"), to deal in the Software without restriction, including
+without limitation the rights to use, copy, modify, merge, publish,
+distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so, subject to
+the following conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/README.md

@@ -8,13 +8,26 @@ 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 do not notice, either that code isn't being covered 
-or it doesn't do anything (useful).
+or it does not have a speced side effect.
 
-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.
+Mutant does currently only support 1.9 mode under Rubinius or MRI. Support for JRuby is planned. 
+
+Also it is easy to write a mutation killer for other test/spec frameworks than rspec2. 
+Just create your own Mutant::Killer subclass, and make sure I get a PR!
 
 See this [ASCII-Cast](http://ascii.io/a/1707) for mutant in action! (v0.2.1)
 
+Using Mutant
+------------
+
+The following projects adopted mutant.
+
+* [dm-mapper](https://github.com/datamapper/dm-mapper)
+* [virtus](https://github.com/solnic/virtus)
+* various small/minor stuff under https://github.com/mbj
+
+Feel free to ping me to add your project to the list!
+
 Installation
 ------------
 
@@ -23,6 +36,8 @@ Install the gem ``mutant`` via your preferred method.
 Examples
 --------
 
+CLI will be simplified in the next releases, but currently stick with this:
+
 ```
 cd virtus
 # Run mutant on virtus namespace (that uses the dm-2 style spec layout)
@@ -39,7 +54,7 @@ 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:
+Mutant currently supports the following buildin strategies for selecting tests/specs.
 
 ### --rspec-dm2
 
@@ -68,6 +83,11 @@ 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.
 
+Alternatives
+------------
+
+* [heckle](https://github.com/seattlerb/heckle)
+
 Credits
 -------
 
@@ -89,23 +109,4 @@ Contributing
 License
 -------
 
-Copyright (c) 2012 Markus Schirp
-
-Permission is hereby granted, free of charge, to any person obtaining
-a copy of this software and associated documentation files (the
-"Software"), to deal in the Software without restriction, including
-without limitation the rights to use, copy, modify, merge, publish,
-distribute, sublicense, and/or sell copies of the Software, and to
-permit persons to whom the Software is furnished to do so, subject to
-the following conditions:
-
-The above copyright notice and this permission notice shall be
-included in all copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
-EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
-MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
-NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
-LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
-OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
-WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+See LICENSE file.

data/TODO

@@ -1,6 +1,5 @@
 Code:
   * Test mutant with dynamically created zombie.
-  * Replace nil or add "do not touch me object" to literal mutations.
   * Fix ugly code within default parameters
 
 AST:
@@ -9,9 +8,18 @@ AST:
 Mutations:
   * Add some kind of a "do not touch me object" that raises on all messages.
     It can be used to make sure each literal value is touched.
+  * Replace nil or add "do not touch me object" to literal mutations.
   * Mutate options on Regexp literals
-  * Make sure loader does not change visibility of injected mutants
   * Add mutations for dynamic regexp symbol and string literals
+  * Mutate "def foo; bar; end" to "def foo; self; end"?
+  * Emit negative and positive mutations 
+   
+    Example of a negative mutation:
+      Mutations on local variables and arguments prefixed with an underscore would be emitted as
+      negative mutations that must be alive.
+
+Loader:
+  * Make sure loader does not change visibility of injected mutants
 
 Killers:
   * Aggregate warnings on missing spec files

data/config/flay.yml

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

data/config/flog.yml

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

data/config/site.reek

@@ -10,8 +10,9 @@ UncommunicativeParameterName:
 LargeClass:
   max_methods: 10
   exclude: 
-  - "Mutant::Matcher::Method" # 13 methods
+  #- "Mutant::Matcher::Method" # 13 methods
   - "Mutant::Reporter::CLI" # 16 methods TODO Reduce!
+  - "Mutant::CLI" # 19 methods and 7 ivars, TODO Reduce!
   enabled: true
   max_instance_variables: 3
 UncommunicativeMethodName:
@@ -50,7 +51,8 @@ UncommunicativeModuleName:
   - !ruby/regexp /[0-9]$/
 NestedIterators:
   ignore_iterators: []
-  exclude: []
+  exclude: 
+  - Mutant#self.define_singleton_subclass
   enabled: true
   max_allowed_nesting: 1
 LongMethod:

data/lib/mutant.rb

@@ -13,12 +13,14 @@ 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
+# Patch ice none to freeze nodes correctly
+class IceNine::Freezer
+  # Rubinius namsepace
+  class Rubinius 
+    # AST namespace
+    class AST < IceNine::Freezer::Object
+      # Node configuration
+      class Node < IceNine::Freezer::Object
       end
     end
   end
@@ -51,8 +53,6 @@ module Mutant
     self
   end
 
-  PID = Process.pid
-
 end
 
 require 'mutant/support/method_object'
@@ -65,6 +65,8 @@ require 'mutant/mutation/filter/code'
 require 'mutant/mutation/filter/whitelist'
 require 'mutant/mutator/registry'
 require 'mutant/mutator/util'
+require 'mutant/mutator/util/array'
+require 'mutant/mutator/util/symbol'
 require 'mutant/mutator/node'
 require 'mutant/mutator/node/literal'
 require 'mutant/mutator/node/literal/boolean'
@@ -81,8 +83,15 @@ require 'mutant/mutator/node/literal/nil'
 require 'mutant/mutator/node/block'
 require 'mutant/mutator/node/noop'
 require 'mutant/mutator/node/send'
-require 'mutant/mutator/node/arguments'
 require 'mutant/mutator/node/define'
+require 'mutant/mutator/node/formal_arguments_19'
+require 'mutant/mutator/node/formal_arguments_19/default_mutations'
+require 'mutant/mutator/node/formal_arguments_19/require_defaults'
+require 'mutant/mutator/node/formal_arguments_19/pattern_argument_expansion'
+require 'mutant/mutator/node/actual_arguments'
+require 'mutant/mutator/node/pattern_arguments'
+require 'mutant/mutator/node/pattern_variable'
+require 'mutant/mutator/node/default_arguments'
 require 'mutant/mutator/node/return'
 require 'mutant/mutator/node/local_variable_assignment'
 require 'mutant/mutator/node/iter_19'

data/lib/mutant/context/scope.rb

@@ -30,12 +30,12 @@ module Mutant
       # @api private
       #
       def self.wrap(scope, node)
-        name = scope.name.split('::').last
+        name = scope.name.split('::').last.to_sym
         case scope
         when ::Class
-          ::Rubinius::AST::Class.new(0, name.to_sym, nil, node)
+          ::Rubinius::AST::Class.new(0, name, nil, node)
         when ::Module
-          ::Rubinius::AST::Module.new(0, name.to_sym, node)
+          ::Rubinius::AST::Module.new(0, name, node)
         else
           raise "Cannot wrap scope: #{scope.inspect}"
         end

data/lib/mutant/differ.rb

@@ -48,10 +48,22 @@ module Mutant
     # @api private
     #
     def initialize(old, new)
-      @old, @new = self.class.lines(old), self.class.lines(new)
+      @old, @new = lines(old), lines(new)
       @diffs = Diff::LCS.diff(@old, @new)
     end
 
+    # Break up sorce into lines
+    #
+    # @param [String] source
+    #
+    # @return [Array<String>]
+    #
+    # @api private
+    #
+    def lines(source)
+      self.class.lines(source)
+    end
+
     # Return length difference
     #
     # @return [Fixnum]

data/lib/mutant/killer.rb

@@ -132,5 +132,6 @@ module Mutant
     # @api private
     #
     abstract_method :run
+
   end
 end

data/lib/mutant/matcher/scope_methods.rb

@@ -55,6 +55,20 @@ module Mutant
         self.class::MATCHER
       end
 
+      # Return method names
+      #
+      # @param [Object] object
+      #
+      # @return [Enumerable<Symbol>]
+      #
+      # @api private
+      #
+      def self.method_names(object)
+        object.public_instance_methods(false)   +
+        object.private_instance_methods(false)  +
+        object.protected_instance_methods(false)
+      end
+
     private
 
       # Initialize object
@@ -118,11 +132,7 @@ module Mutant
         #
         def method_names
           singleton_class = scope.singleton_class
-
-          names = 
-            singleton_class.public_instance_methods(false)   +
-            singleton_class.private_instance_methods(false)  +
-            singleton_class.protected_instance_methods(false)
+          names = self.class.method_names(singleton_class)
 
           names.sort.reject do |name|
             name.to_sym == :__class_init__
@@ -158,12 +168,7 @@ module Mutant
         def method_names
           scope = self.scope
           return [] unless scope.kind_of?(Module)
-
-          names = 
-            scope.public_instance_methods(false)  +
-            scope.private_instance_methods(false) + 
-            scope.protected_instance_methods(false)
-
+          names = self.class.method_names(scope)
           names.uniq.sort
         end
       end

data/lib/mutant/mutator.rb

@@ -182,6 +182,16 @@ module Mutant
       self
     end
 
+    # Run input with mutator
+    #
+    # @return [undefined]
+    #
+    # @api private
+    #
+    def run(mutator)
+      mutator.new(input, method(:emit))
+    end
+
     # Shortcut to create a new unfrozen duplicate of input
     #
     # @return [Object]

data/lib/mutant/mutator/node.rb

@@ -17,7 +17,7 @@ module Mutant
         ToSource.to_source(node)
       end
 
-      private
+    private
 
       # Return mutated node
       #
@@ -89,20 +89,6 @@ module Mutant
         emit(new_self(*arguments))
       end
 
-      # Emit a new node with wrapping class for each entry in values
-      #
-      # @param [Array] values
-      #
-      # @return [undefined]
-      #
-      # @api private
-      #
-      def emit_values(values)
-        values.each do |value|
-          emit_self(value)
-        end
-      end
-
       # Emit body mutations
       #
       # @param [Symbol] name
@@ -111,10 +97,10 @@ module Mutant
       #
       # @api private
       #
-      def emit_attribute_mutations(name)
-        body = node.public_send(name)
+      def emit_attribute_mutations(name, mutator = Mutator)
+        value = node.public_send(name)
 
-        Mutator.each(body) do |mutation|
+        mutator.each(value) do |mutation|
           dup = dup_node
           dup.public_send(:"#{name}=", mutation)
           yield dup if block_given?
@@ -157,6 +143,7 @@ module Mutant
       # @api private
       #
       alias_method :dup_node, :dup_input
+
     end
   end
 end

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

@@ -0,0 +1,25 @@
+module Mutant
+  class Mutator
+    class Node
+
+      # Mutator for arguments
+      class Arguments < self
+
+        handle(Rubinius::AST::ActualArguments)
+
+      private
+
+        # Emit mutations
+        #
+        # @return [undefined]
+        #
+        # @api private
+        #
+        def dispatch
+          emit_attribute_mutations(:array)
+        end
+
+      end
+    end
+  end
+end