tombentley commented on pull request #8808: URL: https://github.com/apache/kafka/pull/8808#issuecomment-642277120
@dajac I tried two different ways of testing this: 1. Capturing standard output as you suggest. For this to work I need to change the test `log4j.properties` so that at least the `org.apache.kafka.common.utils.AppInfoParser` is at INFO. Obviously this stands to affect all tests, though since gradle redirects output it wouldn't be troublesome on the command line. 2. Using the `LogCaptureAppender`, which means I don't need to adjust the `log4j.properties`, but the test still needs to know before hand which classes are the possible emitters of error/warnings messages which ought not to be emitted. Any preference? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
