warn on no constraints (#334)

Author: SentryMan

validator-generator/src/main/java/io/avaje/validation/generator/ClassReader.java

@@ -65,6 +65,10 @@ List<FieldReader> allFields() {
     return allFields;
   }
 
+  boolean hasConstraint() {
+    return !allFields.isEmpty();
+  }
+
   @Override
   public boolean nonAccessibleField() {
     return nonAccessibleField;

validator-generator/src/main/java/io/avaje/validation/generator/ValidationProcessor.java

@@ -2,6 +2,7 @@
 
 import static io.avaje.validation.generator.APContext.asTypeElement;
 import static io.avaje.validation.generator.APContext.logError;
+import static io.avaje.validation.generator.APContext.logWarn;
 import static io.avaje.validation.generator.APContext.typeElement;
 import static io.avaje.validation.generator.ProcessingContext.createMetaInfWriterFor;
 import static java.util.stream.Collectors.joining;
@@ -344,19 +345,24 @@ private void writeAdapter(TypeElement typeElement, BeanReader beanReader) {
       // payloads should be validated - ignore this one
       return;
     }
-    processedAnything = true;
     beanReader.read();
     if (beanReader.nonAccessibleField()) {
       if (beanReader.hasValidationAnnotation()) {
-        logError("Error ValidationAdapter due to nonAccessibleField for %s ", beanReader);
+        logError(typeElement, "Error ValidationAdapter due to nonAccessibleField for %s ", beanReader);
       }
       return;
     }
+
     try {
       final SimpleAdapterWriter beanWriter = new SimpleAdapterWriter(beanReader);
-      if (beanReader instanceof ClassReader) {
+      if (beanReader instanceof ClassReader cr) {
+        if (!cr.hasConstraint()) {
+          logNoConstraint(beanReader, cr);
+          return;
+        }
         writeMeta(typeElement, beanReader, beanWriter);
       }
+      processedAnything = true;
       beanWriter.write();
       allReaders.add(beanReader);
       sourceTypes.add(typeElement.getSimpleName().toString());
@@ -365,6 +371,11 @@ private void writeAdapter(TypeElement typeElement, BeanReader beanReader) {
     }
   }
 
+  private void logNoConstraint(BeanReader beanReader, ClassReader cr) {
+    if (!ControllerPrism.isPresent(cr.beanType()) || !PathPrism.isPresent(cr.beanType()))
+      logWarn(cr.beanType(), "No Constraints Defined", beanReader);
+  }
+
   private void writeMeta(
       TypeElement typeElement, BeanReader beanReader, final SimpleAdapterWriter beanWriter) {
     if (beanReader.isPkgPrivate()) {

validator-generator/src/main/java/io/avaje/validation/generator/package-info.java

@@ -1,3 +1,5 @@
+@GeneratePrism(io.avaje.http.api.Controller.class)
+@GeneratePrism(io.avaje.http.api.Path.class)
 @GeneratePrism(io.avaje.validation.adapter.ConstraintAdapter.class)
 @GeneratePrism(io.avaje.validation.ImportValidPojo.class)
 @GeneratePrism(io.avaje.validation.ValidSubTypes.class)