wx930910 opened a new pull request #584: URL: https://github.com/apache/logging-log4j2/pull/584
### Jira - [LOG4J2-3164](https://issues.apache.org/jira/browse/LOG4J2-3164) ### Description #### Replace test class [StoringStatusListener](https://github.com/apache/logging-log4j2/blob/20f9a97dbe5928c3b5077bcdd2a22ac92e941655/log4j-core/src/test/java/org/apache/logging/log4j/core/config/plugins/validation/validators/ValidatingPluginWithFailoverTest.java#L86) by mocking object and improve test design. <hr> ##### Motivation * Decouple test class `StoringStatusListener` from production interface `StatusListener`. * Remove the redundant test child class `StoringStatusListener`. * Remove the redundant overridden methods `close()`. * Make testing logic more explict by directly verify the input argument through `ArgumentCapture`. <hr> ##### Key changed/added classes in this PR * Created mocking object to replace test subclass `StoringStatusListener`, decoupled test from production code. * Use ArgumentCapture to replace the new field created in `StoringStatusListener`. <hr> -- 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. To unsubscribe, e-mail: notifications-unsubscr...@logging.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
