[SUREFIRE-2242] Plain test report does not include names of the skipp…

…ed tests This closes #752
apache · Jun 27, 2024 · 6ff0f83 · 6ff0f83
1 parent 3a793bb
commit 6ff0f83
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/...n-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/TestSetStats.java b/...n-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/TestSetStats.java
@@ -227,7 +227,7 @@ public List<String> getTestResults() {
             if (testResult.isErrorOrFailure()) {
                 result.add(testResult.getOutput(trimStackTrace));
             } else if (plainFormat && testResult.isSkipped()) {
-                result.add(testResult.getSourceName() + " skipped");
+                result.add(testResult.getFullName() + " skipped");
             } else if (plainFormat && testResult.isSucceeded()) {
                 result.add(testResult.getElapsedTimeSummary());
             }

diff --git a/...fire-common/src/main/java/org/apache/maven/plugin/surefire/report/WrappedReportEntry.java b/...fire-common/src/main/java/org/apache/maven/plugin/surefire/report/WrappedReportEntry.java
@@ -130,6 +130,10 @@ public String getClassMethodName() {
         return original.getSourceName() + "." + original.getName();
     }
 
+    public String getFullName() {
+        return getName() == null ? getSourceName() : getClassMethodName();
+    }
+
     @Override
     public String getGroup() {
         return original.getGroup();
@@ -186,8 +190,7 @@ public String getElapsedTimeVerbose() {
     }
 
     public String getElapsedTimeSummary() {
-        String description = getName() == null ? getSourceName() : getClassMethodName();
-        return description + " -- " + getElapsedTimeVerbose();
+        return getFullName() + " -- " + getElapsedTimeVerbose();
     }
 
     public boolean isErrorOrFailure() {