> 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.
Jonathan Gibbons has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision: - Merge remote-tracking branch 'upstream/master' into 8272853.JavadocTester-runTests - JDK-8272853: improve `JavadocTester.runTests` ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/6983/files - new: https://git.openjdk.java.net/jdk/pull/6983/files/43878135..ee672346 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=6983&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=6983&range=00-01 Stats: 75250 lines in 2147 files changed: 51328 ins; 13254 del; 10668 mod Patch: https://git.openjdk.java.net/jdk/pull/6983.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/6983/head:pull/6983 PR: https://git.openjdk.java.net/jdk/pull/6983
