RFR: 8373935: Migrate java/lang/invoke tests away from TestNG [v2]
Chen Liang
liach at openjdk.org
Tue Jan 20 23:02:50 UTC 2026
On Wed, 17 Dec 2025 23:58:36 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:
>> Chen Liang has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Some omissions
>
> test/jdk/java/lang/invoke/DefineClassTest.java line 211:
>
>> 209:
>> 210: Class<?> clazz = lookup.defineClass(generateClass("java.lang.Foo"));
>> 211: assertEquals("java.lang.Foo", clazz.getName());
>
> Looks like the parameter order was already reversed here.
No? I think my new order is right that the "actual" should be the value from reflection.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28879#discussion_r2710368951
More information about the core-libs-dev
mailing list