From 1c5d4bc331f4273083b547923b122571b8387bfe Mon Sep 17 00:00:00 2001
From: aclement <aclement>
Date: Wed, 10 Dec 2008 20:01:33 +0000
Subject: [PATCH] some annotation matching tests

---
 org.aspectj.matcher/.classpath                |   1 +
 .../weaver/reflect/StandardShadow.java        |   1 -
 .../weaver/tools/StandardPointcutParser.java  |   1 +
 .../testdata/AnnotatedClass.java              |  11 ++++
 .../testdata/MethodLevelAnnotation.java       |   5 ++
 .../testdata/SomeAnnotation.java              |   6 ++
 org.aspectj.matcher/testdata/testdata.jar     | Bin 0 -> 1241 bytes
 ...ommonAdvancedPointcutExpressionTests.java} |  55 +++++++++++++++---
 8 files changed, 70 insertions(+), 10 deletions(-)
 create mode 100644 org.aspectj.matcher/testdata/AnnotatedClass.java
 create mode 100644 org.aspectj.matcher/testdata/MethodLevelAnnotation.java
 create mode 100644 org.aspectj.matcher/testdata/SomeAnnotation.java
 create mode 100644 org.aspectj.matcher/testdata/testdata.jar
 rename org.aspectj.matcher/testsrc/org/aspectj/matcher/tools/{FurtherCommonPointcutExpressionTests.java => CommonAdvancedPointcutExpressionTests.java} (92%)

diff --git a/org.aspectj.matcher/.classpath b/org.aspectj.matcher/.classpath
index 856780f91..544d974e4 100644
--- a/org.aspectj.matcher/.classpath
+++ b/org.aspectj.matcher/.classpath
@@ -8,5 +8,6 @@
 	<classpathentry kind="src" path="/bridge"/>
 	<classpathentry kind="lib" path="/lib/commons/commons.jar" sourcepath="/lib/commons/commons-src.zip"/>
 	<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER"/>
+	<classpathentry kind="lib" path="testdata/testdata.jar"/>
 	<classpathentry kind="output" path="bin"/>
 </classpath>
diff --git a/org.aspectj.matcher/src/org/aspectj/weaver/reflect/StandardShadow.java b/org.aspectj.matcher/src/org/aspectj/weaver/reflect/StandardShadow.java
index 411ccdd91..9a750481a 100644
--- a/org.aspectj.matcher/src/org/aspectj/weaver/reflect/StandardShadow.java
+++ b/org.aspectj.matcher/src/org/aspectj/weaver/reflect/StandardShadow.java
@@ -103,7 +103,6 @@ public class StandardShadow extends Shadow {
 		ResolvedMember[] members = forType.getDeclaredMethods();
 		int clinit = -1;
 		for (int i = 0; i < members.length && clinit == -1; i++) {
-			System.out.println(members[i]);
 			if (members[i].getName().equals("<clinit>")) {
 				clinit = i;
 			}
diff --git a/org.aspectj.matcher/src/org/aspectj/weaver/tools/StandardPointcutParser.java b/org.aspectj.matcher/src/org/aspectj/weaver/tools/StandardPointcutParser.java
index eee063f0f..226ae8b78 100644
--- a/org.aspectj.matcher/src/org/aspectj/weaver/tools/StandardPointcutParser.java
+++ b/org.aspectj.matcher/src/org/aspectj/weaver/tools/StandardPointcutParser.java
@@ -237,6 +237,7 @@ public class StandardPointcutParser {
 		} catch (ParserException pEx) {
 			throw new IllegalArgumentException(buildUserMessageFromParserException(expression, pEx));
 		} catch (ReflectionWorld.ReflectionWorldException rwEx) {
+			rwEx.printStackTrace();
 			throw new IllegalArgumentException(rwEx.getMessage());
 		}
 		return pcExpr;
diff --git a/org.aspectj.matcher/testdata/AnnotatedClass.java b/org.aspectj.matcher/testdata/AnnotatedClass.java
new file mode 100644
index 000000000..80dbda413
--- /dev/null
+++ b/org.aspectj.matcher/testdata/AnnotatedClass.java
@@ -0,0 +1,11 @@
+package testdata;
+
+@SomeAnnotation
+public class AnnotatedClass {
+
+  @MethodLevelAnnotation
+  public void annotatedMethod() { }
+
+  public void nonAnnotatedMethod() {
+  }
+}
diff --git a/org.aspectj.matcher/testdata/MethodLevelAnnotation.java b/org.aspectj.matcher/testdata/MethodLevelAnnotation.java
new file mode 100644
index 000000000..2f46dd8ab
--- /dev/null
+++ b/org.aspectj.matcher/testdata/MethodLevelAnnotation.java
@@ -0,0 +1,5 @@
+package testdata;
+import java.lang.annotation.*;
+
+@Retention(RetentionPolicy.RUNTIME)
+@interface MethodLevelAnnotation {}
diff --git a/org.aspectj.matcher/testdata/SomeAnnotation.java b/org.aspectj.matcher/testdata/SomeAnnotation.java
new file mode 100644
index 000000000..25a4991d6
--- /dev/null
+++ b/org.aspectj.matcher/testdata/SomeAnnotation.java
@@ -0,0 +1,6 @@
+package testdata;
+
+import java.lang.annotation.*;
+
+@Retention(RetentionPolicy.RUNTIME)
+@interface SomeAnnotation {}
diff --git a/org.aspectj.matcher/testdata/testdata.jar b/org.aspectj.matcher/testdata/testdata.jar
new file mode 100644
index 0000000000000000000000000000000000000000..7bbb39068f512b5429dace0324523a439ff8b239
GIT binary patch
literal 1241
zcmWIWW@h1H00H6HE=w>2N^ml;FqEVgm!u??B<lY=#lR2%R4T&20fgvEWr14YN*(j^
z@=Fp+Qd69B5{rxVl0jr?z-h0;4kEVZp2EFXukD(k$8vWiH*=o0lje>tuCQJ6O`82Y
z(-zG9vVWIQ%n!!DD<1i3t>m9`bZ7tP9R{nv{$_8;{^GRb1<MOI3zZ+{UfR{(1?TnE
zSn^YXJYO8~I^AyM|Kv}kO5|3-C~56AKJS;-nrbfmACk4Y$TF}s>14rS?Zw=OGILTB
zgA-l1OgvtsxOr`g!;8$t?fZ=nZ`6DqT)BMJWOrWMQ!arqXOB32xS9Lq#*_Q!_-=i;
zpd@4d$KQTow5rR@3;Gw6jV1?o8(aL5yjtuyb?(LIFI%(=b7S{4PP-$(<hDPqs>fQV
zG%Is|+Q)FQgzu~Qv(wgl{=Y6%ls0>H^WS4N3^hyv-s~K!6gP{rGcqvj24dVHs)8J%
zzNsY{`6)iBWvMyPz|G9hg9L1=-|jpnM;;mH1ZEZ&=`IyfRrQO@yc|_j<StIUpx@5g
zZMHk;T)EJGw}b2-0z3AK#5A`o-KcxM=k4X61uO<G?hBWGSaLzlMBMJT_m9gFYG0P=
zMDL$haPHe-OZ}*x^5(hcOdndFn=>OmZq}7GI@4>sDrSq!I+3-#^UKtZB(XBr?|v(u
z-b#Du_P_7T>FCLCnjMeYZ+`!=#HB*eFLkY}yZL|RlG9T^U+oiC+;eYF(5zn!3xxt)
z1x0QCJePX4aNFS+RvRO!j5uI8uycq^yQX~%=xYnS;ewLVg7b4z(F4T#YyNdRHa0e8
zHoG#myfQYqGPXE3wsjBwTw3%bWc!&pA#Q9BAN;u!a(vI8Ka-A3I`hY7@0uf<T-?}B
zF1WO3(wRF~Hk^5MW{Hf-&m=cCR{051f9D+8Q}WkkM@wBmmS>h+N9x%V2co9U2<9@0
z>20{w@Mq4MCteHKzAXsYFW?^^?#A|U(v?S(rW~2%QXA54XvCp#=9B*^FJHaHlnKYP
zY@8-bF*7SO3kw(<H!uh@bI(XgO<=yX$Dy!H;h9J3gOmqM;Gp2Tb8^A~U{Gj*gMyKX
zivf2I1u8=V@JtBFuZ&D042T?yobo{7hydC^CR{7F{2SnnY7}y0fs!f$)B@cMHwsU}
iMK=-I&!~=@%7kPhwkQwqW@Q68fCUJZfz~fz2Jrws*RtCH

literal 0
HcmV?d00001

diff --git a/org.aspectj.matcher/testsrc/org/aspectj/matcher/tools/FurtherCommonPointcutExpressionTests.java b/org.aspectj.matcher/testsrc/org/aspectj/matcher/tools/CommonAdvancedPointcutExpressionTests.java
similarity index 92%
rename from org.aspectj.matcher/testsrc/org/aspectj/matcher/tools/FurtherCommonPointcutExpressionTests.java
rename to org.aspectj.matcher/testsrc/org/aspectj/matcher/tools/CommonAdvancedPointcutExpressionTests.java
index 03860c008..e5fa8b0c0 100644
--- a/org.aspectj.matcher/testsrc/org/aspectj/matcher/tools/FurtherCommonPointcutExpressionTests.java
+++ b/org.aspectj.matcher/testsrc/org/aspectj/matcher/tools/CommonAdvancedPointcutExpressionTests.java
@@ -25,9 +25,17 @@ import org.aspectj.weaver.tools.StandardPointcutParser;
  * 
  * This is based on the Reflection oriented PointcutExpressionTest in the weaver project.
  * 
+ * TESTDATA. The testdata for these tests is kept in org.aspectj.matcher/testdata. It is a series of .java files that need to be
+ * compiled and available at runtime. Since they are java5 (they include annotations) they cannot be in a source folder for the
+ * project, so they are compiled separately and then jar'd into a testdata.jar file in that folder. This folder is defined to be on
+ * the classpath for org.aspectj.matcher, this enables them to be seen by a simple world that uses the classpath of the matcher
+ * project as the definition of what it can see. Other worlds, for example JDT World, will need to have those types defined in a
+ * project that is accessible in the JDT World instance. Because these tests exercise Java5 matching, the concrete ReflectionWorld
+ * subtype is not defined in here, it is defined in weaver5 (messy, but works well).
+ * 
  * @author Andy Clement
  */
-public abstract class FurtherCommonPointcutExpressionTests extends TestCase {
+public abstract class CommonAdvancedPointcutExpressionTests extends TestCase {
 
 	private World world;
 	private StandardPointcutParser pointcutParser;
@@ -40,15 +48,44 @@ public abstract class FurtherCommonPointcutExpressionTests extends TestCase {
 		pointcutParser = StandardPointcutParser.getPointcutParserSupportingAllPrimitives(world);
 	}
 
-	// public void testResolvingOneType() {
-	// // do it via name
-	// ResolvedType type = world.resolve("java.lang.String");
-	// assertNotNull(type);
-	// // do it via signature
-	// type = world.resolve(UnresolvedType.forSignature("Ljava/lang/String;"));
-	// assertNotNull(type);
-	// }
+	public void testResolvingOneType() {
+		assertFalse(world.resolve("testdata.SomeAnnotation").isMissing());
+		assertFalse(world.resolve("testdata.MethodLevelAnnotation").isMissing());
+		assertFalse(world.resolve("testdata.AnnotatedClass").isMissing());
+	}
+
+	public void testTypeLevelAnnotationMatchingWithStaticInitialization01() {
+		StandardPointcutExpression ex = pointcutParser.parsePointcutExpression("staticinitialization(@testdata.SomeAnnotation *)");
+		ResolvedType jlString = world.resolve("java.lang.String");
+		ResolvedType tAnnotatedClass = world.resolve("testdata.AnnotatedClass");
+
+		assertTrue(ex.matchesStaticInitialization(tAnnotatedClass).alwaysMatches());
+		assertTrue(ex.matchesStaticInitialization(jlString).neverMatches());
+	}
+
+	public void testTypeLevelAnnotationMatchingWithExecution01() {
+		StandardPointcutExpression ex = pointcutParser.parsePointcutExpression("execution(* (@testdata.SomeAnnotation *).*(..))");
+		ResolvedType jlString = world.resolve("java.lang.String");
+		ResolvedType tAnnotatedClass = world.resolve("testdata.AnnotatedClass");
+		assertTrue(ex.matchesMethodExecution(getMethod(tAnnotatedClass, "annotatedMethod", "()V")).alwaysMatches());
+		assertTrue(ex.matchesMethodExecution(getMethod(jlString, "valueOf", "(Z)Ljava/lang/String;")).neverMatches());
+	}
+
+	public void testMethodLevelAnnotationMatchingWithExecution01() {
+		StandardPointcutExpression ex = pointcutParser
+				.parsePointcutExpression("execution(@testdata.MethodLevelAnnotation * *(..))");
+		ResolvedType jlString = world.resolve("java.lang.String");
+		ResolvedType tAnnotatedClass = world.resolve("testdata.AnnotatedClass");
+		assertTrue(ex.matchesMethodExecution(getMethod(tAnnotatedClass, "annotatedMethod", "()V")).alwaysMatches());
+		assertTrue(ex.matchesMethodExecution(getMethod(tAnnotatedClass, "nonAnnotatedMethod", "()V")).neverMatches());
+		assertTrue(ex.matchesMethodExecution(getMethod(jlString, "valueOf", "(Z)Ljava/lang/String;")).neverMatches());
+	}
+
 	//
+	// ResolvedMember stringSplitMethod = getMethod(jlString, "split", "(Ljava/lang/String;I)[Ljava/lang/String;");
+	// ResolvedMember stringValueOfIntMethod = getMethod(jlString, "valueOf", "(I)Ljava/lang/String;");
+	// ResolvedMember listAddMethod = getMethod(juList, "add", "(Ljava/lang/Object;)Z");
+
 	// public void testResolveTypeAndRetrieveMethod() {
 	// ResolvedType type = world.resolve("java.lang.String");
 	// assertNotNull(type);
-- 
2.39.5