On Wed, 14 May 2025 16:39:18 GMT, Lance Andersen <[email protected]> wrote:
>> David Beaumont has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Fix comment based on current behaviour.
>
> test/jdk/jdk/nio/zipfs/NewFileSystemTests.java line 219:
>
>> 217: assertFalse(fs.isReadOnly());
>> 218: if (!"Default
>> version".equals(Files.readString(fs.getPath("file.txt"), UTF_8))) {
>> 219: throw new RuntimeException("unexpected file content");
>
> could also consider
>
> fail("unexpected file content");
Oh thanks for spotting!
That's a hangover from when it was in the old-style test class (ZipFSTester or
whatever it's called).
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25178#discussion_r2091684373
