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