Bug Report: Hibernate Validator Not Working in Script/Lambda Mode (BeanValidator.validate(...))

- fix #3751

Author: jknack

jooby/src/main/java/io/jooby/validation/ValidationContext.java

@@ -97,6 +97,12 @@ public <T> T to(@NonNull Type type) {
       return BeanValidator.apply(ctx, ((Body) delegate).toNullable(type));
     }
 
+    @NonNull @Override
+    public <T> T to(@NonNull Class<T> type) {
+      // Call nullable version to let bean validator to run
+      return BeanValidator.apply(ctx, ((Body) delegate).toNullable(type));
+    }
+
     @Nullable @Override
     public <T> T toNullable(@NonNull Type type) {
       return BeanValidator.apply(ctx, ((Body) delegate).toNullable(type));

tests/src/test/java/io/jooby/i3751/Issue3751.java

@@ -0,0 +1,62 @@
+/*
+ * Jooby https://jooby.io
+ * Apache License Version 2.0 https://jooby.io/LICENSE.txt
+ * Copyright 2014 Edgar Espina
+ */
+package io.jooby.i3751;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+import java.util.List;
+
+import io.jooby.hibernate.validator.HibernateValidatorModule;
+import io.jooby.jackson.JacksonModule;
+import io.jooby.junit.ServerTest;
+import io.jooby.junit.ServerTestRunner;
+import io.jooby.validation.BeanValidator;
+import jakarta.validation.Validator;
+import okhttp3.MediaType;
+import okhttp3.RequestBody;
+
+public class Issue3751 {
+  @ServerTest
+  public void shouldValidateDataBody(ServerTestRunner runner) {
+    runner
+        .define(
+            app -> {
+              app.install(new HibernateValidatorModule());
+              app.install(new JacksonModule());
+              app.post(
+                  "/3751-direct",
+                  (ctx -> {
+                    Validator validator = ctx.require(Validator.class);
+                    return validator.validate(ctx.body(User3721.class));
+                  }));
+              app.post(
+                  "/3751-bean-validator",
+                  BeanValidator.validate(
+                      ctx -> {
+                        ctx.body(User3721.class);
+                        // all good
+                        return List.of();
+                      }));
+            })
+        .ready(
+            http -> {
+              var user =
+                  RequestBody.create(
+                      "{\"password\": \"1234\"}", MediaType.parse("application/json"));
+              http.post(
+                  "/3751-direct",
+                  user,
+                  rsp -> {
+                    http.post(
+                        "/3751-direct",
+                        user,
+                        rsp2 -> {
+                          assertEquals(rsp.body().string(), rsp2.body().string());
+                        });
+                  });
+            });
+  }
+}

tests/src/test/java/io/jooby/i3751/User3721.java

@@ -0,0 +1,20 @@
+/*
+ * Jooby https://jooby.io
+ * Apache License Version 2.0 https://jooby.io/LICENSE.txt
+ * Copyright 2014 Edgar Espina
+ */
+package io.jooby.i3751;
+
+import jakarta.validation.constraints.NotBlank;
+
+public class User3721 {
+  @NotBlank(message = "password can not empty") private String password;
+
+  public String getPassword() {
+    return password;
+  }
+
+  public void setPassword(String password) {
+    this.password = password;
+  }
+}