netty: clean up netty source code

Author: jknack

jooby/src/main/java/io/jooby/internal/Chi.java

@@ -6,9 +6,9 @@
 package io.jooby.internal;
 
 import java.util.Arrays;
+import java.util.HashMap;
 import java.util.Map;
 import java.util.Objects;
-import java.util.concurrent.ConcurrentHashMap;
 import java.util.regex.Pattern;
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
@@ -485,7 +485,7 @@ public StaticRoute get(String path) {
   }
 
   private static class StaticMapN implements StaticMap {
-    private final Map<String, StaticRoute> paths = new ConcurrentHashMap<>(10);
+    private final Map<String, StaticRoute> paths = new HashMap<>(10);
 
     public StaticMapN(StaticMap6 staticMap, String path, StaticRoute staticRoute) {
       put(staticMap.pattern1, staticMap.route1);
@@ -549,7 +549,7 @@ public void clear() {
   }
 
   private static class MultipleMethodMatcher implements MethodMatcher {
-    private final Map<String, StaticRouterMatch> methods = new ConcurrentHashMap<>();
+    private final Map<String, StaticRouterMatch> methods = new HashMap<>();
 
     public MultipleMethodMatcher(SingleMethodMatcher matcher) {
       methods.put(matcher.method, matcher.route);
@@ -890,7 +890,7 @@ void setEndpoint(String method, Route route) {
       Node n = this;
       // Set the handler for the method type on the node
       if (n.endpoints == null) {
-        n.endpoints = new ConcurrentHashMap<>();
+        n.endpoints = new HashMap<>();
       }
 
       //      if ((method & mSTUB) == mSTUB) {

jooby/src/test/java/io/jooby/Issue3607.java

@@ -10,6 +10,7 @@
 import org.junit.jupiter.api.Test;
 
 import io.jooby.output.Output;
+import io.jooby.output.OutputFactory;
 
 public class Issue3607 {
 
@@ -24,6 +25,7 @@ public Output render(Context ctx, ModelAndView<?> modelAndView) throws Exception
   @Test
   public void shouldNotGenerateEmptyFlashMap() throws Exception {
     var ctx = mock(Context.class);
+    when(ctx.getOutputFactory()).thenReturn(OutputFactory.create(false));
 
     var templateEngine = new TemplateEngineImpl();
     templateEngine.encode(ctx, ModelAndView.map("index.html"));

jooby/src/test/java/io/jooby/internal/ChiTest.java

@@ -33,6 +33,17 @@ public void routeOverride() {
     assertEquals(bar, result.route());
   }
 
+  @Test
+  public void staticMap6() {
+    Chi router = new Chi();
+    router.insert(route("GET", "/1", stringHandler("1")));
+    router.insert(route("GET", "/2", stringHandler("2")));
+    router.insert(route("GET", "/3", stringHandler("3")));
+    router.insert(route("GET", "/4", stringHandler("4")));
+    router.insert(route("GET", "/5", stringHandler("5")));
+    router.insert(route("GET", "/6", stringHandler("6")));
+  }
+
   @Test
   public void routeCase() {
     Chi router = new Chi();