(maven-surefire) 01/01: [SUREFIRE-2242] Plain test report does not include names of the skipped tests

[michaelo]

This is an automated email from the ASF dual-hosted git repository.

michaelo pushed a commit to branch SUREFIRE-2242
in repository https://gitbox.apache.org/repos/asf/maven-surefire.git

commit 6ff0f83ea1c9427a88d42b3a8c1b7bdb9e5e3fcb
Author: Michael Osipov <micha...@apache.org>
AuthorDate: Wed Jun 26 23:25:22 2024 +0200

    [SUREFIRE-2242] Plain test report does not include names of the skipped 
tests
    
    This closes #752
---
 .../java/org/apache/maven/plugin/surefire/report/TestSetStats.java | 2 +-
 .../apache/maven/plugin/surefire/report/WrappedReportEntry.java    | 7 +++++--
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git 
a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/TestSetStats.java
 
b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/TestSetStats.java
index 96b4fb451..1431d8fde 100644
--- 
a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/TestSetStats.java
+++ 
b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/TestSetStats.java
@@ -227,7 +227,7 @@ public class TestSetStats {
             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/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/WrappedReportEntry.java
 
b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/WrappedReportEntry.java
index 606936415..f661a003f 100644
--- 
a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/WrappedReportEntry.java
+++ 
b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/WrappedReportEntry.java
@@ -130,6 +130,10 @@ public class WrappedReportEntry implements 
TestSetReportEntry {
         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 class WrappedReportEntry implements 
TestSetReportEntry {
     }
 
     public String getElapsedTimeSummary() {
-        String description = getName() == null ? getSourceName() : 
getClassMethodName();
-        return description + " -- " + getElapsedTimeVerbose();
+        return getFullName() + " -- " + getElapsedTimeVerbose();
     }
 
     public boolean isErrorOrFailure() {