fix: list override (#443)

Author: zombieJ

src/utils/set.ts

@@ -76,14 +76,20 @@ export function merge<T extends object>(...sources: T[]) {
     function internalMerge(path: Path) {
       const value = get(src, path);
 
-      if (isObject(value) || Array.isArray(value)) {
+      const isArr = Array.isArray(value);
+
+      if (isArr || isObject(value)) {
         // Only add not loop obj
         if (!loopSet.has(value)) {
           loopSet.add(value);
 
-          // Init container if not exist
           const originValue = get(clone, path);
-          if (!originValue || typeof originValue !== 'object') {
+
+          if (isArr) {
+            // Array will always be override
+            clone = set(clone, path, []);
+          } else if (!originValue || typeof originValue !== 'object') {
+            // Init container if not exist
             clone = set(clone, path, createEmpty(value));
           }
 

tests/utils.test.js

@@ -107,34 +107,39 @@ describe('utils', () => {
         });
       });
 
-      it('array', () => {
+      it('array is replacement', () => {
         const merged = merge([], [{ a: 1 }], [{ b: 2 }]);
 
         expect(merged).toEqual([
           {
-            a: 1,
             b: 2,
           },
         ]);
       });
 
+      it('array is replacement - sub field', () => {
+        const merged = merge({ a: 1, users: [1, 2] }, { users: [] });
+
+        expect(merged).toEqual({ a: 1, users: [] });
+      });
+
       it('not cover', () => {
         const merged = merge(
-          [],
-          [{ a: { e: 8 }, b: 2 }],
-          [{ a: { f: 9 }, c: 3 }],
+          {},
+          { _: { a: { e: 8 }, b: 2 } },
+          { _: { a: { f: 9 }, c: 3 } },
         );
 
-        expect(merged).toEqual([
-          {
+        expect(merged).toEqual({
+          _: {
             a: {
               e: 8,
               f: 9,
             },
             b: 2,
             c: 3,
           },
-        ]);
+        });
       });
 
       it('DayObject', () => {