On Wed, 7 Jun 2023 19:03:57 GMT, Hannes Wallnöfer <hann...@openjdk.org> wrote:
>> Jim Laskey has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Use checkOutput > > test/langtools/jdk/javadoc/doclet/unnamed/Unnamed.java line 80: > >> 78: ); >> 79: >> 80: if (!result.toFile().exists()) { > > `JavadocTester` has a `checkOutput(String, boolean, String...)` method that > allows to check if a file exists and if it does or does not contain any > number of strings. I think this should be used to make sure the file has the > expected content, such as the doc comment for the main method above. > Otherwise looks good. Updated to use `checkOutput`. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/14356#discussion_r1222067553