On Wed, 1 Oct 2025 16:20:01 GMT, Mikhail Yankelevich <[email protected]>
wrote:
>> Hi all,
>>
>> Currently sun/security/provider/acvp/Launcher.java report fails
>> "java.io.IOException: Cannot find the artifact ACVP-Server". This is not jvm
>> bug cause this failure, so I think sun/security/provider/acvp/Launcher.java
>> test should throw SkippedException when no ACVP-Server available.
>
> test/jdk/sun/security/provider/acvp/Launcher.java line 124:
>
>> 122: } catch (IOException e) {
>> 123: if (e.getMessage().contains("Cannot find the artifact
>> ACVP-Server")) {
>> 124: throw new SkippedException("ACVP-Server not available,
>> skip test.");
>
> nit:
> Suggestion:
>
> throw new SkippedException("ACVP-Server not available.");
Hi @myankelev, can you just take a look at all our tests that call
`ArtifactResolver.fetchOne` to see if it can directly throw a
`SkippedException`? Thanks.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27491#discussion_r2402345258
