fix isTouch bug when emit form.clear()

cantonjs · Jun 29, 2018 · eb32874 · eb32874
1 parent ceb38b4
commit eb32874
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 1 deletion.
diff --git a/src/FormStore/FormStore.js b/src/FormStore/FormStore.js
@@ -84,5 +84,10 @@ export default class FormStore extends ObjectStore {
 		return pristineValue;
 	};
 
-	@action clear = () => (this.pristineValue = {});
+	@action
+	clear = () => {
+		const pristineValue = (this.pristineValue = {});
+		this.touch();
+		return pristineValue;
+	};
 }
diff --git a/test/demon.test.js b/test/demon.test.js
@@ -194,6 +194,22 @@ describe('isTouched', () => {
 			expect.objectContaining({ isTouched: false }),
 		);
 	});
+
+	test('should `isTouched` be true after form.clear()', () => {
+		const value = { hello: 'world' };
+		const formRef = createRef();
+		const children = jest.fn((props) => <input {...props} />);
+		mount(
+			<Form value={value} ref={formRef}>
+				<Demon forwardedProps={{ name: 'hello' }}>{children}</Demon>
+			</Form>,
+		);
+		formRef.current.clear();
+		expect(children).toHaveBeenCalledWith(
+			expect.any(Object),
+			expect.objectContaining({ isTouched: true }),
+		);
+	});
 });
 
 describe('isValid', () => {