RFR: 8005202 - java/util/logging/CheckLockLocationTest.java fail on solars_10

Mandy Chung mandy.chung at oracle.com
Tue Nov 19 20:04:24 UTC 2013


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



More information about the core-libs-dev mailing list