Added full test suite for @Accessors, as well as the new rules for whether or not a method is already there so lombok shouldn't generate it (now number of parameters matters), and added generics testing to RequiredArgsConstructor's static constructor feature.

3 files changed, 51 insertions, 4 deletions

diff --git a/test/transform/resource/after-ecj/Accessors.java b/test/transform/resource/after-ecj/Accessors.java
index 2650b53f..4ee0ebcd 100644
--- a/test/transform/resource/after-ecj/Accessors.java
+++ b/test/transform/resource/after-ecj/Accessors.java
@@ -93,4 +93,34 @@ class AccessorsChain {
   private String getName() {
     return fName;
   }
-}
\ No newline at end of file
+}
+class AccessorsFluentGenerics<T extends Number> {
+  private @lombok.Setter @lombok.experimental.Accessors(fluent = true) String name;
+  public @java.lang.SuppressWarnings("all") AccessorsFluentGenerics<T> name(final String name) {
+    this.name = name;
+    return this;
+  }
+  AccessorsFluentGenerics() {
+    super();
+  }
+}
+class AccessorsFluentNoChaining {
+  private @lombok.Setter @lombok.experimental.Accessors(fluent = true,chain = false) String name;
+  public @java.lang.SuppressWarnings("all") void name(final String name) {
+    this.name = name;
+  }
+  AccessorsFluentNoChaining() {
+    super();
+  }
+}
+class AccessorsFluentStatic<T extends Number> {
+  private static @lombok.Setter @lombok.experimental.Accessors(fluent = true) String name;
+  <clinit>() {
+  }
+  public static @java.lang.SuppressWarnings("all") void name(final String name) {
+    AccessorsFluentStatic.name = name;
+  }
+  AccessorsFluentStatic() {
+    super();
+  }
+}
diff --git a/test/transform/resource/after-ecj/Constructors.java b/test/transform/resource/after-ecj/Constructors.java
index 3b3a14da..e6dd8a21 100644
--- a/test/transform/resource/after-ecj/Constructors.java
+++ b/test/transform/resource/after-ecj/Constructors.java
@@ -40,4 +40,15 @@
   public @java.lang.SuppressWarnings("all") NoArgsConstructor1() {
     super();
   }
+}
+@lombok.RequiredArgsConstructor(staticName = "of") class RequiredArgsConstructorStaticNameGenerics<T extends Number> {
+  final T x;
+  String name;
+  private @java.lang.SuppressWarnings("all") RequiredArgsConstructorStaticNameGenerics(final T x) {
+    super();
+    this.x = x;
+  }
+  public static @java.lang.SuppressWarnings("all") <T extends Number>RequiredArgsConstructorStaticNameGenerics<T> of(final T x) {
+    return new RequiredArgsConstructorStaticNameGenerics<T>(x);
+  }
 }
\ No newline at end of file
diff --git a/test/transform/resource/after-ecj/SetterAlreadyExists.java b/test/transform/resource/after-ecj/SetterAlreadyExists.java
index 69983ac5..4c8a212d 100644
--- a/test/transform/resource/after-ecj/SetterAlreadyExists.java
+++ b/test/transform/resource/after-ecj/SetterAlreadyExists.java
@@ -32,6 +32,9 @@ class Setter4 {
 }
 class Setter5 {
   @lombok.Setter String foo;
+  public @java.lang.SuppressWarnings("all") void setFoo(final String foo) {
+    this.foo = foo;
+  }
   Setter5() {
     super();
   }
@@ -40,6 +43,9 @@ class Setter5 {
 }
 class Setter6 {
   @lombok.Setter String foo;
+  public @java.lang.SuppressWarnings("all") void setFoo(final String foo) {
+    this.foo = foo;
+  }
   Setter6() {
     super();
   }
@@ -51,7 +57,7 @@ class Setter7 {
   Setter7() {
     super();
   }
-  static void setFoo() {
+  void setFoo(String foo, Object... x) {
   }
 }
 class Setter8 {
@@ -59,7 +65,7 @@ class Setter8 {
   Setter8() {
     super();
   }
-  void setIsFoo() {
+  void setIsFoo(boolean foo) {
   }
 }
 class Setter9 {
@@ -67,6 +73,6 @@ class Setter9 {
   Setter9() {
     super();
   }
-  void setFoo() {
+  void setFoo(boolean foo) {
   }
 }
\ No newline at end of file