fixed regression

Author: abdalla-rko

packages/core/src/components/Form.tsx

@@ -441,9 +441,9 @@ export default class Form<
     if (mustValidate) {
       const schemaValidation = this.validate(formData, schema, schemaUtils, _retrievedSchema);
       errors = schemaValidation.errors;
-      // If the schema has changed, we do not merge state.errorSchema.
+      // If retrievedSchema is undefined which means the schema or formData has changed, we do not merge state.
       // Else in the case where it hasn't changed, we merge 'state.errorSchema' with 'schemaValidation.errorSchema.' This done to display the raised field error.
-      if (isSchemaChanged) {
+      if (retrievedSchema === undefined) {
         errorSchema = schemaValidation.errorSchema;
       } else {
         errorSchema = mergeObjects(

packages/core/test/ArrayField.test.jsx

@@ -3219,6 +3219,27 @@ describe('ArrayField', () => {
       });
     });
 
+    it('Check that when formData changes, the form should re-validate', () => {
+      const { node, rerender, compCB } = createFormComponent({
+        schema,
+        formData: [
+          {
+            text: null,
+          },
+        ],
+        liveValidate: true,
+      });
+
+      const errorMessages = node.querySelectorAll('#root_0_text__error');
+      expect(errorMessages).to.have.length(1);
+      const errorMessageContent = node.querySelector('#root_0_text__error .text-danger').textContent;
+      expect(errorMessageContent).to.contain('must be string');
+
+      rerender(compCB({ schema, formData: [{ text: 'test' }], liveValidate: true }));
+
+      expect(node.querySelectorAll('#root_0_text__error')).to.have.length(0);
+    });
+
     it('raise an error and check if the error is displayed', () => {
       const { node } = createFormComponent({
         schema,

packages/core/test/ObjectField.test.jsx

@@ -227,6 +227,25 @@ describe('ObjectField', () => {
       });
     });
 
+    it('Check that when formData changes, the form should re-validate', () => {
+      const { node, rerender, compCB } = createFormComponent({
+        schema,
+        formData: {
+          foo: null,
+        },
+        liveValidate: true,
+      });
+
+      const errorMessages = node.querySelectorAll('#root_foo__error');
+      expect(errorMessages).to.have.length(1);
+      const errorMessageContent = node.querySelector('#root_foo__error .text-danger').textContent;
+      expect(errorMessageContent).to.contain('must be string');
+
+      rerender(compCB({ schema, formData: { foo: 'test' }, liveValidate: true }));
+
+      expect(node.querySelectorAll('#root_foo__error')).to.have.length(0);
+    });
+
     it('raise an error and check if the error is displayed', () => {
       const { node } = createFormComponent({
         schema,

packages/core/test/StringField.test.jsx

@@ -299,6 +299,23 @@ describe('StringField', () => {
       expect(node.querySelector('input').getAttribute('autocomplete')).eql('family-name');
     });
 
+    it('Check that when formData changes, the form should re-validate', () => {
+      const { node, rerender, compCB } = createFormComponent({
+        schema: { type: 'string' },
+        formData: null,
+        liveValidate: true,
+      });
+
+      const errorMessages = node.querySelectorAll('#root__error');
+      expect(errorMessages).to.have.length(1);
+      const errorMessageContent = node.querySelector('#root__error .text-danger').textContent;
+      expect(errorMessageContent).to.contain('must be string');
+
+      rerender(compCB({ schema: { type: 'string' }, formData: 'hello', liveValidate: true }));
+
+      expect(node.querySelectorAll('#root__error')).to.have.length(0);
+    });
+
     it('raise an error and check if the error is displayed', () => {
       const { node } = createFormComponent({
         schema: { type: 'string' },

packages/core/test/test_utils.js

@@ -14,10 +14,11 @@ export function createComponent(Component, props) {
   const onError = sinon.spy();
   const onSubmit = sinon.spy();
   const comp = <Component onSubmit={onSubmit} onError={onError} onChange={onChange} {...props} />;
-  const { container } = render(comp);
+  const compCB = (props) => <Component onSubmit={onSubmit} onError={onError} onChange={onChange} {...props} />;
+  const { container, rerender } = render(comp);
   const node = findDOMNode(container).firstElementChild;
 
-  return { comp, node, onChange, onError, onSubmit };
+  return { comp, compCB, node, onChange, onError, onSubmit, rerender };
 }
 
 export function createFormComponent(props) {