diff --git a/src/components/Footer/NewsletterForm.test.tsx b/src/components/Footer/NewsletterForm.test.tsx index 610b637e..d1ca3c61 100644 --- a/src/components/Footer/NewsletterForm.test.tsx +++ b/src/components/Footer/NewsletterForm.test.tsx @@ -89,6 +89,18 @@ describe("Implementation Requirements", () => { } ); + it("should not submit the newsletter form if the email is invalid (empty)", async () => { + const { getByTestId } = render(); + + userEvent.click( + within(getByTestId("newsletter-form")).getByRole("button", { + name: "Sign up", + }) + ); + + expect(mockWindowOpen).not.toHaveBeenCalled(); + }); + it("should submit the form to the correct URL", async () => { const { getByTestId } = render(); diff --git a/src/components/Footer/NewsletterForm.tsx b/src/components/Footer/NewsletterForm.tsx index 66984c6e..9519a287 100644 --- a/src/components/Footer/NewsletterForm.tsx +++ b/src/components/Footer/NewsletterForm.tsx @@ -77,8 +77,8 @@ const NewsletterForm: FC = forwardRef< className="signUpInputBox" {...register("email", { validate: { - validEmail: isValidEmail, notEmpty: isNotEmpty, + validEmail: isValidEmail, }, required: "This field is required.", })}