Use legacy runfiles to better support ruby

Author: shs96c

.bazelrc

@@ -7,13 +7,15 @@ try-import %workspace%/.bazelrc.windows.local
 # pieces.
 # https://github.com/bazelbuild/bazel/issues/20369
 # https://github.com/bazelbuild/bazel/issues/21491
-
 common --lockfile_mode=off
 
 # We need this for rust and closure targets
 common --enable_workspace
 common --noincompatible_disallow_struct_provider_syntax
 
+# This is required by the ruby targets
+common --legacy_external_runfiles
+
 # Ensure Windows support is accurate.
 
 startup --windows_enable_symlinks

rb/spec/BUILD.bazel

@@ -123,6 +123,6 @@ genrule(
     srcs = [],
     outs = ["java-location.txt"],
     cmd = "echo $(JAVA) > $@",
-    toolchains = ["@bazel_tools//tools/jdk:current_java_runtime"],
+    toolchains = ["@rules_java//toolchains:current_java_runtime"],
     visibility = ["//rb/spec:__subpackages__"],
 )

rb/spec/integration/selenium/webdriver/spec_support/test_environment.rb

@@ -112,7 +112,13 @@ def remote_server
         def bazel_java
           return unless ENV.key?('WD_BAZEL_JAVA_LOCATION')
 
-          File.expand_path(File.read(File.expand_path(ENV.fetch('WD_BAZEL_JAVA_LOCATION'))).chomp)
+          path = File.expand_path(File.read(File.expand_path(ENV.fetch('WD_BAZEL_JAVA_LOCATION'))).chomp)
+
+          if !File.exist?(path)
+            raise "Java not found at #{path}"
+          end
+
+          path
         end
 
         def rbe?

rb/spec/tests.bzl

@@ -174,6 +174,7 @@ def rb_integration_test(name, srcs, deps = [], data = [], browsers = BROWSERS.ke
             data = BROWSERS[browser]["data"] + data + [
                 "//common/src/web",
                 "//java/src/org/openqa/selenium/grid:selenium_server_deploy.jar",
+                "//java/src/org/openqa/selenium/grid:selenium_server",
                 "//rb/spec:java-location",
                 "@bazel_tools//tools/jdk:current_java_runtime",
             ],