On Wed, 20 Aug 2025 19:57:59 GMT, Matthew Donovan <mdono...@openjdk.org> wrote:
> This PR updates `ArtifactResolver.fetchOne()` to throw an IOException instead > of SkippedException. This allows tests to determine of a missing "artifact" > should be treated as a failed or skipped test.`OpensslArtifactFetcher` is > also updated to only throw SkippedException if OpenSSL is not available on > the test platform. test/lib/jdk/test/lib/security/OpensslArtifactFetcher.java line 52: > 50: * > 51: * @return openssl binary path of the current version > 52: * @throws IOException if a valid version of OpenSSL cannot be found This still also throws `SkippedException`: https://github.com/openjdk/jdk/blob/ae26c5a24e50584cfa3784c4b7e4cc84a738a530/test/lib/jdk/test/lib/security/OpensslArtifactFetcher.java#L83-L84 Suggestion: * @throws IOException if a valid version of OpenSSL cannot be found * @throws SkippedException if OpenSSL is not available on the target platform ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/26868#discussion_r2289370584
