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

logging-log4j2 git commit: Improve test clean-up

Repository: logging-log4j2
Updated Branches:
  refs/heads/master a03a3322a -> a853e27e5


Improve test clean-up


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

Branch: refs/heads/master
Commit: a853e27e590b886d30e1ca63f1a89e12e409addd
Parents: a03a332
Author: Mikael Ståldal <mi...@staldal.nu>
Authored: Sun Nov 26 12:25:41 2017 +0100
Committer: Mikael Ståldal <mi...@staldal.nu>
Committed: Sun Nov 26 12:25:41 2017 +0100

----------------------------------------------------------------------
 .../org/apache/log4j/config/Log4j1ConfigurationFactoryTest.java | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/a853e27e/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 b614fe6..8d3e4e2 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
@@ -168,12 +168,15 @@ public class Log4j1ConfigurationFactoryTest {
         try {
             final Configuration configuration = getConfiguration("config-1.2/log4j-system-properties-1.properties");
             final RollingFileAppender appender = configuration.getAppender("RFA");
+			appender.stop(10, TimeUnit.SECONDS);
             System.out.println("expected: " + tempFileName + " Actual: " + appender.getFileName());
             assertEquals(tempFileName, appender.getFileName());
         } finally {
 			try {
 				Files.deleteIfExists(tempFilePath);
-			} catch (FileSystemException ignored) {}
+			} catch (FileSystemException e) {
+				e.printStackTrace();
+			}
         }
 	}