[bazel+js]: Avoid shadowing directory with test name by shs96c · Pull Request #16784 · SeleniumHQ/selenium
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<--
added
B-atoms
labels
Dec 23, 2025
shs96c
deleted the
js-firefox-tests
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters