You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by mi...@apache.org on 2023/05/22 19:04:24 UTC
[maven-surefire] 01/01: fix SAX bug
This is an automated email from the ASF dual-hosted git repository.
michaelo pushed a commit to branch fix-sax-bug
in repository https://gitbox.apache.org/repos/asf/maven-surefire.git
commit 676c9bde96cfac2ca05a982740ed4467b1649adb
Author: Michael Osipov <mi...@apache.org>
AuthorDate: Mon May 22 19:06:39 2023 +0200
fix SAX bug
---
.../apache/maven/plugins/surefire/report/TestSuiteXmlParser.java | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/surefire-report-parser/src/main/java/org/apache/maven/plugins/surefire/report/TestSuiteXmlParser.java b/surefire-report-parser/src/main/java/org/apache/maven/plugins/surefire/report/TestSuiteXmlParser.java
index 83ccc0619..481434b28 100644
--- a/surefire-report-parser/src/main/java/org/apache/maven/plugins/surefire/report/TestSuiteXmlParser.java
+++ b/surefire-report-parser/src/main/java/org/apache/maven/plugins/surefire/report/TestSuiteXmlParser.java
@@ -131,8 +131,6 @@ public final class TestSuiteXmlParser extends DefaultHandler {
classesToSuitesIndex.put(defaultSuite.getFullClassName(), suites.size() - 1);
break;
case "testcase":
- currentElement = new StringBuilder();
-
testCase = new ReportTestCase().setName(attributes.getValue("name"));
String fullClassName = attributes.getValue("classname");
@@ -162,10 +160,14 @@ public final class TestSuiteXmlParser extends DefaultHandler {
}
break;
case "failure":
+ currentElement = new StringBuilder();
+
testCase.setFailure(attributes.getValue("message"), attributes.getValue("type"));
currentSuite.incrementNumberOfFailures();
break;
case "error":
+ currentElement = new StringBuilder();
+
testCase.setError(attributes.getValue("message"), attributes.getValue("type"));
currentSuite.incrementNumberOfErrors();
break;
@@ -181,6 +183,9 @@ public final class TestSuiteXmlParser extends DefaultHandler {
case "failsafe-summary":
valid = false;
break;
+ case "time":
+ currentElement = new StringBuilder();
+ break;
default:
break;
}