Skip abstract classes and interfaces via JavaParser

Author: Donnerbart

src/main/java/de/donnerbart/split/TestSplit.java

@@ -193,11 +193,15 @@ public void run() throws Exception {
                 final var compilationUnit = javaParser.parse(testPath).getResult().orElseThrow();
                 final var declaration = compilationUnit.findFirst(ClassOrInterfaceDeclaration.class).orElseThrow();
                 final var className = declaration.getFullyQualifiedName().orElseThrow();
-                if (declaration.getAnnotations()
+                if (declaration.isInterface()) {
+                    LOG.info("Skipping test interface {}", className);
+                } else if (declaration.isAbstract()) {
+                    LOG.info("Skipping abstract test class {}", className);
+                } else if (declaration.getAnnotations()
                         .stream()
                         .map(AnnotationExpr::getNameAsString)
                         .anyMatch(SKIP_TEST_ANNOTATIONS::contains)) {
-                    LOG.info("Skipping disabled test {}", className);
+                    LOG.info("Skipping disabled test class {}", className);
                 } else {
                     classNames.add(className);
                 }

src/main/java/de/donnerbart/split/TestSplitMain.java

@@ -28,7 +28,6 @@ public static void main(final @Nullable String @NotNull [] args) throws Exceptio
             System.out.println("--split-index must lesser than --split-total");
             System.exit(1);
         }
-        final var excludeGlob = arguments.excludeGlob != null ? arguments.excludeGlob : "**/*Abstract*";
         final var workingDirectory = arguments.workingDirectory != null ?
                 arguments.workingDirectory.toAbsolutePath() :
                 Paths.get(System.getProperty("user.dir")).toAbsolutePath();
@@ -44,7 +43,7 @@ public static void main(final @Nullable String @NotNull [] args) throws Exceptio
         final var testSplit = new TestSplit(arguments.splitIndex,
                 arguments.splitTotal,
                 arguments.glob,
-                excludeGlob,
+                arguments.excludeGlob,
                 arguments.junitGlob,
                 workingDirectory,
                 arguments.debug,

src/test/java/de/donnerbart/split/TestSplitTest.java

@@ -43,7 +43,9 @@ void setUp() throws Exception {
                 .resolve("donnerbart")
                 .resolve("example");
         Files.createDirectories(projectFolder);
+        copyResourceToTarget(projectFolder, "tests/BaseTest.java", "BaseTest.java", PERMISSIONS);
         copyResourceToTarget(projectFolder, "tests/DisabledTest.java", "DisabledTest.java", PERMISSIONS);
+        copyResourceToTarget(projectFolder, "tests/InterfaceTest.java", "InterfaceTest.java", PERMISSIONS);
         copyResourceToTarget(projectFolder, "tests/FastTest.java", "FastTest.java", PERMISSIONS);
         copyResourceToTarget(projectFolder, "tests/NoTimingOneTest.java", "NoTimingOneTest.java", PERMISSIONS);
         copyResourceToTarget(projectFolder, "tests/NoTimingTwoTest.java", "NoTimingTwoTest.java", PERMISSIONS);

src/test/resources/tests/BaseTest.java

@@ -0,0 +1,6 @@
+package de.donnerbart.example;
+
+abstract class BaseTest {
+
+    abstract void implementableMethod();
+}

src/test/resources/tests/InterfaceTest.java

@@ -0,0 +1,6 @@
+package de.donnerbart.example;
+
+interface BaseTest {
+
+    void implementableMethod();
+}