Tibor17 commented on code in PR #3200:
URL: https://github.com/apache/maven-surefire/pull/3200#discussion_r2527913839
##########
maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/StatelessXmlReporter.java:
##########
@@ -161,7 +161,7 @@ public void testSetCompleted(WrappedReportEntry
testSetReportEntry, TestSetStats
OutputStreamWriter fw = getWriter(outputStream)) {
XMLWriter ppw = new PrettyPrintXMLWriter(new PrintWriter(fw),
XML_INDENT, XML_NL, UTF_8.name(), null);
- createTestSuiteElement(ppw, testSetReportEntry, testSetStats); //
TestSuite
+ createTestSuiteElement(ppw, testSetReportEntry,
classMethodStatistics); // TestSuite
Review Comment:
Where this `TestSetStats` come from? Is it the listener event invoked by the
test-provider?
Because, logically this `TestSetStats` makes sense.
Alright, we can accept this, we should consider this as a workaround because
the real fix has to be in the Providers and it is not that easy and definitely
it is not one PR.
I want to continue with the Roadmap in 3.0.0 milestones which solves
multiple issues, and this one too, but it is so complex change that it requires
multiple stable PRs promoted one by one.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
