[🐛 Bug]: There are still some leftovers from the HtmlUnit removal

[rbri]

What happened?

Looks like there are some leftovers from [java] remove the unused htmlunit-driver from the unit tests (#12849).

• "net.sourceforge.htmlunit:htmlunit-core-js:2.70.0" is still referenced from some bzl scripts as dependency
• java\test\org\openqa\selenium\atoms\CompiledAtomsNotLeakingTest.java is base on core-js but the referenced RESOURCE_PATH = "/org/openqa/selenium/atoms/execute_script.js" no longer exists
• java\test\org\openqa\selenium\atoms\InputAtomsTest.java is base on core-js but the referenced RESOURCE_PATH = "/org/openqa/selenium/atoms/atoms_inputs.js" no longer exists

How can we reproduce the issue?

* verify if the tests (CompiledAtomsNotLeakingTest.java, InputAtomsTest.java) are still part of the test suite and passing

Relevant log output

nothing

Operating System

any

Selenium version

any

What are the browser(s) and version(s) where you see this issue?

any

What are the browser driver(s) and version(s) where you see this issue?

any

Are you using Selenium Grid?

no

[github-actions]

@rbri, thank you for creating this issue. We will troubleshoot it as soon as we can.

---

Info for maintainers

Triage this issue by using labels.

If information is missing, add a helpful comment and then I-issue-template label.

If the issue is a question, add the I-question label.

If the issue is valid but there is no time to troubleshoot it, consider adding the help wanted label.

If the issue requires changes or fixes from an external project (e.g., ChromeDriver, GeckoDriver, MSEdgeDriver, W3C), add the applicable G-* label, and it will provide the correct link and auto-close the issue.

After troubleshooting the issue, please add the R-awaiting answer label.

Thank you!

[pujagani]

Thank you for reporting and providing the issue. Would you like to send a PR and help contribute? :)

[github-actions]

This issue is looking for contributors.

Please comment below or reach out to us through our IRC/Slack/Matrix channels if you are interested.

[rbri]

Sorry i really have no time to work on that - maybe @joerg1985 who did the original pr for removing has an idea why we still have this dependency.

[joerg1985]

I will take care of this, as soon as i find the time.

[titusfortner]

@diemol did that PR fix this issue?

[rbri]

@titusfortner sorry but the pr updates the htmlunit dependencies instead of removing it

And btw: Happy new year to you all

[joerg1985]

@rbri i think these files are generated by bazel at build time and should be in place as soon as the tests are running:

selenium/java/test/org/openqa/selenium/atoms/BUILD.bazel

Lines 4 to 16 in b2a8448

	copy_file(
	name = "execute_script",
	src = "//javascript/atoms/fragments:execute-script.js",
	out = "execute_script.js",
	)
	copy_file(
	name = "atoms_inputs",
	src = "//javascript/webdriver/atoms:inputs_bin.js",
	out = "atoms_inputs.js",
	)

After running bazel test --cache_test_results=no --test_size_filters=small grid java/... they are generated:

[rbri]

ok, i see having this dependency to htmlunit-core-js still makes sense.
I think we can close this.

@joerg1985 / @diemol do you like to close or should i do?

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.