From 4a77699d38518de3c031c5a41cfa81ea3568007c Mon Sep 17 00:00:00 2001
From: Chris Beams <cbeams@vmware.com>
Date: Fri, 19 Dec 2008 14:57:35 +0000
Subject: [PATCH] refactoring .testsuite .aop.aspectj tests in preparation for
 migration to .context

---
 .../aop/aspectj/autoproxy/_TestTypes.java     | 109 ++++++++++++++++++
 .../aop/framework/_TestTypes.java             |  41 +++++++
 2 files changed, 150 insertions(+)
 create mode 100644 org.springframework.testsuite/src/test/java/org/springframework/aop/aspectj/autoproxy/_TestTypes.java
 create mode 100644 org.springframework.testsuite/src/test/java/org/springframework/aop/framework/_TestTypes.java

diff --git a/org.springframework.testsuite/src/test/java/org/springframework/aop/aspectj/autoproxy/_TestTypes.java b/org.springframework.testsuite/src/test/java/org/springframework/aop/aspectj/autoproxy/_TestTypes.java
new file mode 100644
index 0000000000..cf76103d34
--- /dev/null
+++ b/org.springframework.testsuite/src/test/java/org/springframework/aop/aspectj/autoproxy/_TestTypes.java
@@ -0,0 +1,109 @@
+/*
+ * Copyright 2002-2006 the original author or authors.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.springframework.aop.aspectj.autoproxy;
+
+import java.lang.annotation.Retention;
+import java.lang.annotation.RetentionPolicy;
+
+import org.aspectj.lang.ProceedingJoinPoint;
+import org.springframework.transaction.annotation.Transactional;
+
+
+/**
+ * Definitions of testing types for use in within this package.
+ * Wherever possible, test types should be defined local to the java
+ * file that makes use of them.  In some cases however, a test type may
+ * need to be shared across tests.  Such types reside here, with the
+ * intention of reducing the surface area of java files within this
+ * package.  This allows developers to think about tests first, and deal
+ * with these second class testing artifacts on an as-needed basis.
+ * 
+ * Types here should be defined as package-private top level classes in
+ * order to avoid needing to fully qualify, e.g.: _TestTypes$Foo.
+ *
+ * @author Chris Beams
+ */
+final class _TestTypes { }
+
+
+/**
+ * @author Adrian Colyer
+ * @since 2.0
+ */
+interface AnnotatedTestBean {
+
+	String doThis();
+
+	String doThat();
+
+	String doTheOther();
+
+	String[] doArray();
+
+}
+
+
+/**
+ * @author Adrian Colyer
+ * @since 2.0
+ */
+@Retention(RetentionPolicy.RUNTIME)
+@interface TestAnnotation {
+	String value() ;
+}
+
+
+/**
+ * @author Adrian Colyer
+ * @since 2.0
+ */
+@Transactional
+class AnnotatedTestBeanImpl implements AnnotatedTestBean {
+
+	@TestAnnotation("this value")
+	public String doThis() {
+		return "doThis";
+	}
+
+	@TestAnnotation("that value")
+	public String doThat() {
+		return "doThat";
+	}
+
+	@TestAnnotation("array value")
+	public String[] doArray() {
+		return new String[] {"doThis", "doThat"};
+	}
+
+	// not annotated
+	public String doTheOther() {
+		return "doTheOther";
+	}
+
+}
+
+
+/**
+ * @author Adrian Colyer
+ */
+class AnnotationBindingTestAspect {
+
+	public String doWithAnnotation(ProceedingJoinPoint pjp, TestAnnotation testAnnotation) throws Throwable {
+		return testAnnotation.value();
+	}
+
+}
diff --git a/org.springframework.testsuite/src/test/java/org/springframework/aop/framework/_TestTypes.java b/org.springframework.testsuite/src/test/java/org/springframework/aop/framework/_TestTypes.java
new file mode 100644
index 0000000000..ea9f35a788
--- /dev/null
+++ b/org.springframework.testsuite/src/test/java/org/springframework/aop/framework/_TestTypes.java
@@ -0,0 +1,41 @@
+package org.springframework.aop.framework;
+
+/**
+ * Definitions of testing types for use in within this package.
+ * Wherever possible, test types should be defined local to the java
+ * file that makes use of them.  In some cases however, a test type may
+ * need to be shared across tests.  Such types reside here, with the
+ * intention of reducing the surface area of java files within this
+ * package.  This allows developers to think about tests first, and deal
+ * with these second class testing artifacts on an as-needed basis.
+ * 
+ * Types here should be defined as package-private top level classes in
+ * order to avoid needing to fully qualify, e.g.: _TestTypes$Foo.
+ *
+ * @author Chris Beams
+ */
+final class _TestTypes { }
+
+
+interface IEcho {
+	int echoException(int i, Throwable t) throws Throwable;
+	int getA();
+	void setA(int a);
+}
+
+
+class Echo implements IEcho {
+	private int a;
+	
+	public int echoException(int i, Throwable t) throws Throwable {
+		if (t != null)
+			throw t;
+		return i;
+	}
+	public void setA(int a) {
+		this.a = a;
+	}
+	public int getA() {
+		return a;
+	}
+}
\ No newline at end of file