RFR: JDK-8272853: improve `JavadocTester.runTests`
Hannes Wallnöfer
hannesw at openjdk.java.net
Wed Mar 2 11:44:59 UTC 2022
On Thu, 6 Jan 2022 20:31:42 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:
> Please review a medium simple test-only fix to improve the `JavadocTester` `runTests` methods.
>
> Currently, there are two overloads: one to invoke no-args test methods, and another to invoke test methods with args, such as a method-specific `Path`. This latter one is slightly inconvenient to use, since it requires the boilerplate use of a function to create the arguments.
>
> The first overload is updated in a backwards compatible way, to examine the parameters of methods annotated with `@Test`, and to recognize common patterns: namely no-args methods and `(Path)` methods, without the need for any additional function. The second overload is retained by backwards compatibility, until if and when we decide it is no longer required.
>
> In addition, two new overloads are added, similar to the first two, but with the addition of an extra parameter to decide which methods should be invoked. This is primarily a debugging aid. Using these overloads, it is easy to specify, on the command line used to run the test, the methods to be executed.
>
> A new test is added to test the new functionality.
Sorry for letting this linger for so long. I asked some questions in the comments, but that was not meant to block the PR. The changes look good to me.
-------------
Marked as reviewed by hannesw (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/6983
More information about the javadoc-dev
mailing list