fix: form::onReset should reset form's "internal" initial state

JakobJingleheimer · Feb 19, 2024 · 98067ce · 98067ce
1 parent 9580f9b
commit 98067ce
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/react/Form/Form.jsx b/lib/react/Form/Form.jsx
@@ -79,7 +79,7 @@ export function Form({
 			onReset={(e) => {
 				props.onReset?.(e);
 				// After everything has succeeded
-				setup(e.currentTarget, initValues);
+				initValues.current = { __proto__: null };
 				is.onSubmit(e);
 			}}
 			onSubmit={(e) => {

diff --git a/lib/react/Form/Form.test.jsx b/lib/react/Form/Form.test.jsx
@@ -140,7 +140,7 @@ describe('<Form>', () => {
 			fireEvent.change(field, { target: { value: sameSearchValue } });
 			fireEvent.submit(form, { elements: [field] });
 			expect(onResetCalled, 'reset count (after subsequent submit)').to.equal(1);
-			expect(onSubmitCalled, 'submit count').to.equal(1);
+			expect(onSubmitCalled, 'submit count').to.equal(2);
 		});
 
 		it('should reset pristine & touched and call handler', () => {