[ 
https://issues.apache.org/jira/browse/SUREFIRE-2224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17799624#comment-17799624
 ] 

ASF GitHub Bot commented on SUREFIRE-2224:
------------------------------------------

kriegaex commented on PR #702:
URL: https://github.com/apache/maven-surefire/pull/702#issuecomment-1867103417

   @michael-o, actually, I am a bit out of context here and not 100% sure what 
that code does and currently am busy otherwise, i.e. I am just looking at the 
code here on the website, which IMO is **not** a good way to conduct a proper 
review. You have to check out the code and see what it does, inspect/run tests 
covering the code etc.
   
   It looks as if in case of reported test errors or failures, it adds 
something like an "error" or "failure" tag to the XML and then, if any stack 
trace exists, also a "stackTrace" element and "system-out"/"system-err".
   
   Am I seeing correctly, that you just refactored the code with no functional 
changes, only reflecting the nesting structure of the XML output in the logical 
structure (`if` block) of the source code? If so, it looks good to me.




> StatelessXmlReporter#getTestProblems() does not properly reflect report 
> schema structure
> ----------------------------------------------------------------------------------------
>
>                 Key: SUREFIRE-2224
>                 URL: https://issues.apache.org/jira/browse/SUREFIRE-2224
>             Project: Maven Surefire
>          Issue Type: Bug
>          Components: xml generation
>    Affects Versions: 3.2.3
>            Reporter: Michael Osipov
>            Assignee: Michael Osipov
>            Priority: Major
>             Fix For: 3.2.4
>
>
> According to 
> https://maven.apache.org/surefire/maven-surefire-plugin/xsd/surefire-test-report.xsd
>  and 
> https://maven.apache.org/surefire/maven-failsafe-plugin/xsd/failsafe-test-report.xsd
>  not all nested elements contain {{stackTrace}}, {{system-out}} and 
> {{system-err}} elements, but the code makes it conditional whether there is a 
> stack trace or not. This does not comply with the schema definition. Either 
> the nested element is simple content or complex. In the latter {{stackTrace}} 
> is mandatory, even if there is no strack trace according to the schema. The 
> system streams are optional.



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

Reply via email to