Integrated: JDK-8273154: Provide a JavadocTester method for non-overlapping, unordered output matching
Jonathan Gibbons
jjg at openjdk.java.net
Wed Nov 10 15:27:47 UTC 2021
On Tue, 28 Sep 2021 23:34:21 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:
> Please review a moderately simple improvement for `JavadocTester` and a related new test.
>
> A new `OutputChecker` class is introduced that mostly supersedes the existing methods to check the output generated by javadoc and the standard doclet. A self-imposed restriction is that no existing tests are modified.
>
> The new class can be used to check files generated by the doclet and the streams written by the tool. It can be configured to check for ordered output or not, overlapping output, and complete coverage, and can search for literal strings and regular expressions.
>
> There is a corresponding new test which is a non-standard use of `JavadocTester`, since it is designed to test `JavadocTester` itself, and not javadoc or the doclet. (Quis custodiet ipsos custodes?) Various methods are overridden so that the operation of the underlying methods can be checked.
>
> Although it is a goal to NOT modify the code of any existing tests, it turns out to be reasonable to adapt some of the existing `check...` methods to use the new `OutputChecker`. All javadoc tests pass, both locally and on all standard platforms. Many/most uses of the existing `checkOutput` method provide "ordered" strings, and are candidates to use the new ordered check. But enough uses are _not_ ordered, so it is not reasonable to change the default at this time. It is noted as a TODO to examine the appropriate test cases, so that we can decide whether to fix those tests and change the default.
This pull request has now been integrated.
Changeset: ce3ed65a
Author: Jonathan Gibbons <jjg at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/ce3ed65ac3411a533052a8c01231f7e540803afb
Stats: 1072 lines in 2 files changed: 918 ins; 90 del; 64 mod
8273154: Provide a JavadocTester method for non-overlapping, unordered output matching
Reviewed-by: prappo
-------------
PR: https://git.openjdk.java.net/jdk/pull/5743
More information about the javadoc-dev
mailing list