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 2014/06/26 05:03:34 UTC

svn commit: r1605654 - /logging/log4j/log4j2/trunk/log4j-web/src/test/java/org/apache/logging/log4j/web/ServletAppenderTest.java

Author: ggregory
Date: Thu Jun 26 03:03:34 2014
New Revision: 1605654

URL: http://svn.apache.org/r1605654
Log:
Test bullet-proofing.

Modified:
    logging/log4j/log4j2/trunk/log4j-web/src/test/java/org/apache/logging/log4j/web/ServletAppenderTest.java

Modified: logging/log4j/log4j2/trunk/log4j-web/src/test/java/org/apache/logging/log4j/web/ServletAppenderTest.java
URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-web/src/test/java/org/apache/logging/log4j/web/ServletAppenderTest.java?rev=1605654&r1=1605653&r2=1605654&view=diff
==============================================================================
--- logging/log4j/log4j2/trunk/log4j-web/src/test/java/org/apache/logging/log4j/web/ServletAppenderTest.java (original)
+++ logging/log4j/log4j2/trunk/log4j-web/src/test/java/org/apache/logging/log4j/web/ServletAppenderTest.java Thu Jun 26 03:03:34 2014
@@ -59,11 +59,11 @@ public class ServletAppenderTest {
             assertNotNull("No ServletAppender", appender);
             final Logger logger = LogManager.getLogger("Test");
             logger.info("This is a test");
-
         } catch (final IllegalStateException e) {
             fail("Failed to initialize Log4j properly." + e.getMessage());
+        } finally {
+            initializer.stop();
+            ContextAnchor.THREAD_CONTEXT.remove();
         }
-        initializer.stop();
-        ContextAnchor.THREAD_CONTEXT.remove();
     }
 }