[jira] [Commented] (SUREFIRE-1744) Enable system-out for successfuly passed tests as well

["Tibor Digana (Jira)" <ji...@apache.org>]

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

Tibor Digana commented on SUREFIRE-1744:
----------------------------------------

The test should declare the behavior by the assertion statement, that's my opinion. But no problem to accept std out in successful tests. Can you pls open a pullrequest on GH?

> Enable system-out for successfuly passed tests as well
> ------------------------------------------------------
>
>                 Key: SUREFIRE-1744
>                 URL: https://issues.apache.org/jira/browse/SUREFIRE-1744
>             Project: Maven Surefire
>          Issue Type: New Feature
>          Components: Maven Surefire Plugin
>            Reporter: Klevis Ramo
>            Priority: Major
>
> *Situation*
> Currently logs during tests execution are exported to *.xml report only for failed tests.
> The xsd schema also implies the same(surefire-test-report-3.0.xsd,legacy-surefire-test-report.xsd) , *system-out* elment is always a child of failure,rerunFailure,flakyFailure,rerunError or flakyError element.
> Some part of the code :
> {code:java}
> StatelessXmlReporter
>  
> private void serializeTestClassWithoutRerun( OutputStream outputStream, OutputStreamWriter fw, XMLWriter ppw,
>                                              List<WrappedReportEntry> methodEntries )
> {
>     for ( WrappedReportEntry methodEntry : methodEntries )
>     {
>         startTestElement( ppw, methodEntry );
>         if ( methodEntry.getReportEntryType() != SUCCESS )
>         {
>             getTestProblems( fw, ppw, methodEntry, trimStackTrace, outputStream,
>                     methodEntry.getReportEntryType().getXmlTag(), false );
>             createOutErrElements( fw, ppw, methodEntry, outputStream );
>         }
>         ppw.endElement();
>     }
> }{code}
> *Problem*
> Because logs are not exported for successfully tests there is no way(outside IDE or complete log) one could nicely explore what a particular test is doing. This is hurting also the documentation part of a test.Imagine those tests are used as end to end tests and business analysts would like to review through CI tools like jenkins.
>  
> *Possible Solution*
> Enable maven sure fire plugin to export logs to system-out also for succefully passed tests through an optional parameter leaving the default behaviour as it is right now(only failed tests have outputs exported). This means of course also enchancing the xsd to support system-out to different parent rather then only failed ones.
>  
> Since it happens I am also a developer I can of course help with the feature implementation if you find this something that makes sense.
>  
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)