[GROOVY-10570] AnnotationCollector produces unspecified errors when used in Java

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 4.0.1
• Fix Version/s: 2.5.17, 3.0.11, 4.0.2
• Component/s: Compiler
• Assignee: Eric Milles
• Reporter: Christopher Smith
• Created: 05/Apr/22 22:02
• Updated: 23/Jul/22 16:27
• Resolved: 18/Apr/22 12:23

Description

I have an AnnotationCollector that refers to some Java classes (type converters). The stub generator has an error generating the stub, so I tried writing the annotation itself in Java. While this compiled correctly, when compiling tests using that annotation I get a BUG! in the Groovy compiler:

BUG! exception in phase 'semantic analysis' in source unit '/home/christopher/git/.../IgnoreMetaClassAttributeConverterProviderTest.groovy' com.example.DynamoDbRecord.value()
...
Caused by: java.lang.NoSuchMethodException: com.example.DynamoDbRecord.value()
    at java.lang.Class.getMethod (Class.java:2108)
    at org.codehaus.groovy.transform.AnnotationCollectorTransform.getTargetListFromClass (AnnotationCollectorTransform.java:276)
    at org.codehaus.groovy.transform.AnnotationCollectorTransform.getMeta (AnnotationCollectorTransform.java:76)
    at org.codehaus.groovy.transform.AnnotationCollectorTransform.getStoredTargetList (AnnotationCollectorTransform.java:226)
    at org.codehaus.groovy.transform.AnnotationCollectorTransform.getTargetAnnotationList (AnnotationCollectorTransform.java:356)
    at org.codehaus.groovy.transform.AnnotationCollectorTransform.visit (AnnotationCollectorTransform.java:383)
    at org.codehaus.groovy.transform.ASTTransformationCollectorCodeVisitor.findCollectedAnnotations (ASTTransformationCollectorCodeVisitor.java:207)
    at org.codehaus.groovy.transform.ASTTransformationCollectorCodeVisitor.visitAnnotations (ASTTransformationCollectorCodeVisitor.java:93)

Adding a String value default "" to the annotation produces an NPE.

It would make sense to me to learn that using AnnotationCollector in Java is unsupported, but the Groovy compiler should probably have a better error report.