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

Alexander Kriegisch edited comment on SUREFIRE-2219 at 12/3/23 3:24 PM:
------------------------------------------------------------------------

[~michael-o], that was not my plan, and HTML layout, CSS and such is not my 
strong suit. Probably the code review would take 7x longer than the 
implementation. Most reviews are not done in a collaborative approach but by 
someone telling me what they don't like and urging me to do something they 
could have done much more quickly by themselves by just committing on top of 
the PR. I do not have time for this at the moment. Maybe if you offer a pair 
programming session with live destop sharing and audio connection. If it was 
something algorithmic only without any GUI or shiny output and a certain 
probability for a smooth review, maybe. But I can test, if someone contributes 
the feature.


was (Author: kriegaex):
[~michael-o], that was not my plan, and HTML layout, CSS and such is not my 
strong suit. Probably the code review would take 7x longer than the 
implementation. Most reviews are not done in a collaborative approach but by 
someone telling me what they don't like and urging me to do something they 
could have done much more quickly but just committing on top of the PR. I do 
not have time for this at the moment. Maybe if you offer a pair programming 
session with live destop sharing and audio connection. If it was something 
algorithmic only without any GUI or shiny output and a certain probability for 
a smooth review, maybe. But I can test, if someone contributes the feature.

> Option to include stdOut/stdErr console logs into test report
> -------------------------------------------------------------
>
>                 Key: SUREFIRE-2219
>                 URL: https://issues.apache.org/jira/browse/SUREFIRE-2219
>             Project: Maven Surefire
>          Issue Type: New Feature
>          Components: Maven Surefire Report Plugin
>    Affects Versions: 3.2.2
>            Reporter: Alexander Kriegisch
>            Priority: Major
>         Attachments: image-2023-12-02-11-16-11-109.png
>
>
> Surefire report XML includes tags capturing console logs, e.g. 
> {{testsuite/testcase/system-err}}:
>  !image-2023-12-02-11-16-11-109.png! 
> As a report reader, I want to see the stdOut/stdErr console logs per test 
> case in the HTML report, ideally in a monotype font to easily be able to read 
> stack traces.
> This requirement has been discussed online a lot, e.g. on Stack Overflow or 
> Reddit. I wonder why such a feature is missing. The HTML report basically 
> being an XML transformation from the surefire report, it should not be super 
> complicated to implement.



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

Reply via email to