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