mvc routes: default to HTTP GET when Path annotation is present fix #370

Author: jknack

coverage-report/src/test/java/org/jooby/issues/Issue370.java

@@ -0,0 +1,30 @@
+package org.jooby.issues;
+
+import org.jooby.mvc.Path;
+import org.jooby.test.ServerFeature;
+import org.junit.Test;
+
+public class Issue370 extends ServerFeature {
+
+  @Path("/")
+  public static class S {
+
+    @Path("/s")
+    public String doSomething() {
+      return "GET";
+    }
+
+  }
+
+  {
+     use(S.class);
+  }
+
+  @Test
+  public void mvcShouldDefaultToGet() throws Exception {
+    request()
+        .get("/s")
+        .expect("GET");
+  }
+
+}

jooby/src/main/java/org/jooby/internal/mvc/MvcRoutes.java

@@ -22,6 +22,7 @@
 import java.lang.reflect.AnnotatedElement;
 import java.lang.reflect.Method;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.HashMap;
 import java.util.LinkedHashMap;
 import java.util.List;
@@ -91,6 +92,8 @@ public static List<Route.Definition> routes(final Env env, final RouteMetadata c
       }
       if (annotations.size() > 0) {
         methods.put(method, annotations);
+      } else if (method.isAnnotationPresent(Path.class)) {
+        methods.put(method, Arrays.asList(GET.class));
       }
     }