Skip to content

[java] add the LICENSE and NOTICE files to the jars created#17601

Open
joerg1985 wants to merge 1 commit into
trunkfrom
issue-16113
Open

[java] add the LICENSE and NOTICE files to the jars created#17601
joerg1985 wants to merge 1 commit into
trunkfrom
issue-16113

Conversation

@joerg1985

@joerg1985 joerg1985 commented May 31, 2026

Copy link
Copy Markdown
Member

🔗 Related Issues

#16113

💥 What does this PR do?

Add the LICENSE and NOTICE files to the jars created and avoid they are disposed while building a merged jar.

🔧 Implementation Notes

The change to java/private/merge_jars.bzl is currently not needed, but this might change as we regulary update the dependencies and nobody is looking to deep into the merged jars after this. Therefore i added a inline comment on the intention of this change.

🤖 AI assistance

  • No substantial AI assistance used
  • AI assisted (complete below)
    • Tool(s):
    • What was generated:
    • I reviewed all AI output and can explain the change

💡 Additional Considerations

When merging the jars we might have duplicates of the files, but this can be ignored in my mind.
We must have them at least once (enforced by the license) but i think there is no limitation.

🔄 Types of changes

  • Cleanup (formatting, renaming)
  • Bug fix (backwards compatible)

@selenium-ci selenium-ci added C-java Java Bindings B-build Includes scripting, bazel and CI integrations labels May 31, 2026
@qodo-code-review

qodo-code-review Bot commented May 31, 2026

Copy link
Copy Markdown
Contributor

Review Summary by Qodo

Include LICENSE and NOTICE files in generated JAR artifacts

✨ Enhancement 🐞 Bug fix

Grey Divider

Walkthroughs

Description 
• Add LICENSE and NOTICE files to generated JAR files
• Create custom java_binary wrapper to include legal files
• Update java_library to support resources parameter
• Preserve license files when merging JARs with duplicate handling
Diagram 
flowchart LR
  A["BUILD.bazel<br/>legal-stuff genrule"] --> B["java_binary.bzl<br/>custom wrapper"]
  A --> C["java_library.bzl<br/>resources param"]
  B --> D["JAR files<br/>with META-INF/LICENSE"]
  C --> D
  E["merge_jars.bzl<br/>duplicate handling"] --> D
Loading

Grey Divider

File Changes

1. java/defs.bzl ⚙️ Configuration changes +1/-1

Redirect java_binary to custom implementation

• Removed _java_binary import from @rules_java//java:defs.bzl
• Added new import of _java_binary from custom //java/private:java_binary.bzl
• Reorganized load statements for better structure

java/defs.bzl

2. java/private/java_binary.bzl ✨ Enhancement +11/-0

New custom java_binary wrapper with legal files

• New file creating custom java_binary wrapper function
• Wraps @contrib_rules_jvm java_binary with automatic resource injection
• Injects //:legal-stuff target containing LICENSE and NOTICE files

java/private/java_binary.bzl

3. java/private/java_library.bzl ✨ Enhancement +2/-0

Add resources parameter with legal files injection

• Added resources parameter to java_library function signature
• Passes resources parameter to underlying _java_library call
• Automatically injects //:legal-stuff target with user-provided resources

java/private/java_library.bzl

View more (2)
4. java/private/merge_jars.bzl 🐞 Bug fix +3/-0

Allow duplicate license files in merged JARs

• Added --allow-duplicate flag to merge_jars command
• Pattern matches COPYRIGHT, LEGAL, LICENSE, PRIVACY, and NOTICE files
• Allows duplicate license files when merging multiple JARs
• Includes inline comment explaining the intention

java/private/merge_jars.bzl

5. BUILD.bazel ✨ Enhancement +14/-0

Add genrule to package legal files for JARs

• Created new legal-stuff genrule target
• Copies LICENSE and NOTICE files to META-INF directory structure
• Makes legal files available as a reusable resource for JAR packaging
• Marked with public visibility for use across build targets

BUILD.bazel

Grey Divider

Qodo Logo

@qodo-code-review

qodo-code-review Bot commented May 31, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Code Review by Qodo

🐞 Bugs (4) 📘 Rule violations (4)

Grey Divider


Action required

1. java_binary macro not buildified 📘 Rule violation ⚙ Maintainability
Description 
The new Starlark file java/private/java_binary.bzl appears to violate repository formatting
expectations (e.g., inconsistent spacing in the function signature). This can cause CI failures if
./go format (which formats Bazel/Starlark files) would produce additional diffs.
Code

java/private/java_binary.bzl[R6-14]

Evidence 
PR Compliance ID 3 requires running ./go format and committing all formatter changes; the added
java_binary macro shows inconsistent spacing that a formatter/buildifier would typically
normalize.

AGENTS.md: Run ./go format After Code Changes and Commit All Formatter Modifications
java/private/java_binary.bzl[6-14]

Agent prompt 
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
New/modified Bazel/Starlark code appears not to be formatter-compliant; CI may fail if `./go format` produces further changes.

## Issue Context
The repository expects formatting to be applied and committed after code changes.

## Fix Focus Areas
- java/private/java_binary.bzl[6-14]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

2. Non-hermetic genrule copy 🐞 Bug ☼ Reliability
Description 
The new //:legal-stuff genrule executes cp ... in cmd, and because it’s appended to
resources in the java_library/java_binary macros, most Java builds now require that shell tool
to be available in the action environment, which can break sandboxed/hermetic or Windows builds.
Code

BUILD.bazel[R26-38]

Evidence 
The genrule’s command explicitly shells out to cp, and the Java macros hardcode //:legal-stuff
into resources, which makes this genrule part of most Java build actions and therefore subject to
whatever tools are (or are not) available in the action environment.

BUILD.bazel[26-38]
java/private/java_library.bzl[39-50]
java/private/java_binary.bzl[6-14]

Agent prompt 
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
`//:legal-stuff` uses `cmd = "cp ..."`, which relies on a shell tool being present in the action environment. Since this target is now injected into most Java builds via `resources`, it expands the surface area of non-hermetic behavior and can cause build failures where `cp` is unavailable.

## Issue Context
This rule is a root-level genrule that will be pulled into most Java `java_library` and `java_binary` targets.

## Fix Focus Areas
- BUILD.bazel[26-38]
- java/private/java_library.bzl[39-50]
- java/private/java_binary.bzl[6-14]

## Suggested fix
- Replace the genrule shell copy with a Bazel-native copy action/rule (e.g., a small Starlark rule using `ctx.actions.copy_file` / `ctx.actions.run` with an explicit tool, or a repository-approved cross-platform copy rule).
- Keep the outputs as `META-INF/LICENSE` and `META-INF/NOTICE`, but implement the copies without depending on `cp` being present implicitly.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

3. java_binary() wrapper narrows signature 📘 Rule violation ≡ Correctness
Description 
The new java_binary wrapper only declares name and resources explicitly, so a second
positional argument will now be treated as resources instead of matching the underlying rule’s
expected parameters. This can break existing callers and violates build API compatibility
expectations for the exported java_binary macro.
Code

java/private/java_binary.bzl[R6-10]

Evidence 
PR Compliance ID 1 requires avoiding breaking changes to publicly used surfaces. The new wrapper’s
signature changes positional argument binding semantics and can break existing build definitions
that previously relied on the underlying java_binary callable’s parameter behavior.

AGENTS.md: Maintain API/ABI Compatibility
java/private/java_binary.bzl[6-10]
java/defs.bzl[10-14]

Agent prompt 
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The wrapper `java_binary(name, resources = [], **kwargs)` changes how positional arguments bind compared to the underlying `java_binary` rule/macro. Any call that previously (even accidentally) used a second positional argument will now be interpreted as `resources`, potentially causing errors or silent misconfiguration.

## Issue Context
`java_binary` is re-exported from `//java:defs.bzl`, making it part of the public build surface. The wrapper should avoid narrowing/changing call semantics and should instead read/merge `resources` from `kwargs`.

## Fix Focus Areas
- java/private/java_binary.bzl[6-11]
- java/defs.bzl[1-14]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

View more (2)
4. Broken genrule output paths 🐞 Bug ≡ Correctness
Description 
The new //:legal-stuff genrule uses $(location META-INF/LICENSE) / `$(location
META-INF/NOTICE), but these are declared outputs (not labels in srcs/tools`), so Bazel won’t
expand them and the action won’t produce the declared outputs, breaking dependent targets. Since
//:legal-stuff is appended to resources in the Java macros, this can break most Java builds.
Code

BUILD.bazel[R26-37]

Evidence 
The new genrule’s cmd references outs via $(location ...), which elsewhere in this repo is not
how outputs are addressed; other genrules write outputs via $@/$(@D). Additionally, the Java
macros now include //:legal-stuff in resources, amplifying the impact of a broken genrule.

BUILD.bazel[26-38]
java/private/java_binary.bzl[6-11]
java/private/java_library.bzl[6-51]
javascript/selenium-webdriver/BUILD.bazel[197-206]
java/src/org/openqa/selenium/BUILD.bazel[102-120]
common/defs.bzl[1-5]

Agent prompt 
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
`//:legal-stuff` is a genrule with two outputs under `META-INF/`, but its `cmd` tries to copy into those outputs using `$(location META-INF/LICENSE)` and `$(location META-INF/NOTICE)`. `$(location ...)` only expands for labels present in `srcs`/`tools` (not for `outs` paths), so the command won’t write the declared output files and the action will fail.

## Issue Context
This target is appended to `resources` for all `java_library` and `java_binary` macros, so a broken genrule can cascade into widespread Java build failures.

## Fix Focus Areas
- BUILD.bazel[26-38]

## Suggested fix
Update the genrule command to write into the declared outputs using `$(@D)` (and create the `META-INF` directory), e.g.:

```bzl
cmd = "mkdir -p $(@D)/META-INF && " +
     "cp $(location //:LICENSE) $(@D)/META-INF/LICENSE && " +
     "cp $(location //:NOTICE) $(@D)/META-INF/NOTICE"
```

Alternatively, replace the genrule with two `copy_file` rules (from `//common:defs.bzl`) and a filegroup aggregating `META-INF/LICENSE` and `META-INF/NOTICE`.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

5. java_library() args order changed 📘 Rule violation ≡ Correctness
Description 
The java_library macro adds a new resources parameter in the middle of the argument list, which
can break any callers that pass arguments positionally beyond srcs. This risks Bazel build/API
compatibility for downstream consumers loading this macro.
Code

java/private/java_library.bzl[R7-13]

Evidence 
PR Compliance ID 1 requires preserving backward compatibility. The signature change adds resources
before exports, altering positional argument binding for any existing positional callers and thus
can break existing builds.

AGENTS.md: Maintain API/ABI Compatibility
java/private/java_library.bzl[6-13]
java/private/java_library.bzl[39-45]

Agent prompt 
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
`java_library()` introduced `resources` as a new parameter inserted between existing parameters (`srcs` and `exports`). This can break callers using positional arguments by shifting how values bind to parameters.

## Issue Context
This macro is part of the build API (re-exported via `//java:defs.bzl`) and may be used by external build files. Adding new optional parameters should not change positional argument bindings.

## Fix Focus Areas
- java/private/java_library.bzl[6-16]
- java/private/java_library.bzl[39-50]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Remediation recommended

6. Overbroad duplicate allow regex 🐞 Bug ≡ Correctness ⭐ New
Description 
merge_jars now allows duplicates for any entry name containing LICENSE/NOTICE/etc anywhere in its
path, which can mask unintended duplicate resources outside META-INF across all merged jars. The
pattern is case-sensitive, so it may still fail to match and preserve common variants like
META-INF/license* or LICENSE.txt.
Code

java/private/merge_jars.bzl[R8-9]

Evidence 
The PR introduces a global --allow-duplicate regex in the merge_jars rule implementation, so any
entry whose name matches that regex is treated as an allowed duplicate in all merges. The rule is
used to merge deploy jars (e.g., :selenium_server_deploy.jar), so the regex applies broadly to
merged jar contents, not just the legal files added by this PR.

java/private/merge_jars.bzl[1-17]
java/src/org/openqa/selenium/grid/BUILD.bazel[181-187]

Agent prompt 
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

### Issue description
`java/private/merge_jars.bzl` adds a global `--allow-duplicate` regex that is (1) too broad (matches anywhere in the entry path) and (2) case-sensitive. This can hide unintended duplicate resources outside `META-INF` and may still miss legitimate legal-file naming variants.

### Issue Context
This regex is applied to *every* `merge_jars(...)` invocation and therefore affects merges of real deploy jars as well as other jar-like inputs.

### Fix Focus Areas
- java/private/merge_jars.bzl[8-10]

### What to change
- Restrict the regex to `META-INF/` (or whatever exact prefix the project intends), instead of `.*...*`.
- Make matching case-insensitive (prefer `(?i)` if the underlying tool uses Java regex), or explicitly include lowercase alternatives.
- Consider matching only expected legal filenames (optionally with common suffixes), e.g. `(?i)^META-INF/(LICENSE|NOTICE|COPYRIGHT)(\..*)?$` (adjust to the actual set you want to preserve).

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

7. No tests for jar legality 📘 Rule violation ☼ Reliability
Description 
This PR changes Java build outputs by injecting META-INF/LICENSE/META-INF/NOTICE resources into
jars via shared macros, but does not add tests to verify the new jar contents. Without a small test,
regressions (e.g., missing resources in some jar types) may go unnoticed.
Code

java/private/java_library.bzl[R41-45]

Evidence 
PR Compliance ID 4 requires adding/updating tests when feasible. The diff shows jar resources are
being injected globally via the java_library/java_binary macros, but no accompanying test
changes are present in the PR diff to validate the new jar contents.

AGENTS.md: Add/Update Tests for Implemented Changes, Prefer Small Unit Tests, and Avoid Mocks
java/private/java_library.bzl[40-45]
java/private/java_binary.bzl[6-13]

Agent prompt 
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
Build behavior changed to include legal files in produced jars, but there is no test coverage confirming the resulting jar contains `META-INF/LICENSE` and `META-INF/NOTICE`.

## Issue Context
This behavior is enforced for all targets using these macros, so a small, fast test should validate jar contents to prevent accidental regressions.

## Fix Focus Areas
- java/private/java_library.bzl[40-45]
- java/private/java_binary.bzl[6-13]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

8. Root package analysis fanout 🐞 Bug ➹ Performance
Description 
Hardcoding "//:legal-stuff" into the Java macros forces Bazel to load/analyze the repo root
package for essentially all Java builds, coupling them to an unusually heavy root BUILD.bazel
(e.g., npm_link_all_packages).
Code

java/private/java_library.bzl[R41-45]

Evidence 
The Java macros explicitly add //:legal-stuff to their resources, and that target is defined in
the root package. The root package’s BUILD file contains unrelated heavy top-level definitions like
npm_link_all_packages, so loading it becomes a prerequisite for analyzing Java targets that use
these macros.

java/private/java_library.bzl[39-50]
BUILD.bazel[2-16]
BUILD.bazel[26-38]

Agent prompt 
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
Injecting `//:legal-stuff` from the root package into all Java targets forces the root package to be loaded/analyzed for most Java builds, increasing analysis overhead and coupling Java-only builds to unrelated root-level build definitions.

## Issue Context
The root `BUILD.bazel` defines a variety of non-Java targets and loads, including `npm_link_all_packages`, but the Java macros now depend on a root target.

## Fix Focus Areas
- java/private/java_library.bzl[39-50]
- java/private/java_binary.bzl[6-14]
- BUILD.bazel[2-16]
- BUILD.bazel[26-38]

## Suggested fix
- Move the legal-file-producing target into a lightweight, dedicated package (with a minimal BUILD file) and update Java macros to reference that package.
- If the goal is to avoid loading the root package entirely for Java builds, place the source legal files in that dedicated package as well (and update existing references accordingly), so Java targets don’t need to resolve labels in `//:`.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

Grey Divider

Previous review results

Review updated until commit 993fd49

Results up to commit 4c0721f


🐞 Bugs (1) 📘 Rule violations (2) 📎 Requirement gaps (0)


Action required
1. java_library() args order changed 📘 Rule violation ≡ Correctness
Description 
The java_library macro adds a new resources parameter in the middle of the argument list, which
can break any callers that pass arguments positionally beyond srcs. This risks Bazel build/API
compatibility for downstream consumers loading this macro.
Code

java/private/java_library.bzl[R7-13]

Evidence 
PR Compliance ID 1 requires preserving backward compatibility. The signature change adds resources
before exports, altering positional argument binding for any existing positional callers and thus
can break existing builds.

AGENTS.md: Maintain API/ABI Compatibility
java/private/java_library.bzl[6-13]
java/private/java_library.bzl[39-45]

Agent prompt 
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
`java_library()` introduced `resources` as a new parameter inserted between existing parameters (`srcs` and `exports`). This can break callers using positional arguments by shifting how values bind to parameters.

## Issue Context
This macro is part of the build API (re-exported via `//java:defs.bzl`) and may be used by external build files. Adding new optional parameters should not change positional argument bindings.

## Fix Focus Areas
- java/private/java_library.bzl[6-16]
- java/private/java_library.bzl[39-50]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

2. java_binary() wrapper narrows signature 📘 Rule violation ≡ Correctness
Description 
The new java_binary wrapper only declares name and resources explicitly, so a second
positional argument will now be treated as resources instead of matching the underlying rule’s
expected parameters. This can break existing callers and violates build API compatibility
expectations for the exported java_binary macro.
Code

java/private/java_binary.bzl[R6-10]

Evidence 
PR Compliance ID 1 requires avoiding breaking changes to publicly used surfaces. The new wrapper’s
signature changes positional argument binding semantics and can break existing build definitions
that previously relied on the underlying java_binary callable’s parameter behavior.

AGENTS.md: Maintain API/ABI Compatibility
java/private/java_binary.bzl[6-10]
java/defs.bzl[10-14]

Agent prompt 
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The wrapper `java_binary(name, resources = [], **kwargs)` changes how positional arguments bind compared to the underlying `java_binary` rule/macro. Any call that previously (even accidentally) used a second positional argument will now be interpreted as `resources`, potentially causing errors or silent misconfiguration.

## Issue Context
`java_binary` is re-exported from `//java:defs.bzl`, making it part of the public build surface. The wrapper should avoid narrowing/changing call semantics and should instead read/merge `resources` from `kwargs`.

## Fix Focus Areas
- java/private/java_binary.bzl[6-11]
- java/defs.bzl[1-14]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

3. Broken genrule output paths 🐞 Bug ≡ Correctness
Description 
The new //:legal-stuff genrule uses $(location META-INF/LICENSE) / `$(location
META-INF/NOTICE), but these are declared outputs (not labels in srcs/tools`), so Bazel won’t
expand them and the action won’t produce the declared outputs, breaking dependent targets. Since
//:legal-stuff is appended to resources in the Java macros, this can break most Java builds.
Code

BUILD.bazel[R26-37]

Evidence 
The new genrule’s cmd references outs via $(location ...), which elsewhere in this repo is not
how outputs are addressed; other genrules write outputs via $@/$(@D). Additionally, the Java
macros now include //:legal-stuff in resources, amplifying the impact of a broken genrule.

BUILD.bazel[26-38]
java/private/java_binary.bzl[6-11]
java/private/java_library.bzl[6-51]
javascript/selenium-webdriver/BUILD.bazel[197-206]
java/src/org/openqa/selenium/BUILD.bazel[102-120]
common/defs.bzl[1-5]

Agent prompt 
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
`//:legal-stuff` is a genrule with two outputs under `META-INF/`, but its `cmd` tries to copy into those outputs using `$(location META-INF/LICENSE)` and `$(location META-INF/NOTICE)`. `$(location ...)` only expands for labels present in `srcs`/`tools` (not for `outs` paths), so the command won’t write the declared output files and the action will fail.

## Issue Context
This target is appended to `resources` for all `java_library` and `java_binary` macros, so a broken genrule can cascade into widespread Java build failures.

## Fix Focus Areas
- BUILD.bazel[26-38]

## Suggested fix
Update the genrule command to write into the declared outputs using `$(@D)` (and create the `META-INF` directory), e.g.:

```bzl
cmd = "mkdir -p $(@D)/META-INF && " +
     "cp $(location //:LICENSE) $(@D)/META-INF/LICENSE && " +
     "cp $(location //:NOTICE) $(@D)/META-INF/NOTICE"
```

Alternatively, replace the genrule with two `copy_file` rules (from `//common:defs.bzl`) and a filegroup aggregating `META-INF/LICENSE` and `META-INF/NOTICE`.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

Results up to commit b11efab


🐞 Bugs (2) 📘 Rule violations (0) 📎 Requirement gaps (0)


Action required
1. Non-hermetic genrule copy 🐞 Bug ☼ Reliability
Description 
The new //:legal-stuff genrule executes cp ... in cmd, and because it’s appended to
resources in the java_library/java_binary macros, most Java builds now require that shell tool
to be available in the action environment, which can break sandboxed/hermetic or Windows builds.
Code

BUILD.bazel[R26-38]

Evidence 
The genrule’s command explicitly shells out to cp, and the Java macros hardcode //:legal-stuff
into resources, which makes this genrule part of most Java build actions and therefore subject to
whatever tools are (or are not) available in the action environment.

BUILD.bazel[26-38]
java/private/java_library.bzl[39-50]
java/private/java_binary.bzl[6-14]

Agent prompt 
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
`//:legal-stuff` uses `cmd = "cp ..."`, which relies on a shell tool being present in the action environment. Since this target is now injected into most Java builds via `resources`, it expands the surface area of non-hermetic behavior and can cause build failures where `cp` is unavailable.

## Issue Context
This rule is a root-level genrule that will be pulled into most Java `java_library` and `java_binary` targets.

## Fix Focus Areas
- BUILD.bazel[26-38]
- java/private/java_library.bzl[39-50]
- java/private/java_binary.bzl[6-14]

## Suggested fix
- Replace the genrule shell copy with a Bazel-native copy action/rule (e.g., a small Starlark rule using `ctx.actions.copy_file` / `ctx.actions.run` with an explicit tool, or a repository-approved cross-platform copy rule).
- Keep the outputs as `META-INF/LICENSE` and `META-INF/NOTICE`, but implement the copies without depending on `cp` being present implicitly.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Remediation recommended
2. Root package analysis fanout 🐞 Bug ➹ Performance
Description 
Hardcoding "//:legal-stuff" into the Java macros forces Bazel to load/analyze the repo root
package for essentially all Java builds, coupling them to an unusually heavy root BUILD.bazel
(e.g., npm_link_all_packages).
Code

java/private/java_library.bzl[R41-45]

Evidence 
The Java macros explicitly add //:legal-stuff to their resources, and that target is defined in
the root package. The root package’s BUILD file contains unrelated heavy top-level definitions like
npm_link_all_packages, so loading it becomes a prerequisite for analyzing Java targets that use
these macros.

java/private/java_library.bzl[39-50]
BUILD.bazel[2-16]
BUILD.bazel[26-38]

Agent prompt 
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
Injecting `//:legal-stuff` from the root package into all Java targets forces the root package to be loaded/analyzed for most Java builds, increasing analysis overhead and coupling Java-only builds to unrelated root-level build definitions.

## Issue Context
The root `BUILD.bazel` defines a variety of non-Java targets and loads, including `npm_link_all_packages`, but the Java macros now depend on a root target.

## Fix Focus Areas
- java/private/java_library.bzl[39-50]
- java/private/java_binary.bzl[6-14]
- BUILD.bazel[2-16]
- BUILD.bazel[26-38]

## Suggested fix
- Move the legal-file-producing target into a lightweight, dedicated package (with a minimal BUILD file) and update Java macros to reference that package.
- If the goal is to avoid loading the root package entirely for Java builds, place the source legal files in that dedicated package as well (and update existing references accordingly), so Java targets don’t need to resolve labels in `//:`.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

Results up to commit e26a0df


🐞 Bugs (0) 📘 Rule violations (2) 📎 Requirement gaps (0)


Action required
1. java_binary macro not buildified 📘 Rule violation ⚙ Maintainability
Description 
The new Starlark file java/private/java_binary.bzl appears to violate repository formatting
expectations (e.g., inconsistent spacing in the function signature). This can cause CI failures if
./go format (which formats Bazel/Starlark files) would produce additional diffs.
Code

java/private/java_binary.bzl[R6-14]

Evidence 
PR Compliance ID 3 requires running ./go format and committing all formatter changes; the added
java_binary macro shows inconsistent spacing that a formatter/buildifier would typically
normalize.

AGENTS.md: Run ./go format After Code Changes and Commit All Formatter Modifications
java/private/java_binary.bzl[6-14]

Agent prompt 
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
New/modified Bazel/Starlark code appears not to be formatter-compliant; CI may fail if `./go format` produces further changes.

## Issue Context
The repository expects formatting to be applied and committed after code changes.

## Fix Focus Areas
- java/private/java_binary.bzl[6-14]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Remediation recommended
2. No tests for jar legality 📘 Rule violation ☼ Reliability
Description 
This PR changes Java build outputs by injecting META-INF/LICENSE/META-INF/NOTICE resources into
jars via shared macros, but does not add tests to verify the new jar contents. Without a small test,
regressions (e.g., missing resources in some jar types) may go unnoticed.
Code

java/private/java_library.bzl[R41-45]

Evidence 
PR Compliance ID 4 requires adding/updating tests when feasible. The diff shows jar resources are
being injected globally via the java_library/java_binary macros, but no accompanying test
changes are present in the PR diff to validate the new jar contents.

AGENTS.md: Add/Update Tests for Implemented Changes, Prefer Small Unit Tests, and Avoid Mocks
java/private/java_library.bzl[40-45]
java/private/java_binary.bzl[6-13]

Agent prompt 
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
Build behavior changed to include legal files in produced jars, but there is no test coverage confirming the resulting jar contains `META-INF/LICENSE` and `META-INF/NOTICE`.

## Issue Context
This behavior is enforced for all targets using these macros, so a small, fast test should validate jar contents to prevent accidental regressions.

## Fix Focus Areas
- java/private/java_library.bzl[40-45]
- java/private/java_binary.bzl[6-13]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

Qodo Logo

qodo-code-review[bot]
qodo-code-review Bot reviewed May 31, 2026
View reviewed changes
Comment thread java/private/java_library.bzl
Comment thread java/private/java_binary.bzl Outdated
Comment thread BUILD.bazel
@joerg1985 joerg1985 force-pushed the issue-16113 branch 3 times, most recently from da90638 to b11efab Compare May 31, 2026 15:40
@qodo-code-review

qodo-code-review Bot commented May 31, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Code review by qodo was updated up to the latest commit b11efab

qodo-code-review[bot]
qodo-code-review Bot reviewed May 31, 2026
View reviewed changes
Comment thread BUILD.bazel
@qodo-code-review

qodo-code-review Bot commented May 31, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Code review by qodo was updated up to the latest commit 6cea6a9

@joerg1985 joerg1985 force-pushed the issue-16113 branch 2 times, most recently from 391e19f to e26a0df Compare May 31, 2026 15:58
@qodo-code-review

qodo-code-review Bot commented May 31, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Code review by qodo was updated up to the latest commit e26a0df

qodo-code-review[bot]
qodo-code-review Bot reviewed May 31, 2026
View reviewed changes
Comment thread java/private/java_binary.bzl Outdated
@qodo-code-review

qodo-code-review Bot commented May 31, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Code review by qodo was updated up to the latest commit 993fd49

@qodo-code-review

qodo-code-review Bot commented May 31, 2026

Copy link
Copy Markdown
Contributor

CI Feedback 🧐

A test triggered by this PR failed. Here is an AI-generated analysis of the failure:

Action: Test / All RBE tests

Failed stage: Run Bazel [❌]

Failed test name: //java/src/dev/selenium/tools/javadoc:javadoc-spotbugs

Failure summary:

The GitHub Action failed because Bazel reported 3 failing SpotBugs test targets (static analysis
violations), causing the build to exit with code 3.
-
//java/src/dev/selenium/tools/javadoc:javadoc-spotbugs failed (Exit 1) due to SpotBugs findings in
JavadocJarMaker.java:
- Possible null pointer dereference in
dev.selenium.tools.javadoc.JavadocJarMaker.readSourceFiles(...) at JavadocJarMaker.java:217
-
Switch statement missing default case in dev.selenium.tools.javadoc.JavadocJarMaker.main(String[])
at JavadocJarMaker.java:63-73
- //java/src/dev/selenium/tools/modules:ModuleGenerator-spotbugs
failed (Exit 1) due to SpotBugs findings in ModuleGenerator.java:
- Possible null pointer
dereference in dev.selenium.tools.modules.ModuleGenerator$1.visitFile(...) at
ModuleGenerator.java:229
- Reliance on platform default encoding in
dev.selenium.tools.modules.ModuleGenerator.main(String[]) at ModuleGenerator.java:204
-
//java/src/org/openqa/selenium/devtools:cdp-client-generator-spotbugs failed (Exit 1) due to
SpotBugs findings in CdpClientGenerator.java:
- Possible null pointer dereference in
org.openqa.selenium.devtools.CdpClientGenerator.main(String[]) at CdpClientGenerator.java:85
-
Format string should use %n rather than \n at CdpClientGenerator.java:1041

Relevant error logs: 
1:  ##[group]Runner Image Provisioner
2:  Hosted Compute Agent
...

907:  Received 188743680 of 201311472 (93.8%), 17.2 MBs/sec
908:  Received 201311472 of 201311472 (100.0%), 17.6 MBs/sec
909:  Cache Size: ~192 MB (201311472 B)
910:  [command]/usr/bin/tar -xf /home/runner/work/_temp/a12880db-eeaa-41c0-9715-9df3f15c365e/cache.tzst -P -C /home/runner/work/selenium/selenium --use-compress-program unzstd
911:  Cache restored successfully
912:  Successfully restored cache from setup-bazel-1-linux-x64-external-rules_java++toolchains+remotejdk25_linux-39089f465ddd75c69cdd29b9d9f9856a609fcfcedd6da879f872c41a097e1052
913:  ##[endgroup]
914:  Cache restored successfully
915:  Successfully restored cache from setup-bazel-1-linux-x64-external-rules_java++toolchains+remotejdk21_linux-39089f465ddd75c69cdd29b9d9f9856a609fcfcedd6da879f872c41a097e1052
916:  ##[endgroup]
917:  Cache restored successfully
918:  Successfully restored cache from setup-bazel-1-linux-x64-external-rules_dotnet++dotnet+dotnet_x86_64-unknown-linux-gnu-39089f465ddd75c69cdd29b9d9f9856a609fcfcedd6da879f872c41a097e1052
919:  ##[endgroup]
920:  ##[group]Run if [[ "success" == "failure" ]]; then
921:  �[36;1mif [[ "success" == "failure" ]]; then�[0m
922:  �[36;1m  echo "::warning::Bazel cache setup failed — fell back to setup without cache"�[0m
923:  �[36;1mfi�[0m
...

932:  GEM_HOST_API_KEY: 
933:  NUGET_API_KEY: 
934:  BAZELISK_GITHUB_TOKEN: ***
935:  ##[endgroup]
936:  === Disk space ===
937:  Filesystem      Size  Used Avail Use% Mounted on
938:  /dev/root       145G   34G  112G  23% /
939:  Available: 111GB
940:  === Bazel cache sizes ===
941:  External:                 3.6G
942:  Repository Cache:         2.7G
943:  Bazelisk:                 63M
944:  ##[group]Run . ./scripts/github-actions/disk-status.sh
945:  �[36;1m. ./scripts/github-actions/disk-status.sh�[0m
946:  �[36;1mif [ "$AVAIL_GB" -lt 10 ]; then�[0m
947:  �[36;1m  echo "::error::Insufficient disk space: ${AVAIL_GB}GB available (need >=10GB)"�[0m
948:  �[36;1m  exit 1�[0m
...

1457:  (16:06:08) �[32mINFO: �[0mFrom Building java/src/org/openqa/selenium/bidi/storage/libstorage-class.jar (11 source files):
1458:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
1459:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
1460:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
1461:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
1462:  (16:06:09) �[32mINFO: �[0mFrom Building java/src/org/openqa/selenium/bidi/module/libmodule-class.jar (10 source files):
1463:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
1464:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
1465:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
1466:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
1467:  (16:06:09) �[32mINFO: �[0mFrom Building java/src/org/openqa/selenium/remote/libapi-class.jar (70 source files):
1468:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
1469:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
1470:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
1471:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
1472:  java/src/org/openqa/selenium/remote/ErrorHandler.java:49: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1473:  private final ErrorCodes errorCodes;
1474:  ^
1475:  java/src/org/openqa/selenium/remote/ErrorHandler.java:63: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1476:  this.errorCodes = new ErrorCodes();
1477:  ^
1478:  java/src/org/openqa/selenium/remote/ErrorHandler.java:71: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1479:  public ErrorHandler(ErrorCodes codes, boolean includeServerErrors) {
1480:  ^
1481:  java/src/org/openqa/selenium/remote/RemoteWebDriverBuilder.java:309: warning: [removal] <T>sendNative(HttpRequest,BodyHandler<T>) in HttpClient has been deprecated and marked for removal
1482:  public <T> java.net.http.HttpResponse<T> sendNative(
1483:  ^
1484:  where T is a type-variable:
1485:  T extends Object declared in method <T>sendNative(HttpRequest,BodyHandler<T>)
1486:  java/src/org/openqa/selenium/remote/RemoteWebDriverBuilder.java:302: warning: [removal] <T>sendAsyncNative(HttpRequest,BodyHandler<T>) in HttpClient has been deprecated and marked for removal
1487:  sendAsyncNative(
1488:  ^
1489:  where T is a type-variable:
1490:  T extends Object declared in method <T>sendAsyncNative(HttpRequest,BodyHandler<T>)
1491:  java/src/org/openqa/selenium/remote/Response.java:99: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1492:  ErrorCodes errorCodes = new ErrorCodes();
1493:  ^
1494:  java/src/org/openqa/selenium/remote/Response.java:99: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1495:  ErrorCodes errorCodes = new ErrorCodes();
1496:  ^
1497:  java/src/org/openqa/selenium/remote/ProtocolHandshake.java:182: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1498:  response.setStatus(ErrorCodes.SUCCESS);
1499:  ^
1500:  java/src/org/openqa/selenium/remote/ProtocolHandshake.java:183: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1501:  response.setState(ErrorCodes.SUCCESS_STRING);
1502:  ^
1503:  java/src/org/openqa/selenium/remote/W3CHandshakeResponse.java:54: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1504:  new ErrorCodes().toStatus((String) rawError, Optional.of(tuple.getStatusCode())));
1505:  ^
1506:  java/src/org/openqa/selenium/remote/W3CHandshakeResponse.java:57: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1507:  new ErrorCodes().getExceptionType((String) rawError);
1508:  ^
1509:  java/src/org/openqa/selenium/remote/codec/AbstractHttpResponseCodec.java:43: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1510:  private final ErrorCodes errorCodes = new ErrorCodes();
1511:  ^
1512:  java/src/org/openqa/selenium/remote/codec/AbstractHttpResponseCodec.java:43: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1513:  private final ErrorCodes errorCodes = new ErrorCodes();
1514:  ^
1515:  java/src/org/openqa/selenium/remote/codec/AbstractHttpResponseCodec.java:55: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1516:  int status = responseStatus == ErrorCodes.SUCCESS ? HTTP_OK : HTTP_INTERNAL_ERROR;
1517:  ^
1518:  java/src/org/openqa/selenium/remote/codec/AbstractHttpResponseCodec.java:100: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1519:  response.setStatus(ErrorCodes.UNKNOWN_COMMAND);
1520:  ^
1521:  java/src/org/openqa/selenium/remote/codec/AbstractHttpResponseCodec.java:102: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1522:  response.setStatus(ErrorCodes.UNHANDLED_ERROR);
1523:  ^
1524:  java/src/org/openqa/selenium/remote/codec/AbstractHttpResponseCodec.java:118: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1525:  response.setStatus(ErrorCodes.SUCCESS);
1526:  ^
1527:  java/src/org/openqa/selenium/remote/codec/AbstractHttpResponseCodec.java:119: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1528:  response.setState(errorCodes.toState(ErrorCodes.SUCCESS));
1529:  ^
1530:  java/src/org/openqa/selenium/remote/codec/AbstractHttpResponseCodec.java:125: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1531:  response.setState(errorCodes.toState(ErrorCodes.SUCCESS));
1532:  ^
1533:  java/src/org/openqa/selenium/remote/codec/w3c/W3CHttpResponseCodec.java:70: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1534:  private final ErrorCodes errorCodes = new ErrorCodes();
1535:  ^
1536:  java/src/org/openqa/selenium/remote/codec/w3c/W3CHttpResponseCodec.java:70: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1537:  private final ErrorCodes errorCodes = new ErrorCodes();
1538:  ^
1539:  java/src/org/openqa/selenium/remote/codec/w3c/W3CHttpResponseCodec.java:98: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1540:  response.setStatus(ErrorCodes.UNKNOWN_COMMAND);
1541:  ^
1542:  java/src/org/openqa/selenium/remote/codec/w3c/W3CHttpResponseCodec.java:103: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1543:  response.setStatus(ErrorCodes.UNHANDLED_ERROR);
1544:  ^
1545:  java/src/org/openqa/selenium/remote/codec/w3c/W3CHttpResponseCodec.java:150: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
1546:  response.setStatus(ErrorCodes.SUCCESS);
1547:  ^
...

2592:  (16:06:27) �[32mINFO: �[0mFrom Checking 1 JS files in @@//third_party/closure/goog/structs:collection:
2593:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
2594:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/bazel-out/platform-opt-exec/bin/external/rules_closure+/java/io/bazel/rules/closure/ClosureWorker.runfiles/protobuf+/java/core/liblite_runtime_only.jar)
2595:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
2596:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
2597:  (16:06:27) �[32mINFO: �[0mFrom Checking 1 JS files in @@//third_party/closure/goog/dom:inputtype:
2598:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
2599:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/bazel-out/platform-opt-exec/bin/external/rules_closure+/java/io/bazel/rules/closure/ClosureWorker.runfiles/protobuf+/java/core/liblite_runtime_only.jar)
2600:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
2601:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
2602:  (16:06:27) �[32mINFO: �[0mFrom Checking 1 JS files in @@//third_party/closure/goog/math:irect:
2603:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
2604:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/bazel-out/platform-opt-exec/bin/external/rules_closure+/java/io/bazel/rules/closure/ClosureWorker.runfiles/protobuf+/java/core/liblite_runtime_only.jar)
2605:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
2606:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
2607:  (16:06:27) �[32mINFO: �[0mFrom Checking 1 JS files in @@//third_party/closure/goog/debug:errorcontext:
2608:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
...

2617:  (16:06:27) �[32mINFO: �[0mFrom Checking 1 JS files in @@//third_party/closure/goog/math:size:
2618:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
2619:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/bazel-out/platform-opt-exec/bin/external/rules_closure+/java/io/bazel/rules/closure/ClosureWorker.runfiles/protobuf+/java/core/liblite_runtime_only.jar)
2620:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
2621:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
2622:  (16:06:27) �[32mINFO: �[0mFrom Checking 1 JS files in @@//third_party/closure/goog/object:object:
2623:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
2624:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/bazel-out/platform-opt-exec/bin/external/rules_closure+/java/io/bazel/rules/closure/ClosureWorker.runfiles/protobuf+/java/core/liblite_runtime_only.jar)
2625:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
2626:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
2627:  (16:06:27) �[32mINFO: �[0mFrom Checking 1 JS files in @@//javascript/atoms:bot:
2628:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
2629:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/bazel-out/platform-opt-exec/bin/external/rules_closure+/java/io/bazel/rules/closure/ClosureWorker.runfiles/protobuf+/java/core/liblite_runtime_only.jar)
2630:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
2631:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
2632:  (16:06:27) �[32mINFO: �[0mFrom Checking 2 JS files in @@//javascript/atoms:errors:
2633:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
2634:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/bazel-out/platform-opt-exec/bin/external/rules_closure+/java/io/bazel/rules/closure/ClosureWorker.runfiles/protobuf+/java/core/liblite_runtime_only.jar)
2635:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
2636:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
2637:  (16:06:27) �[32mINFO: �[0mFrom Checking 1 JS files in @@//third_party/closure/goog/disposable:idisposable:
2638:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
2639:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/bazel-out/platform-opt-exec/bin/external/rules_closure+/java/io/bazel/rules/closure/ClosureWorker.runfiles/protobuf+/java/core/liblite_runtime_only.jar)
2640:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
2641:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
2642:  (16:06:27) �[32mINFO: �[0mFrom Checking 1 JS files in @@//third_party/closure/goog/debug:error:
2643:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
...

3023:  (16:06:34) �[32mINFO: �[0mFrom Building java/src/org/openqa/selenium/devtools/v148/libv148-lib-class.jar (8 source files) and running annotation processors (AutoServiceProcessor):
3024:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
3025:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
3026:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
3027:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
3028:  (16:06:37) �[32mAnalyzing:�[0m 3188 targets (1971 packages loaded, 84368 targets configured, 948 aspect applications)
3029:  �[32m[16,033 / 19,768]�[0m 111 / 2814 tests;�[0m Testing //java/src/dev/selenium/tools/modules:ModuleGenerator-spotbugs; 18s remote, remote-cache ... (35 actions, 3 running)
3030:  (16:06:40) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/grid/web/ReverseProxyHandlerTest-class.jar (1 source file):
3031:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
3032:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
3033:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
3034:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
3035:  (16:06:42) �[32mAnalyzing:�[0m 3188 targets (1971 packages loaded, 87046 targets configured, 948 aspect applications)
3036:  �[32m[16,189 / 19,909]�[0m 119 / 2814 tests;�[0m Testing //java/src/dev/selenium/tools/modules:ModuleGenerator-spotbugs; 24s remote, remote-cache ... (37 actions, 3 running)
3037:  (16:06:45) �[31m�[1mFAIL: �[0m//java/src/dev/selenium/tools/javadoc:javadoc-spotbugs (Exit 1) (see /home/runner/.bazel/execroot/_main/bazel-out/k8-fastbuild/testlogs/java/src/dev/selenium/tools/javadoc/javadoc-spotbugs/test.log)
3038:  �[31m�[1mFAILED: �[0m//java/src/dev/selenium/tools/javadoc:javadoc-spotbugs (Summary)
3039:  /home/runner/.bazel/execroot/_main/bazel-out/k8-fastbuild/testlogs/java/src/dev/selenium/tools/javadoc/javadoc-spotbugs/test.log
3040:  /home/runner/.bazel/execroot/_main/bazel-out/k8-fastbuild/testlogs/java/src/dev/selenium/tools/javadoc/javadoc-spotbugs/test_attempts/attempt_1.log
3041:  (16:06:45) �[32mINFO: �[0mFrom Testing //java/src/dev/selenium/tools/javadoc:javadoc-spotbugs:
3042:  ==================== Test output for //java/src/dev/selenium/tools/javadoc:javadoc-spotbugs:
3043:  M D NP: Possible null pointer dereference in dev.selenium.tools.javadoc.JavadocJarMaker.readSourceFiles(Path, StandardJavaFileManager, Set, Set, Set, Set) due to return value of called method  Method invoked at JavadocJarMaker.java:[line 217]
3044:  M D SF: Switch statement found in dev.selenium.tools.javadoc.JavadocJarMaker.main(String[]) where default case is missing  At JavadocJarMaker.java:[lines 63-73]
3045:  Remote server execution message: Execution result: https://gypsum.cluster.engflow.com/actions/executions/ChDglrpT6wRaJJOMK0DOW3zqEgdkZWZhdWx0GiUKIPDKWVn2JbiF0VdvcZ6JFYm_-MOiUdRSoIiL_DXXzXcIELwD
3046:  ================================================================================
3047:  ==================== Test output for //java/src/dev/selenium/tools/javadoc:javadoc-spotbugs:
3048:  M D NP: Possible null pointer dereference in dev.selenium.tools.javadoc.JavadocJarMaker.readSourceFiles(Path, StandardJavaFileManager, Set, Set, Set, Set) due to return value of called method  Method invoked at JavadocJarMaker.java:[line 217]
3049:  M D SF: Switch statement found in dev.selenium.tools.javadoc.JavadocJarMaker.main(String[]) where default case is missing  At JavadocJarMaker.java:[lines 63-73]
3050:  Remote server execution message: Execution result: https://gypsum.cluster.engflow.com/actions/executions/ChDglrpT6wRaJJOMK0DOW3zqEgdkZWZhdWx0GiUKIPDKWVn2JbiF0VdvcZ6JFYm_-MOiUdRSoIiL_DXXzXcIELwD
3051:  ================================================================================
3052:  (16:06:47) �[32mAnalyzing:�[0m 3188 targets (1975 packages loaded, 89312 targets configured, 948 aspect applications)
3053:  �[32m[16,342 / 20,057]�[0m 122 / 2814 tests, �[31m�[1m1 failed�[0m;�[0m Testing //java/src/dev/selenium/tools/modules:ModuleGenerator-spotbugs; 29s remote, remote-cache ... (40 actions, 2 running)
3054:  (16:06:50) �[32mINFO: �[0mFrom Building java/src/org/openqa/selenium/firefox/libfirefox-lib-class.jar (21 source files) and running annotation processors (AutoServiceProcessor):
3055:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
3056:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
3057:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
3058:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
3059:  (16:06:52) �[32mAnalyzing:�[0m 3188 targets (2007 packages loaded, 93478 targets configured, 948 aspect applications)
3060:  �[32m[16,414 / 20,113]�[0m 122 / 2833 tests, �[31m�[1m1 failed�[0m;�[0m Testing //java/src/dev/selenium/tools/modules:ModuleGenerator-spotbugs; 34s remote, remote-cache ... (32 actions, 2 running)
3061:  (16:06:57) �[32mAnalyzing:�[0m 3188 targets (2017 packages loaded, 98460 targets configured, 948 aspect applications)
3062:  �[32m[16,463 / 20,165]�[0m 122 / 2833 tests, �[31m�[1m1 failed�[0m;�[0m Testing //java/src/dev/selenium/tools/modules:ModuleGenerator-spotbugs; 39s remote, remote-cache ... (38 actions, 2 running)
3063:  (16:07:02) �[32mAnalyzing:�[0m 3188 targets (2019 packages loaded, 103710 targets configured, 948 aspect applications)
3064:  �[32m[16,513 / 20,214]�[0m 124 / 2833 tests, �[31m�[1m1 failed�[0m;�[0m Testing //java/src/dev/selenium/tools/modules:ModuleGenerator-spotbugs; 44s remote, remote-cache ... (40 actions, 2 running)
3065:  (16:07:07) �[32mAnalyzing:�[0m 3188 targets (2021 packages loaded, 109668 targets configured, 969 aspect applications)
3066:  �[32m[16,529 / 20,235]�[0m 124 / 2854 tests, �[31m�[1m1 failed�[0m;�[0m Testing //java/src/dev/selenium/tools/modules:ModuleGenerator-spotbugs; 49s remote, remote-cache ... (50 actions, 2 running)
3067:  (16:07:08) �[32mINFO: �[0mFrom Building java/src/org/openqa/selenium/grid/server/libserver-class.jar (8 source files) and running annotation processors (AutoServiceProcessor):
...

3073:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
3074:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
3075:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
3076:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
3077:  (16:07:09) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/grid/web/TeeReaderTest-class.jar (1 source file):
3078:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
3079:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
3080:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
3081:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
3082:  (16:07:12) �[32mINFO: �[0mFrom Building java/src/org/openqa/selenium/support/ui/libwait-class.jar (5 source files):
3083:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
3084:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
3085:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
3086:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
3087:  (16:07:13) �[32mAnalyzing:�[0m 3188 targets (2028 packages loaded, 117269 targets configured, 969 aspect applications)
3088:  �[32m[16,538 / 20,235]�[0m 128 / 2854 tests, �[31m�[1m1 failed�[0m;�[0m Testing //java/src/dev/selenium/tools/modules:ModuleGenerator-spotbugs; 54s remote, remote-cache ... (50 actions, 2 running)
3089:  (16:07:17) �[31m�[1mFAIL: �[0m//java/src/dev/selenium/tools/modules:ModuleGenerator-spotbugs (Exit 1) (see /home/runner/.bazel/execroot/_main/bazel-out/k8-fastbuild/testlogs/java/src/dev/selenium/tools/modules/ModuleGenerator-spotbugs/test.log)
3090:  �[31m�[1mFAILED: �[0m//java/src/dev/selenium/tools/modules:ModuleGenerator-spotbugs (Summary)
3091:  /home/runner/.bazel/execroot/_main/bazel-out/k8-fastbuild/testlogs/java/src/dev/selenium/tools/modules/ModuleGenerator-spotbugs/test.log
3092:  /home/runner/.bazel/execroot/_main/bazel-out/k8-fastbuild/testlogs/java/src/dev/selenium/tools/modules/ModuleGenerator-spotbugs/test_attempts/attempt_1.log
3093:  (16:07:17) �[32mINFO: �[0mFrom Testing //java/src/dev/selenium/tools/modules:ModuleGenerator-spotbugs:
3094:  ==================== Test output for //java/src/dev/selenium/tools/modules:ModuleGenerator-spotbugs:
3095:  M D NP: Possible null pointer dereference in dev.selenium.tools.modules.ModuleGenerator$1.visitFile(Path, BasicFileAttributes) due to return value of called method  Dereferenced at ModuleGenerator.java:[line 229]
3096:  H I Dm: Found reliance on default encoding in dev.selenium.tools.modules.ModuleGenerator.main(String[]): new java.io.PrintStream(OutputStream)  At ModuleGenerator.java:[line 204]
3097:  Remote server execution message: Execution result: https://gypsum.cluster.engflow.com/actions/executions/ChDvT6z3BEhVsolan_lPp5dvEgdkZWZhdWx0GiUKIFfsAoP5EV0EX9m4u8nxqbO96P6iSyW2-XUfyh-_SuATELwD
3098:  ================================================================================
3099:  ==================== Test output for //java/src/dev/selenium/tools/modules:ModuleGenerator-spotbugs:
3100:  M D NP: Possible null pointer dereference in dev.selenium.tools.modules.ModuleGenerator$1.visitFile(Path, BasicFileAttributes) due to return value of called method  Dereferenced at ModuleGenerator.java:[line 229]
3101:  H I Dm: Found reliance on default encoding in dev.selenium.tools.modules.ModuleGenerator.main(String[]): new java.io.PrintStream(OutputStream)  At ModuleGenerator.java:[line 204]
3102:  Remote server execution message: Execution result: https://gypsum.cluster.engflow.com/actions/executions/ChDvT6z3BEhVsolan_lPp5dvEgdkZWZhdWx0GiUKIFfsAoP5EV0EX9m4u8nxqbO96P6iSyW2-XUfyh-_SuATELwD
3103:  ================================================================================
3104:  (16:07:18) �[32mAnalyzing:�[0m 3188 targets (2031 packages loaded, 124383 targets configured, 969 aspect applications)
3105:  �[32m[16,541 / 20,235]�[0m 131 / 2854 tests, �[31m�[1m2 failed�[0m;�[0m Testing //java/src/org/openqa/selenium/devtools:cdp-client-generator-spotbugs; 59s remote, remote-cache ... (50 actions, 1 running)
3106:  (16:07:19) �[31m�[1mFAIL: �[0m//java/src/org/openqa/selenium/devtools:cdp-client-generator-spotbugs (Exit 1) (see /home/runner/.bazel/execroot/_main/bazel-out/k8-fastbuild/testlogs/java/src/org/openqa/selenium/devtools/cdp-client-generator-spotbugs/test.log)
3107:  �[31m�[1mFAILED: �[0m//java/src/org/openqa/selenium/devtools:cdp-client-generator-spotbugs (Summary)
3108:  /home/runner/.bazel/execroot/_main/bazel-out/k8-fastbuild/testlogs/java/src/org/openqa/selenium/devtools/cdp-client-generator-spotbugs/test.log
3109:  /home/runner/.bazel/execroot/_main/bazel-out/k8-fastbuild/testlogs/java/src/org/openqa/selenium/devtools/cdp-client-generator-spotbugs/test_attempts/attempt_1.log
3110:  (16:07:19) �[32mINFO: �[0mFrom Testing //java/src/org/openqa/selenium/devtools:cdp-client-generator-spotbugs:
3111:  ==================== Test output for //java/src/org/openqa/selenium/devtools:cdp-client-generator-spotbugs:
3112:  M D NP: Possible null pointer dereference in org.openqa.selenium.devtools.CdpClientGenerator.main(String[]) due to return value of called method  Method invoked at CdpClientGenerator.java:[line 85]
3113:  M B FS: Format string should use %n rather than \n in org.openqa.selenium.devtools.CdpClientGenerator$EnumType.toTypeDeclaration()  At CdpClientGenerator.java:[line 1041]
3114:  Remote server execution message: Execution result: https://gypsum.cluster.engflow.com/actions/executions/ChCNUf4XLJBYvrFTa5kpkrutEgdkZWZhdWx0GiUKIHKDwRizX0yFGYh6_KdLTbe7RWtBEsJ_hnlpHgsPIq3hELwD
3115:  ================================================================================
3116:  ==================== Test output for //java/src/org/openqa/selenium/devtools:cdp-client-generator-spotbugs:
3117:  M D NP: Possible null pointer dereference in org.openqa.selenium.devtools.CdpClientGenerator.main(String[]) due to return value of called method  Method invoked at CdpClientGenerator.java:[line 85]
3118:  M B FS: Format string should use %n rather than \n in org.openqa.selenium.devtools.CdpClientGenerator$EnumType.toTypeDeclaration()  At CdpClientGenerator.java:[line 1041]
3119:  Remote server execution message: Execution result: https://gypsum.cluster.engflow.com/actions/executions/ChCNUf4XLJBYvrFTa5kpkrutEgdkZWZhdWx0GiUKIHKDwRizX0yFGYh6_KdLTbe7RWtBEsJ_hnlpHgsPIq3hELwD
3120:  ================================================================================
3121:  (16:07:23) �[32mAnalyzing:�[0m 3188 targets (2034 packages loaded, 132192 targets configured, 969 aspect applications)
3122:  �[32m[16,543 / 20,235]�[0m 132 / 2854 tests, �[31m�[1m3 failed�[0m;�[0m Testing //py:test/selenium/webdriver/common/upload_tests-chrome; 38s remote, remote-cache ... (50 actions, 0 running)
3123:  (16:07:24) �[32mINFO: �[0mFrom Building java/src/org/openqa/selenium/support/locators/liblocators-class.jar (4 source files) and running annotation processors (AutoServiceProcessor):
...

3414:  java/test/org/openqa/selenium/grid/testing/PassthroughHttpClient.java:52: warning: [removal] <T>sendAsyncNative(HttpRequest,BodyHandler<T>) in HttpClient has been deprecated and marked for removal
3415:  public <T> java.util.concurrent.CompletableFuture<java.net.http.HttpResponse<T>> sendAsyncNative(
3416:  ^
3417:  where T is a type-variable:
3418:  T extends Object declared in method <T>sendAsyncNative(HttpRequest,BodyHandler<T>)
3419:  (16:07:28) �[32mINFO: �[0mFrom Building java/src/org/openqa/selenium/grid/node/remote/libremote-class.jar (2 source files):
3420:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
3421:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
3422:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
3423:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
3424:  (16:07:28) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/atoms/InputAtomsTest-class.jar (1 source file):
3425:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
3426:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
3427:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
3428:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
3429:  (16:07:28) �[32m[17,037 / 20,958]�[0m 300 / 3188 tests, �[31m�[1m3 failed�[0m;�[0m Writing repo mapping manifest for //rust/tests:integration_browser_download_tests_test-fmt; 0s local ... (50 actions, 1 running)
3430:  (16:07:28) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/grid/node/relay/RelayOptionsTest.jar (1 source file):
...

3830:  java/test/org/openqa/selenium/grid/node/RemoteNodeTest.java:140: warning: [removal] <T>sendAsyncNative(HttpRequest,BodyHandler<T>) in HttpClient has been deprecated and marked for removal
3831:  java.util.concurrent.CompletableFuture<java.net.http.HttpResponse<T>> sendAsyncNative(
3832:  ^
3833:  where T is a type-variable:
3834:  T extends Object declared in method <T>sendAsyncNative(HttpRequest,BodyHandler<T>)
3835:  (16:07:33) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/support/decorators/DecoratedNavigationTest.jar (1 source file):
3836:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
3837:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
3838:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
3839:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
3840:  (16:07:33) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/support/decorators/IntegrationTest.jar (1 source file):
3841:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
3842:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
3843:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
3844:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
3845:  (16:07:33) �[32m[17,876 / 21,488]�[0m 607 / 3188 tests, �[31m�[1m3 failed�[0m;�[0m Writing repo mapping manifest for //rust/tests:integration_safari_tests_test-fmt; 0s local ... (50 actions, 1 running)
3846:  (16:07:33) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/support/pagefactory/UsingPageFactoryTest.jar (1 source file):
...

4181:  (16:07:36) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/support/pagefactory/AnnotationsTest.jar (1 source file):
4182:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
4183:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
4184:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
4185:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
4186:  (16:07:36) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/io/FileHandlerTest.jar (1 source file):
4187:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
4188:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
4189:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
4190:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
4191:  (16:07:36) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/remote/codec/w3c/W3CHttpResponseCodecTest.jar (1 source file):
4192:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
4193:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
4194:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
4195:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
4196:  java/test/org/openqa/selenium/remote/codec/w3c/W3CHttpResponseCodecTest.java:26: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
4197:  import static org.openqa.selenium.remote.ErrorCodes.METHOD_NOT_ALLOWED;
4198:  ^
...

4274:  (16:07:37) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/remote/http/BinaryMessageTest.jar (1 source file) and running annotation processors (AutoServiceProcessor):
4275:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
4276:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
4277:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
4278:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
4279:  (16:07:37) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/devtools/JavascriptExceptionsTest-edge-remote.jar (1 source file):
4280:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
4281:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
4282:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
4283:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
4284:  (16:07:37) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/remote/libsmall-tests-test-lib.jar (5 source files) and running annotation processors (AutoServiceProcessor):
4285:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
4286:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
4287:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
4288:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
4289:  java/test/org/openqa/selenium/remote/WebDriverFixture.java:170: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
4290:  response.setStatus(new ErrorCodes().toStatus(state, Optional.of(400)));
4291:  ^
...

4377:  (16:07:38) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/remote/RemoteWebDriverBuilderTest.jar (1 source file) and running annotation processors (AutoServiceProcessor):
4378:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
4379:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
4380:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
4381:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
4382:  (16:07:38) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/edge/EdgeOptionsTest.jar (1 source file):
4383:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
4384:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
4385:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
4386:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
4387:  (16:07:38) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/devtools/CdpFacadeTest-edge-remote.jar (1 source file):
4388:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
4389:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
4390:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
4391:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
4392:  (16:07:38) �[32m[19,021 / 22,366]�[0m 895 / 3188 tests, �[31m�[1m3 failed�[0m;�[0m Testing //java/test/org/openqa/selenium/support/pagefactory:UsingPageFactoryTest; 1s remote, remote-cache ... (50 actions, 3 running)
4393:  (16:07:38) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/remote/AugmenterTest.jar (1 source file) and running annotation processors (AutoServiceProcessor):
...

4778:  (16:07:40) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/devtools/DevToolsReuseTest.jar (1 source file):
4779:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
4780:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
4781:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
4782:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
4783:  (16:07:40) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/devtools/JavascriptExceptionsTest-chrome-beta-remote.jar (1 source file):
4784:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
4785:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
4786:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
4787:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
4788:  (16:07:40) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/grid/router/RemoteWebDriverDownloadTest.jar (1 source file):
4789:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
4790:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
4791:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
4792:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
4793:  (16:07:40) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/remote/ErrorCodecTest.jar (1 source file) and running annotation processors (AutoServiceProcessor):
4794:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
...

4818:  java/test/org/openqa/selenium/remote/http/NativeHttpClientMethodsTest.java:195: warning: [removal] <T>sendAsyncNative(HttpRequest,BodyHandler<T>) in HttpClient has been deprecated and marked for removal
4819:  public <T> CompletableFuture<java.net.http.HttpResponse<T>> sendAsyncNative(
4820:  ^
4821:  where T is a type-variable:
4822:  T extends Object declared in method <T>sendAsyncNative(HttpRequest,BodyHandler<T>)
4823:  (16:07:40) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/devtools/NetworkInterceptorTest-remote.jar (1 source file):
4824:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
4825:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
4826:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
4827:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
4828:  (16:07:40) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/remote/RemotableByTest.jar (1 source file) and running annotation processors (AutoServiceProcessor):
4829:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
4830:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
4831:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
4832:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
4833:  java/test/org/openqa/selenium/remote/RemotableByTest.java:23: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
4834:  import static org.openqa.selenium.remote.ErrorCodes.SUCCESS_STRING;
4835:  ^
4836:  java/test/org/openqa/selenium/remote/RemotableByTest.java:23: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
4837:  import static org.openqa.selenium.remote.ErrorCodes.SUCCESS_STRING;
4838:  ^
4839:  java/test/org/openqa/selenium/remote/RemotableByTest.java:23: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
4840:  import static org.openqa.selenium.remote.ErrorCodes.SUCCESS_STRING;
4841:  ^
4842:  java/test/org/openqa/selenium/remote/RemotableByTest.java:45: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
4843:  private final ErrorCodes errorCodes = new ErrorCodes();
4844:  ^
4845:  java/test/org/openqa/selenium/remote/RemotableByTest.java:45: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
4846:  private final ErrorCodes errorCodes = new ErrorCodes();
4847:  ^
4848:  java/test/org/openqa/selenium/remote/RemotableByTest.java:45: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
4849:  private final ErrorCodes errorCodes = new ErrorCodes();
4850:  ^
4851:  java/test/org/openqa/selenium/remote/RemotableByTest.java:45: warning: [removal] ErrorCodes in org.openqa.selenium.remote has been deprecated and marked for removal
4852:  private final ErrorCodes errorCodes = new ErrorCodes();
4853:  ^
...

4859:  java/test/org/openqa/selenium/remote/ProtocolHandshakeTest.java:194: warning: [removal] <T>sendNative(HttpRequest,BodyHandler<T>) in HttpClient has been deprecated and marked for removal
4860:  public <T> java.net.http.HttpResponse<T> sendNative(
4861:  ^
4862:  where T is a type-variable:
4863:  T extends Object declared in method <T>sendNative(HttpRequest,BodyHandler<T>)
4864:  java/test/org/openqa/selenium/remote/ProtocolHandshakeTest.java:188: warning: [removal] <T>sendAsyncNative(HttpRequest,BodyHandler<T>) in HttpClient has been deprecated and marked for removal
4865:  java.util.concurrent.CompletableFuture<java.net.http.HttpResponse<T>> sendAsyncNative(
4866:  ^
4867:  where T is a type-variable:
4868:  T extends Object declared in method <T>sendAsyncNative(HttpRequest,BodyHandler<T>)
4869:  (16:07:40) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/remote/NewSessionPayloadTest.jar (1 source file) and running annotation processors (AutoServiceProcessor):
4870:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
4871:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
4872:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
4873:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
4874:  (16:07:40) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/remote/ErrorHandlerTest.jar (1 source file) and running annotation processors (AutoServiceProcessor):
4875:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
...

5024:  (16:07:42) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/ElementAriaRoleTest-firefox-beta.jar (1 source file):
5025:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
5026:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
5027:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
5028:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
5029:  (16:07:42) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/grid/router/StressTest.jar (1 source file):
5030:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
5031:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
5032:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
5033:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
5034:  (16:07:42) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/ElementAriaRoleTest.jar (1 source file):
5035:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
5036:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
5037:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
5038:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
5039:  (16:07:43) �[32m[19,474 / 22,522]�[0m 1073 / 3188 tests, �[31m�[1m3 failed�[0m;�[0m Testing //java/test/org/openqa/selenium/support/pagefactory:UsingPageFactoryTest; 6s remote, remote-cache ... (50 actions, 9 running)
5040:  (16:07:44) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/ScriptPinningTest.jar (1 source file):
...

5540:  (16:07:48) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/bidi/storage/StorageCommandsTest-remote.jar (1 source file):
5541:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
5542:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
5543:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
5544:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
5545:  (16:07:48) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/ReferrerTest-chrome.jar (1 source file):
5546:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
5547:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
5548:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
5549:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
5550:  (16:07:48) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/AtomsInjectionTest-chrome.jar (1 source file):
5551:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
5552:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
5553:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
5554:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
5555:  (16:07:48) �[32m[20,391 / 23,088]�[0m 1212 / 3188 tests, �[31m�[1m3 failed�[0m;�[0m Testing //java/test/org/openqa/selenium/support/pagefactory:UsingPageFactoryTest; 11s remote, remote-cache ... (49 actions, 8 running)
5556:  (16:07:48) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/ProxySettingTest-chrome-beta.jar (1 source file):
...

5761:  (16:07:50) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/ElementSelectingTest-edge.jar (1 source file):
5762:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
5763:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
5764:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
5765:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
5766:  (16:07:50) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/bidi/input/DefaultWheelTest-remote.jar (1 source file):
5767:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
5768:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
5769:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
5770:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
5771:  (16:07:50) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/SvgDocumentTest-edge.jar (1 source file):
5772:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
5773:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
5774:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
5775:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
5776:  (16:07:50) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/ErrorsTest-edge.jar (1 source file):
5777:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
...

6192:  (16:07:53) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/ElementDomPropertyTest.jar (1 source file):
6193:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
6194:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
6195:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
6196:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
6197:  (16:07:53) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/ElementEqualityTest.jar (1 source file):
6198:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
6199:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
6200:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
6201:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
6202:  (16:07:53) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/bidi/permissions/PermissionsTest-remote.jar (1 source file):
6203:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
6204:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
6205:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
6206:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
6207:  (16:07:53) �[32m[20,992 / 23,286]�[0m 1237 / 3188 tests, �[31m�[1m3 failed�[0m;�[0m Testing //java/test/org/openqa/selenium/support/locators:RelativeLocatorTest; 14s remote, remote-cache ... (48 actions, 8 running)
6208:  (16:07:53) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/bidi/browsingcontext/CaptureScreenshotParametersTest-edge.jar (1 source file):
...

6238:  (16:07:54) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/ElementFindingTest-firefox-beta.jar (1 source file):
6239:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
6240:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
6241:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
6242:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
6243:  (16:07:54) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/bidi/script/ScriptEventsTest-remote.jar (1 source file):
6244:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
6245:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
6246:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
6247:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
6248:  (16:07:54) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/ElementSelectingTest.jar (1 source file):
6249:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
6250:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
6251:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
6252:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
6253:  (16:07:54) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/ErrorsTest-firefox-beta.jar (1 source file):
6254:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
6255:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/1/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
6256:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
6257:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
6258:  (16:07:54) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/ErrorsTest.jar (1 source file):
6259:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
...

6823:  (16:07:58) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/bidi/emulation/SetNetworkConditionsTest-chrome.jar (1 source file):
6824:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
6825:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
6826:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
6827:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
6828:  (16:07:58) �[32mINFO: �[0mFrom Building java/test/org/openqa/selenium/AtomsInjectionTest-chrome-beta.jar (1 source file):
6829:  WARNING: A terminally deprecated method in sun.misc.Unsafe has been called
6830:  WARNING: sun.misc.Unsafe::arrayBaseOffset has been called by com.google.protobuf.UnsafeUtil$MemoryAccessor (file:/mnt/engflow/worker/work/0/exec/external/rules_java++toolchains+remote_java_tools/java_tools/JavaBuilder_deploy.jar)
6831:  WARNING: Please consider reporting this to the maintainers of class com.google.protobuf.UnsafeUtil$MemoryAccessor
6832:  WARNING: sun.misc.Unsafe::arrayBaseOffset will be removed in a future release
6833:  (16:07:58) �[32mINFO: �[...

@joerg1985

joerg1985 commented May 31, 2026

Copy link
Copy Markdown
Member Author

looks like the SpotBugs findings are not related to this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@qodo-code-review qodo-code-review[bot] qodo-code-review[bot] left review comments

Assignees

No one assigned

Labels

B-build Includes scripting, bazel and CI integrations C-java Java Bindings

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@joerg1985 @selenium-ci