You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by "mcmellawatt (GitHub)" <gi...@apache.org> on 2018/11/27 00:35:40 UTC

[GitHub] [geode] mcmellawatt commented on issue #2896: GEODE-6092: Simplify SingleHopClientExecutorLoggingTest

I think this is good conceptually, but I think we identified that Log4J is storing a static reference to System.out which may differ from the one referenced by SystemOutRule on repeated **unit** test runs.  This was not a problem with **integration** tests since each repeated test will get executed in a new JVM.  Maybe we can compromise for now and delete the unnecessary code in the test, but leave it as an integration test?  Or do you think it worth figuring out how to deal with this interaction between Log4J and SystemOutRule?

[ Full content available at: https://github.com/apache/geode/pull/2896 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org