[js + bazel]: Run Node browser tests on the RBE (#14194)

If you `cd` into `javascript/node/selenium-webdriver` a `pnpm`-based workflow should still work.

This PR also removes a now unused ci-javascript.yml file: everything that this used to do is now handled by the RBE.

Author: shs96c

.bazelrc

@@ -39,6 +39,8 @@ build --nobuild_runfile_links
 
 # More JS magic
 build --experimental_allow_unresolved_symlinks
+# Avoid a warning about directory tracking being unsound
+startup --host_jvm_args=-DBAZEL_TRACK_SOURCE_DIRECTORIES=1
 
 # Required for faster TS builds
 build --@aspect_rules_ts//ts:skipLibCheck=always

.github/workflows/ci-javascript.yml

@@ -59,17 +59,6 @@ jobs:
       contains(join(github.event.commits.*.message), '[java]') ||
       contains(github.event.pull_request.title, '[java]')
 
-  javascript:
-    name: JavaScript
-    needs: check
-    uses: ./.github/workflows/ci-javascript.yml
-    if: >
-      github.event_name == 'schedule' ||
-      github.event_name == 'workflow_dispatch' ||
-      contains(needs.check.outputs.targets, '//javascript') ||
-      contains(join(github.event.commits.*.message), '[js]') ||
-      contains(github.event.pull_request.title, '[js]')
-
   python:
     name: Python
     needs: check

.github/workflows/ci.yml

@@ -19,6 +19,14 @@
 -//javascript/atoms:test-chrome
 -//javascript/atoms:test-edge
 -//javascript/atoms:test-firefox-beta
+-//javascript/node/selenium-webdriver:test-bidi-network-test.js-chrome
+-//javascript/node/selenium-webdriver:test-builder-test.js-chrome
+-//javascript/node/selenium-webdriver:test-builder-test.js-firefox
+-//javascript/node/selenium-webdriver:test-chrome-devtools-test.js-chrome
+-//javascript/node/selenium-webdriver:test-chrome-options-test.js-chrome
+-//javascript/node/selenium-webdriver:test-chrome-service-test.js-chrome
+-//javascript/node/selenium-webdriver:test-firefox-options-test.js-firefox
+-//javascript/node/selenium-webdriver:test-lib-capabilities-test.js-chrome
 -//py:test-chrome-test/selenium/webdriver/chrome/chrome_launcher_tests.py
 -//py:test-chrome-test/selenium/webdriver/chrome/chrome_service_tests.py
 -//py:test-chrome-test/selenium/webdriver/chrome/proxy_tests.py

.skipped-tests

@@ -39,7 +39,7 @@ linter.configure(
 linter.register(name = "rust-rustfmt")
 
 node = use_extension("@rules_nodejs//nodejs:extensions.bzl", "node")
-node.toolchain(node_version = "18.17.0")
+node.toolchain(node_version = "20.9.0")
 
 pnpm = use_extension("@aspect_rules_js//npm:extensions.bzl", "pnpm", dev_dependency = True)
 use_repo(pnpm, "pnpm")

MODULE.bazel

@@ -202,7 +202,7 @@ js_library(
         url = "https://storage.googleapis.com/chrome-for-testing-public/126.0.6478.62/linux64/chrome-linux64.zip",
         sha256 = "6f42045c9134bdd3cfcea03fb54876ad061da998cabd624c92a25ac6811cd737",
         build_file_content = """
-load("@aspect_rules_js//js:defs.bzl", "js_library")
+load("@aspect_bazel_lib//lib:copy_to_bin.bzl", "copy_to_bin")
 package(default_visibility = ["//visibility:public"])
 
 filegroup(
@@ -212,9 +212,9 @@ filegroup(
 
 exports_files(["chrome-linux64/chrome"])
 
-js_library(
+copy_to_bin(
     name = "chrome-js",
-    data = [":files"],
+    srcs = [":files"],
 )
 """,
     )
@@ -229,14 +229,14 @@ js_library(
             "mv 'Chrome.app/Contents/MacOS/Google Chrome for Testing' Chrome.app/Contents/MacOS/Chrome",
         ],
         build_file_content = """
-load("@aspect_rules_js//js:defs.bzl", "js_library")
+load("@aspect_bazel_lib//lib:copy_to_bin.bzl", "copy_to_bin")
 package(default_visibility = ["//visibility:public"])
 
 exports_files(["Chrome.app"])
 
-js_library(
+copy_to_bin(
     name = "chrome-js",
-    data = glob(["Chrome.app/**/*"]),
+    srcs = glob(["Chrome.app/**/*"]),
 )
 """,
     )
@@ -247,14 +247,14 @@ js_library(
         sha256 = "a304e692480c726bae846bf6dee36316305d0b8f4826dfafeab8d6bbbc6e7214",
         strip_prefix = "chromedriver-linux64",
         build_file_content = """
-load("@aspect_rules_js//js:defs.bzl", "js_library")
+load("@aspect_bazel_lib//lib:copy_to_bin.bzl", "copy_to_bin")
 package(default_visibility = ["//visibility:public"])
 
 exports_files(["chromedriver"])
 
-js_library(
+copy_to_bin(
     name = "chromedriver-js",
-    data = ["chromedriver"],
+    srcs = ["chromedriver"],
 )
 """,
     )
@@ -265,14 +265,14 @@ js_library(
         sha256 = "05665a6b5fb71141b5d519998e36f01b650f661942a56dd7f6b929896ae38333",
         strip_prefix = "chromedriver-mac-x64",
         build_file_content = """
-load("@aspect_rules_js//js:defs.bzl", "js_library")
+load("@aspect_bazel_lib//lib:copy_to_bin.bzl", "copy_to_bin")
 package(default_visibility = ["//visibility:public"])
 
 exports_files(["chromedriver"])
 
-js_library(
+copy_to_bin(
     name = "chromedriver-js",
-    data = ["chromedriver"],
+    srcs = ["chromedriver"],
 )
 """,
     )

common/repositories.bzl

@@ -242,7 +242,7 @@ void canLocateNodesInAGivenSandbox() {
 
     String sharedId = (String) ((RemoteValue) sharedIdMap.get("sharedId")).getValue().get();
     assertThat(sharedId).isEqualTo(nodeId);
-  } 
+  }
 
   @AfterEach
   public void quitDriver() {

java/test/org/openqa/selenium/bidi/browsingcontext/LocateNodesTest.java

@@ -7,6 +7,7 @@ load("@npm//javascript/node/selenium-webdriver:prettier/package_json.bzl", prett
 load("@rules_pkg//pkg:pkg.bzl", "pkg_tar")
 load("//common:defs.bzl", "copy_file")
 load("//javascript:defs.bzl", "mocha_test")
+load("//javascript/private:browsers.bzl", "BROWSERS")
 
 npm_link_all_packages(name = "node_modules")
 
@@ -39,6 +40,7 @@ js_library(
         "bidi/*.js",
     ]),
     deps = [
+        ":node_modules/@bazel/runfiles",
         ":node_modules/jszip",
         ":node_modules/tmp",
         ":node_modules/ws",
@@ -117,6 +119,9 @@ js_library(
             "test/**/*_test.js",
         ],
     ),
+    deps = [
+        ":node_modules/@bazel/runfiles",
+    ],
 )
 
 mocha_test(
@@ -152,41 +157,31 @@ mocha_test(
             ],
             chdir = package_name(),
             data = [
-                "test-data",
                 ":large-test-srcs",
-                ":node_modules/@bazel/runfiles",
                 ":node_modules/express",
-                ":node_modules/jszip",
+                ":node_modules/mocha",
+                ":node_modules/mocha-junit-reporter",
                 ":node_modules/multer",
                 ":node_modules/serve-index",
                 ":node_modules/sinon",
-                ":node_modules/tmp",
-                ":node_modules/ws",
+                ":test-data",
                 "//:node_modules/selenium-webdriver",
                 "//common/extensions:js-lib",
                 "//common/src/web:js-lib",
-                "//javascript/node/selenium-webdriver/lib/atoms:find-elements",
-                "//javascript/node/selenium-webdriver/lib/atoms:get_attribute",
-                "//javascript/node/selenium-webdriver/lib/atoms:is_displayed",
-                "//javascript/node/selenium-webdriver/lib/atoms:mutation-listener",
-            ],
+            ] + BROWSERS[browser]["data"],
             env = {
                 "SELENIUM_BROWSER": browser,
-            },
+            } | BROWSERS[browser]["env"],
             tags = [
                 browser,
                 "browser-test",
                 "no-sandbox",
                 "requires-network",
-                "skip-remote",
             ],
         ),
     ]
     for test in LARGE_TESTS
-    for browser in [
-        "chrome",
-        "firefox",
-    ]
+    for browser in BROWSERS.keys()
 ]
 
 # npm_package does not pick up filegroup sources.

javascript/node/selenium-webdriver/BUILD.bazel

@@ -11,7 +11,7 @@
     "destination": "../../../build/docs/api/javascript/",
     "recurse": true,
     "verbose": true,
-    "template": "../../../node_modules/clean-jsdoc-theme",
+    "template": "./node_modules/clean-jsdoc-theme",
     "theme_opts": {
       "default_theme": "dark",
       "displayModuleHeader": true,

javascript/node/selenium-webdriver/jsdoc_conf.json

@@ -20,6 +20,7 @@ const crypto = require('node:crypto')
 class PinnedScript {
   constructor(script) {
     this.scriptSource_ = script
+    // eslint-disable-next-line
     this.scriptHandle_ = crypto.randomUUID().replace(/-/gi, '')
   }