Migrate DefaultRouteScriptAppenderTest to JUnit 5

Author: jcoglan

log4j-core-test/src/test/java/org/apache/logging/log4j/core/appender/routing/DefaultRouteScriptAppenderTest.java

@@ -16,40 +16,41 @@
  */
 package org.apache.logging.log4j.core.appender.routing;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import java.util.Map;
 import java.util.concurrent.ConcurrentMap;
 import org.apache.logging.log4j.Marker;
 import org.apache.logging.log4j.MarkerManager;
+import org.apache.logging.log4j.core.Appender;
 import org.apache.logging.log4j.core.Logger;
+import org.apache.logging.log4j.core.LoggerContext;
 import org.apache.logging.log4j.core.config.AppenderControl;
 import org.apache.logging.log4j.core.test.appender.ListAppender;
-import org.apache.logging.log4j.core.test.junit.LoggerContextRule;
 import org.apache.logging.log4j.core.util.Constants;
-import org.junit.Assert;
-import org.junit.BeforeClass;
-import org.junit.Rule;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Test;
 
 /**
  *
  */
 public abstract class DefaultRouteScriptAppenderTest {
 
-    @BeforeClass
+    @BeforeAll
     public static void beforeClass() {
         System.setProperty(Constants.SCRIPT_LANGUAGES, "Groovy, Javascript");
     }
 
-    @Rule
-    public final LoggerContextRule loggerContextRule;
+    public final LoggerContext loggerContext;
 
     private final boolean expectBindingEntries;
 
-    public DefaultRouteScriptAppenderTest(final String configLocation, final boolean expectBindingEntries) {
-        this.loggerContextRule = new LoggerContextRule(configLocation);
+    public DefaultRouteScriptAppenderTest(final LoggerContext context, boolean expectBindingEntries) {
+        this.loggerContext = context;
         this.expectBindingEntries = expectBindingEntries;
     }
 
@@ -65,60 +66,60 @@ private void checkStaticVars() {
     private ListAppender getListAppender() {
         final String key = "Service2";
         final RoutingAppender routingAppender = getRoutingAppender();
-        Assert.assertTrue(routingAppender.isStarted());
+        assertTrue(routingAppender.isStarted());
         final Map<String, AppenderControl> appenders = routingAppender.getAppenders();
         final AppenderControl appenderControl = appenders.get(key);
-        assertNotNull("No appender control generated for '" + key + "'; appenders = " + appenders, appenderControl);
+        assertNotNull(appenderControl, "No appender control generated for '" + key + "'; appenders = " + appenders);
         final ListAppender listAppender = (ListAppender) appenderControl.getAppender();
         return listAppender;
     }
 
     private RoutingAppender getRoutingAppender() {
-        return loggerContextRule.getRequiredAppender("Routing", RoutingAppender.class);
+        return (RoutingAppender) loggerContext.getConfiguration().getAppenders().get("Routing");
     }
 
     private void logAndCheck() {
         final Marker marker = MarkerManager.getMarker("HEXDUMP");
-        final Logger logger = loggerContextRule.getLogger(DefaultRouteScriptAppenderTest.class);
+        final Logger logger = (Logger) loggerContext.getLogger(DefaultRouteScriptAppenderTest.class);
         logger.error("Hello");
         final ListAppender listAppender = getListAppender();
-        assertEquals("Incorrect number of events", 1, listAppender.getEvents().size());
+        assertEquals(1, listAppender.getEvents().size(), "Incorrect number of events");
         logger.error("World");
-        assertEquals("Incorrect number of events", 2, listAppender.getEvents().size());
+        assertEquals(2, listAppender.getEvents().size(), "Incorrect number of events");
         logger.error(marker, "DEADBEEF");
-        assertEquals("Incorrect number of events", 3, listAppender.getEvents().size());
+        assertEquals(3, listAppender.getEvents().size(), "Incorrect number of events");
     }
 
-    @Test(expected = AssertionError.class)
-    public void testAppenderAbsence() {
-        loggerContextRule.getListAppender("List1");
+    @Test
+    public void testNotListAppender() {
+        Appender appender = loggerContext.getConfiguration().getAppender("List1");
+        assertNotNull(appender);
+        assertFalse(appender instanceof ListAppender);
     }
 
     @Test
     public void testListAppenderPresence() {
         // No appender until an event is routed, even thought we initialized the default route on startup.
-        Assert.assertNull(
-                "No appender control generated",
-                getRoutingAppender().getAppenders().get("Service2"));
+        assertNull(getRoutingAppender().getAppenders().get("Service2"), "No appender control generated");
     }
 
     @Test
     public void testNoPurgePolicy() {
         // No PurgePolicy in this test
-        Assert.assertNull("Unexpected PurgePolicy", getRoutingAppender().getPurgePolicy());
+        assertNull(getRoutingAppender().getPurgePolicy(), "Unexpected PurgePolicy");
     }
 
     @Test
     public void testNoRewritePolicy() {
         // No RewritePolicy in this test
-        Assert.assertNull("Unexpected RewritePolicy", getRoutingAppender().getRewritePolicy());
+        assertNull(getRoutingAppender().getRewritePolicy(), "Unexpected RewritePolicy");
     }
 
     @Test
     public void testRoutingAppenderDefaultRouteKey() {
         final RoutingAppender routingAppender = getRoutingAppender();
-        Assert.assertNotNull(routingAppender.getDefaultRouteScript());
-        Assert.assertNotNull(routingAppender.getDefaultRoute());
+        assertNotNull(routingAppender.getDefaultRouteScript());
+        assertNotNull(routingAppender.getDefaultRoute());
         assertEquals("Service2", routingAppender.getDefaultRoute().getKey());
     }
 

log4j-core-test/src/test/java/org/apache/logging/log4j/core/appender/routing/DefaultRouteScriptGroovyAppenderTest.java

@@ -16,8 +16,12 @@
  */
 package org.apache.logging.log4j.core.appender.routing;
 
+import org.apache.logging.log4j.core.LoggerContext;
+import org.apache.logging.log4j.core.test.junit.LoggerContextSource;
+
+@LoggerContextSource("log4j-routing-default-route-script-groovy.xml")
 public class DefaultRouteScriptGroovyAppenderTest extends DefaultRouteScriptAppenderTest {
-    public DefaultRouteScriptGroovyAppenderTest() {
-        super("log4j-routing-default-route-script-groovy.xml", false);
+    public DefaultRouteScriptGroovyAppenderTest(LoggerContext context) {
+        super(context, false);
     }
 }

log4j-core-test/src/test/java/org/apache/logging/log4j/core/appender/routing/DefaultRouteScriptJavaScriptAppenderTest.java

@@ -16,8 +16,12 @@
  */
 package org.apache.logging.log4j.core.appender.routing;
 
+import org.apache.logging.log4j.core.LoggerContext;
+import org.apache.logging.log4j.core.test.junit.LoggerContextSource;
+
+@LoggerContextSource("log4j-routing-default-route-script-javascript.xml")
 public class DefaultRouteScriptJavaScriptAppenderTest extends DefaultRouteScriptAppenderTest {
-    public DefaultRouteScriptJavaScriptAppenderTest() {
-        super("log4j-routing-default-route-script-javascript.xml", false);
+    public DefaultRouteScriptJavaScriptAppenderTest(LoggerContext context) {
+        super(context, false);
     }
 }

log4j-core-test/src/test/java/org/apache/logging/log4j/core/appender/routing/ScriptStaticVarsGroovyAppenderTest.java

@@ -16,8 +16,12 @@
  */
 package org.apache.logging.log4j.core.appender.routing;
 
+import org.apache.logging.log4j.core.LoggerContext;
+import org.apache.logging.log4j.core.test.junit.LoggerContextSource;
+
+@LoggerContextSource("log4j-routing-script-staticvars-groovy.xml")
 public class ScriptStaticVarsGroovyAppenderTest extends DefaultRouteScriptAppenderTest {
-    public ScriptStaticVarsGroovyAppenderTest() {
-        super("log4j-routing-script-staticvars-groovy.xml", true);
+    public ScriptStaticVarsGroovyAppenderTest(LoggerContext context) {
+        super(context, true);
     }
 }

log4j-core-test/src/test/java/org/apache/logging/log4j/core/appender/routing/ScriptStaticVarsJavaScriptAppenderTest.java

@@ -16,8 +16,12 @@
  */
 package org.apache.logging.log4j.core.appender.routing;
 
+import org.apache.logging.log4j.core.LoggerContext;
+import org.apache.logging.log4j.core.test.junit.LoggerContextSource;
+
+@LoggerContextSource("log4j-routing-script-staticvars-javascript.xml")
 public class ScriptStaticVarsJavaScriptAppenderTest extends DefaultRouteScriptAppenderTest {
-    public ScriptStaticVarsJavaScriptAppenderTest() {
-        super("log4j-routing-script-staticvars-javascript.xml", true);
+    public ScriptStaticVarsJavaScriptAppenderTest(LoggerContext context) {
+        super(context, true);
     }
 }