RFR: 8373935: Migrate java/lang/invoke tests away from TestNG [v2]
Chen Liang
liach at openjdk.org
Tue Jan 20 23:55:04 UTC 2026
On Wed, 17 Dec 2025 23:52:44 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:
>> test/jdk/java/lang/invoke/AccessControlTest.java line 217:
>>
>>> 215: boolean sameClass = (c1 == c2);
>>> 216: Assertions.assertTrue(samePackage || !sameTopLevel);
>>> 217: Assertions.assertTrue(sameTopLevel || !sameClass);
>>
>> Suggestion:
>>
>> assertTrue(samePackage || !sameTopLevel);
>> assertTrue(sameTopLevel || !sameClass);
>
> Looks like you can do this cleanup in other places too.
Done, purged qualified calls and replaced with static imports consistently (except the javadoc example test where there is a name clash)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28879#discussion_r2710459676
More information about the core-libs-dev
mailing list