Transient fields should by default be excluded from equals and hashCode. Fixes #1724

6 files changed, 203 insertions, 0 deletions

diff --git a/test/transform/resource/after-delombok/EqualsAndHashCodeAutoExclude.java b/test/transform/resource/after-delombok/EqualsAndHashCodeAutoExclude.java
new file mode 100644
index 00000000..711b588d
--- /dev/null
+++ b/test/transform/resource/after-delombok/EqualsAndHashCodeAutoExclude.java
@@ -0,0 +1,64 @@
+class EqualsAndHashCodeAutoExclude {
+	int x;
+	String $a;
+	transient String b;
+	@java.lang.Override
+	@java.lang.SuppressWarnings("all")
+	public boolean equals(final java.lang.Object o) {
+		if (o == this) return true;
+		if (!(o instanceof EqualsAndHashCodeAutoExclude)) return false;
+		final EqualsAndHashCodeAutoExclude other = (EqualsAndHashCodeAutoExclude) o;
+		if (!other.canEqual((java.lang.Object) this)) return false;
+		if (this.x != other.x) return false;
+		return true;
+	}
+	@java.lang.SuppressWarnings("all")
+	protected boolean canEqual(final java.lang.Object other) {
+		return other instanceof EqualsAndHashCodeAutoExclude;
+	}
+	@java.lang.Override
+	@java.lang.SuppressWarnings("all")
+	public int hashCode() {
+		final int PRIME = 59;
+		int result = 1;
+		result = result * PRIME + this.x;
+		return result;
+	}
+}
+class EqualsAndHashCodeAutoExclude2 {
+	int x;
+	String $a;
+	transient String b;
+	@java.lang.Override
+	@java.lang.SuppressWarnings("all")
+	public boolean equals(final java.lang.Object o) {
+		if (o == this) return true;
+		if (!(o instanceof EqualsAndHashCodeAutoExclude2)) return false;
+		final EqualsAndHashCodeAutoExclude2 other = (EqualsAndHashCodeAutoExclude2) o;
+		if (!other.canEqual((java.lang.Object) this)) return false;
+		if (this.x != other.x) return false;
+		final java.lang.Object this$$a = this.$a;
+		final java.lang.Object other$$a = other.$a;
+		if (this$$a == null ? other$$a != null : !this$$a.equals(other$$a)) return false;
+		final java.lang.Object this$b = this.b;
+		final java.lang.Object other$b = other.b;
+		if (this$b == null ? other$b != null : !this$b.equals(other$b)) return false;
+		return true;
+	}
+	@java.lang.SuppressWarnings("all")
+	protected boolean canEqual(final java.lang.Object other) {
+		return other instanceof EqualsAndHashCodeAutoExclude2;
+	}
+	@java.lang.Override
+	@java.lang.SuppressWarnings("all")
+	public int hashCode() {
+		final int PRIME = 59;
+		int result = 1;
+		result = result * PRIME + this.x;
+		final java.lang.Object $$a = this.$a;
+		result = result * PRIME + ($$a == null ? 43 : $$a.hashCode());
+		final java.lang.Object $b = this.b;
+		result = result * PRIME + ($b == null ? 43 : $b.hashCode());
+		return result;
+	}
+}
\ No newline at end of file
diff --git a/test/transform/resource/after-delombok/ToStringAutoExclude.java b/test/transform/resource/after-delombok/ToStringAutoExclude.java
new file mode 100644
index 00000000..2beee756
--- /dev/null
+++ b/test/transform/resource/after-delombok/ToStringAutoExclude.java
@@ -0,0 +1,20 @@
+class ToStringAutoExclude {
+	int x;
+	String $a;
+	transient String b;
+	@java.lang.Override
+	@java.lang.SuppressWarnings("all")
+	public java.lang.String toString() {
+		return "ToStringAutoExclude(x=" + this.x + ", b=" + this.b + ")";
+	}
+}
+class ToStringAutoExclude2 {
+	int x;
+	String $a;
+	transient String b;
+	@java.lang.Override
+	@java.lang.SuppressWarnings("all")
+	public java.lang.String toString() {
+		return "ToStringAutoExclude2(x=" + this.x + ", $a=" + this.$a + ", b=" + this.b + ")";
+	}
+}
\ No newline at end of file
diff --git a/test/transform/resource/after-ecj/EqualsAndHashCodeAutoExclude.java b/test/transform/resource/after-ecj/EqualsAndHashCodeAutoExclude.java
new file mode 100644
index 00000000..7e6e171b
--- /dev/null
+++ b/test/transform/resource/after-ecj/EqualsAndHashCodeAutoExclude.java
@@ -0,0 +1,70 @@
+@lombok.EqualsAndHashCode class EqualsAndHashCodeAutoExclude {
+  int x;
+  String $a;
+  transient String b;
+  EqualsAndHashCodeAutoExclude() {
+    super();
+  }
+  public @java.lang.Override @java.lang.SuppressWarnings("all") boolean equals(final java.lang.Object o) {
+    if ((o == this))
+        return true;
+    if ((! (o instanceof EqualsAndHashCodeAutoExclude)))
+        return false;
+    final EqualsAndHashCodeAutoExclude other = (EqualsAndHashCodeAutoExclude) o;
+    if ((! other.canEqual((java.lang.Object) this)))
+        return false;
+    if ((this.x != other.x))
+        return false;
+    return true;
+  }
+  protected @java.lang.SuppressWarnings("all") boolean canEqual(final java.lang.Object other) {
+    return (other instanceof EqualsAndHashCodeAutoExclude);
+  }
+  public @java.lang.Override @java.lang.SuppressWarnings("all") int hashCode() {
+    final int PRIME = 59;
+    int result = 1;
+    result = ((result * PRIME) + this.x);
+    return result;
+  }
+}
+@lombok.EqualsAndHashCode class EqualsAndHashCodeAutoExclude2 {
+  int x;
+  @lombok.EqualsAndHashCode.Include String $a;
+  transient @lombok.EqualsAndHashCode.Include String b;
+  EqualsAndHashCodeAutoExclude2() {
+    super();
+  }
+  public @java.lang.Override @java.lang.SuppressWarnings("all") boolean equals(final java.lang.Object o) {
+    if ((o == this))
+        return true;
+    if ((! (o instanceof EqualsAndHashCodeAutoExclude2)))
+        return false;
+    final EqualsAndHashCodeAutoExclude2 other = (EqualsAndHashCodeAutoExclude2) o;
+    if ((! other.canEqual((java.lang.Object) this)))
+        return false;
+    if ((this.x != other.x))
+        return false;
+    final java.lang.Object this$$a = this.$a;
+    final java.lang.Object other$$a = other.$a;
+    if (((this$$a == null) ? (other$$a != null) : (! this$$a.equals(other$$a))))
+        return false;
+    final java.lang.Object this$b = this.b;
+    final java.lang.Object other$b = other.b;
+    if (((this$b == null) ? (other$b != null) : (! this$b.equals(other$b))))
+        return false;
+    return true;
+  }
+  protected @java.lang.SuppressWarnings("all") boolean canEqual(final java.lang.Object other) {
+    return (other instanceof EqualsAndHashCodeAutoExclude2);
+  }
+  public @java.lang.Override @java.lang.SuppressWarnings("all") int hashCode() {
+    final int PRIME = 59;
+    int result = 1;
+    result = ((result * PRIME) + this.x);
+    final java.lang.Object $$a = this.$a;
+    result = ((result * PRIME) + (($$a == null) ? 43 : $$a.hashCode()));
+    final java.lang.Object $b = this.b;
+    result = ((result * PRIME) + (($b == null) ? 43 : $b.hashCode()));
+    return result;
+  }
+}
diff --git a/test/transform/resource/after-ecj/ToStringAutoExclude.java b/test/transform/resource/after-ecj/ToStringAutoExclude.java
new file mode 100644
index 00000000..c25b5bfd
--- /dev/null
+++ b/test/transform/resource/after-ecj/ToStringAutoExclude.java
@@ -0,0 +1,22 @@
+@lombok.ToString class ToStringAutoExclude {
+  int x;
+  String $a;
+  transient String b;
+  ToStringAutoExclude() {
+    super();
+  }
+  public @java.lang.Override @java.lang.SuppressWarnings("all") java.lang.String toString() {
+    return (((("ToStringAutoExclude(x=" + this.x) + ", b=") + this.b) + ")");
+  }
+}
+@lombok.ToString class ToStringAutoExclude2 {
+  int x;
+  @lombok.ToString.Include String $a;
+  transient String b;
+  ToStringAutoExclude2() {
+    super();
+  }
+  public @java.lang.Override @java.lang.SuppressWarnings("all") java.lang.String toString() {
+    return (((((("ToStringAutoExclude2(x=" + this.x) + ", $a=") + this.$a) + ", b=") + this.b) + ")");
+  }
+}
\ No newline at end of file
diff --git a/test/transform/resource/before/EqualsAndHashCodeAutoExclude.java b/test/transform/resource/before/EqualsAndHashCodeAutoExclude.java
new file mode 100644
index 00000000..ef672833
--- /dev/null
+++ b/test/transform/resource/before/EqualsAndHashCodeAutoExclude.java
@@ -0,0 +1,14 @@
+@lombok.EqualsAndHashCode
+class EqualsAndHashCodeAutoExclude {
+	int x;
+	String $a;
+	transient String b;
+}
+@lombok.EqualsAndHashCode
+class EqualsAndHashCodeAutoExclude2 {
+	int x;
+	@lombok.EqualsAndHashCode.Include
+	String $a;
+	@lombok.EqualsAndHashCode.Include
+	transient String b;
+}
diff --git a/test/transform/resource/before/ToStringAutoExclude.java b/test/transform/resource/before/ToStringAutoExclude.java
new file mode 100644
index 00000000..49695390
--- /dev/null
+++ b/test/transform/resource/before/ToStringAutoExclude.java
@@ -0,0 +1,13 @@
+@lombok.ToString
+class ToStringAutoExclude {
+	int x;
+	String $a;
+	transient String b;
+}
+@lombok.ToString
+class ToStringAutoExclude2 {
+	int x;
+	@lombok.ToString.Include
+	String $a;
+	transient String b;
+}