wip

Author: onacit

src/test/java/com/github/jinahya/assertj/validation/ValidationAssertionsTestUtils.java

@@ -0,0 +1,29 @@
+package com.github.jinahya.assertj.validation;
+
+import javax.validation.ConstraintViolation;
+import java.util.function.Consumer;
+
+import static org.mockito.Mockito.spy;
+
+public final class ValidationAssertionsTestUtils {
+
+    public static Consumer<ConstraintViolation<?>> violationConsumerSpy(
+            final Consumer<? super ConstraintViolation<?>> consumer) {
+        final Consumer<ConstraintViolation<?>> wrapper = new Consumer<ConstraintViolation<?>>() {
+            @Override
+            public void accept(final ConstraintViolation<?> constraintViolation) {
+                consumer.accept(constraintViolation);
+            }
+        };
+        return spy(wrapper);
+    }
+
+    public static Consumer<ConstraintViolation<?>> violationConsumerSpy() {
+        return violationConsumerSpy(cv -> {
+        });
+    }
+
+    private ValidationAssertionsTestUtils() {
+        throw new AssertionError("instantiation is not allowed");
+    }
+}

src/test/java/com/github/jinahya/assertj/validation/example/user/User_IsValid_Test.java

@@ -20,6 +20,7 @@
  * #L%
  */
 
+import com.github.jinahya.assertj.validation.ValidationAssertionsTestUtils;
 import lombok.extern.slf4j.Slf4j;
 import org.junit.jupiter.api.Test;
 import org.mockito.ArgumentCaptor;
@@ -131,11 +132,8 @@ void __Invalid() {
     void WithConsumer__Invalid() {
         final var bean = User.newInstance(false, false);
         final var assertion = assertThatBean(bean);
-        final Consumer<ConstraintViolation<?>> consumer = spy(new Consumer<ConstraintViolation<?>>() {
-            @Override
-            public void accept(final ConstraintViolation<?> constraintViolation) {
-                log.debug("violation: {}", constraintViolation);
-            }
+        final var consumer = ValidationAssertionsTestUtils.violationConsumerSpy(cv -> {
+            log.debug("violation: {}", cv);
         });
         // WHEN
         assertThatThrownBy(() -> assertion.consumingViolations(consumer).isValid()).isInstanceOf(AssertionError.class);

src/test/java/com/github/jinahya/assertj/validation/example/user2/User_IsValidFor_Test.java

@@ -41,16 +41,28 @@ class AgeTest {
         void __AgeIsValidForJunior() {
             final var age = UserConstants.MAX_AGE_FOR_JUNIOR_EXCLUSIVE - 1;
             final var assertion = assertThatProperty(age);
-            assertion.isValidFor(User2.class, User.PROPERTY_NAME_AGE);
-            assertion.targetingGroups(Junior.class).isValidFor(User2.class, User.PROPERTY_NAME_AGE);
+            assertion.consumingViolations(cv -> {
+                        log.debug("cv: {}", cv);
+                    })
+                    .isValidFor(User2.class, User.PROPERTY_NAME_AGE);
+            assertion.consumingViolations(cv -> {
+                        log.debug("cv: {}", cv);
+                    })
+                    .targetingGroups(Junior.class).isValidFor(User2.class, User.PROPERTY_NAME_AGE);
             assertThatThrownBy(
                     () -> assertion
+                            .consumingViolations(cv -> {
+                                log.debug("cv: {}", cv);
+                            })
                             .targetingGroups(Senior.class)
                             .isValidFor(User2.class, User.PROPERTY_NAME_AGE)
             )
                     .isInstanceOf(AssertionError.class);
             assertThatThrownBy(
                     () -> assertion
+                            .consumingViolations(cv -> {
+                                log.debug("cv: {}", cv);
+                            })
                             .targetingGroups(Junior.class, Senior.class)
                             .isValidFor(User2.class, User.PROPERTY_NAME_AGE)
             )
@@ -61,20 +73,38 @@ void __AgeIsValidForJunior() {
         void __AgeIsValidForSenior() {
             final var age = UserConstants.MIN_AGE_FOR_SENIOR_INCLUSIVE;
             final var assertion = assertThatProperty(age);
-            assertion.isValidFor(User2.class, User.PROPERTY_NAME_AGE);
-            assertion.targetingGroups(Senior.class).isValidFor(User2.class, User.PROPERTY_NAME_AGE);
+            assertion
+                    .consumingViolations(cv -> {
+                        log.debug("cv: {}", cv);
+                    })
+                    .isValidFor(User2.class, User.PROPERTY_NAME_AGE)
+            ;
+            assertion.targetingGroups(Senior.class)
+                    .consumingViolations(cv -> {
+                        log.debug("cv: {}", cv);
+                    })
+                    .isValidFor(User2.class, User.PROPERTY_NAME_AGE)
+            ;
             assertThatThrownBy(
                     () -> assertion
+                            .consumingViolations(cv -> {
+                                log.debug("cv: {}", cv);
+                            })
                             .targetingGroups(Junior.class)
                             .isValidFor(User2.class, User.PROPERTY_NAME_AGE)
             )
-                    .isInstanceOf(AssertionError.class);
+                    .isInstanceOf(AssertionError.class)
+            ;
             assertThatThrownBy(
                     () -> assertion
+                            .consumingViolations(cv -> {
+                                log.debug("cv: {}", cv);
+                            })
                             .targetingGroups(Junior.class, Senior.class)
                             .isValidFor(User2.class, User.PROPERTY_NAME_AGE)
             )
-                    .isInstanceOf(AssertionError.class);
+                    .isInstanceOf(AssertionError.class)
+            ;
         }
     }
 }