RFR: 8343875: Minor improvements of jpackage test library [v2]
Alexander Matveev
almatvee at openjdk.org
Wed Nov 13 02:12:33 UTC 2024
On Wed, 13 Nov 2024 02:02:27 GMT, Alexey Semenyuk <asemenyuk at openjdk.org> wrote:
>> The PR affects jpackage test library only. Changes:
>>
>> - Removed dependencies on `java.io.File`. `java.io.File.pathSeparator` is still used as there is no better alternative.
>> - Use `TKit.concatMessages()` instead of `String.format()` in TKit.assert...() methods.
>> - Added unit tests for all TKit.assert...() modified methods.
>> - Added unit tests for `JavaAppDesc` class.
>> - Added `jdk.jpackage.test.TestSuite` class to automatically run all jpackage test library unit tests. This eliminates the need to add jtreg comments to every jpackage test library unit test java source.
>
> Alexey Semenyuk has updated the pull request incrementally with one additional commit since the last revision:
>
> Remove redundant space char
Marked as reviewed by almatvee (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/21995#pullrequestreview-2431546000
More information about the core-libs-dev
mailing list