API change: Replace Body with Result #51

Author: jknack

coverage-report/src/test/java/org/jooby/BufferSizeFeature.java

@@ -2,7 +2,6 @@
 
 import java.util.Optional;
 
-import org.jooby.Body;
 import org.jooby.test.ServerFeature;
 import org.junit.Before;
 import org.junit.Test;
@@ -23,7 +22,7 @@ public class BufferSizeFeature extends ServerFeature {
       String value = req.param("data").value();
       Optional<Long> len = req.param("len").toOptional(Long.class);
       Optional<Boolean> chunked = req.param("chunked").toOptional(Boolean.class);
-      Body rsp = Body.ok(value);
+      Result rsp = Results.ok(value);
       len.ifPresent(l -> rsp.header("Content-Length", l));
       chunked.ifPresent(c -> rsp.header("Transfer-Encoding", "Chunked"));
       return rsp;

coverage-report/src/test/java/org/jooby/RedirectFeature.java

@@ -1,6 +1,5 @@
 package org.jooby;
 
-import org.jooby.Body;
 import org.jooby.mvc.GET;
 import org.jooby.mvc.Path;
 import org.jooby.test.ServerFeature;
@@ -13,26 +12,26 @@ public static class Resource {
 
     @Path("/redirect")
     @GET
-    public Body redirect() {
-      return Body.redirect("/r/here");
+    public Result redirect() {
+      return Results.redirect("/r/here");
     }
 
     @Path("/seeOther")
     @GET
-    public Body seeOther() {
-      return Body.seeOther("/r/here");
+    public Result seeOther() {
+      return Results.seeOther("/r/here");
     }
 
     @Path("/tempRedirect")
     @GET
-    public Body tempRedirect() {
-      return Body.tempRedirect("/r/here");
+    public Result tempRedirect() {
+      return Results.tempRedirect("/r/here");
     }
 
     @Path("/moved")
     @GET
-    public Body moved() {
-      return Body.moved("/r/here");
+    public Result moved() {
+      return Results.moved("/r/here");
     }
 
     @Path("/here")

coverage-report/src/test/java/org/jooby/ResponseBodyFeature.java

@@ -1,6 +1,5 @@
 package org.jooby;
 
-import org.jooby.Body;
 import org.jooby.mvc.GET;
 import org.jooby.mvc.Path;
 import org.jooby.test.ServerFeature;
@@ -12,26 +11,26 @@ public static class Resource {
 
     @GET
     @Path("/200")
-    public Body ok() {
-      return Body.ok();
+    public Result ok() {
+      return Results.ok();
     }
 
     @GET
     @Path("/200/body")
-    public Body okWithBody() {
-      return Body.ok("***");
+    public Result okWithBody() {
+      return Results.ok("***");
     }
 
     @GET
     @Path("/204")
-    public Body noContent() {
-      return Body.noContent();
+    public Result noContent() {
+      return Results.noContent();
     }
 
     @GET
     @Path("/headers")
-    public Body headers() {
-      return Body.ok().header("x", "y");
+    public Result headers() {
+      return Results.ok().header("x", "y");
     }
 
   }

coverage-report/src/test/java/org/jooby/hbm/HbmCustomFeature.java

@@ -5,7 +5,7 @@
 import javax.persistence.EntityManager;
 import javax.persistence.Query;
 
-import org.jooby.Body;
+import org.jooby.Results;
 import org.jooby.hbm.data.Member;
 import org.jooby.test.ServerFeature;
 import org.junit.Test;
@@ -36,7 +36,7 @@ public class HbmCustomFeature extends ServerFeature {
         throw new IllegalArgumentException("Rollback on err");
       }
       // we do this way just to make sure the correct delegate got executed
-        rsp.send(Body.body(member));
+        rsp.send(Results.with(member));
         chain.next(req, rsp);
       });
   }

coverage-report/src/test/java/org/jooby/hbm/HbmFeature.java

@@ -5,8 +5,7 @@
 import javax.persistence.EntityManager;
 import javax.persistence.Query;
 
-import org.jooby.Body;
-import org.jooby.hbm.Hbm;
+import org.jooby.Results;
 import org.jooby.hbm.data.Member;
 import org.jooby.test.ServerFeature;
 import org.junit.Test;
@@ -35,7 +34,7 @@ public class HbmFeature extends ServerFeature {
         throw new IllegalArgumentException("Rollback on err");
       }
       // we do this way just to make sure the correct delegate got executed
-        rsp.send(Body.body(member));
+        rsp.send(Results.with(member));
         chain.next(req, rsp);
       });
   }

coverage-report/src/test/java/org/jooby/hbm/HbmProdEnvFeature.java

@@ -5,8 +5,7 @@
 import javax.persistence.EntityManager;
 import javax.persistence.Query;
 
-import org.jooby.Body;
-import org.jooby.hbm.Hbm;
+import org.jooby.Results;
 import org.jooby.hbm.data.Member;
 import org.jooby.test.ServerFeature;
 import org.junit.Test;
@@ -37,7 +36,7 @@ public class HbmProdEnvFeature extends ServerFeature {
         throw new IllegalArgumentException("Rollback on err");
       }
       // we do this way just to make sure the correct delegate got executed
-        rsp.send(Body.body(member));
+        rsp.send(Results.with(member));
         chain.next(req, rsp);
       });
   }

coverage-report/src/test/java/org/jooby/hbm/HbmScanFeature.java

@@ -5,8 +5,7 @@
 import javax.persistence.EntityManager;
 import javax.persistence.Query;
 
-import org.jooby.Body;
-import org.jooby.hbm.Hbm;
+import org.jooby.Results;
 import org.jooby.hbm.data.Member;
 import org.jooby.test.ServerFeature;
 import org.junit.Test;
@@ -35,7 +34,7 @@ public class HbmScanFeature extends ServerFeature {
         throw new IllegalArgumentException("Rollback on err");
       }
       // we do this way just to make sure the correct delegate got executed
-        rsp.send(Body.body(member));
+        rsp.send(Results.with(member));
         chain.next(req, rsp);
       });
   }

jooby-hbm/src/main/java/org/jooby/hbm/Hbm.java

@@ -36,9 +36,9 @@
 import org.hibernate.Session;
 import org.hibernate.engine.spi.SessionImplementor;
 import org.hibernate.jpa.AvailableSettings;
-import org.jooby.Body;
 import org.jooby.Env;
 import org.jooby.Response;
+import org.jooby.Result;
 import org.jooby.Route;
 import org.jooby.internal.hbm.HbmProvider;
 import org.jooby.internal.hbm.HbmUnitDescriptor;
@@ -184,9 +184,9 @@ public void send(final Object body) throws Exception {
           }
 
           @Override
-          public void send(final Body body) throws Exception {
+          public void send(final Result result) throws Exception {
             transactionalSend(() -> {
-              super.send(body);
+              super.send(result);
               return null;
             });
           }

jooby-servlet/src/test/java/org/jooby/servlet/WebXmlTest.java

@@ -35,8 +35,7 @@ public class WebXmlTest {
       "    <!-- MultiPart setup -->\n" +
       "    <multipart-config>\n" +
       "      <file-size-threshold>0</file-size-threshold>\n" +
-      "      <!-- 200k -->\n" +
-      "      <max-request-size>204800</max-request-size>\n" +
+      "      <max-request-size>${war.maxRequestSize}</max-request-size>\n" +
       "    </multipart-config>\n" +
       "  </servlet>\n" +
       "\n" +