[jdk21u-dev] RFR: 8310049: Refactor Charset tests to use JUnit
Goetz Lindenmaier
goetz at openjdk.org
Wed Oct 8 06:42:10 UTC 2025
On Tue, 7 Oct 2025 17:29:40 GMT, Justin Lu <jlu at openjdk.org> wrote:
>> I backport this for parity with 21.0.10-oracle
>
> test/jdk/java/nio/charset/Charset/IllegalCharsetName.java line 52:
>
>> 50: assertThrows(IllegalCharsetNameException.class,
>> 51: () -> Charset.forName(name));
>> 52: assertThrows(IllegalCharsetNameException.class,
>
> In _illegalCharsetsTest_ and _emptyCharsetsTest_, please consider changing the second instance of `Charset.forName` to `Charset.isSupported` which was a faulty conversion made in the original change. Otherwise if not in this PR, please consider back-porting [JDK-8369078](https://bugs.openjdk.org/browse/JDK-8369078) as well, thank you.
Thanks for pointing me to this. I will backport 8369078, too.
-------------
PR Review Comment: https://git.openjdk.org/jdk21u-dev/pull/2296#discussion_r2412731399
More information about the jdk-updates-dev
mailing list