From 3dbd6a76dffcce8335dc021c7773338cb849bc14 Mon Sep 17 00:00:00 2001 From: "Adolfo R. Brandes" Date: Fri, 6 Mar 2026 18:59:31 -0300 Subject: [PATCH] fix: pass correct arguments to setEmailSuggestionContext in handleSuggestionClick The function expects two string arguments (suggestion, type) but was being called with a single object, corrupting the email suggestion state. Co-Authored-By: Claude Opus 4.6 --- src/register/RegistrationFields/EmailField/EmailField.jsx | 2 +- .../RegistrationFields/EmailField/EmailField.test.jsx | 4 ++++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/src/register/RegistrationFields/EmailField/EmailField.jsx b/src/register/RegistrationFields/EmailField/EmailField.jsx index db2118a8..e786d19b 100644 --- a/src/register/RegistrationFields/EmailField/EmailField.jsx +++ b/src/register/RegistrationFields/EmailField/EmailField.jsx @@ -84,7 +84,7 @@ const EmailField = (props) => { handleErrorChange('email', ''); handleChange({ target: { name: 'email', value: emailSuggestion.suggestion } }); setEmailSuggestion({ suggestion: '', type: '' }); - setEmailSuggestionContext({ suggestion: '', type: '' }); + setEmailSuggestionContext('', ''); }; const handleSuggestionClosed = () => setEmailSuggestion({ suggestion: '', type: '' }); diff --git a/src/register/RegistrationFields/EmailField/EmailField.test.jsx b/src/register/RegistrationFields/EmailField/EmailField.test.jsx index 759c39ac..d1a8f425 100644 --- a/src/register/RegistrationFields/EmailField/EmailField.test.jsx +++ b/src/register/RegistrationFields/EmailField/EmailField.test.jsx @@ -182,6 +182,10 @@ describe('EmailField', () => { expect(props.handleChange).toHaveBeenCalledWith( { target: { name: 'email', value: 'john@hotmail.com' } }, ); + expect(mockRegisterContext.setEmailSuggestionContext).toHaveBeenCalledWith( + expect.any(String), + expect.any(String), + ); }); it('should give error for common top level domain mistakes', () => {