Permalink
Browse files

[#5406 state:resolved] calling the correct method on minitest to obta…

…in the test name
  • Loading branch information...
tenderlove committed Oct 4, 2010
1 parent 86571a1 commit 4a77431b9a9b60a0280cee7e88a5ef5974057015
Showing with 38 additions and 0 deletions.
  1. +38 −0 activesupport/test/test_case_test.rb
@@ -0,0 +1,38 @@
+require 'abstract_unit'
+
+module ActiveSupport
+ class TestCaseTest < ActiveSupport::TestCase
+ class FakeRunner
+ attr_reader :puked
+
+ def initialize
+ @puked = []
+ end
+
+ def puke(klass, name, e)
+ @puked << [klass, name, e]
+ end
+ end
+
+ if defined?(MiniTest::Assertions) && TestCase < MiniTest::Assertions
+ def test_callback_with_exception
+ tc = Class.new(TestCase) do
+ setup :bad_callback
+ def bad_callback; raise 'oh noes' end
+ def test_true; assert true end
+ end
+
+ test_name = 'test_true'
+ fr = FakeRunner.new
+
+ test = tc.new test_name
+ test.run fr
+ klass, name, exception = *fr.puked.first
+
+ assert_equal tc, klass
+ assert_equal test_name, name
+ assert_equal 'oh noes', exception.message
+ end
+ end
+ end
+end

2 comments on commit 4a77431

@rdp

This comment has been minimized.

Show comment Hide comment
@rdp

rdp Nov 22, 2010

so is this a bug in minitest that was being worked around?

rdp replied Nov 22, 2010

so is this a bug in minitest that was being worked around?

@tenderlove

This comment has been minimized.

Show comment Hide comment
@tenderlove

tenderlove Nov 22, 2010

Member

No. I was adding test coverage for a way that Rails uses minitest.

Member

tenderlove replied Nov 22, 2010

No. I was adding test coverage for a way that Rails uses minitest.

Please sign in to comment.