[bazel+js]: Avoid shadowing directory with test name by shs96c · Pull Request #16784 · SeleniumHQ/selenium

@shs96c

The test target name `test` collides with the source directory
`test/` containing the test HTML files. When Bazel builds the
runfiles tree:

* The default `test` target (named `test`) places its binary at
  `_main/javascript/atoms/test` (a file)
* The test HTML files should be at
  `_main/javascript/atoms/test/*.html` (in a directory)
* Since the binary occupies the test path as a file, the `test/`
  directory with HTML files cannot exist in the runfiles
  `Files.find()` returns 0 test files because the test directory
   doesn't exist (it's a file instead)

Renaming the tests avoids this problem.

Insert facepalm -->here<--

@selenium-ci added B-atoms

JavaScript chunks generated by Google closure

B-build

Includes scripting, bazel and CI integrations

labels

Dec 23, 2025

@shs96c shs96c deleted the js-firefox-tests branch

December 23, 2025 11:27