one more test and some comments

Author: rbri

src/main/java/org/htmlunit/html/HtmlInput.java

@@ -1044,6 +1044,14 @@ public boolean isTooShortValidityState() {
         return getValue().length() < getMinLength();
     }
 
+    // no need to override isTooLongValidityState()
+    // The HTML spec (§4.10.18.5) has a deliberate rule: tooLong only fires
+    // if the user has interacted with the field ("the element has a dirty value flag").
+    // A value set via JS (elem.value = '...') that was never touched by the user does
+    // not set the dirty flag, so tooLong stays false regardless of the value length.
+    // see HtmlTextInputTest
+    // maxLengthValidationInvalid()/maxLengthValidationInvalidInitial()/maxLengthValidationValid()
+
     @Override
     public boolean isValidValidityState() {
         return !isCustomErrorValidityState()

src/test/java/org/htmlunit/html/HtmlTextInputTest.java

@@ -909,6 +909,21 @@ public void minLengthValidationValid() throws Exception {
         validation("<input type='text' minlength='5' id='e1' name='k'>\n", "", "abcdefghi");
     }
 
+    /**
+     * @throws Exception if an error occurs
+     */
+    @Test
+    @Alerts({"null",
+             "",
+             "true",
+             "false-false-false-false-false-false-false-false-false-true-false",
+             "true",
+             "§§URL§§?k=x", "2"})
+    public void validationMinLengthJs() throws Exception {
+        validation("<input type='text' id='e1' name='k' minlength='2'>\n",
+                "elem.value = 'x';", null);
+    }
+
     /**
      * @throws Exception if an error occurs
      */
@@ -997,6 +1012,64 @@ public void validationEmpty() throws Exception {
         validation("<input type='text' id='e1' name='k'>\n", "", null);
     }
 
+    /**
+     * @throws Exception if an error occurs
+     */
+    @Test
+    @Alerts({"null",
+             "x",
+             "false",
+             "false-false-false-false-false-false-false-true-false-false-false",
+             "true",
+             "§§URL§§", "1"})
+    public void validationMinLength() throws Exception {
+        validation("<input type='text' id='e1' name='k' minlength='2'>\n", "", "x");
+    }
+
+    /**
+     * @throws Exception if an error occurs
+     */
+    @Test
+    @Alerts({"null",
+             "xy",
+             "true",
+             "false-false-false-false-false-false-false-false-false-true-false",
+             "true",
+             "§§URL§§?k=xy", "2"})
+    public void validationMaxLength() throws Exception {
+        validation("<input type='text' id='e1' name='k' maxlength='2'>\n", "", "xyz");
+    }
+
+    /**
+     * @throws Exception if an error occurs
+     */
+    @Test
+    @Alerts({"null",
+             "",
+             "true",
+             "false-false-false-false-false-false-false-false-false-true-false",
+             "true",
+             "§§URL§§?k=toooLong", "2"})
+    public void validationMaxLengthJs() throws Exception {
+        validation("<input type='text' id='e1' name='k' maxlength='2'>\n",
+                "elem.value = 'toooLong';", null);
+    }
+
+    /**
+     * @throws Exception if an error occurs
+     */
+    @Test
+    @Alerts({"null",
+             "",
+             "true",
+             "false-false-false-false-false-false-false-false-false-true-false",
+             "true",
+             "§§URL§§?k=toooLong", "2"})
+    public void validationMaxLengthJsEvent() throws Exception {
+        validation("<input type='text' id='e1' name='k' maxlength='2'>\n",
+                "elem.value = 'toooLong';elem.dispatchEvent(new Event('input'));", null);
+    }
+
     /**
      * @throws Exception if an error occurs
      */