On Wed, 1 May 2024 20:12:33 GMT, Naoto Sato <[email protected]> wrote:
>> Justin Lu has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Reflect review comments
>
> test/jdk/java/util/Base64/TestEncodingDecodingLength.java line 75:
>
>> 73: } catch (InvocationTargetException ex) {
>> 74: Throwable rootEx = ex.getCause();
>> 75: assertEquals(OutOfMemoryError.class, rootEx.getClass(),
>> "00ME should be thrown");
>
> Sorry if it is intentional, but I wonder if you meant "OOME" instead of
> "00ME" here?
Good catch, you sometimes see what you want to see
> test/jdk/java/util/Base64/TestEncodingDecodingLength.java line 100:
>
>> 98: m.invoke(DECODER, src, -LARGE_MEM_SIZE + 1, 1);
>> 99: } catch (InvocationTargetException ex) {
>> 100: fail("Decode should neither throw NASE or OOME: " +
>> ex.getCause());
>
> Should we check the cause is either NASE or OOME here?
The original test just validated there was no exception thrown, so I think we
can just update the message to indicate an unexpected Exception.
I had added this as a comment but must have missed hitting the comment button
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19036#discussion_r1586789013
PR Review Comment: https://git.openjdk.org/jdk/pull/19036#discussion_r1586788635
