allow path operator to globally set route properties fix #925

Author: jknack

jooby/src/main/java/org/jooby/Jooby.java

@@ -907,11 +907,11 @@ public Jooby(final String prefix) {
   }
 
   @Override
-  public Jooby path(String path, Runnable action) {
+  public Route.Collection path(String path, Runnable action) {
     this.path = Route.normalize(path);
-    action.run();
+    Route.Collection collection = with(action);
     this.path = null;
-    return this;
+    return collection;
   }
 
   @Override

jooby/src/main/java/org/jooby/Router.java

@@ -264,7 +264,7 @@ static String decode(String path) {
    * @param action Router action.
    * @return This router.
    */
-  Router path(String path, Runnable action);
+  Route.Collection path(String path, Runnable action);
 
   /**
    * Import content from provide application (routes, parsers/renderers, start/stop callbacks, ...

modules/coverage-report/src/test/java/org/jooby/issues/Issue900.java

@@ -15,8 +15,6 @@ public class Issue900 extends ServerFeature {
 
   @Test
   public void shouldNotEncodingPlusSign() throws Exception {
-    int ch = '+';
-    System.out.println(Integer.toHexString(ch));
     request().get("/900/a+b")
         .expect("a b");
   }

modules/coverage-report/src/test/java/org/jooby/issues/Issue925.java

@@ -0,0 +1,24 @@
+package org.jooby.issues;
+
+import org.jooby.test.ServerFeature;
+import org.junit.Test;
+
+public class Issue925 extends ServerFeature {
+
+  {
+    path("/925/api/some", () -> {
+      get(req -> req.route().produces());
+      post(req -> req.route().consumes());
+    }).produces("json")
+      .consumes("json");
+  }
+
+  @Test
+  public void pathOperatorWithRouteProperties() throws Exception {
+    request().get("/925/api/some")
+        .expect("[application/json]");
+    request().post("/925/api/some")
+        .expect("[application/json]");
+  }
+
+}