On Thu, 2 Sep 2021 13:33:30 GMT, Fernando Guallini <fguall...@openjdk.org> 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 <fguall...@openjdk.org> Committer: Rajan Halade <rhal...@openjdk.org> 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