You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by gg...@apache.org on 2016/09/23 22:55:31 UTC
logging-log4j2 git commit: Better assert message.
Repository: logging-log4j2
Updated Branches:
refs/heads/master fc2a2edd7 -> e58fab172
Better assert message.
Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo
Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/e58fab17
Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/e58fab17
Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/e58fab17
Branch: refs/heads/master
Commit: e58fab172feb3ad828f0b3fa7c399794088a2f7a
Parents: fc2a2ed
Author: Gary Gregory <gg...@apache.org>
Authored: Fri Sep 23 15:55:30 2016 -0700
Committer: Gary Gregory <gg...@apache.org>
Committed: Fri Sep 23 15:55:30 2016 -0700
----------------------------------------------------------------------
.../config/Log4j1ConfigurationFactoryTest.java | 28 +++++++++++---------
1 file changed, 15 insertions(+), 13 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/e58fab17/log4j-1.2-api/src/test/java/org/apache/log4j/config/Log4j1ConfigurationFactoryTest.java
----------------------------------------------------------------------
diff --git a/log4j-1.2-api/src/test/java/org/apache/log4j/config/Log4j1ConfigurationFactoryTest.java b/log4j-1.2-api/src/test/java/org/apache/log4j/config/Log4j1ConfigurationFactoryTest.java
index cb58dbe..3e650ee 100644
--- a/log4j-1.2-api/src/test/java/org/apache/log4j/config/Log4j1ConfigurationFactoryTest.java
+++ b/log4j-1.2-api/src/test/java/org/apache/log4j/config/Log4j1ConfigurationFactoryTest.java
@@ -47,19 +47,21 @@ import org.junit.Test;
public class Log4j1ConfigurationFactoryTest {
- private Layout<?> testConsole(final String configResource) throws Exception {
- final Configuration configuration = configure(configResource);
- final ConsoleAppender appender = configuration.getAppender("Console");
- assertNotNull(appender);
- assertEquals(Target.SYSTEM_ERR, appender.getTarget());
- //
- final LoggerConfig loggerConfig = configuration.getLoggerConfig("com.example.foo");
- assertNotNull(loggerConfig);
- assertEquals(Level.DEBUG, loggerConfig.getLevel());
- configuration.start();
- configuration.stop();
- return appender.getLayout();
- }
+ private Layout<?> testConsole(final String configResource) throws Exception {
+ final Configuration configuration = configure(configResource);
+ final String name = "Console";
+ final ConsoleAppender appender = configuration.getAppender(name);
+ assertNotNull("Missing appender '" + name + "' in configuration " + configResource + " \u2192 " + configuration,
+ appender);
+ assertEquals(Target.SYSTEM_ERR, appender.getTarget());
+ //
+ final LoggerConfig loggerConfig = configuration.getLoggerConfig("com.example.foo");
+ assertNotNull(loggerConfig);
+ assertEquals(Level.DEBUG, loggerConfig.getLevel());
+ configuration.start();
+ configuration.stop();
+ return appender.getLayout();
+ }
private Layout<?> testFile(final String configResource) throws Exception {
final Configuration configuration = configure(configResource);