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