[jdk17] Integrated: JDK-8249899: jdk/javadoc/tool/InlineTagsWithBraces.java uses @ignore w/o bug-id

Jonathan Gibbons jjg at openjdk.java.net
Thu Jun 17 14:12:12 UTC 2021


On Thu, 17 Jun 2021 01:35:58 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:

> 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.

This pull request has now been integrated.

Changeset: 69d01b6b
Author:    Jonathan Gibbons <jjg at openjdk.org>
URL:       https://git.openjdk.java.net/jdk17/commit/69d01b6bcabda177f5e27f6c7b141be57cd00619
Stats:     333 lines in 4 files changed: 0 ins; 333 del; 0 mod

8249899: jdk/javadoc/tool/InlineTagsWithBraces.java uses @ignore w/o bug-id
8249897: jdk/javadoc/tool/LangVers.java uses @ignore w/o bug-id
8249898: jdk/javadoc/tool/6176978/T6176978.java uses @ignore w/o bug-id

Reviewed-by: hannesw

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

PR: https://git.openjdk.java.net/jdk17/pull/87


More information about the javadoc-dev mailing list