mutant 0.3.0.rc1 → 0.3.0.rc2

data/spec/unit/mutant/mutator/node/literal/string_spec.rb

@@ -12,7 +12,7 @@ describe Mutant::Mutator::Node::Literal, 'string' do
   end
 
   before do
-    Mutant::Random.stub(:hex_string => random_string)
+    Mutant::Random.stub(hex_string: random_string)
   end
 
   it_should_behave_like 'a mutator'

data/spec/unit/mutant/mutator/node/literal/symbol_spec.rb

@@ -12,7 +12,7 @@ describe Mutant::Mutator::Node::Literal, 'symbol' do
   end
 
   before do
-    Mutant::Random.stub(:hex_string => random_string)
+    Mutant::Random.stub(hex_string: random_string)
   end
 
   it_should_behave_like 'a mutator'

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

@@ -3,8 +3,12 @@
 require 'spec_helper'
 
 describe Mutant::Mutator, 'masgn' do
-  let(:source)    { 'a, b = c, d' }
-  let(:mutations) { []            }
+  let(:source) { 'a, b = c, d' }
+
+  let(:mutations) do
+    mutations = []
+    mutations << 'nil'
+  end
 
   it_should_behave_like 'a mutator'
 end

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

@@ -14,6 +14,7 @@ describe Mutant::Mutator::Node::Generic, 'match_current_line' do
     mutations << 'true if nil'
     mutations << 'true if !//'
     mutations << 'true if /a\A/'
+    mutations << 'nil'
   end
 
   it_should_behave_like 'a mutator'

data/spec/unit/mutant/mutator/node/named_value/access/mutation_spec.rb

@@ -4,7 +4,7 @@ require 'spec_helper'
 
 describe Mutant::Mutator::Node::NamedValue::Access, 'mutations' do
   before do
-    Mutant::Random.stub(:hex_string => 'random')
+    Mutant::Random.stub(hex_string: 'random')
   end
 
   context 'global variable' do
@@ -17,6 +17,7 @@ describe Mutant::Mutator::Node::NamedValue::Access, 'mutations' do
       mutants << '$a'
       mutants << '$a = ::Object.new; $a'
       mutants << '$srandom = nil; $a'
+      mutants << 'nil; $a'
     end
 
     it_should_behave_like 'a mutator'
@@ -32,6 +33,7 @@ describe Mutant::Mutator::Node::NamedValue::Access, 'mutations' do
       mutants << '@@a'
       mutants << '@@a = ::Object.new; @@a'
       mutants << '@@srandom = nil; @@a'
+      mutants << 'nil; @@a'
     end
   end
 
@@ -45,6 +47,7 @@ describe Mutant::Mutator::Node::NamedValue::Access, 'mutations' do
       mutants << '@a'
       mutants << '@a = ::Object.new; @a'
       mutants << '@srandom = nil; @a'
+      mutants << 'nil; @a'
     end
 
     it_should_behave_like 'a mutator'
@@ -60,6 +63,7 @@ describe Mutant::Mutator::Node::NamedValue::Access, 'mutations' do
       mutants << 'a'
       mutants << 'a = ::Object.new; a'
       mutants << 'srandom = nil; a'
+      mutants << 'nil; a'
     end
 
     it_should_behave_like 'a mutator'

data/spec/unit/mutant/mutator/node/named_value/constant_assignment/mutation_spec.rb

@@ -4,14 +4,13 @@ require 'spec_helper'
 
 describe Mutant::Mutator::Node::NamedValue::VariableAssignment, 'mutations' do
   before do
-    Mutant::Random.stub(:hex_string => 'random')
+    Mutant::Random.stub(hex_string: 'random')
   end
 
   let(:source) { 'A = true' }
 
   let(:mutations) do
     mutations = []
-
     mutations << 'SRANDOM = true'
     mutations << 'A = false'
     mutations << 'A = nil'

data/spec/unit/mutant/mutator/node/named_value/variable_assignment/mutation_spec.rb

@@ -4,7 +4,7 @@ require 'spec_helper'
 
 describe Mutant::Mutator::Node::NamedValue::VariableAssignment, 'mutations' do
   before do
-    Mutant::Random.stub(:hex_string => 'random')
+    Mutant::Random.stub(hex_string: 'random')
   end
 
   context 'global variable' do
@@ -12,10 +12,10 @@ describe Mutant::Mutator::Node::NamedValue::VariableAssignment, 'mutations' do
 
     let(:mutations) do
       mutations = []
-
       mutations << '$srandom = true'
       mutations << '$a = false'
       mutations << '$a = nil'
+      mutations << 'nil'
     end
 
     it_should_behave_like 'a mutator'
@@ -26,10 +26,10 @@ describe Mutant::Mutator::Node::NamedValue::VariableAssignment, 'mutations' do
 
     let(:mutations) do
       mutations = []
-
       mutations << '@@srandom = true'
       mutations << '@@a = false'
       mutations << '@@a = nil'
+      mutations << 'nil'
     end
 
     it_should_behave_like 'a mutator'
@@ -40,10 +40,10 @@ describe Mutant::Mutator::Node::NamedValue::VariableAssignment, 'mutations' do
 
     let(:mutations) do
       mutations = []
-
       mutations << '@srandom = true'
       mutations << '@a = false'
       mutations << '@a = nil'
+      mutations << 'nil'
     end
 
     it_should_behave_like 'a mutator'
@@ -54,10 +54,10 @@ describe Mutant::Mutator::Node::NamedValue::VariableAssignment, 'mutations' do
 
     let(:mutations) do
       mutations = []
-
       mutations << 'srandom = true'
       mutations << 'a = false'
       mutations << 'a = nil'
+      mutations << 'nil'
     end
 
     it_should_behave_like 'a mutator'

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

@@ -5,7 +5,7 @@ require 'spec_helper'
 describe Mutant::Mutator::Node::Generic, 'op_asgn' do
   let(:random_fixnum) { 5 }
 
-  let(:source)  { '@a.b += 1' }
+  let(:source) { '@a.b += 1' }
 
   let(:mutations) do
     mutations = []
@@ -16,10 +16,11 @@ describe Mutant::Mutator::Node::Generic, 'op_asgn' do
     mutations << '@a += 1'
     mutations << '@a.b += 5'
     mutations << 'nil.b += 1'
+    mutations << 'nil'
   end
 
   before do
-    Mutant::Random.stub(:fixnum => random_fixnum)
+    Mutant::Random.stub(fixnum: random_fixnum)
   end
 
   it_should_behave_like 'a mutator'

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

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-describe Mutant::Mutator::Node::Generic, 'or_asgn' do
+describe Mutant::Mutator::Node::OpAsgn, 'or_asgn' do
   let(:random_fixnum) { 5        }
   let(:random_string) { 'random' }
 
@@ -16,10 +16,11 @@ describe Mutant::Mutator::Node::Generic, 'or_asgn' do
     mutations << 'a ||= -1'
     mutations << 'a ||= 2'
     mutations << 'a ||= 5'
+    mutations << 'nil'
   end
 
   before do
-    Mutant::Random.stub(:fixnum => random_fixnum, :hex_string => random_string)
+    Mutant::Random.stub(fixnum: random_fixnum, hex_string: random_string)
   end
 
   it_should_behave_like 'a mutator'

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

@@ -15,7 +15,7 @@ describe Mutant::Mutator::Node::Generic, 'rescue' do
   end
 
   before do
-    Mutant::Random.stub(:hex_string => 'random')
+    Mutant::Random.stub(hex_string: 'random')
   end
 
   pending do

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

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-describe Mutant::Mutator::Node::Generic, 'restarg' do
+describe Mutant::Mutator::Node::Restarg, 'restarg' do
   let(:source) { 'foo(*bar)' }
 
   let(:mutations) do
@@ -10,6 +10,8 @@ describe Mutant::Mutator::Node::Generic, 'restarg' do
     mutants << 'foo'
     mutants << 'foo(nil)'
     mutants << 'foo(bar)'
+    mutants << 'foo(*nil)'
+    mutants << 'nil'
   end
 
   it_should_behave_like 'a mutator'

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

@@ -7,18 +7,22 @@ describe Mutant::Mutator, 'return' do
   context 'return without value' do
     let(:source) { 'return' }
 
-    let(:mutations) { ['nil'] }
+    let(:mutations) do
+      mutations = []
+      mutations << 'nil'
+    end
 
     it_should_behave_like 'a mutator'
   end
 
   context 'return with value' do
-    let(:source) { 'return nil' }
+    let(:source) { 'return foo' }
 
     let(:mutations) do
       mutations = []
+      mutations << 'foo'
+      mutations << 'return nil'
       mutations << 'nil'
-      mutations << 'return ::Object.new'
     end
 
     it_should_behave_like 'a mutator'

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

@@ -15,6 +15,10 @@ describe Mutant::Mutator, 'send' do
       mutations << 'foo.gsub(b)'
       mutations << 'foo.gsub'
       mutations << 'foo.sub(a, b)'
+      mutations << 'foo.gsub(a, nil)'
+      mutations << 'foo.gsub(nil, b)'
+      mutations << 'nil.gsub(a, b)'
+      mutations << 'nil'
     end
 
     it_should_behave_like 'a mutator'
@@ -29,6 +33,9 @@ describe Mutant::Mutator, 'send' do
       mutations << 'foo.public_send(bar)'
       mutations << 'bar'
       mutations << 'foo'
+      mutations << 'foo.send(nil)'
+      mutations << 'nil.send(bar)'
+      mutations << 'nil'
     end
 
     it_should_behave_like 'a mutator'
@@ -40,7 +47,9 @@ describe Mutant::Mutator, 'send' do
     let(:mutations) do
       mutations = []
       mutations << 'foo ||= expression'
+      mutations << 'self.foo ||= nil'
       mutations << 'nil.foo ||= expression'
+      mutations << 'nil'
     end
 
     it_should_behave_like 'a mutator'
@@ -52,6 +61,7 @@ describe Mutant::Mutator, 'send' do
     let(:mutations) do
       mutations = []
       mutations << 'foo'
+      mutations << 'nil'
     end
 
     it_should_behave_like 'a mutator'
@@ -65,6 +75,8 @@ describe Mutant::Mutator, 'send' do
       mutations << 'foo'
       mutations << 'foo(nil)'
       mutations << 'foo(bar)'
+      mutations << 'foo(*nil)'
+      mutations << 'nil'
     end
 
     it_should_behave_like 'a mutator'
@@ -76,6 +88,7 @@ describe Mutant::Mutator, 'send' do
     let(:mutations) do
       mutations = []
       mutations << 'foo'
+      mutations << 'nil'
     end
 
     it_should_behave_like 'a mutator'
@@ -87,6 +100,7 @@ describe Mutant::Mutator, 'send' do
     let(:mutations) do
       mutations = []
       mutations << 'foo'
+      mutations << 'nil'
     end
 
     it_should_behave_like 'a mutator'
@@ -96,7 +110,12 @@ describe Mutant::Mutator, 'send' do
     context 'implicit' do
       let(:source) { 'foo' }
 
-      it_should_behave_like 'a noop mutator'
+      let(:mutations) do
+        mutations = []
+        mutations << 'nil'
+      end
+
+      it_should_behave_like 'a mutator'
     end
 
     context 'explict receiver' do
@@ -107,6 +126,7 @@ describe Mutant::Mutator, 'send' do
         mutations << 'foo'
         mutations << 'self'
         mutations << 'nil.foo'
+        mutations << 'nil'
       end
 
       it_should_behave_like 'a mutator'
@@ -132,6 +152,8 @@ describe Mutant::Mutator, 'send' do
       let(:mutations) do
         mutations = []
         mutations << 'foo'
+        mutations << 'nil.bar'
+        mutations << 'nil'
       end
 
       it_should_behave_like 'a mutator'
@@ -145,6 +167,8 @@ describe Mutant::Mutator, 'send' do
         mutations << 'self.class'
         mutations << 'self.foo'
         mutations << 'nil.class.foo'
+        mutations << 'nil.foo'
+        mutations << 'nil'
       end
 
       it_should_behave_like 'a mutator'
@@ -191,8 +215,9 @@ describe Mutant::Mutator, 'send' do
             mutations = []
             mutations << "foo.#{keyword}"
             mutations << 'foo'
-            mutations << 'nil'
             mutations << "foo.#{keyword}(::Object.new)"
+            mutations << "nil.#{keyword}(nil)"
+            mutations << 'nil'
           end
 
           it_should_behave_like 'a mutator'
@@ -209,6 +234,7 @@ describe Mutant::Mutator, 'send' do
         mutations << 'foo(nil)'
         mutations << 'foo(::Object.new, nil)'
         mutations << 'foo(nil, ::Object.new)'
+        mutations << 'nil'
       end
 
       it_should_behave_like 'a mutator'
@@ -224,6 +250,11 @@ describe Mutant::Mutator, 'send' do
           mutations << '(left - right)'
           mutations << 'left / foo'
           mutations << 'right / foo'
+          mutations << '(left - right) / nil'
+          mutations << '(left - nil) / foo'
+          mutations << '(nil - right) / foo'
+          mutations << 'nil / foo'
+          mutations << 'nil'
         end
 
         it_should_behave_like 'a mutator'
@@ -241,6 +272,7 @@ describe Mutant::Mutator, 'send' do
             mutations << "true  #{operator} nil"
             mutations << 'true'
             mutations << 'false'
+            mutations << 'nil'
           end
 
           it_should_behave_like 'a mutator'
@@ -252,6 +284,9 @@ describe Mutant::Mutator, 'send' do
             mutations = []
             mutations << 'left'
             mutations << 'right'
+            mutations << "left #{operator} nil"
+            mutations << "nil #{operator} right"
+            mutations << 'nil'
           end
 
           it_should_behave_like 'a mutator'

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

@@ -21,6 +21,7 @@ describe Mutant::Mutator, 'super' do
     let(:mutations) do
       mutations = []
       mutations << 'super'
+      mutations << 'nil'
     end
 
     it_should_behave_like 'a mutator'
@@ -35,6 +36,9 @@ describe Mutant::Mutator, 'super' do
       mutations << 'super()'
       mutations << 'super(foo)'
       mutations << 'super(bar)'
+      mutations << 'super(foo, nil)'
+      mutations << 'super(nil, bar)'
+      mutations << 'nil'
     end
 
     it_should_behave_like 'a mutator'

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

@@ -14,6 +14,9 @@ describe Mutant::Mutator::Node::While do
       mutations << 'while true; end'
       mutations << 'while false; foo; bar; end'
       mutations << 'while nil;   foo; bar; end'
+      mutations << 'while true;  foo; nil; end'
+      mutations << 'while true;  nil; bar; end'
+      mutations << 'nil'
     end
 
     it_should_behave_like 'a mutator'

data/spec/unit/mutant/predicate_spec.rb

@@ -0,0 +1,135 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+filter_helpers = proc do
+  let(:input_a) { double('Input A', foo: 'bar') }
+  let(:input_b) { double('Input B', foo: 'baz') }
+
+  let(:filter_a) do
+    input_a = self.input_a
+    Module.new do
+      define_singleton_method(:match?) do |input|
+        input == input_a
+      end
+    end
+  end
+
+  subject { object.match?(input) }
+end
+
+describe Mutant::Predicate::Whitelist do
+  instance_eval(&filter_helpers)
+
+  let(:object) { described_class.new(whitelist) }
+
+  describe '#match?' do
+
+    context 'with empty whitelist' do
+      let(:whitelist) { [] }
+
+      it 'accepts all inputs' do
+        expect(object.match?(input_a)).to be(false)
+        expect(object.match?(input_b)).to be(false)
+      end
+    end
+
+    context 'with non empty whitelist' do
+      let(:whitelist) { [filter_a] }
+
+      context 'with whitelisted input' do
+        let(:input) { input_a }
+
+        it { should be(true) }
+      end
+
+      context 'with non whitelisted input' do
+        let(:input) { input_b }
+
+        it { should be(false) }
+      end
+    end
+  end
+end
+
+describe Mutant::Predicate::Blacklist do
+  instance_eval(&filter_helpers)
+
+  let(:object) { described_class.new(whitelist) }
+
+  describe '#match?' do
+
+    context 'with empty whitelist' do
+      let(:whitelist) { [] }
+
+      it 'accepts all inputs' do
+        expect(object.match?(input_a)).to be(true)
+        expect(object.match?(input_b)).to be(true)
+      end
+    end
+
+    context 'with non empty whitelist' do
+      let(:whitelist) { [filter_a] }
+
+      context 'with whitelisted input' do
+        let(:input) { input_a }
+
+        it { should be(false) }
+      end
+
+      context 'with non whitelisted input' do
+        let(:input) { input_b }
+
+        it { should be(true) }
+      end
+    end
+  end
+end
+
+describe Mutant::Predicate::Attribute::Equality do
+  instance_eval(&filter_helpers)
+
+  let(:object) { described_class.new(attribute_name, expected_value) }
+  let(:input)   { double('Input', attribute_name => actual_value)      }
+
+  let(:attribute_name) { :foo    }
+  let(:expected_value) { 'value' }
+
+  describe '#match?' do
+
+    context 'not matching' do
+      let(:actual_value) { 'other-value' }
+      it { should be(false) }
+    end
+
+    context 'matching' do
+      let(:actual_value) { 'value' }
+      it { should be(true) }
+    end
+
+  end
+end
+
+describe Mutant::Predicate::Attribute::Regexp do
+  instance_eval(&filter_helpers)
+
+  let(:object) { described_class.new(attribute_name, expectation) }
+  let(:input)   { double('Input', attribute_name => actual_value)   }
+
+  let(:attribute_name) { :foo    }
+  let(:expectation)    { /\Avalue\z/ }
+
+  describe '#match?' do
+
+    context 'not matching' do
+      let(:actual_value) { 'other-value' }
+      it { should be(false) }
+    end
+
+    context 'matching' do
+      let(:actual_value) { 'value' }
+      it { should be(true) }
+    end
+
+  end
+end

data/spec/unit/mutant/runner/config/subjects_spec.rb

@@ -10,24 +10,24 @@ describe Mutant::Runner::Config, '#subjects' do
   let(:config) do
     double(
       'Config',
-      :class    => Mutant::Config,
-      :subjects => [subject_a, subject_b],
-      :strategy => strategy,
-      :reporter => reporter
+      class:    Mutant::Config,
+      subjects: [subject_a, subject_b],
+      strategy: strategy,
+      reporter: reporter
     )
   end
 
-  let(:reporter)  { double('Reporter')                   }
-  let(:strategy)  { double('Strategy')                   }
-  let(:subject_a) { double('Subject A')                  }
-  let(:subject_b) { double('Subject B')                  }
-  let(:runner_a)  { double('Runner A', :stop? => stop_a) }
-  let(:runner_b)  { double('Runner B', :stop? => stop_b) }
+  let(:reporter)  { double('Reporter')                }
+  let(:strategy)  { double('Strategy')                }
+  let(:subject_a) { double('Subject A')               }
+  let(:subject_b) { double('Subject B')               }
+  let(:runner_a)  { double('Runner A', stop?: stop_a) }
+  let(:runner_b)  { double('Runner B', stop?: stop_b) }
 
   before do
     strategy.stub(:setup)
     strategy.stub(:teardown)
-    reporter.stub(:report => reporter)
+    reporter.stub(report: reporter)
     Mutant::Runner.stub(:run).with(config, subject_a).and_return(runner_a)
     Mutant::Runner.stub(:run).with(config, subject_b).and_return(runner_b)
   end

data/spec/unit/mutant/runner/config/success_predicate_spec.rb

@@ -10,9 +10,9 @@ describe Mutant::Runner::Config, '#success?' do
   let(:config) do
     double(
       'Config',
-      :reporter => reporter,
-      :strategy => strategy,
-      :subjects => [subject_a, subject_b]
+      reporter: reporter,
+      strategy: strategy,
+      subjects: [subject_a, subject_b]
     )
   end
 
@@ -22,15 +22,15 @@ describe Mutant::Runner::Config, '#success?' do
   let(:subject_b) { double('Subject B') }
 
   let(:runner_a) do
-    double('Runner A', :stop? => stop_a, :success? => success_a)
+    double('Runner A', stop?: stop_a, success?: success_a)
   end
 
   let(:runner_b) do
-    double('Runner B', :stop? => stop_b, :success? => success_b)
+    double('Runner B', stop?: stop_b, success?: success_b)
   end
 
   before do
-    reporter.stub(:report => reporter)
+    reporter.stub(report: reporter)
     strategy.stub(:setup)
     strategy.stub(:teardown)
     Mutant::Runner.stub(:run).with(config, subject_a).and_return(runner_a)

data/spec/unit/mutant/runner/mutation/killer_spec.rb

@@ -8,24 +8,24 @@ describe Mutant::Runner::Mutation, '#killer' do
   let(:config) do
     double(
       'Config',
-      :fail_fast => fail_fast,
-      :reporter  => reporter,
-      :strategy  => strategy
+      fail_fast: fail_fast,
+      reporter:  reporter,
+      strategy:  strategy
     )
   end
 
-  let(:reporter)  { double('Reporter')                             }
-  let(:mutation)  { double('Mutation', :class => Mutant::Mutation) }
-  let(:strategy)  { double('Strategy')                             }
-  let(:killer)    { double('Killer', :success? => success)         }
-  let(:fail_fast) { false                                          }
-  let(:success)   { false                                          }
+  let(:reporter)  { double('Reporter')                          }
+  let(:mutation)  { double('Mutation', class: Mutant::Mutation) }
+  let(:strategy)  { double('Strategy')                          }
+  let(:killer)    { double('Killer', success?: success)         }
+  let(:fail_fast) { false                                       }
+  let(:success)   { false                                       }
 
   subject { object.killer }
 
   before do
-    reporter.stub(:report => reporter)
-    strategy.stub(:kill => killer)
+    reporter.stub(report: reporter)
+    strategy.stub(kill: killer)
   end
 
   it 'should call configuration to identify strategy' do