RFR: 8370942: test/jdk/java/security/Provider/NewInstance.java and /test/jdk/java/security/cert/CertStore/NoLDAP.java may skip without notifying [v2]

Neha Joshi duke at openjdk.org
Mon Nov 3 16:49:46 UTC 2025


On Mon, 3 Nov 2025 16:22:19 GMT, Mikhail Yankelevich <myankelevich at openjdk.org> wrote:

>> We should never silently ignore exceptions so , it better to log it for making it easier to detect and fix problems.
>
> This is expected behaviour, the test shouldn't run without it, so if you add exception logging for expected step it may be confusing. All I propose is to change the log to something like this
> Suggestion:
> 
>             System.err.println("Expected: class not found exception " + ignore.getMessage());

Sounds good , will update the log.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28112#discussion_r2487163904


More information about the security-dev mailing list