On Thu, 20 Feb 2025 07:51:44 GMT, Alan Bateman <al...@openjdk.org> wrote:
>> Brian Burkhalter has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8024695: Extend and clean up test > > test/jdk/java/io/File/EmptyPath.java line 260: > >> 258: assertFalse(f.canRead()); >> 259: assertTrue(f.setReadable(true)); >> 260: assertTrue(f.canRead()); > > Need try-finally in the setter tests to ensure that the directory's > permissions are restored in the event that the test fails. So changed in a3b7977. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22821#discussion_r1964415260
