From 4ca8479d63c935c1942a41b31d7f6a0a26781dd3 Mon Sep 17 00:00:00 2001 From: acolyer Date: Wed, 9 Mar 2005 15:03:50 +0000 Subject: [PATCH] remove accidental 1.5 dependency in tests module --- .../aspectMembers/a/AnnotatedAspect05.aj | 14 +++++++++++++- .../aspectj/systemtest/ajc150/Annotations.java | 18 ------------------ .../org/aspectj/systemtest/ajc150/ajc150.xml | 3 ++- 3 files changed, 15 insertions(+), 20 deletions(-) diff --git a/tests/java5/annotations/aspectMembers/a/AnnotatedAspect05.aj b/tests/java5/annotations/aspectMembers/a/AnnotatedAspect05.aj index f54f15e8d..9739187f0 100644 --- a/tests/java5/annotations/aspectMembers/a/AnnotatedAspect05.aj +++ b/tests/java5/annotations/aspectMembers/a/AnnotatedAspect05.aj @@ -1,6 +1,7 @@ package a; import org.aspectj.lang.annotation.SuppressAjWarnings; +import java.lang.annotation.Annotation; @TypeAnnotation public aspect AnnotatedAspect05 { @@ -21,7 +22,18 @@ public aspect AnnotatedAspect05 { @MethodAnnotation @SuppressAjWarnings - void around() : call(new(..)) { proceed(); } + Object around() : call(new(..)) { return proceed(); } + public static void main(String[] args) { + java.lang.reflect.Method[] methods = AnnotatedAspect05.class.getDeclaredMethods(); + for (int i = 0; i < methods.length; i++) { + if (methods[i].getName().startsWith("ajc$afterThrowing")) { + Annotation annotation = methods[i].getAnnotation(AnyAnnotation.class); + if (annotation == null) { + throw new RuntimeException("advice should be annotated"); + } + } + } + } } diff --git a/tests/src/org/aspectj/systemtest/ajc150/Annotations.java b/tests/src/org/aspectj/systemtest/ajc150/Annotations.java index 4e0a16d45..4cf7d0a7b 100644 --- a/tests/src/org/aspectj/systemtest/ajc150/Annotations.java +++ b/tests/src/org/aspectj/systemtest/ajc150/Annotations.java @@ -11,9 +11,6 @@ package org.aspectj.systemtest.ajc150; import java.io.File; -import java.lang.annotation.Annotation; -import java.net.URL; -import java.net.URLClassLoader; import junit.framework.Test; @@ -88,21 +85,6 @@ public class Annotations extends XMLBasedAjcTestCase { public void testAnnotatedAdvice() { runTest("annotated advice"); - try { - File classFile = new File(ajc.getSandboxDirectory(),"a/AnnotatedAspect05.class"); - ClassLoader cl = new URLClassLoader(new URL[] {ajc.getSandboxDirectory().toURI().toURL()}); - Class c = cl.loadClass("a.AnnotatedAspect05"); - Class ann = cl.loadClass("a.AnyAnnotation"); - java.lang.reflect.Method[] methods = c.getDeclaredMethods(); - for (int i = 0; i < methods.length; i++) { - if (methods[i].getName().startsWith("ajc$afterThrowing")) { - Annotation annotation = methods[i].getAnnotation(ann); - assertNotNull("Should have @AnyAnnotation",annotation); - } - } - } catch (Exception ex) { - fail(ex.getMessage()); - } } public void testAnnotatedAdviceWithWrongAnnotationType() { diff --git a/tests/src/org/aspectj/systemtest/ajc150/ajc150.xml b/tests/src/org/aspectj/systemtest/ajc150/ajc150.xml index 258145e5c..094ddf858 100644 --- a/tests/src/org/aspectj/systemtest/ajc150/ajc150.xml +++ b/tests/src/org/aspectj/systemtest/ajc150/ajc150.xml @@ -994,8 +994,9 @@ - + + -- 2.39.5