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

[Jonathan Gibbons]

On Mon, 4 Oct 2021 17:26:27 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 feedbakc.

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

> 67:                                                          BasicFileAttributes attrs) {
> 68:                                    File file = path.toFile();
> 69:                                    if (file.getName().endsWith(".class")) {

You can avoid the `File` with:

    if (path.getFileName().toString().endsWith(".class"))

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

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