[8u] RFR: 8177776: Create an equivalent test case for JDK9's SupplementalJapaneseEraTest
Naoto Sato
naoto.sato at oracle.com
Thu Apr 6 15:46:29 UTC 2017
Thanks for reviewing, Roger.
On 4/6/17 8:39 AM, Roger Riggs wrote:
> Hi Naoto,
>
> Thanks for replacing the shell script with Java code.
>
> Given the size of the JDK, I'd suggest removing the copy at the end of
> the test
> unless you can rely on jtreg to remove it promptly.
I was thinking about using @AfterTest to do it, but then realized that
jtreg automatically cleans up the "scratch" directory after the test,
where the test JDK is copied into. Here is the quote from jtreg's page [1]:
---
jtreg will automatically clean up any files written in the scratch
directory. In general, you should not clean these files up within the
test because they may provide infomation useful to diagnose a test
failure, should that be necessary.
---
Naoto
[1] http://openjdk.java.net/jtreg/writetests.html
>
> The rest looks fine, Roger
>
>
> On 4/5/2017 5:14 PM, Naoto Sato wrote:
>> I revised the test case not to rely on shell script. Please review.
>>
>> http://cr.openjdk.java.net/~naoto/8177776/webrev.01/
>>
>> Naoto
>>
>> On 3/30/17 2:10 PM, Naoto Sato wrote:
>>> Hello,
>>>
>>> Please review the changes to the following issue:
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8177776
>>>
>>> The proposed change is located at:
>>>
>>> http://cr.openjdk.java.net/~naoto/8177776/webrev.00/
>>>
>>> This is for backporting fixes for JapaneseEra related issues (8054214,
>>> 8173423). The original fixes in JDK9 included a test case,
>>> SupplementalJapaneseEraTest which is intended for the system property
>>> testing. The above proposed fix intends to adapt that test case into
>>> JDK8, where calendars.properties file is used instead of the system
>>> property. The test is pretty much identical to JDK9's. The difference is
>>> to deal with the calendars.properties file and removed some
>>> non-applicable cases in JDK8.
>>>
>>> Naoto
>
More information about the core-libs-dev
mailing list