mutant 0.2.9 → 0.2.11

data/.travis.yml

@@ -4,12 +4,15 @@ rvm:
   - 1.8.7
   - 1.9.2
   - 1.9.3
+  - ruby-head
   - rbx-19mode
 matrix:
   allow_failures:
     # No mutators for 1.8 specifc AST nodes
     - rvm: rbx-18mode  
     - rvm: 1.8.7
+    # We'll see what happens
+    - rvm: ruby-head
 notifications:
   email:
     - mbj@seonic.net

data/Changelog.md

@@ -1,3 +1,16 @@
+# v0.2.11 2013-01-03
+
+* [change] Handle binary operator methods in dedicated mutator
+* [fixed] Do not crash when mutating binary operator method
+
+[Compare v0.2.10..v0.2.11](https://github.com/mbj/mutant/compare/v0.2.10...v0.2.11)
+
+# v0.2.10 2013-01-03
+
+* [fixed] Do not mutate receivers away when message name is a keyword
+
+[Compare v0.2.9..v0.2.10](https://github.com/mbj/mutant/compare/v0.2.9...v0.2.10)
+
 # v0.2.9 2013-01-02
 
 * [feature] Mutate instance and global variable assignments

data/TODO

@@ -1,11 +1,13 @@
 Code:
   * Test mutant with dynamically created zombie.
-  * Fix ugly code within default parameters
+  * Fix ugly code within default parameter mutations
+  * Break up lib/mutant/mutator/node/send.rb in class specific files
 
 AST:
   * Fix the rubinius AST to allow setting @vcall_style variable in Rubinius::AST::Send nodes.
 
 Mutations:
+  * Add binary operator specific mutations (YAY, finally reached this point)
   * 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.
@@ -29,6 +31,10 @@ Killers:
   * Add a general master <=> killer IPC interface. So different strategies of isolation 
     (fork, vs jruby runtime creation) will work without big impact.
 
+Strategy:
+  * Provide "expicit files to kill with" strategy
+  * Automatically load ./spec/spec_helper.rb for rspec strategies (No need to specify -I and -r anymore)
+
 Matcher:
   * Allow matches on attr_reader with literal name argument(s)?
   * Allow matches on define_method with literal name argument?

data/lib/mutant.rb

@@ -1,4 +1,5 @@
 require 'backports'
+require 'set'
 require 'adamantium'
 require 'ice_nine'
 require 'abstract_type'
@@ -29,6 +30,53 @@ end
 # Library namespace
 module Mutant
 
+  # The list of ruby kewords from http://ruby-doc.org/docs/keywords/1.9/
+  KEYWORDS = %w(
+    BEGIN END __ENCODING__ __END__ __FILE__
+    __LINE__ alias and begin break case class
+    def define do else elsif end ensure false
+    for if in module next nil not or redo
+    rescue retry return self super then true
+    undef unless until when while yield
+  ).map(&:to_sym).to_set.freeze
+
+  BINARY_METHOD_OPERATOR_EXPANSIONS = {
+    :<=>  => :spaceship_operator,
+    :===  => :case_equality_operator,
+    :[]=  => :element_writer,
+    :[]   => :element_reader,
+    :<=   => :less_than_or_equal_to_operator,
+    :>=   => :greater_than_or_equal_to_operator,
+    :==   => :equality_operator,
+    :'!~' => :nomatch_operator,
+    :'!=' => :inequality_operator,
+    :=~   => :match_operator,
+    :<<   => :left_shift_operator,
+    :>>   => :right_shift_operator,
+    :**   => :exponentation_operator,
+    :*    => :multiplication_operator,
+    :%    => :modulo_operator,
+    :/    => :division_operator,
+    :|    => :bitwise_or_operator,
+    :^    => :bitwise_xor_operator,
+    :&    => :bitwise_and_operator,
+    :<    => :less_than_operator,
+    :>    => :greater_than_operator,
+    :+    => :addition_operator,
+    :-    => :substraction_operator
+  }.freeze
+
+  UNARY_METHOD_OPERATOR_EXPANSIONS = {
+    :~@   => :unary_match_operator,
+    :+@   => :unary_addition_operator,
+    :-@   => :unary_substraction_operator,
+    :'!'  => :negation_operator
+  }.freeze
+
+  BINARY_METHOD_OPERATORS = BINARY_METHOD_OPERATOR_EXPANSIONS.keys.to_set.freeze
+
+  OPERATOR_EXPANSIONS = BINARY_METHOD_OPERATOR_EXPANSIONS.merge(UNARY_METHOD_OPERATOR_EXPANSIONS).freeze
+
   # Define instance of subclassed superclass as constant
   #
   # @param [Class] superclass

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

@@ -18,9 +18,9 @@ module Mutant
         #
         # @api private
         #
-        def self.self_class?(node)
+        def self.keyword_name?(node)
           node.kind_of?(Rubinius::AST::Send) && 
-          node.name == :class                && 
+          Mutant::KEYWORDS.include?(node.name) && 
           node.receiver.kind_of?(Rubinius::AST::Self)
         end
 
@@ -73,7 +73,7 @@ module Mutant
         def emit_receiver_mutations
           util = self.class
 
-          unless to_self? or util.self_class?(receiver) 
+          unless to_self? or util.keyword_name?(receiver) 
             emit_attribute_mutations(:receiver) 
           end
         end
@@ -151,7 +151,7 @@ module Mutant
         #
         def emit_implicit_self_receiver
           return unless to_self? 
-          return if self.class.self_class?(node)
+          return if self.class.keyword_name?(node)
           mutant = dup_node
           mutant.privately = true
           # TODO: Fix rubinius to allow this as an attr_accessor
@@ -163,6 +163,48 @@ module Mutant
           
           handle(Rubinius::AST::SendWithArguments)
 
+          class BinaryOperatorMethod < Node
+
+          private
+
+            # Emit mutations
+            #
+            # @return [undefined]
+            #
+            # @api private
+            #
+            def dispatch
+              emit_left_mutations
+              emit_right_mutations
+            end
+
+            # Emit left mutations
+            #
+            # @return [undefined]
+            #
+            # @api private
+            #
+            def emit_left_mutations
+              emit_attribute_mutations(:receiver)
+            end
+
+            # Emit right mutations
+            #
+            # @return [undefined]
+            #
+            # @api private
+            #
+            def emit_right_mutations
+              right = node.arguments.array.first
+              Mutator.each(right).each do |mutated|
+                dup = dup_node
+                dup.arguments.array[0] = mutated
+                emit(dup)
+              end
+            end
+
+          end
+
         private
 
           # Emit mutations
@@ -174,6 +216,37 @@ module Mutant
           def dispatch
             super
             emit_call_remove_mutation
+            emit_argument_mutations
+          end
+
+          # Test if message is a binary operator
+          #
+          # @return [true]
+          #   if message is a binary operator
+          #
+          # @return [false]
+          #   otherwise
+          #
+          # @api private
+          #
+          def binary_operator?
+            Mutant::BINARY_METHOD_OPERATORS.include?(node.name)
+          end
+
+          # Emit argument mutations
+          #
+          # @api private
+          #
+          # @return [undefined]
+          #
+          # @api private
+          #
+          def emit_argument_mutations
+            if binary_operator?
+              run(BinaryOperatorMethod)
+              return
+            end
+
             emit_attribute_mutations(:arguments)
           end
 

data/lib/mutant/strategy/rspec/example_lookup.rb

@@ -56,36 +56,6 @@ module Mutant
           mutation.subject.matcher
         end
 
-        MAPPING = {
-          :<=>  => :spaceship_operator,
-          :===  => :case_equality_operator,
-          :[]=  => :element_writer,
-          :[]   => :element_reader,
-          :<=   => :less_than_or_equal_to_operator,
-          :>=   => :greater_than_or_equal_to_operator,
-          :~@   => :unary_match_operator,
-          :+@   => :unary_addition_operator,
-          :-@   => :unary_substraction_operator,
-          :==   => :equality_operator,
-          :'!~' => :nomatch_operator,
-          :'!=' => :inequality_operator,
-          :=~   => :match_operator,
-          :<<   => :left_shift_operator,
-          :>>   => :right_shift_operator,
-          :**   => :exponentation_operator,
-          :*    => :multiplication_operator,
-          :%    => :modulo_operator,
-          :/    => :division_operator,
-          :|    => :bitwise_or_operator,
-          :'!'  => :negation_operator,
-          :^    => :bitwise_xor_operator,
-          :&    => :bitwise_and_operator,
-          :<    => :less_than_operator,
-          :>    => :greater_than_operator,
-          :+    => :addition_operator,
-          :-    => :substraction_operator
-        }
-
         EXPANSIONS = {
           /\?\z/ => '_predicate',
           /=\z/  => '_writer',
@@ -114,7 +84,7 @@ module Mutant
         # @api private
         #
         def mapped_name
-          MAPPING[method_name]
+          OPERATOR_EXPANSIONS[method_name]
         end
 
         # Return expanded name

data/mutant.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |gem|
   gem.name        = 'mutant'
-  gem.version     = '0.2.9'
+  gem.version     = '0.2.11'
   gem.authors     = [ 'Markus Schirp' ]
   gem.email       = [ 'mbj@seonic.net' ]
   gem.description = 'Mutation testing for ruby under rubinius'
@@ -15,7 +15,7 @@ Gem::Specification.new do |gem|
   gem.extra_rdoc_files = %w[TODO LICENSE]
   gem.executables       = [ 'mutant' ]
 
-  gem.add_runtime_dependency('to_source',           '~> 0.2.7')
+  gem.add_runtime_dependency('to_source',           '~> 0.2.8')
   gem.add_runtime_dependency('ice_nine',            '~> 0.6.0')
   gem.add_runtime_dependency('descendants_tracker', '~> 0.0.1')
   gem.add_runtime_dependency('backports',           '~> 2.6')

data/spec/unit/mutant/mutator/node/send/mutation_spec.rb

@@ -1,5 +1,6 @@
 require 'spec_helper'
 
+# FIXME: This spec needs to be structured better!
 describe Mutant::Mutator, 'send' do
   context 'send without arguments' do
     context 'with self as receiver' do
@@ -20,6 +21,15 @@ describe Mutant::Mutator, 'send' do
 
         it_should_behave_like 'a mutator'
       end
+
+      context 'explicit with keyword message name' do
+        Mutant::KEYWORDS.each do |keyword|
+          context "with keyword: #{keyword}" do
+            let(:source) { "self.#{keyword}" }
+            it_should_behave_like 'a noop mutator'
+          end
+        end
+      end
     end
 
     context 'to some object' do
@@ -33,12 +43,6 @@ describe Mutant::Mutator, 'send' do
       it_should_behave_like 'a mutator'
     end
 
-    context 'to self.class' do
-      let(:source) { 'self.class' }
-
-      it_should_behave_like 'a noop mutator'
-    end
-
     context 'to self.class.foo' do
       let(:source) { 'self.class.foo' }
 
@@ -66,6 +70,56 @@ describe Mutant::Mutator, 'send' do
       it_should_behave_like 'a mutator'
     end
 
+    context 'with explicit self as receiver' do
+      let(:source) { 'self.foo(nil)' }
+
+      let(:mutations) do 
+        mutations = []
+        mutations << 'self.foo'
+        mutations << 'foo(nil)'
+        mutations << 'nil'
+        mutations << 'self.foo(Object.new)'
+      end
+
+      it_should_behave_like 'a mutator'
+    end
+
+    context 'to some object with keyword in method name' do
+      Mutant::KEYWORDS.each do |keyword|
+        context "with keyword #{keyword}" do
+          let(:source) { "foo.#{keyword}(nil)" }
+
+          let(:mutations) do 
+            mutations = []
+            mutations << "foo.#{keyword}"
+            mutations << "foo"
+            mutations << 'nil'
+            mutations << "foo.#{keyword}(Object.new)"
+          end
+
+          it_should_behave_like 'a mutator'
+        end
+      end
+    end
+
+    context 'binary operator methods' do
+      Mutant::BINARY_METHOD_OPERATORS.each do |operator|
+        let(:source) { "true #{operator} false" }
+
+        let(:mutations) do 
+          mutations = []
+          mutations << "((false) #{operator} (false))"
+          mutations << "((nil) #{operator} (false))"
+          mutations << "((true) #{operator} (true))"
+          mutations << "((true) #{operator} (nil))"
+          mutations << 'true'
+          mutations << 'false'
+        end
+
+        it_should_behave_like 'a mutator'
+      end
+    end
+
     context 'two arguments' do
       let(:source) { 'foo(nil, nil)' }
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mutant
 version: !ruby/object:Gem::Version
-  version: 0.2.9
+  version: 0.2.11
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-01-02 00:00:00.000000000 Z
+date: 2013-01-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: to_source
@@ -18,7 +18,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.2.7
+        version: 0.2.8
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -26,7 +26,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.2.7
+        version: 0.2.8
 - !ruby/object:Gem::Dependency
   name: ice_nine
   requirement: !ruby/object:Gem::Requirement