On Tue, 25 Mar 2025 16:28:35 GMT, Artur Barashev <abaras...@openjdk.org> wrote:

>> Anthony Scarpino has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   comments on test
>
> test/jdk/sun/security/util/AlgorithmConstraints/DisabledAlgorithmPermits.java 
> line 143:
> 
>> 141:                 throw new AssertionError("failed.  Expected " +
>> 142:                     tc.expected);
>> 143:             }
> 
> I suggest replacing this check with `assertEquals`.

If I did that, then I couldn't println to stdout.  I'd rather keep it as is.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/23647#discussion_r2012498792

Reply via email to