You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by "Patrick Hunt (JIRA)" <ji...@apache.org> on 2010/01/23 00:46:21 UTC
[jira] Commented: (ZOOKEEPER-654) Tests should not rely on CONSOLE
appender being present
[ https://issues.apache.org/jira/browse/ZOOKEEPER-654?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12803945#action_12803945 ]
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.