Integrated: 8359083: Test jdkCheckHtml.java should report SkippedException rather than report fails when miss tidy
SendaoYan
syan at openjdk.org
Thu Jun 12 08:20:33 UTC 2025
On Tue, 10 Jun 2025 07:21:34 GMT, SendaoYan <syan at openjdk.org> wrote:
> Hi all,
>
> When there is no `tidy` command in PATH, the test test/docs/jdk/javadoc/doccheck/checks/jdkCheckHtml.java report failure, I think it shoule be report jtreg.SkippedException rather than report test failure.
> In file test/docs/jdk/javadoc/doccheck/DocCheck.java, we need to call the `new TidyChecker()` before executing the test so that the `jtreg.SkippedException` exception can be thrown normally, that's why I add `new TidyChecker()` in `init()` function.
>
> Change has been verified locally, only the test test/docs/jdk/javadoc/doccheck/checks/jdkCheckHtml.java observed this failure, test-fix only, no risk.
This pull request has now been integrated.
Changeset: 3e0ef832
Author: SendaoYan <syan at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/3e0ef832cc553c70eb7e8ee713c43aac67994b91
Stats: 10 lines in 3 files changed: 4 ins; 1 del; 5 mod
8359083: Test jdkCheckHtml.java should report SkippedException rather than report fails when miss tidy
Reviewed-by: hannesw
-------------
PR: https://git.openjdk.org/jdk/pull/25711
More information about the javadoc-dev
mailing list