From 1d41626c3d8489acf9a630d9236a9ecdf912363d Mon Sep 17 00:00:00 2001
From: aclement <aclement>
Date: Wed, 1 Feb 2006 15:20:37 +0000
Subject: [PATCH] testcode for 125699 and 125810 from matthew. test for 125699
 commented out.

---
 tests/bugs151/pr125699/AtTestTracing.java        |  7 +++++++
 tests/bugs151/pr125699/TestTracing.aj            |  2 ++
 tests/bugs151/pr125699/Tracing.aj                | 16 ++++++++++++++++
 tests/bugs151/pr125810/SubAspect.aj              |  5 +++++
 tests/bugs151/pr125810/SubAtAspect.java          |  6 ++++++
 tests/bugs151/pr125810/SuperAspect.aj            |  4 ++++
 .../aspectj/systemtest/ajc151/Ajc151Tests.java   | 11 +++++++++++
 .../src/org/aspectj/systemtest/ajc151/ajc151.xml | 13 +++++++++++++
 8 files changed, 64 insertions(+)
 create mode 100644 tests/bugs151/pr125699/AtTestTracing.java
 create mode 100644 tests/bugs151/pr125699/TestTracing.aj
 create mode 100644 tests/bugs151/pr125699/Tracing.aj
 create mode 100644 tests/bugs151/pr125810/SubAspect.aj
 create mode 100644 tests/bugs151/pr125810/SubAtAspect.java
 create mode 100644 tests/bugs151/pr125810/SuperAspect.aj

diff --git a/tests/bugs151/pr125699/AtTestTracing.java b/tests/bugs151/pr125699/AtTestTracing.java
new file mode 100644
index 000000000..32d380f8b
--- /dev/null
+++ b/tests/bugs151/pr125699/AtTestTracing.java
@@ -0,0 +1,7 @@
+import org.aspectj.lang.annotation.Aspect;
+import org.aspectj.lang.annotation.Pointcut;
+
+@Aspect
+public class AtTestTracing extends Tracing {
+
+}
diff --git a/tests/bugs151/pr125699/TestTracing.aj b/tests/bugs151/pr125699/TestTracing.aj
new file mode 100644
index 000000000..1de7054bb
--- /dev/null
+++ b/tests/bugs151/pr125699/TestTracing.aj
@@ -0,0 +1,2 @@
+public aspect TestTracing extends Tracing {
+}
diff --git a/tests/bugs151/pr125699/Tracing.aj b/tests/bugs151/pr125699/Tracing.aj
new file mode 100644
index 000000000..bff3a8101
--- /dev/null
+++ b/tests/bugs151/pr125699/Tracing.aj
@@ -0,0 +1,16 @@
+public abstract aspect Tracing {
+	
+	before (Object obj) : execution(* *(..)) && this(obj) {
+		System.out.println(thisJoinPoint);
+	}
+	
+//	before (Object obj) : execution(* *(..)) && this(obj) {
+	before () : execution(* *(..)) {
+		System.out.println(thisJoinPointStaticPart);
+	}
+	
+//	before (Object obj) : execution(* *(..)) && this(obj) {
+	before () : execution(* *(..)) && this(Object) {
+		System.out.println(thisEnclosingJoinPointStaticPart);
+	}
+}
diff --git a/tests/bugs151/pr125810/SubAspect.aj b/tests/bugs151/pr125810/SubAspect.aj
new file mode 100644
index 000000000..9a3a964a0
--- /dev/null
+++ b/tests/bugs151/pr125810/SubAspect.aj
@@ -0,0 +1,5 @@
+public aspect SubAspect extends SuperAspect {
+
+//	protected pointcut scope();
+
+}
diff --git a/tests/bugs151/pr125810/SubAtAspect.java b/tests/bugs151/pr125810/SubAtAspect.java
new file mode 100644
index 000000000..78e85b5f4
--- /dev/null
+++ b/tests/bugs151/pr125810/SubAtAspect.java
@@ -0,0 +1,6 @@
+import org.aspectj.lang.annotation.Aspect;
+
+@Aspect
+public class SubAtAspect extends SuperAspect {
+
+}
diff --git a/tests/bugs151/pr125810/SuperAspect.aj b/tests/bugs151/pr125810/SuperAspect.aj
new file mode 100644
index 000000000..1f34881a6
--- /dev/null
+++ b/tests/bugs151/pr125810/SuperAspect.aj
@@ -0,0 +1,4 @@
+public abstract aspect SuperAspect {
+
+	protected abstract pointcut scope ();
+}
diff --git a/tests/src/org/aspectj/systemtest/ajc151/Ajc151Tests.java b/tests/src/org/aspectj/systemtest/ajc151/Ajc151Tests.java
index 36b036da4..d47e36d8f 100644
--- a/tests/src/org/aspectj/systemtest/ajc151/Ajc151Tests.java
+++ b/tests/src/org/aspectj/systemtest/ajc151/Ajc151Tests.java
@@ -71,6 +71,17 @@ public class Ajc151Tests extends org.aspectj.testing.XMLBasedAjcTestCase {
   	  assertFalse("printParameters method should have arguments",pe2.getParameterTypes().isEmpty());	  
   }
   
+  /*
+   * @AspectJ bugs and enhancements
+   */
+//  public void testAtAspectInheritsAdviceWithTJPAndThis_pr125699 () {
+//	  runTest("inherit adivce with this() and thisJoinPoint"); 
+//  }
+  
+  public void testAtAspectInheritsAbstractPointcut_pr125810 () {
+	  runTest("warning when inherited pointcut not made concrete"); 
+  }
+  
   /*
    * Load-time weaving bugs and enhancements
    */
diff --git a/tests/src/org/aspectj/systemtest/ajc151/ajc151.xml b/tests/src/org/aspectj/systemtest/ajc151/ajc151.xml
index 99c2698d1..7d5a966c2 100644
--- a/tests/src/org/aspectj/systemtest/ajc151/ajc151.xml
+++ b/tests/src/org/aspectj/systemtest/ajc151/ajc151.xml
@@ -124,6 +124,19 @@
         <compile files="AbstractMethods.aj, ConcreteMethods.aj"/>
         <run class="HelloWorld" ltw="aop-tracing.xml"/>
     </ajc-test>
+    
+    <ajc-test dir="bugs151/pr125699" title="inherit adivce with this() and thisJoinPoint">
+        <compile files="Tracing.aj, TestTracing.aj, AtTestTracing.java" options="-1.5"/>
+    </ajc-test>
+    
+    <ajc-test dir="bugs151/pr125810" title="warning when inherited pointcut not made concrete">
+        <compile files="SuperAspect.aj, SubAspect.aj, SubAtAspect.java" options="-1.5">
+          <message kind="error" line="3" text="inherited abstract pointcut SuperAspect.scope() is not made concrete in SubAspect"/>
+          <message kind="error" line="1" text="inherited abstract pointcut SuperAspect.scope() is not made concrete in SubAspect"/>
+          <message kind="error" line="3" text="inherited abstract pointcut SuperAspect.scope() is not made concrete in SubAtAspect"/>
+          <message kind="error" line="4" text="inherited abstract pointcut SuperAspect.scope() is not made concrete in SubAtAspect"/>
+        </compile>
+    </ajc-test>
 
     
     <!-- New features down here... when they arent big enough to have their own test file -->
-- 
2.39.5