[@mantine/form] Reset should set values to current initialValues (#5122)

mantinedev · Oct 26, 2023 · d1fe092 · d1fe092
1 parent e006b48
commit d1fe092
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/src/mantine-form/src/tests/reset.test.ts b/src/mantine-form/src/tests/reset.test.ts
@@ -45,4 +45,15 @@ describe('@mantine/form/reset', () => {
     expect(hook.result.current.isDirty()).toBe(false);
     expect(hook.result.current.values).toStrictEqual({ a: 1, b: 2 });
   });
+
+  it('resets values correctly after updating initial values', () => {
+    const hook = renderHook(() => useForm({ initialValues: { a: 1, b: 2 } }));
+    const newInitialState = { a: 3, b: 4 };
+
+    act(() => hook.result.current.setValues({ a: 100, b: 200 }));
+    act(() => hook.result.current.setInitialValues(newInitialState));
+    act(() => hook.result.current.reset());
+
+    expect(hook.result.current.values).toStrictEqual(newInitialState);
+  });
 });
diff --git a/src/mantine-form/src/use-form.ts b/src/mantine-form/src/use-form.ts
@@ -71,9 +71,8 @@ export function useForm<
 
   const clearErrors: ClearErrors = useCallback(() => _setErrors({}), []);
   const reset: Reset = useCallback(() => {
-    _setValues(initialValues);
+    _setValues(valuesSnapshot.current);
     clearErrors();
-    setValuesSnapshot(initialValues);
     setDirty({});
     resetTouched();
   }, []);