RFR: 8350964: Add an ArtifactResolver.fetch(clazz) method [v3]
Matthew Donovan
mdonovan at openjdk.org
Wed Mar 12 15:29:36 UTC 2025
On Wed, 12 Mar 2025 13:29:08 GMT, Weijun Wang <weijun at openjdk.org> wrote:
>> It encapsulates all of the logic involved in getting the tests. Specifically, what to do if the tests can't be fetched. It could be done in `main()` but this is a little cleaner.
>
> I'd rather just let `ArtifactResolver.fetchOne` throw the `SkippedException`. Is that always what we need?
I can't think of a reason why a test would want to continue if the file wasn't accessible. I moved the skippedexception into the`fetchOne` method.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23989#discussion_r1991752961
More information about the security-dev
mailing list