RFR: 8368335: Refactor the rest of Locale TestNG based tests to JUnit [v2]
Justin Lu
jlu at openjdk.org
Tue Sep 23 18:25:54 UTC 2025
On Tue, 23 Sep 2025 16:56:37 GMT, Naoto Sato <naoto at openjdk.org> wrote:
>> Justin Lu has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Swap other args in assertions in LocaleMatchingTest
>
> test/jdk/java/util/Locale/LocaleMatchingTest.java line 349:
>
>> 347: assertNotEquals(lr1, lr4, " LR(ja, 1.0).equals(LR(en, 1.0)) should return false.");
>> 348: assertNotNull(lr1, " LR(ja, 1.0).equals(null) should return false.");
>> 349: assertNotEquals("", lr1, " LR(ja, 1.0).equals(\"\") should return false.");
>
> Nit: If we change the unexpected and actual, probably do the same for L346/347
I had originally left L345-347 untouched because it seemed like one of those cases where there wasn't a clear expected/unexpected (compared to the `""` in L349). But you are right that it would be consistent and no reason to not do so considering the opposite _actual_ vs _expected_ positioning of TestNG vs JUnit. Fixed in https://github.com/openjdk/jdk/pull/27441/commits/aef3447704fabe43348dfe2fdfd4d1731515571b.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27441#discussion_r2373123586
More information about the core-libs-dev
mailing list