Added a test to showcase the issue where lombok does not update type mirrors, messing up visibility of lombok-generated stuff to other annotation processors.

4 files changed, 141 insertions, 0 deletions

diff --git a/build.xml b/build.xml
index a4f22545..86487671 100644
--- a/build.xml
+++ b/build.xml
@@ -305,6 +305,7 @@ lombok.launch.AnnotationProcessorHider$ClaimingProcessor</echo>
 				<srcdir dir="src/installer" />
 				<srcdir dir="src/delombok" />
 				<srcdir dir="src/stubs" />
+				<srcdir dir="src/testAP" />
 				<srcdir dir="experimental/src" />
 				<srcdir dir="test/transform/src" test="true" />
 				<srcdir dir="test/core/src" test="true" />
@@ -335,6 +336,7 @@ lombok.launch.AnnotationProcessorHider$ClaimingProcessor</echo>
 			<srcdir dir="src/installer" />
 			<srcdir dir="src/delombok" />
 			<srcdir dir="src/stubs" />
+			<srcdir dir="src/testAP" />
 			<srcdir dir="experimental/src" />
 			<srcdir dir="test/transform/src" />
 			<srcdir dir="test/core/src" />
@@ -766,4 +768,53 @@ You can also create your own by writing a 'testenvironment.properties' file. The
 			<property name="ssh.keyfile" value="${ssh.keyfile}" />
 		</ant>
 	</target>
+
+	<target name="testAp-compile" depends="ensureBuildDeps">
+		<delete file="build/testAP/META-INF/services/javax.annotation.processing.Processor" quiet="true" />
+		<ivy:compile destdir="build/testAP" source="1.7" target="1.7" includeantruntime="false">
+			<src path="src/testAP" />
+		</ivy:compile>
+		
+		<mkdir dir="build/testAP/META-INF" />
+		<mkdir dir="build/testAP/META-INF/services" />
+		<echo file="build/testAP/META-INF/services/javax.annotation.processing.Processor">org.projectlombok.testAp.TestAp</echo>
+	</target>
+
+	<target name="testAp-dist" depends="testAp-compile">
+		<mkdir dir="dist" />
+		<tstamp>
+			<format property="releaseTimestamp" pattern="yyyy-MM-dd" />
+		</tstamp>
+		<zip destfile="dist/testAp-${releaseTimestamp}.jar">
+			<fileset dir="." includes="LICENSE" />
+			<fileset dir="build/testAp" />
+		</zip>
+		<copy file="dist/testAp-${releaseTimestamp}.jar" tofile="dist/testAp.jar" />
+	</target>
+
+	<target name="testAp" depends="testAp-dist, dist">
+		<echo>Running in order: First Lombok, Then testAP</echo>
+
+		<delete dir="build/useTestAp" quiet="true" />
+		<mkdir dir="build/useTestAp" />
+		<javac source="1.7" target="1.7" destdir="build/useTestAp" includeantruntime="false">
+			<src path="src/useTestAP" />
+			<classpath location="dist/lombok.jar" />
+			<classpath location="dist/testAp.jar" />
+			<compilerarg value="-processor" />
+			<compilerarg value="lombok.launch.AnnotationProcessorHider$AnnotationProcessor,org.projectlombok.testAp.TestAp" />
+		</javac>
+
+		<echo>Running in order: First TestAP, Then Lombok</echo>
+
+		<delete dir="build/useTestAp" quiet="true" />
+		<mkdir dir="build/useTestAp" />
+		<javac source="1.7" target="1.7" destdir="build/useTestAp" includeantruntime="false">
+			<src path="src/useTestAP" />
+			<classpath location="dist/lombok.jar" />
+			<classpath location="dist/testAp.jar" />
+			<compilerarg value="-processor" />
+			<compilerarg value="org.projectlombok.testAp.TestAp,lombok.launch.AnnotationProcessorHider$AnnotationProcessor" />
+		</javac>
+	</target>
 </project>
diff --git a/src/testAP/org/projectlombok/testAp/ExampleAnnotation.java b/src/testAP/org/projectlombok/testAp/ExampleAnnotation.java
new file mode 100644
index 00000000..b419326b
--- /dev/null
+++ b/src/testAP/org/projectlombok/testAp/ExampleAnnotation.java
@@ -0,0 +1,10 @@
+package org.projectlombok.testAp;
+
+import java.lang.annotation.ElementType;
+import java.lang.annotation.Retention;
+import java.lang.annotation.RetentionPolicy;
+import java.lang.annotation.Target;
+
+@Retention(RetentionPolicy.CLASS)
+@Target(ElementType.TYPE)
+public @interface ExampleAnnotation {}
diff --git a/src/testAP/org/projectlombok/testAp/TestAp.java b/src/testAP/org/projectlombok/testAp/TestAp.java
new file mode 100644
index 00000000..3deaf1c5
--- /dev/null
+++ b/src/testAP/org/projectlombok/testAp/TestAp.java
@@ -0,0 +1,72 @@
+/*
+ * Copyright (C) 2016 The Project Lombok Authors.
+ * 
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ * 
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ * 
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+ * THE SOFTWARE.
+ */
+package org.projectlombok.testAp;
+
+import java.util.Set;
+
+import javax.annotation.processing.AbstractProcessor;
+import javax.annotation.processing.ProcessingEnvironment;
+import javax.annotation.processing.RoundEnvironment;
+import javax.annotation.processing.SupportedAnnotationTypes;
+import javax.lang.model.SourceVersion;
+import javax.lang.model.element.Element;
+import javax.lang.model.element.ElementKind;
+import javax.lang.model.element.TypeElement;
+
+@SupportedAnnotationTypes("org.projectlombok.testAp.ExampleAnnotation")
+public final class TestAp extends AbstractProcessor {
+	private int roundCounter = 0;
+	private static final long START = System.currentTimeMillis();
+	
+	private void log(String txt) {
+		System.out.printf("***[%3d]: %s\n", System.currentTimeMillis() - START, txt);
+	}
+	
+	@Override public void init(ProcessingEnvironment processingEnv) {
+		log("TestAP in init");
+		super.init(processingEnv);
+	}
+	
+	@Override public boolean process(Set<? extends TypeElement> annotations, RoundEnvironment roundEnv) {
+		roundCounter++;
+		log("TestAP in round " + roundCounter);
+		boolean foundGetTest = false;
+		int annotatedElemCount = 0;
+		for (Element annotated : roundEnv.getElementsAnnotatedWith(ExampleAnnotation.class)) {
+			annotatedElemCount++;
+			for (Element child : annotated.getEnclosedElements()) {
+				if (child.getSimpleName().equals("getTest") && child.getKind() == ElementKind.METHOD) foundGetTest = true;
+				System.out.println(child);
+			}
+		}
+		
+		if (foundGetTest) log("RESULT: POSITIVE -- found the getTest method");
+		else if (annotatedElemCount > 0) log("RESULT: NEGATIVE-T1 -- found the example class but there's no getTest method in it according to the type mirror.");
+		else log("RESULT: NEGATIVE-T2 -- The example class is not provided by 'getElementsAnnotatedWith' in this rond.");
+		
+		return false;
+	}
+	
+	@Override public SourceVersion getSupportedSourceVersion() {
+		return SourceVersion.latestSupported();
+	}
+}
diff --git a/src/useTestAP/UseTestAp.java b/src/useTestAP/UseTestAp.java
new file mode 100644
index 00000000..08ea11df
--- /dev/null
+++ b/src/useTestAP/UseTestAp.java
@@ -0,0 +1,8 @@
+@org.projectlombok.testAp.ExampleAnnotation
+public class UseTestAp {
+	@lombok.Getter String test;
+
+	public void confirmGetTestExists() {
+		System.out.println(getTest());
+	}
+}