On Tue, 19 Sep 2023 20:15:20 GMT, Alexey Semenyuk <asemen...@openjdk.org> wrote:

> Strip
> 
> /*
>  * @test
>  * @modules jdk.jpackage
>  * @compile --patch-module jdk.jpackage=${test.src} --add-reads 
> jdk.jpackage=ALL-UNNAMED --add-exports 
> jdk.jpackage/jdk.jpackage.internal=ALL-UNNAMED AppImageFileTest.java
>  * @run junit/othervm --patch-module jdk.jpackage=${test.classes} --add-reads 
> jdk.jpackage=ALL-UNNAMED --add-exports 
> jdk.jpackage/jdk.jpackage.internal=ALL-UNNAMED 
> jdk.jpackage.internal.AppImageFileTest
>  */
> 
> comment from jpackage JUnit tests and instruct jtreg to run them through 
> `JUnit.dirs` property.

Looks good.

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

Marked as reviewed by almatvee (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/15826#pullrequestreview-1634191968

Reply via email to