Don't assume that backtraces are Array subclasses when tweaking them

mhennemeyer · Dec 19, 2008 · 6cb6a63 · 6cb6a63
1 parent 699cd19
commit 6cb6a63
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/spec/runner/backtrace_tweaker.rb b/lib/spec/runner/backtrace_tweaker.rb
@@ -40,7 +40,7 @@ class QuietBacktraceTweaker < BacktraceTweaker
 
       def tweak_backtrace(error)
         return if error.backtrace.nil?
-        error.backtrace.collect! do |message|
+        tweaked = error.backtrace.collect do |message|
           clean_up_double_slashes(message)
           kept_lines = message.split("\n").select do |line|
             IGNORE_PATTERNS.each do |ignore|
@@ -49,7 +49,7 @@ def tweak_backtrace(error)
           end
           kept_lines.empty?? nil : kept_lines.join("\n")
         end
-        error.backtrace.compact!
+        error.set_backtrace(tweaked.select {|line| line})
       end
     end
   end