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

Dawid Weiss commented on LUCENE-5622:
-------------------------------------

I've added a patch that does *not* check for sysouts by default -- this would 
be very annoying for people using IDEs. The ant test also doesn't check for 
these by default (to simplify the build file in case of filtering, etc.). You 
can turn it on by passing -Dtests.sysouts=no to ant (we could modify jenkins 
runs to do it, perhaps).

Again, didn't check Solr at all. There's still something odd going on with 
those nested tests but they did build over 5 times for me now so we'll see how 
it goes.

> Fail tests if they print, and tests.verbose is not set
> ------------------------------------------------------
>
>                 Key: LUCENE-5622
>                 URL: https://issues.apache.org/jira/browse/LUCENE-5622
>             Project: Lucene - Core
>          Issue Type: Bug
>            Reporter: Robert Muir
>            Assignee: Dawid Weiss
>         Attachments: LUCENE-5622.patch, LUCENE-5622.patch, LUCENE-5622.patch, 
> LUCENE-5622.patch
>
>
> Some tests print so much stuff they are now undebuggable (see LUCENE-5612).
> I think its bad that the testrunner hides this stuff, we used to stay on top 
> of it. Instead, whne tests.verbose is false, we should install a printstreams 
> (system.out/err) that fail the test instantly because they are noisy. 
> This will ensure that our tests don't go out of control.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to