Skip to content

Commit 391229e

Browse files
committed
Fix force_skip patch
1 parent ad343c5 commit 391229e

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
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) || force_skip("'#{file}' was already loaded")
43+
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: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
# frozen_string_literal: true
22

3+
ActiveSupport::TestCase.alias_method :force_skip, :skip
4+
35
if ENV["BUILDKITE"]
46
require "minitest-ci"
57
ENV.delete("CI") # CI has affect on the applications, and we don't want it applied to the apps.
@@ -14,6 +16,5 @@ def skip(message = nil, *)
1416
"This should never happen on CI."
1517
end
1618
end
17-
ActiveSupport::TestCase.alias_method :skip, :force_skip
1819
ActiveSupport::TestCase.include(DisableSkipping)
1920
end

0 commit comments

Comments
 (0)