RFR: 8349648: Test tools/jpackage/share/JLinkOptionsTest.java fails with --enable-linkable-runtime set after JDK-8346434

[Severin Gehwolf]

On Fri, 7 Feb 2025 13:13:24 GMT, Christoph Langer <clanger at openjdk.org> wrote:

> The change for JDK-8346434 added a new test case to tools/jpackage/share/JLinkOptionsTest.java which does not respect the constraint of the linkable runtime (JEP 493) that no jdk.jlink module can be part of the target image.
> This can be circumvented by limiting the modules.

test/jdk/tools/jpackage/share/JLinkOptionsTest.java line 145:

> 143:     public void testNoBindServicesByDefault() {
> 144:         final var defaultModules = getModulesInRuntime("--limit-modules java.smartcardio,jdk.crypto.cryptoki,java.desktop");
> 145:         final var modulesWithBindServices = getModulesInRuntime("--bind-services --limit-modules java.smartcardio,jdk.crypto.cryptoki,java.desktop");

Pre-existing: It would be good to document which additional modules in the set get dragged in by `--bind-services`. That would allow future readers of that test to figure out why a test starts to fail (e.g. by a future change removing a provided service in a module).

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/23514#discussion_r1946847340