RFR: JDK-8249903: jdk/javadoc/doclet/testSerializedForm/TestSerializedForm.java needs to be updated after 8146022 got closed

[Jonathan Gibbons]

On Thu, 29 Apr 2021 08:00:18 GMT, Hannes Wallnöfer <hannesw at openjdk.org> wrote:

> I don't know why this test was disabled because of JDK-8146022, in fact I wasn't able to understand what JDK-8146022 was all about as I didn't find a commit for it. However, the test runs fine when updated to the current output format, so I guess whatever bug caused it to fail has been fixed in the meantime. I also removed some vestiges of an old test framework nearby that has been commented out from time immemorial.

FWIW, the way I see the history here ... a bug was filed because of a resource issue, and this test was disabled because of the bug. Using `@ignore` was a precursor/alternate to using the problem list file.
Separately, refactoring and cleanup fixed the resource issue, using separate independent cleanup RFEs.  So, JDK-8146022 was closed (without needing a changeset), and without removing the `@ignore` here.

Lines 49-51 are a reminder of life before `JavadocTester`.

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

Marked as reviewed by jjg (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/3791