[SUREFIRE-2166] Use ChoiceFormat to selectively render percentage and elapsed time in SurefireReportRenderer

• 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: Maven Surefire Report Plugin
• Assignee: Michael Osipov
• Reporter: Michael Osipov
• Created: 24/May/23 19:23
• Updated: 11/Jun/23 15:19
• Resolved: 28/May/23 09:48

Description

The changes applied in SUREFIRE-2153 were in complete in terms of output formatting. It was done halfheartedly. This ticket aims to improve this:

• Get rid of NumberFormat and use choice format with the resource bundles where the former will do all necessary magic for us
• Selective scale the percentage to one decimal position if the value is neither 0 nor 100, both will display as integer value only
• Scale the elapsed time of a test set to exactly four digits including fractional values which improves the harmony of readability (see attachment) and with a increasing decimal position (times 10) one decimal (fractional) position is displayed less, e.g., no one cares about microseconds if your test runs hundreds of seconds.

Note: Though the attached file refers to log output, the formatting is consistent with the report and aims to unify both renderings.

Issue Links

• causes: SUREFIRE-2170 Use ChoiceFormat to selectively render elapsed time in WrappedReportEntry
• is caused by: SUREFIRE-2153 Replace SurefireReportGenerator with a new SurefireReportRenderer
• relates to: MINVOKER-345 Use ChoiceFormat to selectively render percentage and elapsed time in InvokerReportRenderer
• links to: GitHub Pull Request #639