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