Remove JMockTestWithoutRunWithOrRuleAnnotation, which isn't enabled.

I spot checked some cases this would fire on, and it seems that Mockery is used a _lot_ in JUnit3 tests, for which this check would false positive. PiperOrigin-RevId: 439277961
google · Apr 4, 2022 · 3cc8786 · 3cc8786
1 parent 1be56c2
commit 3cc8786
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 245 deletions.
diff --git a/.../main/java/com/google/errorprone/bugpatterns/JMockTestWithoutRunWithOrRuleAnnotation.java b/.../main/java/com/google/errorprone/bugpatterns/JMockTestWithoutRunWithOrRuleAnnotation.java
diff --git a/core/src/main/java/com/google/errorprone/scanner/BuiltInCheckerSuppliers.java b/core/src/main/java/com/google/errorprone/scanner/BuiltInCheckerSuppliers.java
@@ -176,7 +176,6 @@
 import com.google.errorprone.bugpatterns.IsInstanceOfClass;
 import com.google.errorprone.bugpatterns.IterableAndIterator;
 import com.google.errorprone.bugpatterns.IterablePathParameter;
-import com.google.errorprone.bugpatterns.JMockTestWithoutRunWithOrRuleAnnotation;
 import com.google.errorprone.bugpatterns.JUnit3FloatingPointComparisonWithoutDelta;
 import com.google.errorprone.bugpatterns.JUnit3TestNotRun;
 import com.google.errorprone.bugpatterns.JUnit4ClassAnnotationNonStatic;
@@ -1049,7 +1048,6 @@ public static ScannerSupplier errorChecks() {
           InterfaceWithOnlyStatics.class,
           InterruptedExceptionSwallowed.class,
           IterablePathParameter.class,
-          JMockTestWithoutRunWithOrRuleAnnotation.class,
           Java7ApiChecker.class,
           Java8ApiChecker.class,
           LambdaFunctionalInterface.class,

diff --git a/...t/java/com/google/errorprone/bugpatterns/JMockTestWithoutRunWithOrRuleAnnotationTest.java b/...t/java/com/google/errorprone/bugpatterns/JMockTestWithoutRunWithOrRuleAnnotationTest.java
diff --git a/docs/bugpattern/JMockTestWithoutRunWithOrRuleAnnotation.md b/docs/bugpattern/JMockTestWithoutRunWithOrRuleAnnotation.md