Skip to content

Commit ad343c5

Browse files
committed
Allow to force skip tests
This is useful for when a test depends on code not being loaded that might be loaded by another test.
1 parent 4544b0d commit ad343c5

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

activesupport/test/core_ext/object/json_gem_encoding_test.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ def to_json(*)
4040

4141
private
4242
def require_or_skip(file)
43-
require(file) || skip("'#{file}' was already loaded")
43+
require(file) || force_skip("'#{file}' was already loaded")
4444
end
4545

4646
def assert_same_with_or_without_active_support(subject)

tools/test_common.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,5 +14,6 @@ def skip(message = nil, *)
1414
"This should never happen on CI."
1515
end
1616
end
17+
ActiveSupport::TestCase.alias_method :skip, :force_skip
1718
ActiveSupport::TestCase.include(DisableSkipping)
1819
end

0 commit comments

Comments
 (0)