RFR: 8265828: [TestBug] Save and restore the default Locale in javafx.base unit test LocalDateTimeStringConverterTest [v3]

[Andy Goryachev]

On Mon, 28 Nov 2022 14:47:16 GMT, Kevin Rushforth <kcr at 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