On Fri, 14 Apr 2023 21:59:33 GMT, Lance Andersen <[email protected]> wrote:
>> Please review this trivial change when ZipFS returns the wrong >> java.nio.file.FileSystemException message due the the parameters being >> reversed. >> >> I also included a simple junit test as part of the fix. >> >> Mach5 tiers1-3 are clean >> >> Best >> Lance > > Lance Andersen has updated the pull request incrementally with one additional > commit since the last revision: > > Address additional feedback test/jdk/jdk/nio/zipfs/ZipFSDirectoryExceptionMessageTest.java line 96: > 94: var file = zipfs.getPath(DIRECTORY_NAME); > 95: var x = assertThrows(FileSystemException.class, () -> > Files.newInputStream(file)); > 96: assertEquals(x.getMessage(), DIR_EXCEPTION_MESSAGE); JUnit asserts have the expected parameter first followed by the actual. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/13482#discussion_r1167305606
