Jex generation using {} rather than colon based path parameters

Author: rbygrave

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

@@ -31,7 +31,7 @@ class ControllerMethodWriter {
   void write(boolean requestScoped) {
 
     final PathSegments segments = method.getPathSegments();
-    final String fullPath = segments.fullPathColon();
+    final String fullPath = segments.fullPath();
 
     writer.append("    routing.%s(\"%s\", ctx -> {", webMethod.name().toLowerCase(), fullPath).eol();
     writer.append("      ctx.status(%s);", method.getStatusCode()).eol();
@@ -95,9 +95,9 @@ private void writeContextReturn() {
     } else if (produces.equalsIgnoreCase(MediaType.TEXT_HTML)) {
       writer.append("ctx.html(");
     } else if (produces.equalsIgnoreCase(MediaType.TEXT_PLAIN)) {
-      writer.append("ctx.contentType(\"text/plain\").result(");
+      writer.append("ctx.text(");
     } else {
-      writer.append("ctx.contentType(\"%s\").result(", produces);
+      writer.append("ctx.contentType(\"%s\").write(", produces);
     }
   }
 }

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

@@ -3,6 +3,7 @@
 import io.avaje.http.api.Controller;
 import io.avaje.http.api.Get;
 import io.avaje.http.api.Path;
+import io.avaje.http.api.Produces;
 
 @Controller
 @Path("/")
@@ -15,4 +16,16 @@ HelloDto getHello() {
     dto.name = "rob";
     return dto;
   }
+
+  @Produces("text/plain")
+  @Get("plain")
+  String getText() {
+    return "something";
+  }
+
+  @Produces("text/plain")
+  @Get("other/{name}")
+  String name(String name) {
+    return "hi "+name;
+  }
 }

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

@@ -3,6 +3,9 @@
 import io.avaje.http.client.HttpClientContext;
 import org.junit.jupiter.api.Test;
 
+import java.net.http.HttpResponse;
+
+import static org.assertj.core.api.Assertions.assertThat;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
 class HelloControllerTest extends BaseWebTest {
@@ -16,4 +19,20 @@ void getHello() {
     assertEquals(42, hello.id);
     assertEquals("rob", hello.name);
   }
+
+  @Test
+  void getPlain() {
+
+    final HttpResponse<String> res = client.request().path("plain").get().asString();
+
+    assertEquals("something", res.body());
+    assertThat(res.headers().firstValue("content-type").get()).startsWith("text/plain;");
+  }
+
+  @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());
+  }
 }