Refactor output formatting; enhance module header display and adjust indentation settings

Author: oubiwann

src/main/java/io/github/clojang/sophodromos/OutputPatternMatcher.java

@@ -125,7 +125,7 @@ protected String tryMatchMethodExecution(final String line, final TestOutputForm
       final String methodWithParens = methodName + "()";
       result =
           formatter.formatTestResult(
-              className, methodWithParens, "SUCCESS", (long) (timeElapsed * 1000));
+              "  " + className, methodWithParens, "SUCCESS", (long) (timeElapsed * 1000));
     }
     return result;
   }
@@ -179,15 +179,15 @@ protected String tryMatchTestResults(
    */
   protected String formatModuleHeader(
       final String artifactId, final TestOutputFormatter formatter) {
-    final String moduleHeader = colors.colorize(
-            artifactId, AnsiColors.BOLD + AnsiColors.BRIGHT_YELLOW);
-    return formatter.formatProgressLine(moduleHeader);
+    return colors.colorize(artifactId, AnsiColors.BOLD + AnsiColors.BRIGHT_YELLOW);
   }
 
   private String getSimpleClassName(final String testClass) {
     return testClass.substring(testClass.lastIndexOf('.') + 1);
   }
 
+  @SuppressWarnings("PMD.UnusedFormalParameter")
+  // Parameters intentionally unused in current implementation
   private String formatTestResults(
       final int testsRun,
       final int failures,
@@ -197,27 +197,27 @@ private String formatTestResults(
       final TestOutputFormatter formatter) {
     final StringBuilder result = new StringBuilder();
 
-    //    if (failures == 0 && errors == 0) {
-    //      result.append("✅ ");
-    //    } else {
-    //      result.append("❌ ");
-    //    }
+    // if (failures == 0 && errors == 0) {
+    //   result.append("✅ ");
+    // } else {
+    //   result.append("❌ ");
+    // }
 
-    //result.append(String.format("Tests: %d", testsRun));
+    // result.append(String.format("Tests: %d", testsRun));
 
-    //if (failures > 0) {
-    //  result.append(String.format(", Failures: %d", failures));
-    //}
+    // if (failures > 0) {
+    //   result.append(String.format(", Failures: %d", failures));
+    // }
 
-    //if (errors > 0) {
-    //  result.append(String.format(", Errors: %d", errors));
-    //}
+    // if (errors > 0) {
+    //   result.append(String.format(", Errors: %d", errors));
+    // }
 
-    //if (skipped > 0) {
-    //  result.append(String.format(", Skipped: %d", skipped));
-    //}
+    // if (skipped > 0) {
+    //   result.append(String.format(", Skipped: %d", skipped));
+    // }
 
-    //result.append(String.format(" (%.3fs)", timeElapsed));
+    // result.append(String.format(" (%.3fs)", timeElapsed));
 
     return formatter.formatProgressLine(result.toString());
   }

src/main/java/io/github/clojang/sophodromos/SophoDromosTestMojo.java

@@ -121,7 +121,8 @@ private void displayHeader() {
   @SuppressWarnings("PMD.SystemPrintln") // Intentional console output for clean formatting
   private void displayModuleHeader() {
     final String artifactId = project.getArtifactId();
-    final String moduleHeader = ":" + artifactId + ":test";
+    final OutputPatternMatcher patternMatcher = new OutputPatternMatcher(formatter.getColors());
+    final String moduleHeader = patternMatcher.formatModuleHeader(artifactId, formatter);
     System.out.println(moduleHeader);
   }
 

src/main/java/io/github/clojang/sophodromos/TestResultFormatter.java

@@ -7,8 +7,8 @@
 /** Handles individual test result formatting. */
 @SuppressWarnings("PMD.TestClassWithoutTestCases") // This is not a test class
 class TestResultFormatter {
-  private static final String INDENT = "    ";
-  private static final int DOTS_BASE = 76;
+  private static final String INDENT = "";
+  private static final int DOTS_BASE = 78;
 
   private final AnsiColors colors;
   private final GradlDromusExtension extension;