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

Reply via email to