[jdk17] RFR: JDK-8249899: jdk/javadoc/tool/InlineTagsWithBraces.java uses @ignore w/o bug-id
Jonathan Gibbons
jjg at openjdk.java.net
Thu Jun 17 01:44:34 UTC 2021
Also:
JDK-8249897: jdk/javadoc/tool/LangVers.java uses @ignore w/o bug-id
JDK-8249898: jdk/javadoc/tool/6176978/T6176978.java uses @ignore w/o bug-id
Please review a trivial PR for 17 to remove some tests which have been `@ignore`d since at least JDK 9.
The tests were previously written for the old API and doclet, and were at best incompletely updated for the new world.
The functionality covered by these tests is now better covered in other non-ignored tests, and there seems little benefit to trying to update them for the new world.
* InlineTagsWithBraces.java was about checking nested `{ }` inside inline tags, which is now covered by `doctree` tests for the `DocCommentParser`.
* LangVers.java was a JDK5-era test, ensuring that the right language level was assumed. This is now covered by delegating the relevant options (`-source`, `--release`, etc) to javac
* T6176978.java was about testing classloading of doclets. While we do not have any direct replacement tests, we have a variety of tests that load custom doclets, and javadoc uses the JavaFileManager API to get a class loader for the appropriate path.
-------------
Commit messages:
- JDK-8249899: jdk/javadoc/tool/InlineTagsWithBraces.java uses @ignore w/o bug-id
Changes: https://git.openjdk.java.net/jdk17/pull/87/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk17&pr=87&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8249899
Stats: 333 lines in 4 files changed: 0 ins; 333 del; 0 mod
Patch: https://git.openjdk.java.net/jdk17/pull/87.diff
Fetch: git fetch https://git.openjdk.java.net/jdk17 pull/87/head:pull/87
PR: https://git.openjdk.java.net/jdk17/pull/87
More information about the javadoc-dev
mailing list