fix: update schema builder add/set actions to have unique errors

Author: guilhermecastros

packages/utils/src/ErrorSchemaBuilder.ts

@@ -75,9 +75,9 @@ export default class ErrorSchemaBuilder<T = any> {
     }
 
     if (Array.isArray(errorOrList)) {
-      errorsList.push(...errorOrList);
+      set(errorBlock, ERRORS_KEY, [...new Set([...errorsList, ...errorOrList])]);
     } else {
-      errorsList.push(errorOrList);
+      set(errorBlock, ERRORS_KEY, [...new Set([...errorsList, errorOrList])]);
     }
     return this;
   }
@@ -93,7 +93,7 @@ export default class ErrorSchemaBuilder<T = any> {
   setErrors(errorOrList: string | string[], pathOfError?: string | (string | number)[]) {
     const errorBlock: ErrorSchema = this.getOrCreateErrorBlock(pathOfError);
     // Effectively clone the array being given to prevent accidental outside manipulation of the given list
-    const listToAdd = Array.isArray(errorOrList) ? [...errorOrList] : [errorOrList];
+    const listToAdd = Array.isArray(errorOrList) ? [...new Set([...errorOrList])] : [errorOrList];
     set(errorBlock, ERRORS_KEY, listToAdd);
     return this;
   }

packages/utils/test/ErrorSchemaBuilder.test.ts

@@ -240,6 +240,24 @@ describe('ErrorSchemaBuilder', () => {
     it('resetting error restores things back to an empty object', () => {
       expect(builder.resetAllErrors().ErrorSchema).toEqual({});
     });
+    it('adding duplicated error string with a path puts only the unique errors at the path', () => {
+      builder.addErrors([AN_ERROR], STRING_PATH);
+      builder.addErrors([AN_ERROR], STRING_PATH);
+      expect(builder.ErrorSchema).toEqual({
+        [STRING_PATH]: {
+          [ERRORS_KEY]: [AN_ERROR],
+        },
+      });
+    });
+
+    it('setting duplicated error string with a path puts only the unique errors at the path', () => {
+      builder.setErrors([AN_ERROR, AN_ERROR], STRING_PATH);
+      expect(builder.ErrorSchema).toEqual({
+        [STRING_PATH]: {
+          [ERRORS_KEY]: [AN_ERROR],
+        },
+      });
+    });
   });
   describe('using initial schema', () => {
     let builder: ErrorSchemaBuilder;