> Doccheck's human-generated reports are great at previewing a "chessboard" of > results. Giving reader a quick glimpse at the quality/health of the > documentation. But these tests needed to be automated and they didn't easily > translate to something that can be integrated into a CI. > > This PR includes an HTML and internal link test on `api/java.base` and a > BadChars and Doctype test on the entire generated documentation bundle. > > Here is an example of the output after running all tests on `api/java.base` > > Note: There is an active PR to fix the broken anchors left in `java.base` so > this is not a blocker. > > > > STDOUT: > STDERR: > test: test > Tidy found errors in the generated HTML > /Users/nizarbenalla/Work/jdk-repos/jdk1/build/macosx-aarch64/images/docs/api/java.base/java/lang/Class.html:323:87: > Warning: <a> anchor "nest" already defined > Tidy output end. > > > api/java.base/java/util/concurrent/StructuredTaskScope.ShutdownOnFailure.html:245: > id not found: > api/java.base/java/util/concurrent/StructuredTaskScope.ShutdownOnFailure.html#TreeStructure > api/java.base/java/util/concurrent/StructuredTaskScope.ShutdownOnSuccess.html:242: > id not found: > api/java.base/java/util/concurrent/StructuredTaskScope.ShutdownOnSuccess.html#TreeStructure > api/java.base/java/lang/Class.html:323: name already declared: nest > api/java.base/java/lang/Module.html:291: id not found: > api/java.base/java/lang/foreign/package-summary.html#restricted > api/java.base/java/lang/Module.html:434: id not found: > api/java.base/java/lang/foreign/package-summary.html#restricted > api/java.base/java/lang/foreign/MemorySegment.html:725: id not found: > api/java.base/java/lang/foreign/package-summary.html#restricted > > Link Checker Report > Checked 3446 files. > Found 445059 references to 48205 anchors in 5770 files and 64 other URIs. > 1 duplicate ids > 3 missing ids > > Hosts > 20 docs.oracle.com > 1 tools.ietf.org > 1 www.ietf.org > 1 jcp.org > 4 www.rfc-editor.org > 7 unicode.org > 10 www.unicode.org > 20 www.w3.org > Exception running test test: java.lang.Exception: One or more HTML checkers > failed: [java.lang.RuntimeException: Tidy found errors in the generated HTML, > java.lang.RuntimeException: LinkChecker encountered errors. Duplicate IDs: 1, > Missing IDs: 3, Missing Files: 0, Bad Schemes: 0] > java.lang.Exception: One or more HTML checkers failed: > [java.lang.RuntimeException: Tidy found errors in the generated HTML, > java.lang.RuntimeException: LinkChecker encountered errors. Duplicate IDs: 1, > Missing IDs: 3, Mi...
Nizar Benalla has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains eight additional commits since the last revision: - Add a test for external links separate checks into different jtreg tests fix typos - Merge remote-tracking branch 'upstream/master' into new-docs-tests-suit - Merge remote-tracking branch 'upstream/master' into new-docs-tests-suit - add file with all vetted links - improve some parts based on review comments - Merge remote-tracking branch 'upstream/master' into new-docs-tests-suit - Merge remote-tracking branch 'upstream/master' into new-docs-tests-suit - Convert parts of doccheck into tests ------------- Changes: - all: https://git.openjdk.org/jdk/pull/21879/files - new: https://git.openjdk.org/jdk/pull/21879/files/8174c67f..a5efb313 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=21879&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=21879&range=01-02 Stats: 68435 lines in 3533 files changed: 51634 ins; 9139 del; 7662 mod Patch: https://git.openjdk.org/jdk/pull/21879.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/21879/head:pull/21879 PR: https://git.openjdk.org/jdk/pull/21879
