On Wed, 17 Dec 2025 21:41:50 GMT, Chen Liang <[email protected]> wrote:
>> Refactor java/lang/invoke tests to use JUnit instead of TestNG.
>> This is done by:
>> 1. First a round of automatic conversion
>> 2. Simplify exception handling tests
>> 3. Replacing `assert` keyword and switching to better assertion APIs for
>> equality etc.
>> 4. Some other random cleanups, such as module status
>>
>> Testing: java/lang/invoke on Linux-x64. I un-problemlisted the updated
>> `java/lang/invoke/lambda/LambdaFileEncodingSerialization.java` too.
>
> Chen Liang has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Some omissions
test/jdk/java/lang/invoke/DropLookupModeTest.java line 158:
> 156: Integer.MIN_VALUE,
> 157: })
> 158: public void testBadInput(Integer modeToDrop) {
The ValueSource produces ints but the parameter to testBadInput is Integer. The
original data provider test used Integer too so I assume this was just the
conversion.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28879#discussion_r2629914346
