On Wed, 20 Aug 2025 14:58:44 GMT, Mikhail Yankelevich
<[email protected]> 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