#72 - Add support for withDefault() with Jex generation

Author: rbygrave

http-generator-jex/src/main/java/io/avaje/http/generator/jex/JexAdapter.java

@@ -59,6 +59,6 @@ public void writeReadParameter(Append writer, ParamType paramType, String paramN
 
   @Override
   public void writeReadParameter(Append writer, ParamType paramType, String paramName, String paramDefault) {
-    writer.append("ctx.%s(\"%s\",\"%s\")", paramType, paramName, paramDefault);
+    writer.append("withDefault(ctx.%s(\"%s\"), \"%s\")", paramType, paramName, paramDefault);
   }
 }

tests/test-jex/src/main/java/org/example/web/HelloController.java

@@ -31,6 +31,12 @@ String name(String name) {
     return "hi " + name;
   }
 
+  @Produces("text/plain")
+  @Get("withDefault/{name}")
+  String withDefault(String name, @Default("42") String limit) {
+    return "name|" + name+";limit|"+limit;
+  }
+
   @Produces("text/plain")
   @Get("splat/{name}/<s0>/other/<s1>")
   String splat(String name, Context ctx) {

tests/test-jex/src/main/resources/public/openapi.json

@@ -157,6 +157,39 @@
           }
         }
       }
+    },
+    "/withDefault/{name}" : {
+      "get" : {
+        "tags" : [ ],
+        "summary" : "",
+        "description" : "",
+        "parameters" : [ {
+          "name" : "name",
+          "in" : "path",
+          "required" : true,
+          "schema" : {
+            "type" : "string"
+          }
+        }, {
+          "name" : "limit",
+          "in" : "query",
+          "schema" : {
+            "type" : "string"
+          }
+        } ],
+        "responses" : {
+          "200" : {
+            "description" : "",
+            "content" : {
+              "text/plain" : {
+                "schema" : {
+                  "type" : "string"
+                }
+              }
+            }
+          }
+        }
+      }
     }
   },
   "components" : {

tests/test-jex/src/test/java/org/example/web/HelloControllerTest.java

@@ -24,7 +24,6 @@ void getHello() {
 
   @Test
   void getPlain() {
-
     final HttpResponse<String> res = client.request().path("plain").GET().asString();
 
     assertEquals("something", res.body());
@@ -33,11 +32,17 @@ void getPlain() {
 
   @Test
   void getName() {
-
     assertEquals("hi bazz", client.request().path("other/bazz").GET().asString().body());
     assertEquals("hi bax", client.request().path("other/bax").GET().asString().body());
   }
 
+  @Test
+  void withDefault() {
+    assertEquals("name|bazz;limit|42", client.request().path("withDefault/bazz").GET().asString().body());
+    assertEquals("name|bazz;limit|10", client.request().path("withDefault/bazz").queryParam("limit", 10).GET().asString().body());
+    assertEquals("name|foo;limit|11", client.request().path("withDefault/foo").queryParam("limit", 11).queryParam("limit", 12).GET().asString().body());
+  }
+
   @Test
   void splat() {
     assertEquals("got name:one splat0:a/b splat1:x/y/z", client.request().path("splat/one/a/b/other/x/y/z").GET().asString().body());
@@ -48,10 +53,8 @@ void splat2() {
     assertEquals("got name:one splat0:a/b splat1:x/y/z", client.request().path("splat2/one/a/b/other/x/y/z").GET().asString().body());
   }
 
-
   @Test
   void validation() {
-
     HelloDto helloDto = new HelloDto();
     helloDto.id = 42;
 
@@ -65,7 +68,6 @@ void validation() {
 
   @Test
   void validation_expect_HttpException() {
-
     HelloDto helloDto = new HelloDto();
     helloDto.id = 42;