RFR: 8263536: Add missing @compile tags to jpackage tests [v2]

Alexey Semenyuk asemenyuk at openjdk.java.net
Mon Mar 15 19:46:19 UTC 2021


On Sat, 13 Mar 2021 08:38:16 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> Alexey Semenyuk has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains one commit:
>> 
>>   8263536: Add missing @compile tags to jpackage tests
>
> Changes requested by iklam (Reviewer).

@kevinrushforth I plan to resolve JDK-8263474 manually as "Delivered". Would this work?

> test/jdk/tools/jpackage/windows/WinDirChooserTest.java line 44:
> 
>> 42:  * @requires (os.family == "windows")
>> 43:  * @modules jdk.jpackage/jdk.jpackage.internal
>> 44:  * @compile WinDirChooserTest.java
> 
> Changes like this one is not necessary. `@run` will build WinDirChooserTest automatically.
> 
> `@compile` should be added only when necessary, like WinInstallerUiTest.java

It is better to have `@compile` everywhere for consistency. 

The proper way to run tests is by passing test class name as an argument for test runner, test class should not have `main()` and should have `@Test` annotation for test methods. Some tests have not been updated to follow this design and adding `@compile` to them doesn't make a difference now. But eventually they will be updated and they will need `@compile` anyways.

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

PR: https://git.openjdk.java.net/jdk/pull/2975


More information about the core-libs-dev mailing list