RFR: 8346434: Add test for non-automatic service binding [v4]

Alexey Semenyuk asemenyuk at openjdk.org
Thu Feb 6 02:29:49 UTC 2025


> Added a test case to JLinkOptionsTest to test that jpackage doesn't bind services in app's runtime by default.
> 
> The test builds two app images for the same app. One with the default jlink options and another with `--bind-services` jlink option. The test compares module lists from both runtimes. The expected result is that the runtime built with `--bind-services` jlink option has additional modules.
> 
> The PR must be integrated after https://github.com/openjdk/jdk/pull/22644. Otherwise, the newly added test will fail.

Alexey Semenyuk has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains nine additional commits since the last revision:

 - Merge branch 'master' into JDK-8346434
 - Merge branch 'master' into JDK-8346434
 - Merge branch 'master' into JDK-8346434
 - Use HelloApp.assertApp() to run a launcher
 - Merge branch 'master' into JDK-8346434
 - Trailing whitespace removed
 - Merge branch 'master' into JDK-8346434
 - Copyright year updated
 - 8346434: Add test for non-automatic service binding

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/23047/files
  - new: https://git.openjdk.org/jdk/pull/23047/files/6d6e0442..6a54a352

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=23047&range=03
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=23047&range=02-03

  Stats: 91 lines in 11 files changed: 18 ins; 20 del; 53 mod
  Patch: https://git.openjdk.org/jdk/pull/23047.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/23047/head:pull/23047

PR: https://git.openjdk.org/jdk/pull/23047


More information about the core-libs-dev mailing list