RFR: 8274244: ReportOnImportedModuleAnnotation.java fails on rerun [v3]

[Jonathan Gibbons]

On Tue, 5 Oct 2021 17:22:29 GMT, Joe Darcy <darcy at openjdk.org> wrote:

>> In terms of jtreg build tags, semantically this test wants to:
>> 
>> * build annotation processing sources
>> * compile module sources, running build annotation processor
>> 
>> In particular, the test wants to always compile the module sources, even if class files are newer than the module-info.java files. Since the test is not written in terms of those tags, I've added a pre-step which delete any existing class files from the output directory. If needed, some extra care could be taken to only delete files from the one module.
>> 
>> With this addition, the test will pass even when run with an already populated JTwork directory. At present, the test fails since the implicit compilation of module-info.java is *not* done when the class files in the output directory are newer than the sources.
>
> Joe Darcy has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Respond to review feedback.

test/langtools/tools/javac/processing/ReportOnImportedModuleAnnotation/ReportOnImportedModuleAnnotation.java line 43:

> 41: import java.nio.file.Files;
> 42: import java.nio.file.Path;
> 43: import java.nio.file.SimpleFileVisitor;

Consider removing redundant imports.

test/langtools/tools/javac/processing/ReportOnImportedModuleAnnotation/ReportOnImportedModuleAnnotation.java line 63:

> 61:         // Clean any existing files in output directory
> 62:         (new ToolBox()).cleanDirectory(testOutputPath);
> 63: 

Nice.

-------------

PR: https://git.openjdk.java.net/jdk/pull/5802