/*
 * Licensed to the Apache Software Foundation (ASF) under one or more
 * contributor license agreements. See the NOTICE file distributed with
 * this work for additional information regarding copyright ownership.
 * The ASF licenses this file to You under the Apache license, Version 2.0
 * (the "License"); you may not use this file except in compliance with
 * the License. You may obtain a copy of the License at
 *
 *      http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the license for the specific language governing permissions and
 * limitations under the license.
 */
package org.apache.log4j.config;

import org.apache.log4j.ListAppender;
import org.apache.log4j.LogManager;
import org.apache.log4j.Logger;
import org.apache.log4j.bridge.AppenderAdapter;
import org.apache.log4j.spi.LoggingEvent;
import org.apache.logging.log4j.core.Appender;
import org.apache.logging.log4j.core.LoggerContext;
import org.apache.logging.log4j.core.config.Configuration;
import org.apache.logging.log4j.core.config.ConfigurationSource;
import org.apache.logging.log4j.core.config.Configurator;
import org.junit.Test;

import java.io.File;
import java.io.FileInputStream;
import java.io.InputStream;
import java.util.List;
import java.util.Map;

import static org.junit.Assert.assertNotNull;
import static org.junit.Assert.assertTrue;

/**
 * Test configuration from Properties.
 */
public class PropertiesConfigurationTest {

    @Test
    public void testProperties() throws Exception {
        configure("target/test-classes/log4j1-file.properties");
        Logger logger = LogManager.getLogger("test");
        logger.debug("This is a test of the root logger");
        File file = new File("target/temp.A1");
        assertTrue("File A1 was not created", file.exists());
        assertTrue("File A1 is empty", file.length() > 0);
        file = new File("target/temp.A2");
        assertTrue("File A2 was not created", file.exists());
        assertTrue("File A2 is empty", file.length() > 0);
    }

    @Test
    public void testListAppender() throws Exception {
        LoggerContext loggerContext = configure("target/test-classes/log4j1-list.properties");
        Logger logger = LogManager.getLogger("test");
        logger.debug("This is a test of the root logger");
        Configuration configuration = loggerContext.getConfiguration();
        Map<String, Appender> appenders = configuration.getAppenders();
        ListAppender eventAppender = null;
        ListAppender messageAppender = null;
        for (Map.Entry<String, Appender> entry : appenders.entrySet()) {
            if (entry.getKey().equals("list")) {
                messageAppender = (ListAppender) ((AppenderAdapter.Adapter) entry.getValue()).getAppender();
            } else if (entry.getKey().equals("events")) {
                eventAppender = (ListAppender) ((AppenderAdapter.Adapter) entry.getValue()).getAppender();
            }
        }
        assertNotNull("No Event Appender", eventAppender);
        assertNotNull("No Message Appender", messageAppender);
        List<LoggingEvent> events = eventAppender.getEvents();
        assertTrue("No events", events != null && events.size() > 0);
        List<String> messages = messageAppender.getMessages();
        assertTrue("No messages", messages != null && messages.size() > 0);
    }

    private LoggerContext configure(String configLocation) throws Exception {
        File file = new File(configLocation);
        InputStream is = new FileInputStream(file);
        ConfigurationSource source = new ConfigurationSource(is, file);
        LoggerContext context = (LoggerContext) org.apache.logging.log4j.LogManager.getContext(false);
        Configuration configuration = new PropertiesConfigurationFactory().getConfiguration(context, source);
        assertNotNull("No configuration created", configuration);
        Configurator.reconfigure(configuration);
        return context;
    }

}

@@ -1706,16 +1706,51 @@
 %0A       
+ try (LoggerContext loggerContext =
  configu
@@ -1793,34 +1793,38 @@
 properties%22)
-;%0A
+) %7B%0A  
         
 Logger logge
@@ -1803,32 +1803,34 @@
 %22)) %7B%0A          
+  
 Logger logger = 
@@ -1851,32 +1851,36 @@
 Logger(%22test%22);%0A
+    
         logger.d
@@ -1914,32 +1914,36 @@
  root logger%22);%0A
+    
         File fil
@@ -1974,32 +1974,36 @@
 p.A1%22);%0A        
+    
 assertTrue(%22File
@@ -2040,32 +2040,36 @@
 sts());%0A        
+    
 assertTrue(%22File
@@ -2111,16 +2111,20 @@
         
+  
+  
 file = n
@@ -2146,24 +2146,28 @@
 /temp.A2%22);%0A
+    
         asse
@@ -2208,32 +2208,36 @@
 file.exists());%0A
+    
         assertTr
@@ -2271,32 +2271,42 @@
 .length() %3E 0);%0A
+        %7D%0A
     %7D%0A%0A    @Test
@@ -3617,24 +3617,58 @@
  0);%0A    %7D%0A%0A
+    @SuppressWarnings(%22resource%22)%0A
     private 
@@ -4263,9 +4263,8 @@
     %7D%0A%0A%7D
-%0A

Use try-with-resources.