RFR: 8350964: Add an ArtifactResolver.fetch(clazz) method [v4]

Weijun Wang weijun at openjdk.org
Wed Mar 12 20:37:54 UTC 2025


On Wed, 12 Mar 2025 20:28:31 GMT, Matthew Donovan <mdonovan at openjdk.org> wrote:

>> In this PR, I created a new method, `ArtifactResolver.fetchOne()`, to consolidate duplicate code across tests.
>
> Matthew Donovan has updated the pull request incrementally with one additional commit since the last revision:
> 
>   updated code to remove extraneous null checks

test/jdk/sun/security/pkcs11/PKCS11Test.java line 1:

> 1: /*

There are 2 useless imports now:

import java.nio.file.Paths;
import jdk.test.lib.artifacts.ArtifactResolverException;

test/jdk/sun/security/pkcs11/SecmodTest.java line 1:

> 1: /*

Copyright year.

test/jdk/sun/security/pkcs12/KeytoolOpensslInteropTest.java line 1:

> 1: /*

`import jtreg.SkippedException;` is useless.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23989#discussion_r1992250415
PR Review Comment: https://git.openjdk.org/jdk/pull/23989#discussion_r1992251730
PR Review Comment: https://git.openjdk.org/jdk/pull/23989#discussion_r1992248430


More information about the security-dev mailing list