RFR: 8362598: [macos] Add tests for custom info plist files

Alexey Semenyuk asemenyuk at openjdk.org
Fri Sep 26 11:32:57 UTC 2025


On Fri, 26 Sep 2025 02:54:32 GMT, Alexander Matveev <almatvee at openjdk.org> wrote:

> - Added test for custom info plist to cover app image including embedded runtime and runtime installer.
> - Fixed bug in `writePList`. It was missing `writeStartDocument()/writeEndDocument()` and `DOCTYPE` should be full xml string.

test/jdk/tools/jpackage/helpers/jdk/jpackage/test/MacHelper.java line 178:

> 176:     }
> 177: 
> 178:     public static Path createInputRuntimeImage() throws IOException {

This is not mac-specific helper function. You'd rather move it to JPackageCommand class.

BTW, there is a bunch of `createInputRuntimeImage()` functions already:
 - https://github.com/openjdk/jdk/blob/7bfdb0120752d01da96c19e8037a6e909847d63c/test/jdk/tools/jpackage/macosx/SigningRuntimeImagePackageTest.java#L97
 - https://github.com/openjdk/jdk/blob/7bfdb0120752d01da96c19e8037a6e909847d63c/test/jdk/tools/jpackage/share/RuntimePackageTest.java#L176
 - https://github.com/openjdk/jdk/blob/7bfdb0120752d01da96c19e8037a6e909847d63c/test/jdk/tools/jpackage/share/RuntimeImageTest.java#L51

Instead of adding another one, let's consolidate them in one.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27509#discussion_r2382089629


More information about the core-libs-dev mailing list