JDK 9 RFR of JDK-8038330: tools/jar/JarEntryTime.java fails intermittently on checking extracted file last modified values are the current times
Amy Lu
amy.lu at oracle.com
Tue Mar 1 13:27:47 UTC 2016
On 3/1/16 7:41 PM, Peter Levart wrote:
> Hi Amy,
>
> I think that the following test:
>
> 178 if (!(Math.abs(now - start) >= 0L && Math.abs(end - now)
> >= 0L)) {
>
> ...will always be false. Therefore, the test will always succeed.
>
> Perhaps you wanted to test the following:
>
> assert start <= end;
> if (start > now || now > end) { ...
Thank you Peter for reviewing.
My bad ... I'm updating the webrev and will send updated version tomorrow.
Thanks,
Amy
>
>
> Regards, Peter
>
> On 03/01/2016 07:11 AM, Amy Lu wrote:
>> Please review the patch for test tools/jar/JarEntryTime.java
>>
>> In which two issues fixed:
>>
>> 1. Test fails intermittently on checking the extracted files'
>> last-modified-time are the current times.
>> Instead of compare the file last-modified-time with pre-saved time
>> value “now” (which is the time *before* current time, especially in a
>> slow run, the time diff of “now” and current time is possible greater
>> than 2 seconds precision (PRECISION)), test now compares the
>> extracted file’s last-modified-time with newly created file
>> last-modified-time.
>> 2. Test may fail if run during the Daylight Saving Time change.
>>
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8038330
>> webrev: http://cr.openjdk.java.net/~amlu/8038330/webrev.00/
>>
>> Thanks,
>> Amy
>
More information about the core-libs-dev
mailing list