I noted that in b0ee1bdf2650d7a8380d4e9be58bba8d9c5bd40e and related commits that the extensions to TestCase are getting pulled out; what's the correct thing to put into test/test_helper.rb?
--Matt Jones
I noted that in b0ee1bdf2650d7a8380d4e9be58bba8d9c5bd40e and related commits that the extensions to TestCase are getting pulled out; what's the correct thing to put into test/test_helper.rb?
--Matt Jones
Use ActiveSupport::TestCase instead of Test::Unit::TestCase
jeremy
Are you sure? When I changed that (in test/test_helper.rb), none of
the test methods ran, and I
don't see any activity in the log.
Just so you know, I have edge frozen from around five days ago and I
went through and replaced all Test::Unit::TestCase instances in my
test/ directory and all my tests still pass.
Found it - in activesupport/lib/active_support/test_case.rb, test/unit
needs to be required,
or the tests don't get run. Add "require 'test/unit'" after "rescue
LoadError".
--Matt
It's required in railties' test_help.
Otherwise, requiring 'active_support/test_case' will tests even if you didn't mean to.
jeremy
The relevant line (from railties test_help.rb) was deleted in commit
582aa2ead58eacffca13e7efe94235958ee4db1b
on November 7 - should that be put back?
--Matt
Yes; thanks: 5a4789e
jeremy