michael-o commented on PR #702:
URL: https://github.com/apache/maven-surefire/pull/702#issuecomment-1867589786

   > @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.
   
   Just refactoring and abiding to the XML schemas: 
https://maven.apache.org/surefire/maven-surefire-plugin/xsd/surefire-test-report.xsd
 and 
https://maven.apache.org/surefire/maven-surefire-plugin/xsd/surefire-test-report.xsd


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to