<i18n dev> RFR: 8369590: LocaleEnhanceTest has incorrectly passing test case
Justin Lu
jlu at openjdk.org
Mon Oct 13 21:27:14 UTC 2025
This PR corrects _test/jdk/java/util/Locale/LocaleEnhanceTest.java_, which has two test cases under `testBuilderSetLanguageTag()` which accidentally pass. One checks that Locale.setLanguageTag(String) throws ILE for duplicate extensions and the other for duplicate U-extension keys. The test cases are updated to actually test the provided code. When the test cases are fixed, they now fail.
Fixing the behavior to match the expectation of those test cases is consistent with the specification.
>From `Locale.forLanguageTag(String)`,
>
> * <p>If the specified language tag contains any ill-formed subtags,
> * the first such subtag and all following subtags are ignored. Compare
> * to {@link Locale.Builder#setLanguageTag(String)} which throws an exception
> * in this case.
and the RFC specification
> Each singleton subtag MUST appear at most one time in each tag
> (other than as a private use subtag). That is, singleton subtags
> MUST NOT be repeated. For example, the tag "en-a-bbb-a-ccc" is
> invalid because the subtag 'a' appears twice.
Since duplicate extensions (and Unicode keys/attributes) are invalid, throwing `IllformedLocaleException` in (the strict) `Locale.Builder` and ignoring in (the lenient) `Locale.forLanguageTag` for such tags would be appropriate. This PR updates the implementation as such.
-------------
Commit messages:
- Add duplicate checking for Locale.Builder.setLanguageTag(String)
- Add duplicate U extension attributes test case
- Remove testing classes in favor or JUnit (which fixes unintentional passing)
Changes: https://git.openjdk.org/jdk/pull/27775/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=27775&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8369590
Stats: 686 lines in 4 files changed: 75 ins; 169 del; 442 mod
Patch: https://git.openjdk.org/jdk/pull/27775.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/27775/head:pull/27775
PR: https://git.openjdk.org/jdk/pull/27775
More information about the i18n-dev
mailing list