emberflow 1.3.7 → 1.3.9

package/lib/tests/index-utils.test.js

@@ -32,8 +32,9 @@ const security_1 = require("../sample-custom/security");
 const validators_1 = require("../sample-custom/validators");
 var Timestamp = firebase_admin_1.firestore.Timestamp;
 const paths_1 = require("../utils/paths");
-const adminClient = __importStar(require("emberflow-admin-client/lib"));
 const batch_1 = require("../utils/batch");
+const distribution = __importStar(require("../utils/distribution"));
+const forms = __importStar(require("../utils/forms"));
 jest.spyOn(console, "log").mockImplementation();
 jest.spyOn(console, "info").mockImplementation();
 const projectConfig = {
@@ -55,9 +56,123 @@ jest.mock("../utils/paths", () => {
     const originalModule = jest.requireActual("../utils/paths");
     return Object.assign(Object.assign({}, originalModule), { expandAndGroupDocPathsByEntity: jest.fn() });
 });
+describe("distributeDoc", () => {
+    let dbSpy;
+    let queueInstructionsSpy;
+    let docSetMock;
+    let docDeleteMock;
+    const batch = batch_1.BatchUtil.getInstance();
+    jest.spyOn(batch_1.BatchUtil, "getInstance").mockImplementation(() => batch);
+    beforeEach(() => {
+        docSetMock = jest.fn().mockResolvedValue({});
+        docDeleteMock = jest.fn().mockResolvedValue({});
+        const dbDoc = {
+            set: docSetMock,
+            delete: docDeleteMock,
+            id: "test-doc-id",
+        };
+        dbSpy = jest.spyOn(admin.firestore(), "doc").mockReturnValue(dbDoc);
+        queueInstructionsSpy = jest.spyOn(distribution, "queueInstructions").mockResolvedValue();
+    });
+    afterEach(() => {
+        dbSpy.mockRestore();
+        queueInstructionsSpy.mockRestore();
+    });
+    it("should delete a document from dstPath", async () => {
+        const logicResultDoc = {
+            action: "delete",
+            priority: "normal",
+            dstPath: "/users/test-user-id/documents/test-doc-id",
+        };
+        await (0, index_utils_1.distributeDoc)(logicResultDoc);
+        expect(admin.firestore().doc).toHaveBeenCalledTimes(1);
+        expect(admin.firestore().doc).toHaveBeenCalledWith("/users/test-user-id/documents/test-doc-id");
+        expect(docDeleteMock).toHaveBeenCalledTimes(1);
+        expect(docDeleteMock).toHaveBeenCalled();
+    });
+    it("should delete documents in batch", async () => {
+        const batchDeleteSpy = jest.spyOn(batch, "deleteDoc").mockResolvedValue(undefined);
+        const logicResultDoc = {
+            action: "delete",
+            priority: "normal",
+            dstPath: "/users/test-user-id/documents/test-doc-id",
+        };
+        await (0, index_utils_1.distributeDoc)(logicResultDoc, batch);
+        expect(admin.firestore().doc).toHaveBeenCalledTimes(1);
+        expect(admin.firestore().doc).toHaveBeenCalledWith("/users/test-user-id/documents/test-doc-id");
+        expect(batchDeleteSpy).toHaveBeenCalledTimes(1);
+        batchDeleteSpy.mockRestore();
+    });
+    it("should merge a document to dstPath", async () => {
+        const logicResultDoc = {
+            action: "merge",
+            priority: "normal",
+            dstPath: "/users/test-user-id/documents/test-doc-id",
+            doc: { name: "test-doc-name-updated" },
+        };
+        const expectedData = Object.assign(Object.assign({}, logicResultDoc.doc), { "@id": "test-doc-id" });
+        await (0, index_utils_1.distributeDoc)(logicResultDoc);
+        expect(admin.firestore().doc).toHaveBeenCalledTimes(1);
+        expect(admin.firestore().doc).toHaveBeenCalledWith("/users/test-user-id/documents/test-doc-id");
+        expect(docSetMock).toHaveBeenCalledTimes(1);
+        expect(docSetMock).toHaveBeenCalledWith(expectedData, { merge: true });
+    });
+    it("should merge a document to dstPath and queue instructions", async () => {
+        const logicResultDoc = {
+            action: "merge",
+            priority: "normal",
+            doc: { name: "test-doc-name-updated" },
+            instructions: {
+                "count": "++",
+                "score": "+5",
+                "minusCount": "--",
+                "minusScore": "-3",
+            },
+            dstPath: "/users/test-user-id/documents/test-doc-id",
+        };
+        const expectedData = Object.assign(Object.assign({}, logicResultDoc.doc), { "@id": "test-doc-id" });
+        await (0, index_utils_1.distributeDoc)(logicResultDoc);
+        expect(admin.firestore().doc).toHaveBeenCalledTimes(1);
+        expect(admin.firestore().doc).toHaveBeenCalledWith("/users/test-user-id/documents/test-doc-id");
+        expect(queueInstructionsSpy).toHaveBeenCalledTimes(1);
+        expect(queueInstructionsSpy).toHaveBeenCalledWith("/users/test-user-id/documents/test-doc-id", logicResultDoc.instructions);
+        expect(docSetMock).toHaveBeenCalledTimes(1);
+        expect(docSetMock).toHaveBeenCalledWith(expectedData, { merge: true });
+    });
+    it("should merge documents in batch", async () => {
+        const batchSetSpy = jest.spyOn(batch, "set").mockResolvedValue(undefined);
+        const logicResultDoc = {
+            action: "merge",
+            priority: "normal",
+            dstPath: "/users/test-user-id/documents/test-doc-id",
+        };
+        await (0, index_utils_1.distributeDoc)(logicResultDoc, batch);
+        expect(admin.firestore().doc).toHaveBeenCalledTimes(1);
+        expect(admin.firestore().doc).toHaveBeenCalledWith("/users/test-user-id/documents/test-doc-id");
+        expect(batchSetSpy).toHaveBeenCalledTimes(1);
+        batchSetSpy.mockRestore();
+    });
+    it("should queue a document to submit form", async () => {
+        const queueSubmitFormSpy = jest.spyOn(forms, "queueSubmitForm").mockResolvedValue("test-message-id");
+        const logicResultDoc = {
+            action: "submit-form",
+            priority: "normal",
+            doc: { name: "test-doc-name-updated" },
+            dstPath: "/users/test-user-id/documents/test-doc-id",
+        };
+        const formData = Object.assign({ "@docPath": logicResultDoc.dstPath, "@actionType": "create" }, logicResultDoc.doc);
+        await (0, index_utils_1.distributeDoc)(logicResultDoc, batch);
+        expect(admin.firestore().doc).toHaveBeenCalledTimes(1);
+        expect(admin.firestore().doc).toHaveBeenCalledWith("/users/test-user-id/documents/test-doc-id");
+        expect(queueSubmitFormSpy).toHaveBeenCalledTimes(1);
+        expect(queueSubmitFormSpy).toHaveBeenCalledWith(formData);
+        queueSubmitFormSpy.mockRestore();
+    });
+});
 describe("distribute", () => {
     let dbSpy;
     let colSpy;
+    let queueInstructionsSpy;
     const batch = batch_1.BatchUtil.getInstance();
     jest.spyOn(batch_1.BatchUtil, "getInstance").mockImplementation(() => batch);
     beforeEach(() => {
@@ -65,17 +180,20 @@ describe("distribute", () => {
             get: jest.fn().mockResolvedValue({ exists: true, data: () => ({}) }),
             set: jest.fn().mockResolvedValue({}),
             delete: jest.fn().mockResolvedValue({}),
+            id: "test-doc-id",
         };
         dbSpy = jest.spyOn(admin.firestore(), "doc").mockReturnValue(dbDoc);
         colSpy = jest.spyOn(admin.firestore(), "collection").mockReturnValue({
             doc: jest.fn(() => dbDoc),
         });
+        queueInstructionsSpy = jest.spyOn(distribution, "queueInstructions").mockResolvedValue();
     });
     afterEach(() => {
         dbSpy.mockRestore();
         colSpy.mockRestore();
+        queueInstructionsSpy.mockRestore();
     });
-    it("should merge a document to dstPath with instructions", async () => {
+    it("should merge a document to dstPath and queue instructions", async () => {
         const batchSetSpy = jest.spyOn(batch, "set").mockResolvedValue(undefined);
         const userDocsByDstPath = new Map([[
                 "/users/test-user-id/documents/test-doc-id",
@@ -96,12 +214,16 @@ describe("distribute", () => {
         await (0, index_utils_1.distribute)(userDocsByDstPath);
         expect(admin.firestore().doc).toHaveBeenCalledTimes(1);
         expect(admin.firestore().doc).toHaveBeenCalledWith("/users/test-user-id/documents/test-doc-id");
+        expect(queueInstructionsSpy).toHaveBeenCalledTimes(1);
+        expect(queueInstructionsSpy).toHaveBeenCalledWith("/users/test-user-id/documents/test-doc-id", {
+            "count": "++",
+            "score": "+5",
+            "minusCount": "--",
+            "minusScore": "-3",
+        });
         expect(batchSetSpy.mock.calls[0][1]).toEqual({
+            "@id": "test-doc-id",
             "name": "test-doc-name-updated",
-            "count": admin.firestore.FieldValue.increment(1),
-            "score": admin.firestore.FieldValue.increment(5),
-            "minusCount": admin.firestore.FieldValue.increment(-1),
-            "minusScore": admin.firestore.FieldValue.increment(-3),
         });
         expect(batchSetSpy).toHaveBeenCalledTimes(1);
         batchSetSpy.mockRestore();
@@ -123,15 +245,15 @@ describe("distribute", () => {
     });
 });
 describe("distributeLater", () => {
-    let submitFormSpy;
+    let queueForDistributionLaterSpy;
     beforeEach(() => {
-        submitFormSpy = jest.spyOn(adminClient, "submitForm").mockImplementation();
         (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, security_1.securityConfig, validators_1.validatorConfig, []);
+        queueForDistributionLaterSpy = jest.spyOn(distribution, "queueForDistributionLater").mockResolvedValue();
     });
     afterEach(() => {
-        submitFormSpy.mockRestore();
+        queueForDistributionLaterSpy.mockRestore();
     });
-    it("should update docs priority from normal to high and submit form for later", async () => {
+    it("should queue docs for distribution later", async () => {
         const doc1 = {
             action: "merge",
             priority: "normal",
@@ -140,36 +262,7 @@ describe("distributeLater", () => {
         };
         const doc2 = {
             action: "merge",
-            priority: "normal",
-            doc: { name: "test-doc-name-updated" },
-            dstPath: "/users/test-user-id/documents/doc2",
-        };
-        const usersDocsByDstPath = new Map([
-            ["/users/test-user-id/documents/doc1", [doc1]],
-            ["/users/test-user-id/documents/doc2", [doc2]],
-        ]);
-        const formId = "formId";
-        await (0, index_utils_1.distributeLater)(usersDocsByDstPath, formId);
-        expect(submitFormSpy).toHaveBeenCalledTimes(1);
-        expect(submitFormSpy.mock.calls[0][0]).toEqual({
-            "@docPath": `@internal/forDistribution/distributions/${formId}`,
-            "@actionType": "create",
-            "logicResultDocs": [
-                Object.assign(Object.assign({}, doc1), { priority: "high" }),
-                Object.assign(Object.assign({}, doc2), { priority: "high" }),
-            ],
-        });
-    });
-    it("should update docs priority from low to normal and submit form for later", async () => {
-        const doc1 = {
-            action: "merge",
-            priority: "low",
-            doc: { name: "test-doc-name-updated" },
-            dstPath: "/users/test-user-id/documents/doc1",
-        };
-        const doc2 = {
-            action: "merge",
-            priority: "low",
+            priority: "high",
             doc: { name: "test-doc-name-updated" },
             dstPath: "/users/test-user-id/documents/doc2",
         };
@@ -177,17 +270,9 @@ describe("distributeLater", () => {
             ["/users/test-user-id/documents/doc1", [doc1]],
             ["/users/test-user-id/documents/doc2", [doc2]],
         ]);
-        const formId = "formId";
-        await (0, index_utils_1.distributeLater)(usersDocsByDstPath, formId);
-        expect(submitFormSpy).toHaveBeenCalledTimes(1);
-        expect(submitFormSpy.mock.calls[0][0]).toEqual({
-            "@docPath": `@internal/forDistribution/distributions/${formId}`,
-            "@actionType": "create",
-            "logicResultDocs": [
-                Object.assign(Object.assign({}, doc1), { priority: "normal" }),
-                Object.assign(Object.assign({}, doc2), { priority: "normal" }),
-            ],
-        });
+        await (0, index_utils_1.distributeLater)(usersDocsByDstPath);
+        expect(queueForDistributionLaterSpy).toHaveBeenCalledTimes(1);
+        expect(queueForDistributionLaterSpy).toHaveBeenCalledWith(doc1, doc2);
     });
 });
 describe("validateForm", () => {
@@ -481,13 +566,13 @@ describe("groupDocsByUserAndDstPath", () => {
         const userId = "user123";
         const expectedResults = {
             userDocsByDstPath: new Map([
-                ["users/user123/document1", { action: "merge", priority: "normal", dstPath: "users/user123/document1", doc: { field1: "value1", field2: "value2" } }],
-                ["users/user123/document2", { action: "merge", priority: "normal", dstPath: "users/user123/document2", doc: { field3: "value3", field6: "value6" } }],
+                ["users/user123/document1", [{ action: "merge", priority: "normal", dstPath: "users/user123/document1", doc: { field1: "value1", field2: "value2" } }]],
+                ["users/user123/document2", [{ action: "merge", priority: "normal", dstPath: "users/user123/document2", doc: { field3: "value3", field6: "value6" } }]],
             ]),
             otherUsersDocsByDstPath: new Map([
-                ["users/user456/document3", { action: "merge", priority: "normal", dstPath: "users/user456/document3", doc: { field4: "value4" } }],
-                ["users/user789/document4", { action: "delete", priority: "normal", dstPath: "users/user789/document4" }],
-                ["othercollection/document5", { action: "merge", priority: "normal", dstPath: "othercollection/document5", doc: { field5: "value5" } }],
+                ["users/user456/document3", [{ action: "merge", priority: "normal", dstPath: "users/user456/document3", doc: { field4: "value4" } }]],
+                ["users/user789/document4", [{ action: "delete", priority: "normal", dstPath: "users/user789/document4" }]],
+                ["othercollection/document5", [{ action: "merge", priority: "normal", dstPath: "othercollection/document5", doc: { field5: "value5" } }]],
             ]),
         };
         const results = (0, index_utils_1.groupDocsByUserAndDstPath)(docsByDstPath, userId);
@@ -555,8 +640,10 @@ describe("expandConsolidateAndGroupByDstPath", () => {
                 timeFinished: firebase_admin_1.firestore.Timestamp.now(),
                 status: "finished",
                 documents: [
+                    { action: "create", priority: "normal", dstPath: "path8/doc8", doc: { field1: "value1" }, instructions: {} },
                     { action: "merge", priority: "normal", dstPath: "path1/doc1", doc: { field1: "value1" }, instructions: { field2: "++" } },
                     { action: "delete", priority: "normal", dstPath: "path2/doc2" },
+                    { action: "merge", priority: "normal", dstPath: "path11/doc11", doc: { field3: "value3" } },
                 ],
             },
             {
@@ -564,11 +651,13 @@ describe("expandConsolidateAndGroupByDstPath", () => {
                 timeFinished: firebase_admin_1.firestore.Timestamp.now(),
                 status: "finished",
                 documents: [
+                    { action: "merge", priority: "normal", dstPath: "path8/doc8", doc: { field3: "value3" }, instructions: { field4: "--" } },
                     { action: "merge", priority: "normal", dstPath: "path1/doc1", doc: { field1: "value1a" }, instructions: { field2: "--" } },
                     { action: "merge", priority: "normal", dstPath: "path1/doc1", doc: { field3: "value3" }, instructions: { field4: "--" } },
                     { action: "copy", priority: "normal", srcPath: "path3/doc3", dstPath: "path4/doc4" },
                     { action: "merge", priority: "normal", dstPath: "path2/doc2", doc: { field4: "value4" } },
                     { action: "merge", priority: "normal", dstPath: "path7/doc7", doc: { field6: "value7" } },
+                    { action: "create", priority: "normal", dstPath: "path10/doc10", doc: { field10: "value10" }, instructions: { field6: "++" } },
                 ],
             },
             {
@@ -580,6 +669,8 @@ describe("expandConsolidateAndGroupByDstPath", () => {
                     { action: "delete", priority: "normal", dstPath: "path7/doc7" },
                     { action: "delete", priority: "normal", dstPath: "path7/doc7" },
                     { action: "copy", priority: "normal", srcPath: "path3/doc3", dstPath: "path4/doc4" },
+                    { action: "create", priority: "normal", dstPath: "path9/doc9", instructions: { field4: "++" } },
+                    { action: "create", priority: "normal", dstPath: "path10/doc10", instructions: { field3: "--" } },
                 ],
             },
             {
@@ -589,6 +680,8 @@ describe("expandConsolidateAndGroupByDstPath", () => {
                 documents: [
                     { action: "delete", priority: "normal", dstPath: "path2/doc2" },
                     { action: "copy", priority: "normal", srcPath: "path3/doc3", dstPath: "path7/doc7" },
+                    { action: "merge", priority: "normal", dstPath: "path9/doc9", doc: { field9: "value9" } },
+                    { action: "merge", priority: "normal", dstPath: "path11/doc11", doc: { field1: "value1" }, instructions: { field6: "++" } },
                 ],
             },
         ];
@@ -597,11 +690,15 @@ describe("expandConsolidateAndGroupByDstPath", () => {
         const result = await (0, index_utils_1.expandConsolidateAndGroupByDstPath)(logicResultDocs);
         // Assert
         const expectedResult = new Map([
-            ["path1/doc1", { action: "merge", priority: "normal", dstPath: "path1/doc1", doc: { field1: "value1a", field3: "value3" }, instructions: { field2: "--", field4: "--" } }],
-            ["path2/doc2", { action: "delete", priority: "normal", dstPath: "path2/doc2" }],
-            ["path4/doc4", { action: "merge", priority: "normal", dstPath: "path4/doc4", doc: {}, instructions: {} }],
-            ["path6/doc6", { action: "merge", priority: "normal", dstPath: "path6/doc6", doc: {} }],
-            ["path7/doc7", { action: "delete", priority: "normal", dstPath: "path7/doc7" }],
+            ["path1/doc1", [{ action: "merge", priority: "normal", dstPath: "path1/doc1", doc: { field1: "value1a", field3: "value3" }, instructions: { field2: "--", field4: "--" } }]],
+            ["path2/doc2", [{ action: "delete", priority: "normal", dstPath: "path2/doc2" }]],
+            ["path4/doc4", [{ action: "merge", priority: "normal", dstPath: "path4/doc4", doc: {}, instructions: {} }]],
+            ["path7/doc7", [{ action: "delete", priority: "normal", dstPath: "path7/doc7" }]],
+            ["path6/doc6", [{ action: "merge", priority: "normal", dstPath: "path6/doc6", doc: {} }]],
+            ["path8/doc8", [{ action: "create", priority: "normal", dstPath: "path8/doc8", doc: { field1: "value1", field3: "value3" }, instructions: { field4: "--" } }]],
+            ["path9/doc9", [{ action: "create", priority: "normal", dstPath: "path9/doc9", doc: { field9: "value9" }, instructions: { field4: "++" } }]],
+            ["path10/doc10", [{ action: "create", priority: "normal", dstPath: "path10/doc10", doc: { field10: "value10" }, instructions: { field3: "--", field6: "++" } }]],
+            ["path11/doc11", [{ action: "merge", priority: "normal", dstPath: "path11/doc11", doc: { field3: "value3", field1: "value1" }, instructions: { field6: "++" } }]],
         ]);
         expect(result).toEqual(expectedResult);
         // Verify that console.warn was called
@@ -609,11 +706,11 @@ describe("expandConsolidateAndGroupByDstPath", () => {
         // Verify that console.warn was called with the correct message
         expect(consoleWarnSpy.mock.calls[0][0]).toBe("Overwriting key \"field1\" in doc for dstPath \"path1/doc1\"");
         expect(consoleWarnSpy.mock.calls[1][0]).toBe("Overwriting key \"field2\" in instructions for dstPath \"path1/doc1\"");
-        expect(consoleWarnSpy.mock.calls[2][0]).toBe("Action \"merge\" ignored because a \"delete\" for dstPath \"path2/doc2\" already exists");
-        expect(consoleWarnSpy.mock.calls[3][0]).toBe("Action \"merge\" for dstPath \"path7/doc7\" is being overwritten by action \"delete\"");
-        expect(consoleWarnSpy.mock.calls[4][0]).toBe("Action \"delete\" ignored because a \"delete\" for dstPath \"path7/doc7\" already exists");
-        expect(consoleWarnSpy.mock.calls[5][0]).toBe("Action \"delete\" ignored because a \"delete\" for dstPath \"path2/doc2\" already exists");
-        expect(consoleWarnSpy.mock.calls[6][0]).toBe("Action \"merge\" ignored because a \"delete\" for dstPath \"path7/doc7\" already exists");
+        expect(consoleWarnSpy.mock.calls[2][0]).toBe("Action merge ignored because a \"delete\" for dstPath \"path2/doc2\" already exists");
+        expect(consoleWarnSpy.mock.calls[3][0]).toBe("Action merge for dstPath \"path7/doc7\" is being overwritten by action \"delete\"");
+        expect(consoleWarnSpy.mock.calls[4][0]).toBe("Action delete for dstPath \"path7/doc7\" is being overwritten by action \"delete\"");
+        expect(consoleWarnSpy.mock.calls[5][0]).toBe("Action delete for dstPath \"path2/doc2\" is being overwritten by action \"delete\"");
+        expect(consoleWarnSpy.mock.calls[6][0]).toBe("Action merge ignored because a \"delete\" for dstPath \"path7/doc7\" already exists");
     });
     it("should expand recursive-copy logic results documents to merge logic results", async () => {
         const logicResults = [
@@ -639,31 +736,31 @@ describe("expandConsolidateAndGroupByDstPath", () => {
         const result = await (0, index_utils_1.expandConsolidateAndGroupByDstPath)(logicResultDocs);
         // Assert
         const expectedResult = new Map([
-            ["path1/doc1", {
-                    action: "merge",
-                    priority: "normal",
-                    dstPath: "path1/doc1",
-                    doc: { field1: "value1a" },
-                    instructions: { field2: "--" },
-                }],
-            ["users/456/friends/123", {
-                    action: "merge",
-                    priority: "normal",
-                    doc: friend,
-                    dstPath: "users/456/friends/123",
-                }],
-            ["users/456/friends/123/games/1", {
-                    action: "merge",
-                    priority: "normal",
-                    doc: games[0],
-                    dstPath: "users/456/friends/123/games/1",
-                }],
-            ["users/456/friends/123/games/2", {
-                    action: "merge",
-                    priority: "normal",
-                    doc: games[1],
-                    dstPath: "users/456/friends/123/games/2",
-                }],
+            ["path1/doc1", [{
+                        action: "merge",
+                        priority: "normal",
+                        dstPath: "path1/doc1",
+                        doc: { field1: "value1a" },
+                        instructions: { field2: "--" },
+                    }]],
+            ["users/456/friends/123", [{
+                        action: "merge",
+                        priority: "normal",
+                        doc: friend,
+                        dstPath: "users/456/friends/123",
+                    }]],
+            ["users/456/friends/123/games/1", [{
+                        action: "merge",
+                        priority: "normal",
+                        doc: games[0],
+                        dstPath: "users/456/friends/123/games/1",
+                    }]],
+            ["users/456/friends/123/games/2", [{
+                        action: "merge",
+                        priority: "normal",
+                        doc: games[1],
+                        dstPath: "users/456/friends/123/games/2",
+                    }]],
         ]);
         // Checks if "recursive-copy" is removed from the logic results
         expect([...result.values()].every((logicResultDocs) => logicResultDocs.every((doc) => doc.action !== "recursive-copy")))
@@ -702,28 +799,28 @@ describe("expandConsolidateAndGroupByDstPath", () => {
         const result = await (0, index_utils_1.expandConsolidateAndGroupByDstPath)(logicResultDocs);
         // Assert
         const expectedResult = new Map([
-            ["path1/doc1", {
-                    action: "merge",
-                    priority: "normal",
-                    dstPath: "path1/doc1",
-                    doc: { field1: "value1a" },
-                    instructions: { field2: "--" },
-                }],
-            ["users/123/friends/123", {
-                    action: "delete",
-                    priority: "normal",
-                    dstPath: "users/123/friends/123",
-                }],
-            ["users/123/friends/123/games/1", {
-                    action: "delete",
-                    priority: "normal",
-                    dstPath: "users/123/friends/123/games/1",
-                }],
-            ["users/123/friends/123/games/2", {
-                    action: "delete",
-                    priority: "normal",
-                    dstPath: "users/123/friends/123/games/2",
-                }],
+            ["path1/doc1", [{
+                        action: "merge",
+                        priority: "normal",
+                        dstPath: "path1/doc1",
+                        doc: { field1: "value1a" },
+                        instructions: { field2: "--" },
+                    }]],
+            ["users/123/friends/123", [{
+                        action: "delete",
+                        priority: "normal",
+                        dstPath: "users/123/friends/123",
+                    }]],
+            ["users/123/friends/123/games/1", [{
+                        action: "delete",
+                        priority: "normal",
+                        dstPath: "users/123/friends/123/games/1",
+                    }]],
+            ["users/123/friends/123/games/2", [{
+                        action: "delete",
+                        priority: "normal",
+                        dstPath: "users/123/friends/123/games/2",
+                    }]],
         ]);
         // Checks if "recursive-delete" is removed from the logic results
         expect([...result.values()].every((logicResultDocs) => logicResultDocs.every((doc) => doc.action !== "recursive-delete")))
@@ -731,7 +828,7 @@ describe("expandConsolidateAndGroupByDstPath", () => {
         expect(result).toEqual(expectedResult);
         // Verify that console.warn was called
         expect(consoleWarnSpy).toHaveBeenCalledTimes(1);
-        expect(consoleWarnSpy.mock.calls[0][0]).toBe("Action \"merge\" ignored because a \"delete\" for dstPath \"users/123/friends/123\" already exists");
+        expect(consoleWarnSpy.mock.calls[0][0]).toBe("Action merge ignored because a \"delete\" for dstPath \"users/123/friends/123\" already exists");
     });
     it("should convert copy logic results documents to merge logic results", async () => {
         const logicResults = [
@@ -758,25 +855,25 @@ describe("expandConsolidateAndGroupByDstPath", () => {
         const result = await (0, index_utils_1.expandConsolidateAndGroupByDstPath)(logicResultDocs);
         // Assert
         const expectedResult = new Map([
-            ["path1/doc1", {
-                    action: "merge",
-                    priority: "normal",
-                    dstPath: "path1/doc1",
-                    doc: { field1: "value1a" },
-                    instructions: { field2: "--" },
-                }],
-            ["users/456/friends/123/games/1", {
-                    action: "merge",
-                    priority: "normal",
-                    doc: games[0],
-                    dstPath: "users/456/friends/123/games/1",
-                }],
-            ["users/456/friends/123/games/2", {
-                    action: "merge",
-                    priority: "normal",
-                    doc: games[1],
-                    dstPath: "users/456/friends/123/games/2",
-                }],
+            ["path1/doc1", [{
+                        action: "merge",
+                        priority: "normal",
+                        dstPath: "path1/doc1",
+                        doc: { field1: "value1a" },
+                        instructions: { field2: "--" },
+                    }]],
+            ["users/456/friends/123/games/1", [{
+                        action: "merge",
+                        priority: "normal",
+                        doc: games[0],
+                        dstPath: "users/456/friends/123/games/1",
+                    }]],
+            ["users/456/friends/123/games/2", [{
+                        action: "merge",
+                        priority: "normal",
+                        doc: games[1],
+                        dstPath: "users/456/friends/123/games/2",
+                    }]],
         ]);
         // Checks if "copy" is removed from the logic results
         expect([...result.values()].every((logicResultDocs) => logicResultDocs.every((doc) => doc.action !== "copy")))
@@ -806,7 +903,6 @@ describe("runViewLogics", () => {
         jest.spyOn(index_utils_1._mockable, "getViewLogicsConfig").mockReturnValue(customViewLogicsConfig);
     });
     it("should run view logics properly", async () => {
-        const dstPathLogicDocsMap = new Map();
         const logicResult1 = {
             action: "merge",
             priority: "normal",
@@ -818,11 +914,11 @@ describe("runViewLogics", () => {
             priority: "normal",
             dstPath: "users/user124",
         };
-        dstPathLogicDocsMap.set(logicResult1.dstPath, logicResult1);
-        dstPathLogicDocsMap.set(logicResult2.dstPath, logicResult2);
         viewLogicFn1.mockResolvedValue({});
         viewLogicFn2.mockResolvedValue({});
-        const results = await (0, index_utils_1.runViewLogics)(dstPathLogicDocsMap);
+        const results1 = await (0, index_utils_1.runViewLogics)(logicResult1);
+        const results2 = await (0, index_utils_1.runViewLogics)(logicResult2);
+        const results = [...results1, ...results2];
         expect(viewLogicFn1).toHaveBeenCalledTimes(2);
         expect(viewLogicFn1.mock.calls[0][0]).toBe(logicResult1);
         expect(viewLogicFn1.mock.calls[1][0]).toBe(logicResult2);