On Wed, 12 Mar 2025 20:28:31 GMT, Matthew Donovan <mdono...@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
