On Wed, 7 May 2025 17:28:34 GMT, Lance Andersen <[email protected]> wrote:
>> Henry Jen has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Restore Validator access level
>
> test/jdk/tools/jar/ValidatorTest.java line 232:
>
>> 230: var err = e.getMessage();
>> 231: System.out.println(err);
>> 232: Assertions.assertTrue(err.contains("Warning: 2 copies of
>> META-INF/MANIFEST.MF is detected in local file header"));
>
> Given you are importing Assertions, you should be able to just use
> `assertTrue`
I didn't use static import. Since that seems to be preferred, I changed to do
static imports.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24430#discussion_r2092017390