Patrick Hunt commented on ZOOKEEPER-654:

it's difficult to test this particular functionality (ie verifying we log 
something in certain cases) without this approach. 

Are you saying we should have a dedicated log4j configuration file as part of 
test, rather than relying on the one in conf?
That would be fine w/me.

> Tests should not rely on CONSOLE appender being present
> -------------------------------------------------------
>                 Key: ZOOKEEPER-654
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-654
>             Project: Zookeeper
>          Issue Type: Bug
>    Affects Versions: 3.2.1
>            Reporter: Henry Robinson
>            Priority: Minor
> Tests have been failing for us in an environment where we removed the CONSOLE 
> appender from log4j. This breaks a couple of tests in QuorumPeerMainTest at 
> least.
> I have fixed in our builds by replacing CONSOLE with ROLLINGFILE (which we 
> are using) for the time being, but messing with the log config shouldn't 
> break tests. 

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

Reply via email to