On 11/19/2013 11:35 AM, Daniel Fuchs wrote:

I am therefore proposing to add an additional check in the test's
setUp() method, in order to verify that the directory is indeed
not writable.


It may be useful to print the owner of the directory in case if it's running with root permission.

webrev:
http://cr.openjdk.java.net/~dfuchs/webrev_8005202/webrev.00/


 175         final boolean nonWritable = nonWritableDir.setWritable(false);
 176         final boolean isWritable = Files.isWritable(path);
 177         if (nonWritable && !isWritable) {

Perhaps it should assert isWriteable if File.setWriteable returns true.

Mandy

Reply via email to