fix: provide correct validation state in value change listener

Author: ugur-vaadin

vaadin-date-time-picker-flow-parent/vaadin-date-time-picker-flow/src/main/java/com/vaadin/flow/component/datetimepicker/DateTimePicker.java

@@ -376,12 +376,12 @@ private void addValidationListeners() {
     public void setValue(LocalDateTime value) {
         var oldValue = getValue();
         value = sanitizeValue(value);
-        super.setValue(value);
         var shouldFireValidationStatusChangeEvent = oldValue == null
                 && value == null
                 && (isInputUnparsable() || isInputIncomplete());
         synchronizeChildComponentValues(value);
         validate(shouldFireValidationStatusChangeEvent);
+        super.setValue(value);
     }
 
     /**

vaadin-date-time-picker-flow-parent/vaadin-date-time-picker-flow/src/test/java/com/vaadin/flow/component/datetimepicker/validation/BasicValidationTest.java

@@ -18,6 +18,7 @@
 import java.time.LocalDate;
 import java.time.LocalDateTime;
 import java.time.LocalTime;
+import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.concurrent.atomic.AtomicInteger;
 
 import org.junit.Assert;
@@ -234,6 +235,16 @@ public void clearValueProgrammatically_fieldValidatedOnce() {
                 dateTimePicker.getValidationCount());
     }
 
+    @Test
+    public void setValueProgrammatically_invalidStateIsUpdatedInValueChangeListener() {
+        var isInvalid = new AtomicBoolean();
+        testField.addValueChangeListener(
+                e -> isInvalid.set(e.getSource().isInvalid()));
+        testField.setMax(LocalDateTime.now());
+        testField.setValue(LocalDateTime.now().plusDays(1));
+        Assert.assertTrue(isInvalid.get());
+    }
+
     @Override
     protected DateTimePicker createTestField() {
         return new DateTimePicker();