junit: treat test methods in abstract test cases as tests (IDEA-174710)

java/java-impl/src/com/intellij/testIntegration/JavaTestFramework.java

@@ -172,10 +172,6 @@ public abstract class JavaTestFramework implements TestFramework {
     return isTestMethod(element, true);
   }
 
-  public boolean isTestMethod(PsiElement element, boolean checkAbstract) {
-    return isTestMethod(element);
-  }
-
   public boolean isMyConfigurationType(ConfigurationType type) {
     return false;
   }

platform/core-api/src/com/intellij/testIntegration/TestFramework.java

@@ -73,6 +73,10 @@ public interface TestFramework {
    * should be checked for abstract method error
    */
   boolean isTestMethod(PsiElement element);
+  
+  default boolean isTestMethod(PsiElement element, boolean checkAbstract) {
+    return isTestMethod(element);
+  }
 
   @NotNull
   Language getLanguage();

plugins/InspectionGadgets/InspectionGadgetsAnalysis/src/com/siyeh/ig/psiutils/TestUtils.java

@@ -65,7 +65,7 @@ public class TestUtils {
     final PsiClass containingClass = method.getContainingClass();
     if (containingClass == null) return false;
     final TestFramework framework = TestFrameworks.detectFramework(containingClass);
-    return framework != null && framework.getName().startsWith("JUnit") && framework.isTestMethod(method);
+    return framework != null && framework.getName().startsWith("JUnit") && framework.isTestMethod(method, false);
   }
 
   public static boolean isRunnable(PsiMethod method) {

plugins/InspectionGadgets/test/com/siyeh/igtest/junit/test_case_with_no_test_methods/TestCaseWithNoTestMethods.java

@@ -16,4 +16,10 @@ public class <warning descr="JUnit test case 'TestCaseWithNoTestMethods' has no
   public void tearDown() throws Exception {
     super.tearDown();
   }
-}
+}
+
+abstract class AbstractTest extends junit.framework.TestCase {
+  public void testInAbstract() {}
+}
+
+class MyImplTest extends AbstractTest {}

plugins/InspectionGadgets/testsrc/com/siyeh/ig/junit/TestCaseWithNoTestMethodsInspectionTest.java

@@ -29,7 +29,9 @@ public class TestCaseWithNoTestMethodsInspectionTest extends LightInspectionTest
   @Nullable
   @Override
   protected InspectionProfileEntry getInspection() {
-    return new TestCaseWithNoTestMethodsInspection();
+    TestCaseWithNoTestMethodsInspection inspection = new TestCaseWithNoTestMethodsInspection();
+    inspection.ignoreSupers = true;
+    return inspection;
   }
 
   @Override