On Tue, 25 Mar 2025 16:28:35 GMT, Artur Barashev <[email protected]> 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
