diff --git a/extras/src/test/java/com/google/gson/typeadapters/PostConstructAdapterFactoryTest.java b/extras/src/test/java/com/google/gson/typeadapters/PostConstructAdapterFactoryTest.java index 65053bac..a4b9d04e 100644 --- a/extras/src/test/java/com/google/gson/typeadapters/PostConstructAdapterFactoryTest.java +++ b/extras/src/test/java/com/google/gson/typeadapters/PostConstructAdapterFactoryTest.java @@ -57,8 +57,8 @@ public class PostConstructAdapterFactoryTest { assertEquals(sandwiches, sandwichesFromJson); } - @SuppressWarnings("overrides") // for missing hashCode() override - static class Sandwich { + @SuppressWarnings({"overrides", "EqualsHashCode"}) // for missing hashCode() override + static class Sandwich { public String bread; public String cheese; @@ -92,7 +92,7 @@ public class PostConstructAdapterFactoryTest { } } - @SuppressWarnings("overrides") // for missing hashCode() override + @SuppressWarnings({"overrides", "EqualsHashCode"}) // for missing hashCode() override static class MultipleSandwiches { public List sandwiches; diff --git a/gson/src/test/java/com/google/gson/JsonArrayTest.java b/gson/src/test/java/com/google/gson/JsonArrayTest.java index 21deb729..bb531117 100644 --- a/gson/src/test/java/com/google/gson/JsonArrayTest.java +++ b/gson/src/test/java/com/google/gson/JsonArrayTest.java @@ -35,6 +35,7 @@ public final class JsonArrayTest { } @Test + @SuppressWarnings("TruthSelfEquals") public void testEqualsNonEmptyArray() { JsonArray a = new JsonArray(); JsonArray b = new JsonArray(); diff --git a/gson/src/test/java/com/google/gson/JsonObjectTest.java b/gson/src/test/java/com/google/gson/JsonObjectTest.java index d0da6abb..9b37823d 100644 --- a/gson/src/test/java/com/google/gson/JsonObjectTest.java +++ b/gson/src/test/java/com/google/gson/JsonObjectTest.java @@ -163,6 +163,7 @@ public class JsonObjectTest { } @Test + @SuppressWarnings("TruthSelfEquals") public void testEqualsNonEmptyObject() { JsonObject a = new JsonObject(); JsonObject b = new JsonObject(); diff --git a/gson/src/test/java/com/google/gson/common/TestTypes.java b/gson/src/test/java/com/google/gson/common/TestTypes.java index 13807637..9df4a73d 100644 --- a/gson/src/test/java/com/google/gson/common/TestTypes.java +++ b/gson/src/test/java/com/google/gson/common/TestTypes.java @@ -227,7 +227,8 @@ public class TestTypes { } } - @SuppressWarnings("overrides") // for missing hashCode() override + // for missing hashCode() override + @SuppressWarnings({"overrides", "EqualsHashCode"}) public static class ClassWithNoFields { // Nothing here.. @Override diff --git a/gson/src/test/java/com/google/gson/functional/DefaultTypeAdaptersTest.java b/gson/src/test/java/com/google/gson/functional/DefaultTypeAdaptersTest.java index 33ebc682..ee15bfc4 100644 --- a/gson/src/test/java/com/google/gson/functional/DefaultTypeAdaptersTest.java +++ b/gson/src/test/java/com/google/gson/functional/DefaultTypeAdaptersTest.java @@ -103,7 +103,7 @@ public class DefaultTypeAdaptersTest { @Test public void testClassDeserialization() { try { - gson.fromJson("String.class", String.class.getClass()); + gson.fromJson("String.class", Class.class); fail(); } catch (UnsupportedOperationException expected) { } diff --git a/gson/src/test/java/com/google/gson/functional/InstanceCreatorTest.java b/gson/src/test/java/com/google/gson/functional/InstanceCreatorTest.java index 930a18d5..ea3f9799 100644 --- a/gson/src/test/java/com/google/gson/functional/InstanceCreatorTest.java +++ b/gson/src/test/java/com/google/gson/functional/InstanceCreatorTest.java @@ -122,11 +122,11 @@ public class InstanceCreatorTest { Type sortedSetType = new TypeToken>() {}.getType(); SortedSet set = gson.fromJson("[\"a\"]", sortedSetType); - assertThat("a").isEqualTo(set.first()); + assertThat(set.first()).isEqualTo("a"); assertThat(set.getClass()).isEqualTo(SubTreeSet.class); set = gson.fromJson("[\"b\"]", SortedSet.class); - assertThat("b").isEqualTo(set.first()); + assertThat(set.first()).isEqualTo("b"); assertThat(set.getClass()).isEqualTo(SubTreeSet.class); } } diff --git a/gson/src/test/java/com/google/gson/functional/ReflectionAccessFilterTest.java b/gson/src/test/java/com/google/gson/functional/ReflectionAccessFilterTest.java index 67b60e2b..dc58d024 100644 --- a/gson/src/test/java/com/google/gson/functional/ReflectionAccessFilterTest.java +++ b/gson/src/test/java/com/google/gson/functional/ReflectionAccessFilterTest.java @@ -18,7 +18,7 @@ package com.google.gson.functional; import static com.google.common.truth.Truth.assertThat; import static org.junit.Assert.fail; -import static org.junit.Assume.assumeNotNull;; +import static org.junit.Assume.assumeNotNull; import com.google.gson.Gson; import com.google.gson.GsonBuilder; diff --git a/gson/src/test/java/com/google/gson/functional/TypeVariableTest.java b/gson/src/test/java/com/google/gson/functional/TypeVariableTest.java index fdde72c8..c788febe 100644 --- a/gson/src/test/java/com/google/gson/functional/TypeVariableTest.java +++ b/gson/src/test/java/com/google/gson/functional/TypeVariableTest.java @@ -73,7 +73,8 @@ public class TypeVariableTest { assertThat(blue2).isEqualTo(blue1); } - @SuppressWarnings("overrides") // for missing hashCode() override + // for missing hashCode() override + @SuppressWarnings({"overrides", "EqualsHashCode"}) public static class Blue extends Red { public Blue() { super(false); @@ -103,7 +104,7 @@ public class TypeVariableTest { } } - @SuppressWarnings("overrides") // for missing hashCode() override + @SuppressWarnings({"overrides", "EqualsHashCode"}) // for missing hashCode() override public static class Foo extends Red { private S someSField; private T someTField; diff --git a/gson/src/test/java/com/google/gson/stream/JsonReaderTest.java b/gson/src/test/java/com/google/gson/stream/JsonReaderTest.java index 92af5bd1..09c07470 100644 --- a/gson/src/test/java/com/google/gson/stream/JsonReaderTest.java +++ b/gson/src/test/java/com/google/gson/stream/JsonReaderTest.java @@ -1698,7 +1698,8 @@ public final class JsonReaderTest { reader.beginObject(); assertThat(reader.nextName()).isEqualTo("a"); } - assertThat(reader.getPath()); + assertThat(reader.getPath()).isEqualTo("$.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a" + + ".a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a"); assertThat(reader.nextBoolean()).isTrue(); for (int i = 0; i < 40; i++) { reader.endObject(); diff --git a/pom.xml b/pom.xml index 256fd54f..babfbc24 100644 --- a/pom.xml +++ b/pom.xml @@ -94,12 +94,32 @@ true true - true + true + + -XDcompilePolicy=simple + -Xplugin:ErrorProne + -J--add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED + -J--add-exports=jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED + -J--add-exports=jdk.compiler/com.sun.tools.javac.main=ALL-UNNAMED + -J--add-exports=jdk.compiler/com.sun.tools.javac.model=ALL-UNNAMED + -J--add-exports=jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED + -J--add-exports=jdk.compiler/com.sun.tools.javac.processing=ALL-UNNAMED + -J--add-exports=jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED + -J--add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED + -J--add-opens=jdk.compiler/com.sun.tools.javac.code=ALL-UNNAMED + -J--add-opens=jdk.compiler/com.sun.tools.javac.comp=ALL-UNNAMED -Xlint:all,-options + + + com.google.errorprone + error_prone_core + 2.18.0 + + [11,) @@ -271,6 +291,26 @@ + + + jdk15 + + 15 + + + + + org.apache.maven.plugins + maven-compiler-plugin + + + -Xlint:all,-options + + + + + + release