JUnit3 test methods in fact _can_ be static (and will still get run by the test runner!).

PiperOrigin-RevId: 914692547

Author: kluever

check_api/src/main/java/com/google/errorprone/matchers/JUnitMatchers.java

@@ -155,7 +155,6 @@ private static boolean hasJUnitAttr(MethodSymbol methodSym) {
    *   <li>The method's name begins with "test".
    *   <li>The method has no parameters.
    *   <li>The method is public.
-   *   <li>The method is not static.
    *   <li>The method returns void.
    * </ol>
    */
@@ -164,7 +163,6 @@ private static boolean hasJUnitAttr(MethodSymbol methodSym) {
           methodNameStartsWith("test"),
           methodHasNoParameters(),
           Matchers.<MethodTree>hasModifier(Modifier.PUBLIC),
-          not(Matchers.<MethodTree>hasModifier(Modifier.STATIC)),
           methodReturns(VOID_TYPE));
 
   /** Common matcher for possible JUnit setUp/tearDown methods. */

core/src/test/java/com/google/errorprone/bugpatterns/JUnit3TestNotRunTest.java

@@ -215,15 +215,7 @@ public class KansasGlobalMetadataUtilsTest extends TestCase {
               public static void testParseGlobalMetadata() {}
             }
             """)
-        .addOutputLines(
-            "out/KansasGlobalMetadataUtilsTest.java",
-            """
-            import junit.framework.TestCase;
-
-            public class KansasGlobalMetadataUtilsTest extends TestCase {
-              public void testParseGlobalMetadata() {}
-            }
-            """)
+        .expectUnchanged()
         .doTest();
   }