[GROOVY-5740] Cannot use ASTTransformationCustomizer with TimedInterrupt

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.8.6, 2.0.1
• Fix Version/s: 2.2.1
• Assignee: Cédric Champeau
• Reporter: Jim Driscoll
• Created: 02/Oct/12 18:33
• Updated: 15/Oct/15 18:19
• Resolved: 22/Nov/13 08:37

Description

The following code will fail:

def program = "println 'hello world'; while(true) {;}"
def config = new CompilerConfiguration()
config.addCompilationCustomizers(new ASTTransformationCustomizer([value: 1L, unit: TimeUnit.MILLISECONDS],TimedInterrupt))
GroovyShell sh = new GroovyShell(config)
Script script = sh.parse(program)
System.out.println(script.run()) // fail here

With the exception:

org.codehaus.groovy.classgen.ClassGeneratorException: Cannot generate bytecode for constant: MILLISECONDS of type: java.util.concurrent.TimeUnit$3

Per blackdrag, this is apparently a bug in line 197 of TimedInterruptableASTTransformation, which uses ConstantExpression when it should have used PropertyExpression.

As a result of this bug, the only TimeUnit available when using TimedInterrupt is TimeUnit.SECONDS, which is the default. This is probably not too big of an issue for most uses, which is why I've set the priority to Minor.