RFR: 8327096: (fc) java/nio/channels/FileChannel/Size.java fails on partition incapable of creating large files
Daniel Jeliński
djelinski at openjdk.org
Mon Mar 11 19:03:17 UTC 2024
On Mon, 11 Mar 2024 17:56:24 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:
>> test/jdk/java/nio/channels/FileChannel/Size.java line 87:
>>
>>> 85: FileStore store = p.getFileSystem().provider().getFileStore(p);
>>> 86: if ("msdos".equals(store.type())) // FAT32
>>> 87: return; // ignore IOE: file too big for 32 bits
>>
>> Throwing a SkippedException might be more appropriate here
>
> `SkippedException` is EE, no?
>
> I think the main question is whether to ignore or throw.
I was referring to `jtreg.SkippedException`. See: https://openjdk.org/jtreg/faq.html#what-if-a-test-does-not-apply-in-a-given-situation
If you throw that, the test will still pass.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18073#discussion_r1520284094
More information about the nio-dev
mailing list