[SUREFIRE-2164] Simplify serialization of elapsed time in StatelessXmlReporter

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.1.0
• Fix Version/s: 3.1.1, 3.1.2
• Component/s: xml generation
• Assignee: Michael Osipov
• Reporter: Michael Osipov
• Created: 22/May/23 07:12
• Updated: 11/Jun/23 11:48
• Resolved: 28/May/23 09:46

Description

The current code uses a convoluted solution based on NumberFormat which refers to an outdated defintion on Ant's XML report file. These days, we can use String#valueOf() to serialize the string representation of a float value since:

• Easier to understand and this is not human output (separation of concerns)
• Ant's XMLJUnitResultFormatter does exactly the same with a double value.

Issue Links

• causes: SUREFIRE-2175 Turn time attribute to xs:float in Surefire test report schema
• is depended upon by: SUREFIRE-2176 Invalid formatting of time attribute
• is required by: SUREFIRE-1533 surefire-test-report.xsd has too narrow a definition for testcase time attribute
• relates to: MINVOKER-341 Make elapsed time field type consistent with Maven Surefire
• relates to: SUREFIRE-2167 Simplify deserialization of elapsed time in TestSuiteXmlParser
• links to: GitHub Pull Request #645