Merge pull request #6784 from msgehard/active_support_cleanup

Remove extra test case.
rails · Jun 19, 2012 · 82cf95c · 82cf95c
2 parents b2ca26a + db03f13
commit 82cf95c
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/activesupport/test/clean_backtrace_test.rb b/activesupport/test/clean_backtrace_test.rb
@@ -6,8 +6,10 @@ def setup
     @bc.add_filter { |line| line.gsub("/my/prefix", '') }
   end
 
-  test "backtrace should not contain prefix when it has been filtered out" do
+  test "backtrace should filter all lines in a backtrace, removing prefixes" do
-    assert_equal "/my/class.rb", @bc.clean([ "/my/prefix/my/class.rb" ]).first
+    assert_equal \
+        ["/my/class.rb", "/my/module.rb"],
+        @bc.clean(["/my/prefix/my/class.rb", "/my/prefix/my/module.rb"])
   end
 
   test "backtrace cleaner should allow removing filters" do
@@ -19,11 +21,6 @@ def setup
     assert_equal "/my/other_prefix/my/class.rb", @bc.clean([ "/my/other_prefix/my/class.rb" ]).first
   end
 
-  test "backtrace should filter all lines in a backtrace" do
-    assert_equal \
-      ["/my/class.rb", "/my/module.rb"],
-      @bc.clean([ "/my/prefix/my/class.rb", "/my/prefix/my/module.rb" ])
-  end
 end
 
 class BacktraceCleanerSilencerTest < ActiveSupport::TestCase