fix: preserve false should not trigger shouldUpdate rerender (#714)

* test: test driven

* enhance: shouldUpdate false should not trigger rerender

Author: zombieJ

src/Field.tsx

@@ -297,7 +297,10 @@ class Field extends React.Component<InternalFieldProps, FieldState> implements F
        * - Reset A, need clean B, C
        */
       case 'remove': {
-        if (shouldUpdate) {
+        if (
+          shouldUpdate &&
+          requireUpdate(shouldUpdate, prevStore, store, prevValue, curValue, info)
+        ) {
           this.reRender();
           return;
         }

tests/dependencies.test.tsx

@@ -226,4 +226,43 @@ describe('Form.Dependencies', () => {
     // sync end
     expect(spy).toHaveBeenCalledTimes(3);
   });
+
+  it('shouldUpdate false should not update', () => {
+    let counter = 0;
+    const formRef = React.createRef<FormInstance>();
+
+    const { container } = render(
+      <Form ref={formRef}>
+        <Field name="little" preserve={false}>
+          <Input />
+        </Field>
+
+        <Field shouldUpdate={(prev, next) => prev.little !== next.little}>
+          {(_, __, form) => {
+            // Fill to hide
+            if (!form.getFieldValue('little')) {
+              return <InfoField name="bamboo" preserve={false} />;
+            }
+
+            return null;
+          }}
+        </Field>
+
+        <Field shouldUpdate={() => false}>
+          {() => {
+            console.log('render!');
+            counter += 1;
+            return null;
+          }}
+        </Field>
+      </Form>,
+    );
+    expect(counter).toEqual(1);
+    expect(container.querySelectorAll('input')).toHaveLength(2);
+
+    // hide should not re-render
+    fireEvent.change(getInput(container, 0), { target: { value: '1' } });
+    expect(container.querySelectorAll('input')).toHaveLength(1);
+    expect(counter).toEqual(1);
+  });
 });