<i18n dev> RFR: 8311663: Additional refactoring of Locale tests to JUnit
    Andrey Turbanov 
    aturbanov at openjdk.org
       
    Mon Jul 17 11:01:03 UTC 2023
    
    
  
On Thu, 13 Jul 2023 23:23:42 GMT, Justin Lu <jlu at openjdk.org> wrote:
> Please review this PR which refactors more java.util.Locale tests to JUnit with some minor cleanup as well.
> 
> Although some of the files could benefit from being renamed bugNNNNNNN to something more descriptive, this makes reviewing harder, and will be handled separately.
test/jdk/java/util/Locale/Bug8179071.java line 78:
> 76:         return Stream.of(
> 77:                 Arguments.of("pa-PK", "\u0a1c\u0a28"),
> 78:                 Arguments.of("uz-AF" , "yan"),
Suggestion:
                Arguments.of("uz-AF", "yan"),
test/jdk/java/util/Locale/Bug8179071.java line 95:
> 93:                 .forEach(tag -> {if(LegacyAliases.contains(tag)) {invalidTags.add(tag);}});
> 94:         assertEquals(true, invalidTags.isEmpty(),
> 95:                 "Deprecated and Legacy tags found  " + invalidTags  + " in AvailableLocales ");
Suggestion:
                "Deprecated and Legacy tags found  " + invalidTags + " in AvailableLocales ");
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14881#discussion_r1265185559
PR Review Comment: https://git.openjdk.org/jdk/pull/14881#discussion_r1265185856
    
    
More information about the i18n-dev
mailing list