kotovalexarian/free_mutant
1
0
Fork 0
Code Releases Activity

Remove nil => Object.new mutation

Browse source 
Mutation operators should *always* mutate to a less powerfull structure
/ less complex semantics. To force tests to prove complex semantics are
needed. The mutation removed with that commit violates that principle.
This commit is contained in:
Markus Schirp 2014-05-09 20:09:07 +00:00
parent 563075403f
commit a98319650b
4 changed files with 2 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
lib/mutant/mutator/node/literal/nil.rb
View file

@ -18,7 +18,6 @@ module Mutant
# @api private
#
def dispatch
emit(NEW_OBJECT)
end
end # Nil

4
spec/unit/mutant/mutator/node/literal/nil_spec.rb
View file

@ -3,8 +3,8 @@
require 'spec_helper'
describe Mutant::Mutator::Node::Literal, 'nil' do
let(:source) { 'nil' }
let(:mutations) { ['::Object.new'] }
let(:source) { 'nil' }
let(:mutations) { [] }
it_should_behave_like 'a mutator'
end

4
spec/unit/mutant/mutator/node/named_value/access_spec.rb
View file

@ -15,7 +15,6 @@ describe Mutant::Mutator::Node::NamedValue::Access, 'mutations' do
mutants << '$a = nil; nil'
mutants << '$a = nil'
mutants << '$a'
mutants << '$a = ::Object.new; $a'
mutants << '$srandom = nil; $a'
mutants << 'nil; $a'
end
@ -31,7 +30,6 @@ describe Mutant::Mutator::Node::NamedValue::Access, 'mutations' do
mutants << '@@a = nil; nil'
mutants << '@@a = nil'
mutants << '@@a'
mutants << '@@a = ::Object.new; @@a'
mutants << '@@srandom = nil; @@a'
mutants << 'nil; @@a'
end
@ -45,7 +43,6 @@ describe Mutant::Mutator::Node::NamedValue::Access, 'mutations' do
mutants << '@a = nil; nil'
mutants << '@a = nil'
mutants << '@a'
mutants << '@a = ::Object.new; @a'
mutants << '@srandom = nil; @a'
mutants << 'nil; @a'
end
@ -60,7 +57,6 @@ describe Mutant::Mutator::Node::NamedValue::Access, 'mutations' do
mutants = []
mutants << 'a = nil; nil'
mutants << 'a = nil'
mutants << 'a = ::Object.new; a'
# TODO: fix invalid AST
# These ASTs are not valid and should NOT be emitted
# Mutations of lvarasgn need to be special cased to avoid this.

5
spec/unit/mutant/mutator/node/send_spec.rb
View file

@ -237,7 +237,6 @@ describe Mutant::Mutator, 'send' do
mutations = []
mutations << 'foo'
mutations << 'nil'
mutations << 'foo(::Object.new)'
end
it_should_behave_like 'a mutator'
@ -252,7 +251,6 @@ describe Mutant::Mutator, 'send' do
mutations << 'self.foo'
mutations << 'foo(nil)'
mutations << 'nil'
mutations << 'self.foo(::Object.new)'
mutations << 'nil.foo(nil)'
end
@ -268,7 +266,6 @@ describe Mutant::Mutator, 'send' do
mutations = []
mutations << "foo.#{keyword}"
mutations << 'foo'
mutations << "foo.#{keyword}(::Object.new)"
mutations << "nil.#{keyword}(nil)"
mutations << 'nil'
end
@ -285,8 +282,6 @@ describe Mutant::Mutator, 'send' do
mutations = []
mutations << 'foo()'
mutations << 'foo(nil)'
mutations << 'foo(::Object.new, nil)'
mutations << 'foo(nil, ::Object.new)'
mutations << 'nil'
end