On Thu, 2 Sep 2021 13:33:30 GMT, Fernando Guallini <[email protected]>
wrote:
> This trivial change fixes the case 2 in test
> sun/security/pkcs11/rsa/TestP11KeyFactoryGetRSAKeySpec, the method
> testKeySpec is expecting a class object of type KeySpec as second argument in
> order to be reusable for multiple test scenarios, but then instead of using
> that argument the RSAPrivateKeySpec.class is hardcoded:
>
>
> private static void testKeySpec(KeyFactory factory, PrivateKey key, Class<?
> extends KeySpec> specClass) throws Exception {
> try {
> KeySpec spec = factory.getKeySpec(key, RSAPrivateKeySpec.class);
>
>
> it should be:
>
> `KeySpec spec = factory.getKeySpec(key, specClass);
> `
This pull request has now been integrated.
Changeset: 041ae20b
Author: Fernando Guallini <[email protected]>
Committer: Rajan Halade <[email protected]>
URL:
https://git.openjdk.java.net/jdk/commit/041ae20b10e11381415e8f61fd39e9e19aa8d4f2
Stats: 3 lines in 1 file changed: 1 ins; 1 del; 1 mod
8268558: [TESTBUG] Case 2 in TestP11KeyFactoryGetRSAKeySpec is skipped
Reviewed-by: mullan, rhalade
-------------
PR: https://git.openjdk.java.net/jdk/pull/5351
