RFR: JDK-8255055: Create two phase test for case with different names, and fix linux DTI
Alexey Semenyuk
asemenyuk at openjdk.java.net
Mon Nov 16 19:05:02 UTC 2020
On Mon, 16 Nov 2020 15:15:02 GMT, Andy Herrick <herrick at openjdk.org> wrote:
> … fix linux DTI
Changes requested by asemenyuk (Committer).
test/jdk/tools/jpackage/helpers/jdk/jpackage/test/JPackageCommand.java line 484:
> 482: return TKit.getCurrentDefaultAppName();
> 483: }
> 484:
I don't like the idea of introducing this method. Two reasons:
- it adds confusion because we already have name()
- name() is not returning app name any more for the case of two phase jpackage usage as app name is now supposed to be returned by getApplicationName()
- TKit.getCurrentDefaultAppName() should be used only to set the default app name when jpackage default command line is configured by the test.
I suggest instead update name() method. Test is "--app-image" is on the command line, if it is, extract app name of xml file with app image info. This will eliminate need for another method in JPackageCommand interface and make name() method return accurate result without relying on the current usage scenarios of "--app-image" parameter in tests.
-------------
PR: https://git.openjdk.java.net/jdk/pull/1229
More information about the core-libs-dev
mailing list