Github user surli commented on a diff in the pull request:

    https://github.com/apache/maven-surefire/pull/143#discussion_r138012247
  
    --- Diff: 
surefire-report-parser/src/main/java/org/apache/maven/plugins/surefire/report/TestSuiteXmlParser.java
 ---
    @@ -196,7 +196,8 @@ else if ( "failure".equals( qName ) )
                     }
                     else if ( "error".equals( qName ) )
                     {
    -                    testCase.setFailure( attributes.getValue( "message" ), 
attributes.getValue( "type" ) );
    +                    testCase.setFailure( attributes.getValue( "message" ), 
attributes.getValue( "type" ) )
    --- End diff --
    
    > If having an error, then failure and error are set both. Why?
    
    An error is a specific kind of failure: if I start to introduce error 
details, then I should also add error type, message, line, and I'll got every 
field of `ReportTestCase` duplicated. My purpose was to keep the field of 
`ReportTestCase` to report the information around the failure, but to be able 
to flag it as an error instead of a failure. 


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org

Reply via email to