Uploaded image for project: 'Maven Surefire'
  1. Maven Surefire
  2. SUREFIRE-2164

Simplify serialization of elapsed time in StatelessXmlReporter

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 3.1.0
    • 3.1.1, 3.1.2
    • xml generation
    • None

    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.

      Attachments

        Issue Links

          causes

          Task - A task that needs to be done. SUREFIRE-2175 Turn time attribute to xs:float in Surefire test report schema

          • Major - Major loss of function.
          • Closed
          is depended upon by

          Bug - A problem which impairs or prevents the functions of the product. SUREFIRE-2176 Invalid formatting of time attribute

          • Major - Major loss of function.
          • Closed
          is required by

          Improvement - An improvement or enhancement to an existing feature or task. SUREFIRE-1533 surefire-test-report.xsd has too narrow a definition for testcase time attribute

          • Major - Major loss of function.
          • Closed
          relates to

          Improvement - An improvement or enhancement to an existing feature or task. MINVOKER-341 Make elapsed time field type consistent with Maven Surefire

          • Major - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. SUREFIRE-2167 Simplify deserialization of elapsed time in TestSuiteXmlParser

          • Major - Major loss of function.
          • Closed
          links to

          Web Link GitHub Pull Request #645

          Activity

            People

              michael-o Michael Osipov
              michael-o Michael Osipov
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: