mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
Nobody sucks so nobody should call this awful method name
This commit is contained in:
parent
9424b48d39
commit
e81f3c210e
9 changed files with 12 additions and 8 deletions
|
@ -55,4 +55,4 @@ require 'mocha/setup' # FIXME: stop using mocha
|
|||
# FIXME: we have tests that depend on run order, we should fix that and
|
||||
# remove this method call.
|
||||
require 'active_support/test_case'
|
||||
ActiveSupport::TestCase.i_suck_and_my_tests_are_order_dependent!
|
||||
ActiveSupport::TestCase.my_tests_are_order_dependent!
|
||||
|
|
|
@ -501,4 +501,4 @@ end
|
|||
# FIXME: we have tests that depend on run order, we should fix that and
|
||||
# remove this method call.
|
||||
require 'active_support/test_case'
|
||||
ActiveSupport::TestCase.i_suck_and_my_tests_are_order_dependent!
|
||||
ActiveSupport::TestCase.my_tests_are_order_dependent!
|
||||
|
|
|
@ -344,4 +344,4 @@ require 'mocha/setup' # FIXME: stop using mocha
|
|||
# FIXME: we have tests that depend on run order, we should fix that and
|
||||
# remove this method call.
|
||||
require 'active_support/test_case'
|
||||
ActiveSupport::TestCase.i_suck_and_my_tests_are_order_dependent!
|
||||
ActiveSupport::TestCase.my_tests_are_order_dependent!
|
||||
|
|
|
@ -17,4 +17,4 @@ require 'mocha/setup' # FIXME: stop using mocha
|
|||
# FIXME: we have tests that depend on run order, we should fix that and
|
||||
# remove this method call.
|
||||
require 'active_support/test_case'
|
||||
ActiveSupport::TestCase.i_suck_and_my_tests_are_order_dependent!
|
||||
ActiveSupport::TestCase.my_tests_are_order_dependent!
|
||||
|
|
|
@ -205,4 +205,4 @@ require 'mocha/setup' # FIXME: stop using mocha
|
|||
# FIXME: we have tests that depend on run order, we should fix that and
|
||||
# remove this method call.
|
||||
require 'active_support/test_case'
|
||||
ActiveSupport::TestCase.i_suck_and_my_tests_are_order_dependent!
|
||||
ActiveSupport::TestCase.tests_are_order_dependent!
|
||||
|
|
|
@ -1,6 +1,6 @@
|
|||
* Fix ActiveSupport::TestCase not to order users' test cases by default.
|
||||
If this change breaks your tests because your tests are order dependent, you need to explicitly call
|
||||
ActiveSupport::TestCase.i_suck_and_my_tests_are_order_dependent! at the top of your tests.
|
||||
ActiveSupport::TestCase.my_tests_are_order_dependent! at the top of your tests.
|
||||
|
||||
*Akira Matsuda*
|
||||
|
||||
|
|
|
@ -15,6 +15,10 @@ module ActiveSupport
|
|||
class TestCase < ::Minitest::Test
|
||||
Assertion = Minitest::Assertion
|
||||
|
||||
class << self
|
||||
alias :my_tests_are_order_dependent! :i_suck_and_my_tests_are_order_dependent!
|
||||
end
|
||||
|
||||
alias_method :method_name, :name
|
||||
|
||||
include ActiveSupport::Testing::TaggedLogging
|
||||
|
|
|
@ -42,4 +42,4 @@ require 'mocha/setup' # FIXME: stop using mocha
|
|||
# FIXME: we have tests that depend on run order, we should fix that and
|
||||
# remove this method call.
|
||||
require 'active_support/test_case'
|
||||
ActiveSupport::TestCase.i_suck_and_my_tests_are_order_dependent!
|
||||
ActiveSupport::TestCase.my_tests_are_order_dependent!
|
||||
|
|
|
@ -30,7 +30,7 @@ end
|
|||
class ActiveSupport::TestCase
|
||||
# FIXME: we have tests that depend on run order, we should fix that and
|
||||
# remove this method call.
|
||||
self.i_suck_and_my_tests_are_order_dependent!
|
||||
self.my_tests_are_order_dependent!
|
||||
|
||||
private
|
||||
|
||||
|
|
Loading…
Reference in a new issue