mutant 0.2.8 → 0.2.9

data/Changelog.md

@@ -1,3 +1,10 @@
+# v0.2.9 2013-01-02
+
+* [feature] Mutate instance and global variable assignments
+* [feature] Mutate super calls
+
+[Compare v0.2.8..v0.2.9](https://github.com/mbj/mutant/compare/v0.2.8...v0.2.9)
+
 # v0.2.8 2012-12-29
 
 * [feature] Do not mutate argument or local variable names beginning with an underscore

data/TODO

@@ -13,6 +13,9 @@ Mutations:
   * Add mutations for dynamic regexp symbol and string literals
   * Mutate "def foo; bar; end" to "def foo; self; end"?
   * Emit negative and positive mutations 
+  * Mutate Block catch "def foo(&block)" and block pass "foo(&block)"
+  * Mutate super arguments just like send arguments
+  * Add timeout to terminate infinite loops
    
     Example of a negative mutation:
       Mutations on local variables and arguments prefixed with an underscore would be emitted as

data/lib/mutant.rb

@@ -68,6 +68,7 @@ require 'mutant/mutator/util'
 require 'mutant/mutator/util/array'
 require 'mutant/mutator/util/symbol'
 require 'mutant/mutator/node'
+require 'mutant/mutator/node/noop'
 require 'mutant/mutator/node/literal'
 require 'mutant/mutator/node/literal/boolean'
 require 'mutant/mutator/node/literal/range'
@@ -81,8 +82,10 @@ require 'mutant/mutator/node/literal/hash'
 require 'mutant/mutator/node/literal/regex'
 require 'mutant/mutator/node/literal/nil'
 require 'mutant/mutator/node/block'
-require 'mutant/mutator/node/noop'
+require 'mutant/mutator/node/while'
+require 'mutant/mutator/node/super'
 require 'mutant/mutator/node/send'
+require 'mutant/mutator/node/assignment'
 require 'mutant/mutator/node/define'
 require 'mutant/mutator/node/formal_arguments_19'
 require 'mutant/mutator/node/formal_arguments_19/default_mutations'
@@ -93,7 +96,6 @@ 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'
 require 'mutant/mutator/node/if_statement'
 require 'mutant/mutator/node/receiver_case'

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

@@ -0,0 +1,48 @@
+module Mutant
+  class Mutator
+    class Node
+      class Assignment < self
+
+      private
+
+        class Variable < self
+
+          # Emit mutants
+          #
+          # @return [undefined]
+          #
+          # @api private
+          #
+          def dispatch
+            emit_attribute_mutations(:name) do |mutation|
+              mutation.name = "#{self.class::PREFIX}#{mutation.name}".to_sym
+            end
+            emit_attribute_mutations(:value)
+          end
+
+          class Local < self
+            PREFIX = ''.freeze
+            handle(Rubinius::AST::LocalVariableAssignment)
+          end
+
+          class Instance < self
+            PREFIX = '@'.freeze
+            handle(Rubinius::AST::InstanceVariableAssignment)
+          end
+
+          class Class < self
+            PREFIX = '@@'.freeze
+            handle(Rubinius::AST::ClassVariableAssignment)
+          end
+
+          class Global < self
+            PREFIX = '$'.freeze
+            handle(Rubinius::AST::GlobalVariableAssignment)
+          end
+
+        end
+
+      end
+    end
+  end
+end

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

@@ -7,13 +7,12 @@ module Mutant
         # Literal references to self do not need to be mutated?
         handle(Rubinius::AST::Self)
 
-
         # Currently unhandled node classes. Feel free to contribute your mutator!
-        handle(Rubinius::AST::While)
         handle(Rubinius::AST::ElementAssignment)
         handle(Rubinius::AST::AttributeAssignment)
         handle(Rubinius::AST::Not)
         handle(Rubinius::AST::And)
+        handle(Rubinius::AST::Or)
         handle(Rubinius::AST::Defined)
         handle(Rubinius::AST::Super)
         handle(Rubinius::AST::Next)
@@ -22,25 +21,24 @@ module Mutant
         handle(Rubinius::AST::ZSuper)
         handle(Rubinius::AST::MultipleAssignment)
         handle(Rubinius::AST::ScopedConstant)
-        handle(Rubinius::AST::LocalVariableAssignment)
         handle(Rubinius::AST::LocalVariableAccess)
-        handle(Rubinius::AST::InstanceVariableAssignment)
         handle(Rubinius::AST::InstanceVariableAccess)
-        handle(Rubinius::AST::GlobalVariableAssignment)
         handle(Rubinius::AST::GlobalVariableAccess)
+        handle(Rubinius::AST::ClassVariableAccess)
         handle(Rubinius::AST::ToplevelConstant)
         handle(Rubinius::AST::Ensure)
         handle(Rubinius::AST::Rescue)
         handle(Rubinius::AST::DynamicString)
         handle(Rubinius::AST::DynamicSymbol)
-        handle(Rubinius::AST::File)
         handle(Rubinius::AST::DynamicRegex)
+        handle(Rubinius::AST::File)
         handle(Rubinius::AST::OpAssignOr19)
         handle(Rubinius::AST::BlockPass19)
         handle(Rubinius::AST::OpAssign1)
-        handle(Rubinius::AST::Or)
         handle(Rubinius::AST::ConstantAccess)
         handle(Rubinius::AST::Yield)
+        handle(Rubinius::AST::Begin)
+        handle(Rubinius::AST::Rescue)
       
       private
 

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

@@ -0,0 +1,53 @@
+module Mutant
+  class Mutator
+    class Node
+
+      class ZSuper < self
+
+        handle(Rubinius::AST::ZSuper)
+
+        # Emit mutations
+        #
+        # @return [undefined]
+        #
+        # @api private
+        #
+        def dispatch
+          emit_node(Rubinius::AST::Super, new(Rubinius::AST::ActualArguments))
+        end
+
+      end
+
+      class Super < self
+        handle(Rubinius::AST::Super)
+
+      private
+
+        # Emit mutations
+        #
+        # @return [undefined]
+        #
+        # @api private
+        #
+        def dispatch
+          emit_node(Rubinius::AST::ZSuper)
+          emit_without_block
+          emit_attribute_mutations(:block) if node.block
+          emit_attribute_mutations(:arguments) 
+        end
+
+        # Emit without block mutation
+        #
+        # @return [undefined]
+        #
+        # @api private
+        #
+        def emit_without_block
+          dup = dup_node
+          dup.block = nil
+          emit(dup)
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,24 @@
+module Mutant
+  class Mutator
+    class Node
+      class While < self
+
+        handle(Rubinius::AST::While)
+
+      private
+
+        # Emit mutations
+        #
+        # @return [undefined]
+        #
+        # @api private
+        #
+        def dispatch
+          emit_attribute_mutations(:condition)
+          emit_attribute_mutations(:body)
+        end
+
+      end
+    end
+  end
+end

data/mutant.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |gem|
   gem.name        = 'mutant'
-  gem.version     = '0.2.8'
+  gem.version     = '0.2.9'
   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.5')
+  gem.add_runtime_dependency('to_source',           '~> 0.2.7')
   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/assignment/mutation_spec.rb

@@ -0,0 +1,64 @@
+require 'spec_helper'
+
+describe Mutant::Mutator::Node::Assignment, 'mutations' do
+
+  before do
+    Mutant::Random.stub(:hex_string => :random)
+  end
+
+  context 'global variable' do
+    let(:source) { '$a = true' }
+
+    let(:mutations) do
+      mutations = []
+
+      mutations << '$srandom = true'
+      mutations << '$a = false'
+      mutations << '$a = nil'
+    end
+
+    it_should_behave_like 'a mutator'
+  end
+
+  context 'class variable' do
+    let(:source) { '@@a = true' }
+
+    let(:mutations) do
+      mutations = []
+
+      mutations << '@@srandom = true'
+      mutations << '@@a = false'
+      mutations << '@@a = nil'
+    end
+
+    it_should_behave_like 'a mutator'
+  end
+
+  context 'instance variable' do
+    let(:source) { '@a = true' }
+
+    let(:mutations) do
+      mutations = []
+
+      mutations << '@srandom = true'
+      mutations << '@a = false'
+      mutations << '@a = nil'
+    end
+
+    it_should_behave_like 'a mutator'
+  end
+
+  context 'local variable' do
+    let(:source) { 'a = true' }
+
+    let(:mutations) do
+      mutations = []
+
+      mutations << 'srandom = true'
+      mutations << 'a = false'
+      mutations << 'a = nil'
+    end
+
+    it_should_behave_like 'a mutator'
+  end
+end

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

@@ -0,0 +1,73 @@
+require 'spec_helper'
+
+describe Mutant::Mutator, 'super' do
+
+  context 'with no arguments' do
+    let(:source) { 'super' }
+
+    let(:mutations) do
+      mutations = []
+      mutations << 'super()'
+    end
+
+    it_should_behave_like 'a mutator'
+  end
+
+  context 'with explicit empty arguments' do
+    let(:source) { 'super()' }
+
+    let(:mutations) do
+      mutations = []
+      mutations << 'super'
+    end
+
+    it_should_behave_like 'a mutator'
+  end
+
+  context 'with explicit empty arguments and block' do
+    let(:source) { 'super() { foo; bar }' }
+
+    let(:mutations) do
+      mutations = []
+      mutations << 'super() { foo }'
+      mutations << 'super() { bar }'
+      mutations << 'super() { }'
+      mutations << 'super()'
+      mutations << 'super'
+    end
+
+    it_should_behave_like 'a mutator'
+  end
+
+  context 'super with arguments' do
+    let(:source) { 'super(foo, bar)' }
+
+    let(:mutations) do
+      mutations = []
+      mutations << 'super'
+      mutations << 'super()'
+      mutations << 'super(foo)'
+      mutations << 'super(bar)'
+    end
+
+    it_should_behave_like 'a mutator'
+  end
+
+  context 'super with arguments and block' do
+    let(:source) { 'super(foo, bar) { foo; bar }' }
+
+    let(:mutations) do
+      mutations = []
+      mutations << 'super(foo, bar) { foo; }'
+      mutations << 'super(foo, bar) { bar; }'
+      mutations << 'super(foo, bar) { nil }'
+      mutations << 'super(foo, bar)'
+      mutations << 'super'
+      mutations << 'super(foo) { foo; bar }'
+      mutations << 'super(bar) { foo; bar }'
+      mutations << 'super() { foo; bar }'
+    end
+
+    it_should_behave_like 'a mutator'
+  end
+end

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

@@ -0,0 +1,19 @@
+require 'spec_helper'
+
+describe Mutant::Mutator::Node::While do
+
+  context 'with more than one statement' do
+    let(:source) { "while true; foo; bar; end" }
+
+    let(:mutations) do
+      mutations = []
+      mutations << 'while true; bar; end'
+      mutations << 'while true; foo; end'
+      mutations << 'while true; nil; end'
+      mutations << 'while false; foo; bar; end'
+      mutations << 'while nil; foo; bar; end'
+    end
+
+    it_should_behave_like 'a mutator'
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mutant
 version: !ruby/object:Gem::Version
-  version: 0.2.8
+  version: 0.2.9
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-30 00:00:00.000000000 Z
+date: 2013-01-02 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.5
+        version: 0.2.7
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -26,7 +26,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.2.5
+        version: 0.2.7
 - !ruby/object:Gem::Dependency
   name: ice_nine
   requirement: !ruby/object:Gem::Requirement
@@ -212,6 +212,7 @@ files:
 - lib/mutant/mutator/node.rb
 - lib/mutant/mutator/node/actual_arguments.rb
 - lib/mutant/mutator/node/arguments.rb
+- lib/mutant/mutator/node/assignment.rb
 - lib/mutant/mutator/node/block.rb
 - lib/mutant/mutator/node/default_arguments.rb
 - lib/mutant/mutator/node/define.rb
@@ -234,13 +235,14 @@ files:
 - lib/mutant/mutator/node/literal/regex.rb
 - lib/mutant/mutator/node/literal/string.rb
 - lib/mutant/mutator/node/literal/symbol.rb
-- lib/mutant/mutator/node/local_variable_assignment.rb
 - lib/mutant/mutator/node/noop.rb
 - lib/mutant/mutator/node/pattern_arguments.rb
 - lib/mutant/mutator/node/pattern_variable.rb
 - lib/mutant/mutator/node/receiver_case.rb
 - lib/mutant/mutator/node/return.rb
 - lib/mutant/mutator/node/send.rb
+- lib/mutant/mutator/node/super.rb
+- lib/mutant/mutator/node/while.rb
 - lib/mutant/mutator/registry.rb
 - lib/mutant/mutator/util.rb
 - lib/mutant/mutator/util/array.rb
@@ -296,6 +298,7 @@ files:
 - spec/unit/mutant/mutator/each_spec.rb
 - spec/unit/mutant/mutator/emit_new_spec.rb
 - spec/unit/mutant/mutator/emit_spec.rb
+- spec/unit/mutant/mutator/node/assignment/mutation_spec.rb
 - spec/unit/mutant/mutator/node/block/mutation_spec.rb
 - spec/unit/mutant/mutator/node/define/mutation_spec.rb
 - spec/unit/mutant/mutator/node/if_statement/mutation_spec.rb
@@ -313,6 +316,8 @@ files:
 - spec/unit/mutant/mutator/node/receiver_case/mutation_spec.rb
 - spec/unit/mutant/mutator/node/return/mutation_spec.rb
 - spec/unit/mutant/mutator/node/send/mutation_spec.rb
+- spec/unit/mutant/mutator/node/super/mutation_spec.rb
+- spec/unit/mutant/mutator/node/while/mutation_spec.rb
 - spec/unit/mutant/mutator/self_spec.rb
 - spec/unit/mutant/strategy/rspec/example_lookup/spec_file_spec.rb
 - spec/unit/mutant/subject/class_methods/new_spec.rb
@@ -407,6 +412,7 @@ test_files:
 - spec/unit/mutant/mutator/each_spec.rb
 - spec/unit/mutant/mutator/emit_new_spec.rb
 - spec/unit/mutant/mutator/emit_spec.rb
+- spec/unit/mutant/mutator/node/assignment/mutation_spec.rb
 - spec/unit/mutant/mutator/node/block/mutation_spec.rb
 - spec/unit/mutant/mutator/node/define/mutation_spec.rb
 - spec/unit/mutant/mutator/node/if_statement/mutation_spec.rb
@@ -424,6 +430,8 @@ test_files:
 - spec/unit/mutant/mutator/node/receiver_case/mutation_spec.rb
 - spec/unit/mutant/mutator/node/return/mutation_spec.rb
 - spec/unit/mutant/mutator/node/send/mutation_spec.rb
+- spec/unit/mutant/mutator/node/super/mutation_spec.rb
+- spec/unit/mutant/mutator/node/while/mutation_spec.rb
 - spec/unit/mutant/mutator/self_spec.rb
 - spec/unit/mutant/strategy/rspec/example_lookup/spec_file_spec.rb
 - spec/unit/mutant/subject/class_methods/new_spec.rb

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

@@ -1,25 +0,0 @@
-module Mutant
-  class Mutator
-    class Node 
-      class LocalVariableAssignment < self
-
-        handle(Rubinius::AST::LocalVariableAssignment)
-
-      private
-
-        # Emit mutants
-        #
-        # @return [undefined]
-        #
-        # @api private
-        #
-        def dispatch
-          emit_attribute_mutations(:name)
-          emit_attribute_mutations(:value)
-        end
-
-      end
-    end
-  end
-end
-