Merge pull request #14 from sashirestela/13-validate-fields-declared-in-parent-classes

Validate fields declared in parent classes

Author: sashirestela

pom.xml

@@ -6,7 +6,7 @@
 
   <groupId>io.github.sashirestela</groupId>
   <artifactId>slimvalidator</artifactId>
-  <version>1.2.0</version>
+  <version>1.2.1</version>
   <packaging>jar</packaging>
 
   <name>slimvalidator</name>

src/main/java/io/github/sashirestela/slimvalidator/metadata/ClassMetadata.java

@@ -27,7 +27,7 @@ public static class FieldMetadata {
         public Object getValue(Object object) {
             var clazz = object.getClass();
             try {
-                var method = clazz.getDeclaredMethod(getMethodName(name), (Class<?>[]) null);
+                var method = clazz.getMethod(getMethodName(name), (Class<?>[]) null);
                 return method.invoke(object);
             } catch (NoSuchMethodException | SecurityException | IllegalAccessException | IllegalArgumentException
                     | InvocationTargetException e) {

src/test/java/io/github/sashirestela/slimvalidator/ValidatorTest.java

@@ -1,5 +1,6 @@
 package io.github.sashirestela.slimvalidator;
 
+import io.github.sashirestela.slimvalidator.data.AbstractClass.ChildClass;
 import io.github.sashirestela.slimvalidator.data.Address;
 import io.github.sashirestela.slimvalidator.data.Address.Coordinate;
 import io.github.sashirestela.slimvalidator.data.Person;
@@ -75,4 +76,20 @@ void shouldReturnViolationsWhenObjectDoesNotAccomplishConstraints() {
         assertEquals(expectedViolationMessage, actualViolationsMessage);
     }
 
+    @Test
+    void shouldExecuteValidationWhenThereIsClassHierarchy() {
+
+        var childObject = ChildClass.builder()
+                .name("name")
+                .level(12)
+                .category("category")
+                .build();
+        var validator = new Validator();
+        var violations = validator.validate(childObject);
+        var exception = new ConstraintViolationException(violations);
+        var actualViolationsMessage = exception.getMessage();
+        var expectedViolationMessage = "level must be at least 1 at most 10.";
+        assertEquals(expectedViolationMessage, actualViolationsMessage);
+    }
+
 }

src/test/java/io/github/sashirestela/slimvalidator/data/AbstractClass.java

@@ -0,0 +1,27 @@
+package io.github.sashirestela.slimvalidator.data;
+
+import io.github.sashirestela.slimvalidator.constraints.Range;
+import io.github.sashirestela.slimvalidator.constraints.Required;
+import lombok.Getter;
+import lombok.experimental.SuperBuilder;
+
+@Getter
+@SuperBuilder
+public abstract class AbstractClass {
+
+    @Required
+    protected String name;
+
+    @Range(min = 1, max = 10)
+    protected Integer level;
+
+    @Getter
+    @SuperBuilder
+    public static class ChildClass extends AbstractClass {
+
+        @Required
+        private String category;
+
+    }
+
+}