On Tue, 29 Jul 2025 17:11:28 GMT, Alan Bateman <[email protected]> wrote:
>> David Beaumont has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Removing direct references to JrtFS.
>
> test/jdk/jdk/internal/jimage/JImageReadTest.java line 350:
>
>> 348: try (ImageReader badReader = ImageReader.open(imageFile,
>> otherOrder)) {
>> 349: Assert.fail("Reader should not be openable with the wrong
>> byte order.");
>> 350: } catch (IOException expected) {
>
> In passing, you can use `assertThrows(IOException.class(), () ->
> ImageReader.open(..))` here if you want.
I don't think I can, can I? That method takes a Runnable, not some sort of
Callable that throws E.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26054#discussion_r2240563330
