Stylistic changes

PicnicSupermarket · Jun 20, 2022 · 804bebd · 804bebd
1 parent 5399ef4
commit 804bebd
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 11 deletions.
diff --git a/...-contrib/src/main/java/tech/picnic/errorprone/bugpatterns/AssertThatIsNullUsageCheck.java b/...-contrib/src/main/java/tech/picnic/errorprone/bugpatterns/AssertThatIsNullUsageCheck.java
@@ -16,10 +16,7 @@
 import com.google.errorprone.matchers.Matchers;
 import com.sun.source.tree.MethodInvocationTree;
 
-/**
- * A {@link BugChecker} which flags {@code asserThat(someValue).isEqualTo(null)} for further
- * simplification
- */
+/** A {@link BugChecker} which flags {@code Assert.isEqualTo(null)} for further simplification */
 @AutoService(BugChecker.class)
 @BugPattern(
     name = "AssertThatIsNullUsage",

diff --git a/...trib/src/test/java/tech/picnic/errorprone/bugpatterns/AssertThatIsNullUsageCheckTest.java b/...trib/src/test/java/tech/picnic/errorprone/bugpatterns/AssertThatIsNullUsageCheckTest.java
@@ -7,7 +7,6 @@
 final class AssertThatIsNullUsageCheckTest {
   private final CompilationTestHelper compilationTestHelper =
       CompilationTestHelper.newInstance(AssertThatIsNullUsageCheck.class, getClass());
-
   private final BugCheckerRefactoringTestHelper refactoringTestHelper =
       BugCheckerRefactoringTestHelper.newInstance(AssertThatIsNullUsageCheck.class, getClass());
 
@@ -25,10 +24,10 @@ void identification() {
             "    String nullValue = null;",
             "    assertThat(12).isEqualTo(12);",
             "    // BUG: Diagnostic contains: assertThat(...).isNull()",
-            "    assertThat(\"value\").isEqualTo(null);",
+            "    assertThat(\"foo\").isEqualTo(null);",
             "    // BUG: Diagnostic contains: assertThat(...).isNull()",
             "    assertThat(nullValue).isEqualTo(null);",
-            "    isEqualTo(\"foo\");",
+            "    isEqualTo(\"bar\");",
             "    isEqualTo(null);",
             "  }",
             "",
@@ -52,9 +51,9 @@ void replacement() {
             "  public void testAssertThat() {",
             "    String nullValue = null;",
             "    assertThat(12).isEqualTo(12);",
-            "    assertThat(\"value\").isEqualTo(null);",
+            "    assertThat(\"foo\").isEqualTo(null);",
             "    assertThat(nullValue).isEqualTo(null);",
-            "    isEqualTo(\"foo\");",
+            "    isEqualTo(\"bar\");",
             "    isEqualTo(null);",
             "  }",
             "",
@@ -72,9 +71,9 @@ void replacement() {
             "  public void testAssertThat() {",
             "    String nullValue = null;",
             "    assertThat(12).isEqualTo(12);",
-            "    assertThat(\"value\").isNull();",
+            "    assertThat(\"foo\").isNull();",
             "    assertThat(nullValue).isNull();",
-            "    isEqualTo(\"foo\");",
+            "    isEqualTo(\"bar\");",
             "    isEqualTo(null);",
             "  }",
             "",