Permalink
Browse files

Remove extra test case.

Make the test description better reflect what is happening
  • Loading branch information...
mikegehard authored and Mike Gehard committed Jun 19, 2012
1 parent 1aa052c commit db03f13255bae7312e661c9bed56be4709fd51f8
Showing with 4 additions and 7 deletions.
  1. +4 −7 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
- assert_equal "/my/class.rb", @bc.clean([ "/my/prefix/my/class.rb" ]).first
+ test "backtrace should filter all lines in a backtrace, removing prefixes" do
+ 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

0 comments on commit db03f13

Please sign in to comment.