You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Michael Osipov (Jira)" <ji...@apache.org> on 2023/05/28 09:47:00 UTC

[jira] [Closed] (SUREFIRE-2167) Simplify deserialization of elapsed time in TestSuiteXmlParser

     [ https://issues.apache.org/jira/browse/SUREFIRE-2167?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Michael Osipov closed SUREFIRE-2167.
------------------------------------
    Resolution: Fixed

Fixed with [9e971d20c82f4c222b0bd8cb34f646433e00252b|https://gitbox.apache.org/repos/asf?p=maven-surefire.git;a=commit;h=9e971d20c82f4c222b0bd8cb34f646433e00252b].

> Simplify deserialization of elapsed time in TestSuiteXmlParser
> --------------------------------------------------------------
>
>                 Key: SUREFIRE-2167
>                 URL: https://issues.apache.org/jira/browse/SUREFIRE-2167
>             Project: Maven Surefire
>          Issue Type: Improvement
>    Affects Versions: 3.1.0
>            Reporter: Michael Osipov
>            Assignee: Michael Osipov
>            Priority: Major
>             Fix For: 3.1.1
>
>
> The current code uses a convoluted solution based on {{NumberFormat}} which does not even correspond to the serialization code in {{StatelessXmlReporter}}. These days, we can use {{(Float#parseFloat()}} to deserialize the string representation of a float value since:
> * Easier to understand and this is not human output (separation of concerns)
> * It is much stricter than {{NumberFormat}} which ignores multiple decimal separators (This issue caused SUREFIRE-2165)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)