[jdk17u-dev] RFR: 8336942: Improve test coverage for class loading elements with annotations of different retentions [v2]

[Thomas Fitzsimmons]

On Tue, 5 Nov 2024 18:49:17 GMT, Liam Miller-Cushon <cushon at openjdk.org> wrote:

>> This change improves annotation processing test coverage, see [JDK-8336942](https://bugs.openjdk.org/browse/JDK-8336942). I resolved a trivial merge conflict with JDK-8323684, which added the `nameToAnnotation` map. The modified test passes.
>
> Liam Miller-Cushon 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 'origin/master' into backport-JDK-8336942
>  - 8336942: Improve test coverage for class loading elements with annotations of different retentions
>    
>    Reviewed-by: vromero

Liam Miller-Cushon ***@***.***> writes:

>  I think this issue with the test should be fixed at head, I will
>  follow up on that.
>
> I filed https://bugs.openjdk.org/browse/JDK-8343882 for this

Looks good, thank you.  On Monday I will experiment on `jdk17u-dev` with your preliminary fix for the test harness.

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

PR Comment: https://git.openjdk.org/jdk17u-dev/pull/2955#issuecomment-2466468840