mutant 0.2.4 → 0.2.5

data/spec/integration/mutant/loader_spec.rb

@@ -7,11 +7,11 @@ class CodeLoadingSubject
 end
 
 describe Mutant, 'code loading' do
-  let(:context) { Mutant::Context::Scope.build(CodeLoadingSubject,"/some/path") }
-  let(:node)    { 'def foo; :bar; end'.to_ast                                   }
-  let(:root)    { context.root(node)                                            }
+  let(:context) { Mutant::Context::Scope.new(CodeLoadingSubject, "/some/path") }
+  let(:node)    { 'def foo; :bar; end'.to_ast                                  }
+  let(:root)    { context.root(node)                                           }
  
-  subject       { Mutant::Loader::Eval.run(root)                                }
+  subject       { Mutant::Loader::Eval.run(root)                               }
 
   before { subject }
 

data/spec/shared/mutator_behavior.rb

@@ -15,13 +15,25 @@ shared_examples_for 'a mutator' do
 
     it { should be_instance_of(to_enum.class) }
 
+    def assert_transitive(ast)
+      generated = ToSource.to_source(ast)
+      parsed    = generated.to_ast
+      again     = ToSource.to_source(parsed)
+      unless generated == again
+        fail "Untransitive:\n%s\n---\n%s" % [generated, again]
+      end
+    end
+
     unless instance_methods.include?(:expected_mutations)
       let(:expected_mutations)  do
         mutations.map do |mutation|
           case mutation
           when String
-            mutation.to_ast
+            ast = mutation.to_ast
+            assert_transitive(ast)
+            ast
           when Rubinius::AST::Node
+            assert_transitive(mutation)
             mutation
           else
             raise

data/spec/unit/mutant/context/scope/root_spec.rb

@@ -3,20 +3,32 @@ require 'spec_helper'
 describe Mutant::Context::Scope, '#root' do
   subject { object.root(node) }
 
-  let(:object) { described_class.build(TestApp::Literal, path) }
+  let(:object) { described_class.new(TestApp::Literal, path) }
   let(:path)   { mock('Path') }
-  let(:node)   { mock('Node') } 
+  let(:node)   { ':node'.to_ast } 
 
   let(:scope)      { subject.body }
   let(:scope_body) { scope.body    }
 
-  it 'should wrap the ast under constant' do
-    scope.should be_kind_of(Rubinius::AST::ClassScope)
+  let(:expected_source) do
+    ToSource.to_source(<<-RUBY.to_ast)
+      module TestApp
+        class Literal
+          :node
+        end
+      end
+    RUBY
   end
 
-  it 'should place the ast under scope inside of block' do
-    scope_body.should be_a(Rubinius::AST::Block)
-    scope_body.array.should eql([node])
-    scope_body.array.first.should be(node)
+  let(:generated_source) do
+    ToSource.to_source(subject)
+  end
+
+  let(:round_tripped_source) do
+    ToSource.to_source(expected_source.to_ast)
+  end
+
+  it 'should create correct source' do
+    generated_source.should eql(expected_source)
   end
 end

data/spec/unit/mutant/context/scope/unqualified_name_spec.rb

@@ -6,7 +6,7 @@ describe Mutant::Context::Scope, '#unqualified_name' do
   let(:path)   { mock('Path') }
 
   context 'with top level constant name' do
-    let(:object) { described_class.build(TestApp, path) }
+    let(:object) { described_class.new(TestApp, path) }
 
     it 'should return the unqualified name' do
       should eql('TestApp')
@@ -16,7 +16,7 @@ describe Mutant::Context::Scope, '#unqualified_name' do
   end
 
   context 'with scoped constant name' do
-    let(:object) { described_class.build(TestApp::Literal, path) }
+    let(:object) { described_class.new(TestApp::Literal, path) }
 
     it 'should return the unqualified name' do
       should eql('Literal')

data/spec/unit/mutant/killer/rspec/class_methods/new_spec.rb

@@ -4,7 +4,7 @@ describe Mutant::Killer::Rspec, '.new' do
 
   subject { object.new(strategy, mutation) }
 
-  let(:strategy) { mock('Strategy', :spec_files => ['foo']) }
+  let(:strategy) { mock('Strategy', :spec_files => ['foo'], :error_stream => $stderr, :output_stream => $stdout) }
   let(:context)  { mock('Context')  }
   let(:mutation) { mock('Mutation') }
 

data/spec/unit/mutant/matcher/chain/each_spec.rb

@@ -24,8 +24,12 @@ describe Mutant::Matcher::Chain, '#each' do
 
     it { should be_instance_of(to_enum.class) }
 
-    it 'yields the expected values' do
-      subject.to_a.should eql(object.to_a)
+    if defined?(RUBY_ENGINE) && RUBY_ENGINE == 'rbx'
+      pending 'FIX RBX rspec? BUG HERE'
+    else
+      it 'yields the expected values' do
+        subject.to_a.should eql(object.to_a)
+      end
     end
   end
 

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

@@ -23,6 +23,54 @@ describe Mutant::Mutator, 'define' do
     it_should_behave_like 'a mutator'
   end
 
+  context 'with arguments' do
+    let(:source) { 'def foo(a, b); nil; end' }
+
+    before do
+      Mutant::Random.stub(:hex_string => 'random')
+    end
+
+    let(:mutations) do
+      mutations = []
+
+      # Deletion of each argument
+      mutations << 'def foo(a); nil; end'
+      mutations << 'def foo(b); nil; end'
+
+      # Deletion of all arguments
+      mutations << 'def foo; nil; end'
+
+      # Rename each argument
+      mutations << 'def foo(srandom, b); nil; end'
+      mutations << 'def foo(a, srandom); nil; end'
+
+      # Mutation of body
+      mutations << 'def foo(a, b); Object.new; end'
+    end
+
+    it_should_behave_like 'a mutator'
+  end
+
+  context 'default argument' do
+    let(:source) { 'def foo(a = "literal"); end' }
+
+    before do
+      Mutant::Random.stub(:hex_string => 'random')
+    end
+
+    let(:mutations) do 
+      mutations = []
+      mutations << 'def foo(a); end'
+      mutations << 'def foo(); end'
+      mutations << 'def foo(a = "random"); end'
+      mutations << 'def foo(a = nil); end'
+      mutations << 'def foo(a = "literal"); Object.new; end'
+      mutations << 'def foo(srandom = "literal"); nil; end'
+    end
+
+    it_should_behave_like 'a mutator'
+  end
+
   context 'define on singleton' do
     let(:source) { 'def self.foo; self.bar; self.baz; end' }
 
@@ -44,4 +92,32 @@ describe Mutant::Mutator, 'define' do
     it_should_behave_like 'a mutator'
   end
 
+  context 'define on singleton with argument' do
+
+    before do
+      Mutant::Random.stub(:hex_string => 'random')
+    end
+
+    let(:source) { 'def self.foo(a, b); nil; end' }
+
+    let(:mutations) do
+      mutations = []
+
+      # Deletion of each argument
+      mutations << 'def self.foo(a); nil; end'
+      mutations << 'def self.foo(b); nil; end'
+
+      # Deletion of all arguments
+      mutations << 'def self.foo; nil; end'
+
+      # Rename each argument
+      mutations << 'def self.foo(srandom, b); nil; end'
+      mutations << 'def self.foo(a, srandom); nil; end'
+
+      # Mutation of body
+      mutations << 'def self.foo(a, b); Object.new; end'
+    end
+
+    it_should_behave_like 'a mutator'
+  end
 end

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

@@ -1,9 +1,9 @@
 require 'spec_helper'
 
-describe Mutant::Mutator, 'call' do
+describe Mutant::Mutator, 'send' do
   context 'send without arguments' do
     # This could not be reproduced in a test case but happens in the mutant source code?
-    context 'block given' do
+    context 'block_given?' do
       context 'implicit self' do
         let(:source) { 'block_given?' }
 
@@ -47,32 +47,91 @@ describe Mutant::Mutator, 'call' do
 
   context 'send with arguments' do
 
-    context 'with self as receiver' do
-      context 'implicit' do
-        let(:source) { 'foo(nil)' }
+    context 'one argument' do
+      let(:source) { 'foo(nil)' }
 
-        let(:mutations) do 
-          mutations = []
-          mutations << 'foo()'
-          mutations << 'foo(Object.new)'
-        end
+      let(:mutations) do 
+        mutations = []
+        mutations << 'foo()'
+        mutations << 'foo(Object.new)'
+      end
 
-        it_should_behave_like 'a mutator'
+      it_should_behave_like 'a mutator'
+    end
+
+    context 'two arguments' do
+      let(:source) { 'foo(nil, nil)' }
+
+      let(:mutations) do 
+        mutations = []
+        mutations << 'foo()'
+        mutations << 'foo(nil)'
+        mutations << 'foo(Object.new, nil)'
+        mutations << 'foo(nil, Object.new)'
       end
 
-      context 'explicit' do
-        let(:source) { 'self.foo(nil)' }
+      it_should_behave_like 'a mutator'
+    end
 
-        let(:mutations) do
-          mutations = []
-          # with implicit receiver (send privately)
-          mutations << 'foo(nil)'
-          mutations << 'self.foo(Object.new)'
-          mutations << 'self.foo()'
-        end
+    context 'with block' do
+      let(:source) { 'foo() { a; b }' }
 
-        it_should_behave_like 'a mutator'
+      let(:mutations) do 
+        mutations = []
+        mutations << 'foo() { a }'
+        mutations << 'foo() { b }'
+        mutations << 'foo() { }'
+        mutations << 'foo'
+      end
+
+      it_should_behave_like 'a mutator'
+    end
+
+    context 'with block args' do
+
+      let(:source) { 'foo { |a, b| }' }
+
+      before do
+        Mutant::Random.stub(:hex_string => :random)
+      end
+
+      let(:mutations) do 
+        mutations = []
+        mutations << 'foo'
+        mutations << 'foo() { |a, b| Object.new }'
+        mutations << 'foo() { |a, srandom| }'
+        mutations << 'foo() { |srandom, b| }'
+        mutations << 'foo() { |a| }'
+        mutations << 'foo() { |b| }'
+        mutations << 'foo() { || }'
+      end
+
+      it_should_behave_like 'a mutator'
+    end
+
+    context 'with block pattern args' do
+
+      before do
+        Mutant::Random.stub(:hex_string => :random)
       end
+
+      let(:source) { 'foo { |(a, b), c| }' }
+
+      let(:mutations) do 
+        mutations = []
+        mutations << 'foo() { || }'
+        mutations << 'foo() { |a, b, c| }'
+        mutations << 'foo() { |(a, b), c| Object.new }'
+        mutations << 'foo() { |(a, b)| }'
+        mutations << 'foo() { |c| }'
+        mutations << 'foo() { |(srandom, b), c| }'
+        mutations << 'foo() { |(a, srandom), c| }'
+        mutations << 'foo() { |(a, b), srandom| }'
+        mutations << 'foo'
+      end
+
+      it_should_behave_like 'a mutator'
     end
+
   end
 end

data/spec/unit/mutant/strategy/rspec/example_lookup/spec_file_spec.rb

@@ -17,7 +17,7 @@ describe Mutant::Strategy::Rspec::ExampleLookup, '#spec_file' do
   end
 
   context 'negation operator' do
-    let(:method_name) { :! }
+    let(:method_name) { :'!' }
     let(:expected_spec_file) { 'negation_operator_spec.rb' }
 
     it_should_behave_like 'Mutant::Strategy::Rspec::ExampleLookup#spec_file'
@@ -164,7 +164,7 @@ describe Mutant::Strategy::Rspec::ExampleLookup, '#spec_file' do
   end
 
   context 'with nomatch operator method' do
-    let(:method_name) { :!~ }
+    let(:method_name) { :'!~' }
     let(:expected_spec_file) { 'nomatch_operator_spec.rb' }
 
     it_should_behave_like 'Mutant::Strategy::Rspec::ExampleLookup#spec_file'
@@ -178,7 +178,7 @@ describe Mutant::Strategy::Rspec::ExampleLookup, '#spec_file' do
   end
 
   context 'with inequality operator method' do
-    let(:method_name) { :!= }
+    let(:method_name) { :'!=' }
     let(:expected_spec_file) { 'inequality_operator_spec.rb' }
 
     it_should_behave_like 'Mutant::Strategy::Rspec::ExampleLookup#spec_file'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mutant
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.2.5
   prerelease: 
 platform: ruby
 authors:
@@ -18,7 +18,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.2.0
+        version: 0.2.4
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -26,7 +26,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.2.0
+        version: 0.2.4
 - !ruby/object:Gem::Dependency
   name: ice_nine
   requirement: !ruby/object:Gem::Requirement
@@ -91,6 +91,22 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: 0.0.3
+- !ruby/object:Gem::Dependency
+  name: mbj-inflector
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 0.0.1
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 0.0.1
 - !ruby/object:Gem::Dependency
   name: equalizer
   requirement: !ruby/object:Gem::Requirement
@@ -173,10 +189,6 @@ files:
 - lib/mutant/differ.rb
 - lib/mutant/helper.rb
 - lib/mutant/inflector.rb
-- lib/mutant/inflector/defaults.rb
-- lib/mutant/inflector/inflections.rb
-- lib/mutant/inflector/methods.rb
-- lib/mutant/inflector/version.rb
 - lib/mutant/killer.rb
 - lib/mutant/killer/forking.rb
 - lib/mutant/killer/rspec.rb
@@ -201,6 +213,7 @@ files:
 - lib/mutant/mutator/node/block.rb
 - lib/mutant/mutator/node/define.rb
 - lib/mutant/mutator/node/if_statement.rb
+- lib/mutant/mutator/node/iter_19.rb
 - lib/mutant/mutator/node/literal.rb
 - lib/mutant/mutator/node/literal/array.rb
 - lib/mutant/mutator/node/literal/boolean.rb
@@ -214,6 +227,7 @@ 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/receiver_case.rb
 - lib/mutant/mutator/node/return.rb
@@ -231,7 +245,6 @@ files:
 - lib/mutant/strategy/rspec/example_lookup.rb
 - lib/mutant/subject.rb
 - lib/mutant/support/method_object.rb
-- locator.rb
 - mutant.gemspec
 - spec/integration/mutant/differ_spec.rb
 - spec/integration/mutant/loader_spec.rb
@@ -255,7 +268,6 @@ files:
 - spec/unit/mutant/cli/class_methods/new_spec.rb
 - spec/unit/mutant/cli/class_methods/run_spec.rb
 - spec/unit/mutant/context/root_spec.rb
-- spec/unit/mutant/context/scope/class_methods/build_spec.rb
 - spec/unit/mutant/context/scope/root_spec.rb
 - spec/unit/mutant/context/scope/unqualified_name_spec.rb
 - spec/unit/mutant/killer/fail_ques_spec.rb
@@ -368,7 +380,6 @@ test_files:
 - spec/unit/mutant/cli/class_methods/new_spec.rb
 - spec/unit/mutant/cli/class_methods/run_spec.rb
 - spec/unit/mutant/context/root_spec.rb
-- spec/unit/mutant/context/scope/class_methods/build_spec.rb
 - spec/unit/mutant/context/scope/root_spec.rb
 - spec/unit/mutant/context/scope/unqualified_name_spec.rb
 - spec/unit/mutant/killer/fail_ques_spec.rb