set error message to null instead of empty string when clearing the e…

…rror
Khan · Nov 6, 2024 · d2aeaf8 · d2aeaf8
1 parent 116ce7e
commit d2aeaf8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/packages/wonder-blocks-form/src/components/__tests__/text-area.test.tsx b/packages/wonder-blocks-form/src/components/__tests__/text-area.test.tsx
@@ -1546,7 +1546,7 @@ describe("TextArea", () => {
                     expect(field).toHaveAttribute("aria-invalid", "false");
                 });
 
-                it("should call onValidate with an empty string when the user changes the value after there was an error", async () => {
+                it("should call onValidate with an null when the user changes the value after there was an error", async () => {
                     // Arrange
                     const handleValidate = jest.fn();
                     const errorMsg = "error message";
@@ -1570,7 +1570,7 @@ describe("TextArea", () => {
                     // Assert
                     expect(handleValidate.mock.calls).toStrictEqual([
                         [errorMsg],
-                        [""],
+                        [null],
                     ]);
                 });
 

diff --git a/packages/wonder-blocks-form/src/components/text-area.tsx b/packages/wonder-blocks-form/src/components/text-area.tsx
@@ -255,9 +255,9 @@ const TextArea = React.forwardRef<HTMLTextAreaElement, TextAreaProps>(
                     // If instantValidation is false and there is an error
                     // message, error needs to be cleared when the user updates
                     // the value
-                    setErrorMessage("");
+                    setErrorMessage(null);
                     if (onValidate) {
-                        onValidate("");
+                        onValidate(null);
                     }
                 }
             }