HandleDelegate did not remove the annotation @Delegate from javac's AST, so HandleDelegate was called multiple times for the same field resulting in an error saying the delegate method was already defined.

Also added a test for @Delegate that uncovered this issue.

3 files changed, 64 insertions, 0 deletions

diff --git a/test/transform/resource/after-delombok/DelegateTypesAndExcludes.java b/test/transform/resource/after-delombok/DelegateTypesAndExcludes.java
new file mode 100644
index 00000000..3f6f555d
--- /dev/null
+++ b/test/transform/resource/after-delombok/DelegateTypesAndExcludes.java
@@ -0,0 +1,35 @@
+class DelegatePlain {
+	private final BarImpl bar = new BarImpl();
+	private final FooImpl foo = new FooImpl();
+
+	private static class FooImpl implements Foo {
+		public void foo() {
+		}
+
+		public void bar(java.util.ArrayList<java.lang.String> list) {
+		}
+	}
+
+	private static class BarImpl implements Bar {
+		public void bar(java.util.ArrayList<java.lang.String> list) {
+		}
+	}
+
+	private static interface Foo extends Bar {
+		void foo();
+	}
+
+	private static interface Bar {
+		void bar(java.util.ArrayList<java.lang.String> list);
+	}
+
+	@java.lang.SuppressWarnings("all")
+	public void bar(final java.util.ArrayList<java.lang.String> list) {
+		this.bar.bar(list);
+	}
+
+	@java.lang.SuppressWarnings("all")
+	public void foo() {
+		this.foo.foo();
+	}
+}
\ No newline at end of file
diff --git a/test/transform/resource/after-ecj/DelegateTypesAndExcludes.java b/test/transform/resource/after-ecj/DelegateTypesAndExcludes.java
new file mode 100644
index 00000000..cb06d3c1
--- /dev/null
+++ b/test/transform/resource/after-ecj/DelegateTypesAndExcludes.java
@@ -0,0 +1 @@
+//ignore
\ No newline at end of file
diff --git a/test/transform/resource/before/DelegateTypesAndExcludes.java b/test/transform/resource/before/DelegateTypesAndExcludes.java
new file mode 100644
index 00000000..da7fc4cb
--- /dev/null
+++ b/test/transform/resource/before/DelegateTypesAndExcludes.java
@@ -0,0 +1,28 @@
+import lombok.Delegate;
+class DelegatePlain {
+	@Delegate(types = Bar.class)
+	private final BarImpl bar = new BarImpl();
+	@Delegate(types = Foo.class, excludes = Bar.class)
+	private final FooImpl foo = new FooImpl();
+
+	private static class FooImpl implements Foo {
+		public void foo() {
+		}
+
+		public void bar(java.util.ArrayList<java.lang.String> list) {
+		}
+	}
+
+	private static class BarImpl implements Bar {
+		public void bar(java.util.ArrayList<java.lang.String> list) {
+		}
+	}
+
+	private static interface Foo extends Bar {
+		void foo();
+	}
+
+	private static interface Bar {
+		void bar(java.util.ArrayList<java.lang.String> list);
+	}
+}
\ No newline at end of file