On Tue, 26 Aug 2025 14:19:06 GMT, Mikhail Yankelevich
<myankelev...@openjdk.org> wrote:
>> test/jdk/sun/security/pkcs11/KeyStore/CertChainRemoval.java line 130:
>>
>>> 128: printKeyStore("Initial PKCS11 KeyStore: ", p11ks);
>>> 129: } catch (Exception e) {
>>> 130: throw new SkippedException("Skip test, due to " + e);
>>
>> Maybe `e` can be specified as a real `cause` now?
>
> I might be misunderstanding your point, but I think in this case a full
> exception stack trace would be more helpful. It covers everything this way.
The usual way to print exceptions includes the cause too. So if you create
`SkippedException` as `new SkippedException("Skip test", e)` you should see `e`
too in the error.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26942#discussion_r2301251031
