On Wed, 20 Aug 2025 14:58:44 GMT, Mikhail Yankelevich 
<myankelev...@openjdk.org> wrote:

> * test/jdk/sun/security/pkcs11/Provider/ tests skipped without 
> SkippedException:
>   - test/jdk/sun/security/pkcs11/Provider/LoginISE.java 
>   - test/jdk/sun/security/pkcs11/Provider/ConfigShortPath.java
>   - test/jdk/sun/security/pkcs11/Provider/Absolute.java
> * cleanup

test/jdk/sun/security/pkcs11/Provider/LoginISE.java line 47:

> 45:         Provider p = Security.getProvider("SunPKCS11");
> 46:         if (p == null) {
> 47:             throw new SkippedException("No un-initialized PKCS11 provider 
> available; skip");

Maybe this is not needed anymore, since we are wrapping the message in a 
`SkippedException`
Suggestion:

            throw new SkippedException("No un-initialized PKCS11 provider 
available");

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26862#discussion_r2288521930

Reply via email to