Remove literal array mutation that adds an element

It turns out this mutation is not "universal". Meaning not in all scenarios this extra element triggers an edge case the specs have to cover.
mbj · Jul 8, 2013 · 30884c2 · 30884c2
1 parent 5090877
commit 30884c2
Show file tree

Hide file tree

Showing 4 changed files with 1 addition and 22 deletions.
diff --git a/config/flay.yml b/config/flay.yml
@@ -1,3 +1,3 @@
 ---
 threshold: 16
-total_score: 684
+total_score: 680
diff --git a/lib/mutant/mutator/node/literal/array.rb b/lib/mutant/mutator/node/literal/array.rb
@@ -34,7 +34,6 @@ def mutate_body
               emit_self(*dup_children)
               mutate_child(index)
             end
-            emit_self(s(:nil), *children)
           end
 
         end # Array

diff --git a/spec/unit/mutant/mutator/node/literal/array_spec.rb b/spec/unit/mutant/mutator/node/literal/array_spec.rb
@@ -21,9 +21,6 @@
 
     # Empty array
     mutations << '[]'
-
-    # Extra element
-    mutations << '[nil, true, false]'
   end
 
   it_should_behave_like 'a mutator'

diff --git a/spec/unit/mutant/mutator/node/literal/empty_array_spec.rb b/spec/unit/mutant/mutator/node/literal/empty_array_spec.rb