RFR: 8366159: SkippedException is treated as a pass for pkcs11/KeyStore, pkcs11/SecretKeyFactory and pkcs11/SecureRandom [v3]

[Weijun Wang]

On Thu, 4 Sep 2025 15:02:02 GMT, Mikhail Yankelevich <myankelevich at openjdk.org> wrote:

>> Skipped tests are treated as a pass:
>> 
>> * test/jdk/sun/security/pkcs11/KeyStore/ClientAuth.java
>> * test/jdk/sun/security/pkcs11/KeyStore/CertChainRemoval.java
>> * test/jdk/sun/security/pkcs11/SecretKeyFactory/TestGeneral.java
>> * test/jdk/sun/security/pkcs11/SecureRandom/Basic.java
>> * test/jdk/sun/security/pkcs11/SecureRandom/TestDeserialization.java
>
> Mikhail Yankelevich has updated the pull request incrementally with one additional commit since the last revision:
> 
>   waiting for all tests to run before skipping

test/jdk/sun/security/pkcs11/SecretKeyFactory/TestGeneral.java line 128:

> 126:             test("Blowfish", bf_128Key, p, TestResult.PASS);
> 127:         } catch (SkippedException skippedException){
> 128:             skippedList.add(skippedException.getMessage());

Can we add a `skippedException.printStackTrace()` here?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26942#discussion_r2327753529