Suggestion

PicnicSupermarket · Jul 27, 2023 · 2dc88b7 · 2dc88b7
1 parent 5621b94
commit 2dc88b7
Showing 1 changed file with 16 additions and 17 deletions.
diff --git a/core/src/test/java/com/google/errorprone/scanner/ScannerTest.java b/core/src/test/java/com/google/errorprone/scanner/ScannerTest.java
@@ -39,23 +39,6 @@ public class ScannerTest {
   private final CompilationTestHelper compilationHelper =
       CompilationTestHelper.newInstance(ShouldNotUseFoo.class, getClass());
 
-  @Test
-  public void dontRunDisabledChecks() {
-    compilationHelper
-        .addSourceLines(
-            "Test.java",
-            "import com.google.errorprone.scanner.ScannerTest.Foo;",
-            "class Test {",
-            "  Foo foo;",
-            "}")
-        .setArgs(
-            ImmutableList.of(
-                "-XepPatchLocation:IN_PLACE",
-                "-XepPatchChecks:ShouldNotUseFoo",
-                "-Xep:ShouldNotUseFoo:OFF"))
-        .doTest();
-  }
-
   @Test
   public void notSuppressedByAnnotationOnType() {
     compilationHelper
@@ -112,6 +95,22 @@ public void suppressionAnnotationIgnoredWithOptions() {
         .doTest();
   }
 
+  @Test
+  public void dontRunDisabledChecks() {
+    compilationHelper
+        .addSourceLines(
+            "Test.java",
+            "import com.google.errorprone.scanner.ScannerTest.Foo;",
+            "class Test {",
+            "  Foo foo;",
+            "}")
+        .setArgs(
+            "-XepPatchLocation:IN_PLACE",
+            "-XepPatchChecks:ShouldNotUseFoo",
+            "-Xep:ShouldNotUseFoo:OFF")
+        .doTest();
+  }
+
   @OkToUseFoo // Foo can use itself. But this shouldn't suppress errors on *usages* of Foo.
   public static final class Foo<T> {}