You give jruby_skip & rubinius_skip a good name

This hack prevails everywhere in the codebase by being copy & pasted, and it's actually not a negative thing but a necessary thing for framework implementors,
so it should better have a name and be a thing.

And with this commit, activesupport/test/abstract_unit.rb now doesn't silently autoload AS::TestCase,
so we're ready to establish clearner environment for running AS tests (probably in later commits)

1 files changed, 18 insertions, 0 deletions

diff --git a/activesupport/lib/active_support/testing/alternative_runtime_skipper.rb b/activesupport/lib/active_support/testing/alternative_runtime_skipper.rb
new file mode 100644
index 0000000000..f9e6eaff2f
--- /dev/null
+++ b/activesupport/lib/active_support/testing/alternative_runtime_skipper.rb
@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+module ActiveSupport
+  module Testing
+    module AlternativeRuntimeSkipper
+      private
+        # Skips the current run on Rubinius using Minitest::Assertions#skip
+        def rubinius_skip(message = "")
+          skip message if RUBY_ENGINE == "rbx"
+        end
+
+        # Skips the current run on JRuby using Minitest::Assertions#skip
+        def jruby_skip(message = "")
+          skip message if defined?(JRUBY_VERSION)
+        end
+    end
+  end
+end