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

SendaoYan syan at openjdk.org
Tue Nov 4 02:10:03 UTC 2025


On Mon, 3 Nov 2025 16:49:44 GMT, Neha Joshi <duke at openjdk.org> wrote:

>> This PR contain changes to handle skipped exception for below test cases.
>> 
>> -> test/jdk/java/security/cert/CertStore/NoLDAP.java
>> -> test/jdk/java/security/Provider/NewInstance.java
>> 
>> 
>> https://bugs.openjdk.org/browse/JDK-8370942
>
> Neha Joshi has updated the pull request incrementally with one additional commit since the last revision:
> 
>   JDK-8370942 : Reveiw comment fixes.

test/jdk/java/security/cert/CertStore/NoLDAP.java line 39:

> 37:         try {
> 38:             Class.forName("javax.naming.ldap.LdapName");
> 39:             System.out.println("LDAP is present, test skipped");

Maybe we can add a new line to throw a Exception when the expected ClassNotFoundException do not seen.


throw new RuntimeException("The expexted ClassNotFoundException was not observed.");

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

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


More information about the security-dev mailing list