You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by rg...@apache.org on 2017/08/25 23:53:25 UTC

logging-log4j2 git commit: LOG4J2-2009 unit test log files must go in the target directory

Repository: logging-log4j2
Updated Branches:
  refs/heads/master d69d83e56 -> 3aec69d7a


LOG4J2-2009 unit test log files must go in the target directory


Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo
Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/3aec69d7
Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/3aec69d7
Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/3aec69d7

Branch: refs/heads/master
Commit: 3aec69d7aa6e21e7f0abb21a9bf23bbbfbdca556
Parents: d69d83e
Author: Ralph Goers <rg...@nextiva.com>
Authored: Fri Aug 25 16:53:17 2017 -0700
Committer: Ralph Goers <rg...@nextiva.com>
Committed: Fri Aug 25 16:53:17 2017 -0700

----------------------------------------------------------------------
 .../rolling/RollingFileAppenderUpdateDataTest.java    | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/3aec69d7/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/rolling/RollingFileAppenderUpdateDataTest.java
----------------------------------------------------------------------
diff --git a/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/rolling/RollingFileAppenderUpdateDataTest.java b/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/rolling/RollingFileAppenderUpdateDataTest.java
index dad5948..b73b6b1 100644
--- a/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/rolling/RollingFileAppenderUpdateDataTest.java
+++ b/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/rolling/RollingFileAppenderUpdateDataTest.java
@@ -36,12 +36,12 @@ import org.junit.Test;
 public class RollingFileAppenderUpdateDataTest {
 
     private ConfigurationBuilder<BuiltConfiguration> buildConfigA() {
-        return buildConfigurationBuilder("foo.log.%i");
+        return buildConfigurationBuilder("target/rolling-update-date/foo.log.%i");
     }
 
     // rebuild config with date based rollover
     private ConfigurationBuilder<BuiltConfiguration> buildConfigB() {
-        return buildConfigurationBuilder("foo.log.%d{yyyy-MM-dd-HH:mm:ss}.%i");
+        return buildConfigurationBuilder("target/rolling-update-date/foo.log.%d{yyyy-MM-dd-HH:mm:ss}.%i");
     }
 
     private ConfigurationBuilder<BuiltConfiguration> buildConfigurationBuilder(final String filePattern) {
@@ -52,7 +52,7 @@ public class RollingFileAppenderUpdateDataTest {
         builder.add(builder.newAppender("consoleLog", "Console")
             .addAttribute("target", ConsoleAppender.Target.SYSTEM_ERR));
         builder.add(builder.newAppender("fooAppender", "RollingFile")
-                .addAttribute("fileName", "foo.log")
+                .addAttribute("fileName", "target/rolling-update-date/foo.log")
                 .addAttribute("filePattern", filePattern)
                 .addComponent(builder.newComponent("SizeBasedTriggeringPolicy")
                         .addAttribute("size", "10MB")));
@@ -67,12 +67,12 @@ public class RollingFileAppenderUpdateDataTest {
     public void testClosingLoggerContext() {
         // initial config with indexed rollover
         try (LoggerContext loggerContext1 = Configurator.initialize(buildConfigA().build())) {
-            validateAppender(loggerContext1, "foo.log.%i");
+            validateAppender(loggerContext1, "target/rolling-update-date/foo.log.%i");
         }
 
         // rebuild config with date based rollover
         try (LoggerContext loggerContext2 = Configurator.initialize(buildConfigB().build())) {
-            validateAppender(loggerContext2, "foo.log.%d{yyyy-MM-dd-HH:mm:ss}.%i");
+            validateAppender(loggerContext2, "target/rolling-update-date/foo.log.%d{yyyy-MM-dd-HH:mm:ss}.%i");
         }
     }
 
@@ -81,11 +81,11 @@ public class RollingFileAppenderUpdateDataTest {
     public void testNotClosingLoggerContext() {
         // initial config with indexed rollover
         final LoggerContext loggerContext1 = Configurator.initialize(buildConfigA().build());
-        validateAppender(loggerContext1, "foo.log.%i");
+        validateAppender(loggerContext1, "target-rolling-update-date/foo.log.%i");
 
         // rebuild config with date based rollover
         final LoggerContext loggerContext2 = Configurator.initialize(buildConfigB().build());
-        validateAppender(loggerContext2, "foo.log.%d{yyyy-MM-dd-HH:mm:ss}.%i");
+        validateAppender(loggerContext2, "target/rolling-update-date/foo.log.%d{yyyy-MM-dd-HH:mm:ss}.%i");
     }
 
     private void validateAppender(final LoggerContext loggerContext, final String expectedFilePattern) {