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

Mikhail Yankelevich myankelevich at openjdk.org
Mon Sep 8 10:18:48 UTC 2025


On Sat, 6 Sep 2025 15:46:54 GMT, Weijun Wang <weijun at openjdk.org> wrote:

>> 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?

Sure, done in the next commit

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

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


More information about the security-dev mailing list