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

Dawid Weiss commented on SOLR-15815:
------------------------------------

Yup, the approach looks fine to me. The only downside I see is that rules 
defined this way would trigger for all tests (even if they don't use the rule). 
Perhaps the logic inside that rule should be a lazy/ deferred evaluation (so 
that if get() is called, it'll clean up the returned instance but otherwise is 
a noop).

> Add TestRule support for ErrorLogMuter & LogListener
> ----------------------------------------------------
>
>                 Key: SOLR-15815
>                 URL: https://issues.apache.org/jira/browse/SOLR-15815
>             Project: Solr
>          Issue Type: Sub-task
>            Reporter: Chris M. Hostetter
>            Priority: Major
>         Attachments: SOLR-15815.patch
>
>
> sibling sub-tasks implement log muting & inspection helpers that work by 
> implementing AutoClosable, as discussed in parent task we should also expose 
> this functionality in a TestRule



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org

Reply via email to