Please review: 7127906: (launcher) convert the launcher regression tests to java

Kumar Srinivasan kumar.x.srinivasan at oracle.COM
Fri Jan 27 19:36:16 UTC 2012


On 1/27/2012 11:05 AM, Naoto Sato wrote:
> Hi Kumar,
>
> I have a question/suggestion on TestHelper.isEnglishLocale field.
>
> - Is there any chance that the test case changes the default locale by 
> Locale.setDefault() during the test execution? If that's the case, 
> static initialization of the field may not work.

Yes, the tests that do change the default locale  are UnicodeTest.java and
  I18NJarTest.java, however they execute in their own child Process/VM.

I think I should also the save default to a static, such that we can
revert back the locale to the original default.

>
> - If the default locale does not change during the test execution, 
> then I'd suggest to initialize it with something like:
>
> isEnglishLocale = Locale.getDefault().getLanguage().equals("en");

What do you suggest ?

Thanks
Kumar

>
> Otherwise it won't work with other English locales such as en_GB.
>
> Other than that, it looks good to me.
>
> Naoto
>
> On 1/27/12 10:30 AM, Joe Darcy wrote:
>> Hi Kumar,
>>
>> Looks fine; good to see the shell code go away!
>>
>> -Joe
>>
>> On 01/27/2012 10:03 AM, Kumar Srinivasan wrote:
>>> Hi Joe, Naoto, Sherman,
>>>
>>> Here are the highlights of these changes:
>>>
>>> 1. The TestHelper.java is now extensible, this simplifies the usage of
>>> static variables
>>> and methods, added a couple of convenience methods and some minor
>>> refactoring.
>>>
>>> 2. Eliminated some shell tests, (few more remaining, saved for a
>>> future project).
>>> a. UnicodeTest.sh, UnicodeCleanup.java merged into UnicodeTest.java
>>> b. i18nTest.sh, deleteI18n.sh, CreatePlatformFile.java merged into
>>> I18NTest.java
>>> c. ChangeDataModel.sh converted to ChangeDataModel.java
>>> d. unresolvedExceptions.sh converted and merged into
>>> UnresolvedExceptions.java
>>>
>>> 3. Arrrghs.java when testing on Solaris with ja_JP.UTF-8 locale, some
>>> of the tests fail
>>> as the launcher messages in java are localized, therefore these tests
>>> are skipped,
>>> under such conditions.
>>>
>>> Tests: Windows English/Japanese, Solaris C/ja_JP.UTF-8 + usual jprt
>>> systems (undergoing).
>>>
>>> webrev: http://cr.openjdk.java.net/~ksrini/7127906/webrev.0/
>>>
>>> Thanks
>>> Kumar
>>>
>>>
>>
>




More information about the core-libs-dev mailing list