Please review: fix for java.time Issues with French locale on compact1, 2
roger riggs
roger.riggs at oracle.com
Thu Sep 12 14:57:59 UTC 2013
Hi Alan,
For conformance tests, the configurations are more limited (Only the US
locale).
The other java.time tests that refer to specific locales don't rely on
the locale data
only the identity of the locale or depend on the fallback to the US locale.
Otherwise a wider refactoring would be needed and still the configurations
needed for unit testing would need to be more fine grained.
Thanks, Roger
On 9/12/2013 10:32 AM, Alan Bateman wrote:
> On 12/09/2013 15:14, roger riggs wrote:
>> Hi,
>>
>> Testing of java.time has found a problematic test on compact 1 and 2.
>> The test relies on localization data for the French locale that is
>> not present.
>> Changing the tests to the US locale would be sufficient but then are
>> redundant
>> with other tests. The tests are unnecessary and are removed.
>>
>> Please review:
>> http://cr.openjdk.java.net/~rriggs/webrev-nofrench-8024618/
>>
>> Thanks, Roger
> TCK tests can't depend on optional locales so the change looks okay to
> me.
>
> In general however, there are several tests that uses FR and other
> locales. I wasn't aware that we are testing without localedata.jar
> being present. I hope not as it would require more yucky lists in
> TEST.groups.
>
> -Alan
More information about the core-libs-dev
mailing list