Form // fix handling of isValid when the Form has errors. Add missing method mock in FormTest::testIsValid().

Author: Christian Leucht

src/Element/Form.php

@@ -124,7 +124,11 @@ public function isValid(): bool
 
         // Either on "submit" we have erroneous Elements, or the
         // Form itself has errors assigned through Form::withErrors().
-        return $this->hasErrors() || $this->isValid;
+        if ($this->hasErrors()) {
+            return false;
+        }
+
+        return $this->isValid;
     }
 
     /**

tests/phpunit/Unit/Element/FormTest.php

@@ -61,6 +61,8 @@ public function testIsValid(): void
             ->andReturn($expected_value);
         $element->allows('withErrors')
             ->with($expected_error);
+        $element->allows('hasErrors')
+            ->andReturnTrue();
         $element->allows('validate')
             ->andReturnFalse();