From 8217d81d0222d175bb35e743ca180a1f86ad7694 Mon Sep 17 00:00:00 2001 From: Chris Beams Date: Tue, 21 Apr 2009 17:20:39 +0000 Subject: [PATCH] Fixing build breakage with GroovyScriptFactoryTests. The problem was that the Messenger object was not being proxed (assertTrue(AopUtils.isAopProxy(messenger)) was returning false). The cause for this seemed to be that the pointcut was malformed / out of date, reading execution(* org.springframework.scripting.Messenger.*(..)), when the groovy Messenger class is actually declared in the org.springframework.scripting.groovy package. I tried updating the fully-qualified package name in the pointcut expression, and this caused AspectJ matching errors saying that there was not such type that matches that FQ name. So as a final resort, I removed the full-qualification entirely and went with execution(* *..Messenger.*(..)). All tests pass now, but it raises the question, why was AJ having matching errors? Is it because the pointcut matching is being done before the groovy class is loaded? There could be a potential bug here. git-svn-id: https://src.springframework.org/svn/spring-framework/trunk@1040 50f2f4bb-b051-0410-bef5-90022cba6387 --- .../org/springframework/scripting/groovy/groovy-with-xsd.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/org.springframework.context/src/test/java/org/springframework/scripting/groovy/groovy-with-xsd.xml b/org.springframework.context/src/test/java/org/springframework/scripting/groovy/groovy-with-xsd.xml index 1887788bd97..8de3522ab66 100644 --- a/org.springframework.context/src/test/java/org/springframework/scripting/groovy/groovy-with-xsd.xml +++ b/org.springframework.context/src/test/java/org/springframework/scripting/groovy/groovy-with-xsd.xml @@ -8,7 +8,7 @@ - +