On Mon, 28 Nov 2022 14:47:16 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
>> Lukasz Kostyra has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains four commits: >> >> - Merge branch 'master' of https://git.openjdk.org/jfx into >> JDK-8265828-locale >> - Refactor remaining LocalStringConverter tests >> >> Treatment done in this commit is similar to the previous change. >> - LocalDateTimeStringConverterTest: Refactor test to properly utilize Locale >> >> * Locale initialization was moved to @BeforeClass method. >> * DateTimeFormatter objects are allocated after Locale initialization >> * Since LocalDateTimeStringConverter depends on DateTimeFormatter and on >> VM's Locale, >> creation of it was moved to @Before method. >> - 8265828: [TestBug] Save and restore the default Locale in javafx.base >> unit test LocalDateTimeStringConverterTest > > modules/javafx.base/src/test/java/test/javafx/util/converter/LocalDateStringConverterTest.java > line 54: > >> 52: @RunWith(Parameterized.class) >> 53: public class LocalDateStringConverterTest { >> 54: private static final LocalDate VALID_DATE = LocalDate.of(1985, 1, >> 12); > > I presume that this is not locale sensitive? LocalDate is not locale sensitive. ------------- PR: https://git.openjdk.org/jfx/pull/954
