Don't fire PreferJavaTimeOverload on any static methods named assertT…

…hat or assumeThat. Fixes google#1437 and google#1435 ------------- Created by MOE: https://github.com/google/moe MOE_MIGRATED_REVID=284455194
HubSpot · Feb 19, 2020 · 7e50c52 · 7e50c52
1 parent c662a6e
commit 7e50c52
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/core/src/main/java/com/google/errorprone/bugpatterns/time/PreferJavaTimeOverload.java b/core/src/main/java/com/google/errorprone/bugpatterns/time/PreferJavaTimeOverload.java
@@ -111,7 +111,8 @@ public final class PreferJavaTimeOverload extends BugChecker
 
   private static final Matcher<ExpressionTree> IGNORED_APIS =
       anyOf(
-          staticMethod().onClass("org.assertj.core.api.Assertions").named("assertThat"));
+          // For AssertJ
+          staticMethod().anyClass().namedAnyOf("assertThat", "assumeThat"));
 
   private static final Matcher<ExpressionTree> JAVA_DURATION_DECOMPOSITION_MATCHER =
       anyOf(