diff options
author | Jamis Buck <jamis@37signals.com> | 2011-01-19 15:55:30 -0700 |
---|---|---|
committer | Jamis Buck <jamis@37signals.com> | 2011-01-19 15:55:30 -0700 |
commit | 1de1dc02d7e7920bd243da6910baa3cac7002926 (patch) | |
tree | 37490bfcbfa6a372b6f48c6b6c8b7b2270d3a5fc /activesupport/test/test_case_test.rb | |
parent | 41f76946d02e571a2483eccb72632960125e122b (diff) | |
download | rails-1de1dc02d7e7920bd243da6910baa3cac7002926.tar.gz rails-1de1dc02d7e7920bd243da6910baa3cac7002926.tar.bz2 rails-1de1dc02d7e7920bd243da6910baa3cac7002926.zip |
Revert "scrub instance variables from test cases on teardown"
This reverts commit 79a06225ef0e148eb5da541711f74163b5efb18d.
Diffstat (limited to 'activesupport/test/test_case_test.rb')
-rw-r--r-- | activesupport/test/test_case_test.rb | 18 |
1 files changed, 0 insertions, 18 deletions
diff --git a/activesupport/test/test_case_test.rb b/activesupport/test/test_case_test.rb index 740c2351ea..b6a96b2ae7 100644 --- a/activesupport/test/test_case_test.rb +++ b/activesupport/test/test_case_test.rb @@ -74,23 +74,5 @@ module ActiveSupport assert_match %r{oh noes}, exception.message end - - def test_teardown_should_scrub_instance_variables - tc = Class.new(TestCase) do - def test_true; @alpha = "a"; assert_equal "a", @alpha; end - end - - test_name = 'test_true' - fr = FakeRunner.new - - test = tc.new test_name - test.run(fr) {} - - passed_var = IS_MINITEST ? :@passed : :@test_passed - ivars = test.instance_variables.map(&:to_sym) - - assert ivars.include?(passed_var), "#{passed_var} should not have been scrubbed" - assert !ivars.include?(:@alpha), "@alpha should have been scrubbed" - end end end |