RFR: 8023113: tools/jar/ChangeDir.java fails if /tmp/a exists

Xueming Shen xueming.shen at oracle.com
Wed Sep 18 19:45:21 UTC 2013


OK, how about this one? I'm even using lambda:-)

http://cr.openjdk.java.net/~sherman/8023113/webrev/

-Sherman

On 09/18/2013 09:42 AM, Xueming Shen wrote:
> On 09/18/2013 03:31 AM, Alan Bateman wrote:
>> On 17/09/2013 22:55, Xueming Shen wrote:
>>> Hi,
>>>
>>> Please help the small update to the test case tools/jar/ChangeDir.java,
>>> which might fail if the "dedicated" test directory /tmp/a exists and the
>>> user does not have privilege to delete/clean it up.
>>>
>>> The proposed change here is to use a random file name (from File.getTempFile())
>>> for the top test directory (instead of the fixed /tmp/a), and really clean it up
>>> after testing (the old test leaves /tmp/a and those temp files undeleted).
>>>
>>> http://cr.openjdk.java.net/~sherman/8023113/webrev
>> I've run into this a few times so thanks for fixing it.
>>
>> What you have is okay although some of the setup and cleanup could be reduced, for example doTest could setup the test tree with:
>>
>> Path topDir = Files.createTempDirectory("delete");
>> Path testDir = Files.createDirectories(topDir.resolve("a").resolve("b"));
>>
>> It's not important for this fix of course, just an observation when looking at an old test.
>>
> thought about using the nio,  but then decided to keep the test as the classic "old" io,
> otherwise I would end up with rewriting the test:-)
>
> -Sherman
>
>> -Alan.
>




More information about the core-libs-dev mailing list