On Tue, 10 Jun 2025 07:21:34 GMT, SendaoYan <s...@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 <s...@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