On Mon, 29 Mar 2021 16:06:31 GMT, Jonathan Gibbons <j...@openjdk.org> wrote:

>> test/langtools/jdk/javadoc/doclet/testLinkOption/TestRedirectLinks.java line 
>> 110:
>> 
>>> 108:         } catch (UnknownHostException e) {
>>> 109:             out.println("Setup failed (" + testURLHost + " not found); 
>>> this test skipped");
>>> 110:             return;
>> 
>> L98 and L110: can we use `jtreg.SkippedException` instead? That way the 
>> results of the test test run might look more clearly.
>
> No, because that means the entire text was skipped; here we just want to skip 
> one `@Test` method.

It's rather unfortunate that a setup that uses 
`@javadoc.tester.JavadocTester.Test` works differently than that of 
`@org.testng.annotations.Test`.

-------------

PR: https://git.openjdk.java.net/jdk/pull/3137

Reply via email to