On Wed, 23 Sep 2020 18:07:18 GMT, Alexey Semenyuk <asemen...@openjdk.org> wrote:
>> Alexander Matveev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> 8231591: [TESTBUG] Create additional two phase jpackage tests (revision 2)
>
> test/jdk/tools/jpackage/helpers/jdk/jpackage/test/PackageTest.java line 320:
>
>> 318: }
>> 319:
>> 320: public PackageTest addLauncherName(String name) {
>
> There is no need to make this function public and explicitly call it from
> tests.
> It would be better to add call to this function from
> AdditionalLauncher.applyTo(PackageTest).
But test calls AdditionalLauncher.applyTo(JPackageCommand cmd), which does not
have reference to PackageTest. Do you
know why we need two applyTo()?
-------------
PR: https://git.openjdk.java.net/jdk/pull/263
