(10) RFR of JDK-8181478, Refactor java/io shell tests to plain java tests
Hamlin Li
huaming.li at oracle.com
Fri Jun 16 09:09:40 UTC 2017
Ping.
Thank you
-Hamlin
On 2017/6/15 14:29, Hamlin Li wrote:
> Hi Alan, Paul,
>
> Thank you for review, new webrev at:
> http://cr.openjdk.java.net/~mli/8181478/webrev.01/
>
> Please also check my comments inline.
>
>
> On 2017/6/15 1:28, Alan Bateman wrote:
>>
>> On 14/06/2017 18:20, Paul Sandoz wrote:
>>>> On 12 Jun 2017, at 01:00, Hamlin Li <huaming.li at oracle.com> wrote:
>>>>
>>>> Would you please review the below patch?
>>>>
>>>> bug: https://bugs.openjdk.java.net/browse/JDK-8181478
>>>>
>>>> webrev: http://cr.openjdk.java.net/~mli/8181478/webrev.00/
>>>>
>>> It took me a few moments to grok the NonExistentDriver behaviour. If
>>> i got it correct, this is checking that deleteOnExit is working
>>> correctly?
>>>
>>> If so it might be clearer to make this is a separate test that is
>>> run as a driver and a test with different arguments performing the
>>> action and then the checking.
> Agree, it's not that clear. And I just found out there is another test
> java/io/File/DeleteOnExit.java testing the File.deleteOnExit. so I
> just remove NonExistentDriver.java and related code.
>> I agree. Also in FileOpenTest then it can use APIs to set the hidden
>> attribute, no need to launch attrib.exe each time.
> This is a test running only on windows.
> I'm not sure if I understand you correctly. In new patch I use
> Files.setAttribute(path, "dos:hidden", true/false); to hide/un-hide a
> file, and use File.setReadOnly() to make a file read only.
>
> Thank you
> -Hamlin
>>
>> -Alan
>
More information about the core-libs-dev
mailing list