Parameters that are called "name" cannot have other annotations on them besides *Param fix #2352

Author: jknack

tests/src/test/java/io/jooby/i2352/C2352.java

@@ -0,0 +1,21 @@
+package io.jooby.i2352;
+
+import java.util.Optional;
+
+import javax.annotation.Nonnull;
+import javax.annotation.Nullable;
+
+import io.jooby.annotations.FormParam;
+import io.jooby.annotations.POST;
+
+public class C2352 {
+  @POST("/2352/nonnull")
+  public String nonnull(@Nonnull @FormParam String name) {
+    return name;
+  }
+
+  @POST("/2352/nullable")
+  public String nullable(@Nullable @FormParam String name) {
+    return String.valueOf(name);
+  }
+}

tests/src/test/java/io/jooby/i2352/Issue2352.java

@@ -0,0 +1,44 @@
+package io.jooby.i2352;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+import io.jooby.junit.ServerTest;
+import io.jooby.junit.ServerTestRunner;
+import okhttp3.MultipartBody;
+
+public class Issue2352 {
+  @ServerTest
+  public void shouldNotIgnoreAnnotationOnParam(ServerTestRunner runner) {
+    runner.define(app -> {
+      app.mvc(new C2352());
+      app.error((ctx, cause, code) -> {
+        ctx.send(cause.getMessage());
+      });
+    }).ready(http -> {
+
+      String name = getClass().getSimpleName();
+      http.post("/2352/nonnull", new MultipartBody.Builder()
+          .setType(MultipartBody.FORM)
+          .addFormDataPart("noname", name)
+          .build(), rsp -> {
+        assertEquals(400, rsp.code());
+        assertEquals("Missing value: 'name'", rsp.body().string());
+      });
+
+      http.post("/2352/nonnull", new MultipartBody.Builder()
+          .setType(MultipartBody.FORM)
+          .addFormDataPart("name", name)
+          .build(), rsp -> {
+        assertEquals(name, rsp.body().string());
+      });
+
+      http.post("/2352/nullable", new MultipartBody.Builder()
+          .setType(MultipartBody.FORM)
+          .addFormDataPart("noname", name)
+          .build(), rsp -> {
+        assertEquals(200, rsp.code());
+        assertEquals("null", rsp.body().string());
+      });
+    });
+  }
+}