Refactor collection instantiation to use factory methods

microsphere-log4j2/src/main/java/io/microsphere/logging/log4j2/layout/SmartFileAppenderLayout.java

@@ -27,9 +27,9 @@
 import org.apache.logging.log4j.core.layout.ByteBufferDestination;
 
 import java.io.Serializable;
-import java.util.HashMap;
 import java.util.Map;
 
+import static io.microsphere.collection.MapUtils.newHashMap;
 import static io.microsphere.logging.log4j2.appender.InMemoryAppender.NAME;
 import static org.apache.logging.log4j.core.layout.PatternLayout.newBuilder;
 
@@ -66,7 +66,7 @@ public SmartFileAppenderLayout(LoggerContext context) {
     }
 
     private Map<String, Layout> initDelegatingLayouts(LoggerContext context) {
-        Map<String, Layout> delegatingLayouts = new HashMap<>();
+        Map<String, Layout> delegatingLayouts = newHashMap();
         for (Logger logger : context.getLoggers()) {
             String loggerName = logger.getName();
             Layout layout = selectLayout(logger);
@@ -91,7 +91,7 @@ private Appender selectAppender(Logger logger) {
 
     private Appender selectAppender(Logger logger, Class<? extends Appender> appenderClass) {
         Appender targetAppender = null;
-        Map<String, Appender> appendersMap = new HashMap<>(logger.getAppenders());
+        Map<String, Appender> appendersMap = newHashMap(logger.getAppenders());
         appendersMap.remove(NAME);
         if (appendersMap.isEmpty()) {
             Logger parentLogger = logger.getParent();

microsphere-log4j2/src/test/java/io/microsphere/logging/log4j2/appender/InMemoryAppenderTest.java

@@ -23,9 +23,9 @@
 import org.apache.logging.log4j.message.SimpleMessage;
 import org.junit.jupiter.api.Test;
 
-import java.util.ArrayList;
 import java.util.List;
 
+import static io.microsphere.collection.ListUtils.newArrayList;
 import static io.microsphere.logging.log4j2.appender.InMemoryAppender.NAME;
 import static io.microsphere.logging.log4j2.appender.InMemoryAppender.findInMemoryAppender;
 import static java.lang.Thread.sleep;
@@ -111,7 +111,7 @@ private static LogEvent newEvent(String message) throws InterruptedException {
     }
 
     private static class CollectingAppender implements Appender {
-        private final List<LogEvent> events = new ArrayList<>();
+        private final List<LogEvent> events = newArrayList();
 
         @Override
         public void append(LogEvent event) {

microsphere-log4j2/src/test/java/io/microsphere/logging/log4j2/util/Log4j2UtilsTest.java

@@ -31,10 +31,10 @@
 import org.junit.jupiter.api.Test;
 
 import java.io.Serializable;
-import java.util.ArrayList;
 import java.util.List;
 import java.util.concurrent.atomic.AtomicInteger;
 
+import static io.microsphere.collection.ListUtils.newArrayList;
 import static io.microsphere.logging.log4j2.util.Log4j2Utils.addAppender;
 import static io.microsphere.logging.log4j2.util.Log4j2Utils.findAppender;
 import static io.microsphere.logging.log4j2.util.Log4j2Utils.getLevel;
@@ -172,7 +172,7 @@ void shouldCoverDoInLoggerAndFindAppender() {
     @Test
     void shouldCoverAddRemoveAppenderStaticHelpers() {
         LoggerContext context = (LoggerContext) LogManager.getContext(false);
-        List<Logger> targets = new ArrayList<>();
+        List<Logger> targets = newArrayList();
         targets.add(context.getRootLogger());
 
         TestAppender appender = new TestAppender("helperAppender");
@@ -197,7 +197,7 @@ void shouldCoverAddRemoveAppenderStaticHelpers() {
     @Test
     void shouldCoverAddRemoveAppenderNullGuardsAndLifecycleBranches() {
         LoggerContext context = (LoggerContext) LogManager.getContext(false);
-        List<Logger> targets = new ArrayList<>();
+        List<Logger> targets = newArrayList();
         targets.add(context.getRootLogger());
 
         // null guard branches

microsphere-logging-commons/src/main/java/io/microsphere/logging/jmx/LoggingMXBeanRegistrar.java

@@ -26,12 +26,12 @@
 import javax.management.MalformedObjectNameException;
 import javax.management.ObjectInstance;
 import javax.management.ObjectName;
-import java.util.LinkedList;
 import java.util.List;
 import java.util.ServiceLoader;
 import java.util.Set;
 import java.util.logging.LoggingMXBean;
 
+import static io.microsphere.collection.ListUtils.newLinkedList;
 import static io.microsphere.lang.function.ThrowableSupplier.execute;
 import static io.microsphere.logging.LoggerFactory.getLogger;
 import static io.microsphere.logging.LoggingUtils.loadAll;
@@ -75,7 +75,7 @@ public static List<ObjectInstance> registerAll() {
      */
     @Nonnull
     public static List<ObjectInstance> registerAll(ClassLoader classLoader) {
-        List<ObjectInstance> instances = new LinkedList<>();
+        List<ObjectInstance> instances = newLinkedList();
         Iterable<Logging> loggings = loadAll(classLoader);
         for (Logging logging : loggings) {
             ObjectInstance instance = register(logging);

microsphere-logging-test/src/main/java/io/microsphere/logging/test/jupiter/extension/logging/LoggingLevelTemplateInvocationContext.java

@@ -23,9 +23,10 @@
 import org.junit.jupiter.api.extension.ExtensionContext;
 import org.junit.jupiter.api.extension.TestTemplateInvocationContext;
 
-import java.util.ArrayList;
 import java.util.List;
 
+import static io.microsphere.collection.ListUtils.newArrayList;
+
 /**
  * The facade class of {@link ClassTemplateInvocationContext} and {@link TestTemplateInvocationContext} to set the
  * logging level.
@@ -96,7 +97,7 @@ public String getDisplayName(int invocationIndex) {
      */
     @Override
     public List<Extension> getAdditionalExtensions() {
-        List<Extension> extensions = new ArrayList<>(this.isClassTemplate ? 2 : 1);
+        List<Extension> extensions = newArrayList(this.isClassTemplate ? 2 : 1);
         extensions.add(new LoggingLevelCallback(this.loggins, this.loggerName, this.level));
         if (this.isClassTemplate) {
             extensions.add(new LoggingLevelParameterResolver(this.level, this.index));