emberflow 1.4.30 → 1.4.32

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

@@ -38,6 +38,7 @@ const forms = __importStar(require("../utils/forms"));
 const misc = __importStar(require("../utils/misc"));
 const index_utils_1 = require("../index-utils");
 const viewLogics = __importStar(require("../logics/view-logics"));
+const patch_logics_1 = require("../sample-custom/patch-logics");
 // should mock when using initializeEmberFlow and testing db.doc() calls count
 jest.spyOn(indexUtils, "createMetricLogicDoc").mockResolvedValue();
 jest.spyOn(console, "log").mockImplementation();
@@ -479,7 +480,7 @@ describe("distribute", () => {
                         dstPath: "/users/test-user-id/documents/test-doc-id",
                     }],
             ]]);
-        (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, security_1.securityConfig, validators_1.validatorConfig, []);
+        (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, security_1.securityConfigs, validators_1.validatorConfigs, [], patch_logics_1.patchLogicConfigs);
         await indexUtils.distributeFnNonTransactional(userDocsByDstPath);
         expect(admin.firestore().doc).toHaveBeenCalledTimes(1);
         expect(admin.firestore().doc).toHaveBeenCalledWith("/users/test-user-id/documents/test-doc-id");
@@ -516,7 +517,7 @@ describe("distribute", () => {
 describe("distributeLater", () => {
     let queueForDistributionLaterSpy;
     beforeEach(() => {
-        (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, security_1.securityConfig, validators_1.validatorConfig, []);
+        (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, security_1.securityConfigs, validators_1.validatorConfigs, [], patch_logics_1.patchLogicConfigs);
         queueForDistributionLaterSpy = jest.spyOn(distribution, "queueForDistributionLater").mockResolvedValue();
     });
     afterEach(() => {
@@ -539,38 +540,128 @@ describe("distributeLater", () => {
             ["/users/test-user-id/documents/doc1", [doc1]],
             ["/users/test-user-id/documents/doc2", [doc2]],
         ]);
-        await indexUtils.distributeLater(usersDocsByDstPath);
+        const targetVersion = "1.0.0";
+        await indexUtils.distributeLater(usersDocsByDstPath, targetVersion);
         expect(queueForDistributionLaterSpy).toHaveBeenCalledTimes(1);
-        expect(queueForDistributionLaterSpy).toHaveBeenCalledWith(doc1, doc2);
+        expect(queueForDistributionLaterSpy).toHaveBeenCalledWith(targetVersion, doc1, doc2);
     });
 });
 describe("validateForm", () => {
     beforeEach(() => {
-        (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, security_1.securityConfig, validators_1.validatorConfig, []);
+        (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, security_1.securityConfigs, validators_1.validatorConfigs, [], patch_logics_1.patchLogicConfigs);
+    });
+    const targetVersion = "2.5";
+    const entity = "user";
+    it("should call the appropriate version of validator", async () => {
+        const userValidation = jest.fn().mockResolvedValue({});
+        const userValidationV2 = jest.fn().mockResolvedValue({});
+        const userValidationV3 = jest.fn().mockResolvedValue({});
+        const feedValidation = jest.fn().mockResolvedValue({});
+        const feedValidationV2 = jest.fn().mockResolvedValue({});
+        const validatorConfigs = [
+            {
+                entity: db_structure_1.Entity.User,
+                validatorFn: userValidation,
+                version: "01",
+            },
+            {
+                entity: db_structure_1.Entity.User,
+                validatorFn: userValidationV2,
+                version: "02.04.06",
+            },
+            {
+                entity: db_structure_1.Entity.User,
+                validatorFn: userValidationV3,
+                version: "03",
+            },
+            {
+                entity: db_structure_1.Entity.Feed,
+                validatorFn: feedValidation,
+                version: "01",
+            },
+            {
+                entity: db_structure_1.Entity.Feed,
+                validatorFn: feedValidationV2,
+                version: "02.04.06",
+            },
+        ];
+        (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, security_1.securityConfigs, validatorConfigs, [], patch_logics_1.patchLogicConfigs);
+        const document = {
+            name: "John Doe",
+            email: "johndoe@example.com",
+            password: "abc123",
+        };
+        await indexUtils.validateForm(entity, document, targetVersion);
+        expect(userValidation).not.toHaveBeenCalled();
+        expect(userValidationV2).toHaveBeenCalledTimes(1);
+        expect(userValidationV3).not.toHaveBeenCalled();
+        expect(feedValidation).not.toHaveBeenCalled();
+        expect(feedValidationV2).not.toHaveBeenCalled();
     });
     it("returns an object with empty validationResult when document is valid", async () => {
-        const entity = "user";
         const document = {
             name: "John Doe",
             email: "johndoe@example.com",
             password: "abc123",
         };
-        const [hasValidationError, validationResult] = await indexUtils.validateForm(entity, document);
+        const [hasValidationError, validationResult] = await indexUtils.validateForm(entity, document, targetVersion);
         expect(hasValidationError).toBe(false);
         expect(validationResult).toEqual({});
     });
     it("returns an object with validation errors when document is invalid", async () => {
-        const entity = "user";
         const document = {
             name: "",
             email: "johndoe@example.com",
             password: "abc",
         };
-        const [hasValidationError, validationResult] = await indexUtils.validateForm(entity, document);
+        const [hasValidationError, validationResult] = await indexUtils.validateForm(entity, document, targetVersion);
         expect(hasValidationError).toBe(true);
         expect(validationResult).toEqual({ name: ["Name is required"] });
     });
 });
+describe("getSecurityFn", () => {
+    it("should return the appropriate version of security function", async () => {
+        const userSecurityFn = jest.fn().mockResolvedValue({});
+        const userSecurityFnV2 = jest.fn().mockResolvedValue({
+            status: "allowed",
+        });
+        const userSecurityFnV3 = jest.fn().mockResolvedValue({});
+        const feedSecurityFn = jest.fn().mockResolvedValue({});
+        const feedSecurityFnV2 = jest.fn().mockResolvedValue({});
+        const targetVersion = "2.5";
+        const entity = "user";
+        const securityConfigs = [
+            {
+                entity: db_structure_1.Entity.User,
+                securityFn: userSecurityFn,
+                version: "01",
+            },
+            {
+                entity: db_structure_1.Entity.User,
+                securityFn: userSecurityFnV2,
+                version: "02.04.06",
+            },
+            {
+                entity: db_structure_1.Entity.User,
+                securityFn: userSecurityFnV3,
+                version: "03",
+            },
+            {
+                entity: db_structure_1.Entity.Feed,
+                securityFn: feedSecurityFn,
+                version: "01",
+            },
+            {
+                entity: db_structure_1.Entity.Feed,
+                securityFn: feedSecurityFnV2,
+                version: "02.04.06",
+            },
+        ];
+        (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, securityConfigs, validators_1.validatorConfigs, [], patch_logics_1.patchLogicConfigs);
+        const result = await indexUtils.getSecurityFn(entity, targetVersion);
+        expect(result).toEqual(userSecurityFnV2);
+    });
+});
 describe("getFormModifiedFields", () => {
     it("should return an empty object when there are no form fields", () => {
         const document = { name: "John Doe", age: 30 };
@@ -622,6 +713,34 @@ describe("delayFormSubmissionAndCheckIfCancelled", () => {
         expect(formResponseRef.get).toHaveBeenCalledWith();
     });
 });
+describe("groupDocsByTargetDocPath", () => {
+    (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, [], [], [], []);
+    const docsByDstPath = new Map([
+        ["users/user123/document1", [{ action: "merge", priority: "normal", dstPath: "users/user123/document1", doc: { field1: "value1", field2: "value2" } }]],
+        ["users/user123/document1/threads/thread1", [{ action: "merge", priority: "normal", dstPath: "users/user123/document1/threads/thread1", doc: { field3: "value3", field6: "value6" } }]],
+        ["users/user123/document1/messages/message1", [{ action: "merge", priority: "normal", dstPath: "users/user123/document1/messages/message1", doc: { field4: "value4" } }]],
+        ["users/user123/document1/images/image1", [{ action: "merge", priority: "low", dstPath: "users/user123/document1/images/image1", doc: { field7: "value7" } }]],
+        ["othercollection/document2", [{ action: "merge", priority: "normal", dstPath: "othercollection/document2", doc: { field5: "value5" } }]],
+        ["othercollection/document3", [{ action: "delete", priority: "normal", dstPath: "othercollection/document3" }]],
+    ]);
+    it("should group documents by docPath", () => {
+        const docPath = "users/user123/document1";
+        const expectedResults = {
+            docsByDocPath: new Map([
+                ["users/user123/document1", [{ action: "merge", priority: "normal", dstPath: "users/user123/document1", doc: { field1: "value1", field2: "value2" } }]],
+                ["users/user123/document1/threads/thread1", [{ action: "merge", priority: "normal", dstPath: "users/user123/document1/threads/thread1", doc: { field3: "value3", field6: "value6" } }]],
+                ["users/user123/document1/messages/message1", [{ action: "merge", priority: "normal", dstPath: "users/user123/document1/messages/message1", doc: { field4: "value4" } }]],
+                ["users/user123/document1/images/image1", [{ action: "merge", priority: "low", dstPath: "users/user123/document1/images/image1", doc: { field7: "value7" } }]],
+            ]),
+            otherDocsByDocPath: new Map([
+                ["othercollection/document2", [{ action: "merge", priority: "normal", dstPath: "othercollection/document2", doc: { field5: "value5" } }]],
+                ["othercollection/document3", [{ action: "delete", priority: "normal", dstPath: "othercollection/document3" }]],
+            ]),
+        };
+        const results = indexUtils.groupDocsByTargetDocPath(docsByDstPath, docPath);
+        expect(results).toEqual(expectedResults);
+    });
+});
 describe("runBusinessLogics", () => {
     const actionType = "create";
     const formModifiedFields = { field1: "value1", field2: "value2" };
@@ -630,7 +749,6 @@ describe("runBusinessLogics", () => {
         id: "user123",
         name: "John Doe",
     };
-    const metadata = { version: "1.0" };
     const action = {
         user,
         eventContext: {
@@ -650,7 +768,8 @@ describe("runBusinessLogics", () => {
         status: "processing",
         timeCreated: firebase_admin_1.firestore.Timestamp.now(),
         modifiedFields: formModifiedFields,
-        metadata,
+        metadata: {},
+        appVersion: "1.0.0",
     };
     let distributeFn;
     let logicFn1;
@@ -699,7 +818,7 @@ describe("runBusinessLogics", () => {
         distributeFn.mockRestore();
         dbSpy.mockRestore();
     });
-    it("should call all matching logics and pass their results to distributeFn", async () => {
+    it("should call all matching logics then pass their results to distributeFn", async () => {
         const logics = [
             {
                 name: "Logic 1",
@@ -707,6 +826,7 @@ describe("runBusinessLogics", () => {
                 modifiedFields: ["field1"],
                 entities: ["user"],
                 logicFn: logicFn1,
+                version: "1",
             },
             {
                 name: "Logic 2",
@@ -714,6 +834,7 @@ describe("runBusinessLogics", () => {
                 modifiedFields: ["field2"],
                 entities: ["user"],
                 logicFn: logicFn2,
+                version: "1",
             },
             {
                 name: "Logic 3",
@@ -721,57 +842,173 @@ describe("runBusinessLogics", () => {
                 modifiedFields: ["field3"],
                 entities: ["user"],
                 logicFn: logicFn3,
+                version: "1",
             },
+        ];
+        (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, security_1.securityConfigs, validators_1.validatorConfigs, logics, patch_logics_1.patchLogicConfigs);
+        const runStatus = await indexUtils.runBusinessLogics(txnGet, action, "2");
+        expect(logicFn1).toHaveBeenCalledWith(txnGet, action, new Map());
+        expect(logicFn2).toHaveBeenCalledWith(txnGet, action, new Map());
+        expect(logicFn3).not.toHaveBeenCalled();
+        expect(runStatus).toEqual({
+            status: "done",
+            logicResults: [{
+                    status: "finished",
+                    execTime: expect.any(Number),
+                    timeFinished: expect.any(Timestamp),
+                }, {
+                    status: "finished",
+                    execTime: expect.any(Number),
+                    timeFinished: expect.any(Timestamp),
+                }],
+        });
+    });
+    it("should call all matching logics with passing addtnlFilters then pass their results " +
+        "to distributeFn", async () => {
+        const logics = [
             {
                 name: "Logic 4",
                 actionTypes: ["create"],
-                modifiedFields: ["field1"],
+                modifiedFields: ["shouldRun"],
                 entities: ["user"],
                 logicFn: logicFn4,
-                addtlFilterFn(actionType) {
-                    return actionType !== "create";
+                addtlFilterFn(_, modifiedFields) {
+                    return modifiedFields.shouldRun;
                 },
+                version: "1",
             },
             {
                 name: "Logic 5",
                 actionTypes: ["create"],
-                modifiedFields: ["field2"],
+                modifiedFields: ["shouldRun"],
                 entities: ["user"],
                 logicFn: logicFn5,
-                addtlFilterFn(actionType, modifiedFields) {
-                    return !Object.prototype.hasOwnProperty.call(modifiedFields, "field1");
+                addtlFilterFn(_, modifiedFields) {
+                    return !modifiedFields.shouldRun;
                 },
+                version: "1",
             },
             {
                 name: "Logic 6",
-                actionTypes: ["create"],
-                modifiedFields: ["field2"],
+                actionTypes: ["update"],
+                modifiedFields: ["shouldRun"],
                 entities: ["user"],
                 logicFn: logicFn6,
-                addtlFilterFn(actionType, modifiedFields, document) {
-                    return !Object.prototype.hasOwnProperty.call(document, "field3");
+                addtlFilterFn(_, modifiedFields) {
+                    return modifiedFields.shouldRun;
                 },
+                version: "1",
             },
             {
                 name: "Logic 7",
                 actionTypes: ["create"],
-                modifiedFields: ["field2"],
+                modifiedFields: ["all"],
                 entities: ["user"],
                 logicFn: logicFn7,
                 addtlFilterFn(actionType, modifiedFields, document, entity, metadata) {
                     return metadata.version === "2.0";
                 },
+                version: "1",
             },
         ];
-        (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, security_1.securityConfig, validators_1.validatorConfig, logics);
-        const runStatus = await indexUtils.runBusinessLogics(txnGet, action);
-        expect(logicFn1).toHaveBeenCalledWith(txnGet, action, new Map());
-        expect(logicFn2).toHaveBeenCalledWith(txnGet, action, new Map());
-        expect(logicFn3).not.toHaveBeenCalled();
-        expect(logicFn4).not.toHaveBeenCalled();
+        (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, security_1.securityConfigs, validators_1.validatorConfigs, logics, []);
+        const newAction = Object.assign(Object.assign({}, action), { modifiedFields: {
+                shouldRun: true,
+            } });
+        const runStatus = await indexUtils.runBusinessLogics(txnGet, newAction, "1");
+        expect(logicFn4).toHaveBeenCalledWith(txnGet, newAction, new Map());
         expect(logicFn5).not.toHaveBeenCalled();
         expect(logicFn6).not.toHaveBeenCalled();
         expect(logicFn7).not.toHaveBeenCalled();
+        expect(runStatus).toEqual({
+            status: "done",
+            logicResults: [{
+                    status: "finished",
+                    execTime: expect.any(Number),
+                    timeFinished: expect.any(Timestamp),
+                }],
+        });
+    });
+    it("should call all matching logics nearest to the target version then pass their results " +
+        "to distributeFn", async () => {
+        const logicFn1 = jest.fn().mockResolvedValue({ status: "finished" });
+        const logicFn1V2 = jest.fn().mockResolvedValue({ status: "finished" });
+        const logicFn1V3 = jest.fn().mockResolvedValue({ status: "finished" });
+        const logicFn2 = jest.fn().mockResolvedValue({ status: "finished" });
+        const logicFn2V2 = jest.fn().mockResolvedValue({ status: "finished" });
+        const logicFn3 = jest.fn().mockResolvedValue({ status: "finished" });
+        const logicFn3V2 = jest.fn().mockResolvedValue({ status: "error", message: "Error message" });
+        const logics = [
+            {
+                name: "Logic1",
+                actionTypes: ["create"],
+                modifiedFields: "all",
+                entities: ["user"],
+                logicFn: logicFn1,
+                version: "1.2.3",
+            },
+            {
+                name: "Logic1",
+                actionTypes: ["create"],
+                modifiedFields: "all",
+                entities: ["user"],
+                logicFn: logicFn1V2,
+                version: "2.4.6",
+            },
+            {
+                name: "Logic1",
+                actionTypes: ["update"],
+                modifiedFields: "all",
+                entities: ["user"],
+                logicFn: logicFn1V3,
+                version: "3.0.0",
+            },
+            {
+                name: "Logic2",
+                actionTypes: ["create"],
+                modifiedFields: ["field1"],
+                entities: ["user"],
+                logicFn: logicFn2,
+                version: "1.2.3",
+            },
+            {
+                name: "Logic2",
+                actionTypes: ["create"],
+                modifiedFields: ["field1"],
+                entities: ["user"],
+                logicFn: logicFn2V2,
+                version: "2.4.6",
+            },
+            {
+                name: "Logic3",
+                actionTypes: ["update"],
+                modifiedFields: ["field2"],
+                entities: ["user"],
+                logicFn: logicFn3,
+                version: "1.2.3",
+            },
+            {
+                name: "Logic3",
+                actionTypes: ["update"],
+                modifiedFields: ["field2"],
+                entities: ["user"],
+                logicFn: logicFn3V2,
+                version: "2.4.6",
+            },
+        ];
+        (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, security_1.securityConfigs, validators_1.validatorConfigs, logics, []);
+        const newAction = Object.assign(Object.assign({}, action), { modifiedFields: {
+                field1: true,
+            } });
+        const targetVersion = "2.5";
+        const runStatus = await indexUtils.runBusinessLogics(txnGet, newAction, targetVersion);
+        expect(logicFn1).not.toHaveBeenCalled();
+        expect(logicFn1V2).toHaveBeenCalledWith(txnGet, newAction, new Map());
+        expect(logicFn1V3).not.toHaveBeenCalled();
+        expect(logicFn2).not.toHaveBeenCalled();
+        expect(logicFn2V2).toHaveBeenCalledWith(txnGet, newAction, new Map());
+        expect(logicFn3).not.toHaveBeenCalled();
+        expect(logicFn3V2).not.toHaveBeenCalled();
         expect(runStatus).toEqual({
             status: "done",
             logicResults: [{
@@ -794,6 +1031,7 @@ describe("runBusinessLogics", () => {
                 modifiedFields: ["field1"],
                 entities: ["customentity"],
                 logicFn: logicFn1,
+                version: "1",
             },
             {
                 name: "Logic 2",
@@ -801,6 +1039,7 @@ describe("runBusinessLogics", () => {
                 modifiedFields: ["field2"],
                 entities: ["customentity"],
                 logicFn: logicFn2,
+                version: "1",
             },
             {
                 name: "Logic 3",
@@ -808,10 +1047,11 @@ describe("runBusinessLogics", () => {
                 modifiedFields: ["field3"],
                 entities: ["customentity"],
                 logicFn: logicFn3,
+                version: "1",
             },
         ];
-        (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, security_1.securityConfig, validators_1.validatorConfig, logics);
-        const runStatus = await indexUtils.runBusinessLogics(txnGet, action);
+        (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, security_1.securityConfigs, validators_1.validatorConfigs, logics, []);
+        const runStatus = await indexUtils.runBusinessLogics(txnGet, action, "1");
         expect(logicFn1).not.toHaveBeenCalled();
         expect(logicFn2).not.toHaveBeenCalled();
         expect(logicFn3).not.toHaveBeenCalled();
@@ -845,6 +1085,7 @@ describe("runBusinessLogics", () => {
                 modifiedFields: ["field1"],
                 entities: ["user"],
                 logicFn: logicFn1,
+                version: "1",
             },
             {
                 name: "Logic 2",
@@ -852,6 +1093,7 @@ describe("runBusinessLogics", () => {
                 modifiedFields: ["field2"],
                 entities: ["user"],
                 logicFn: logicFn2,
+                version: "1",
             },
             {
                 name: "Logic 3",
@@ -859,10 +1101,11 @@ describe("runBusinessLogics", () => {
                 modifiedFields: ["field2"],
                 entities: ["user"],
                 logicFn: logicFn3,
+                version: "1",
             },
         ];
-        (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, security_1.securityConfig, validators_1.validatorConfig, logics);
-        const runStatus = await indexUtils.runBusinessLogics(txnGet, action);
+        (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, security_1.securityConfigs, validators_1.validatorConfigs, logics, []);
+        const runStatus = await indexUtils.runBusinessLogics(txnGet, action, "1");
         expect(logicFn1).toHaveBeenCalledWith(txnGet, action, expectedSharedMap);
         expect(logicFn2).toHaveBeenCalledWith(txnGet, action, expectedSharedMap);
         expect(logicFn3).toHaveBeenCalledWith(txnGet, action, expectedSharedMap);
@@ -874,34 +1117,6 @@ describe("runBusinessLogics", () => {
         expect(logicResults[2].status).toEqual("finished");
     });
 });
-describe("groupDocsByTargetDocPath", () => {
-    (0, index_1.initializeEmberFlow)(projectConfig, admin, db_structure_1.dbStructure, db_structure_1.Entity, {}, {}, []);
-    const docsByDstPath = new Map([
-        ["users/user123/document1", [{ action: "merge", priority: "normal", dstPath: "users/user123/document1", doc: { field1: "value1", field2: "value2" } }]],
-        ["users/user123/document1/threads/thread1", [{ action: "merge", priority: "normal", dstPath: "users/user123/document1/threads/thread1", doc: { field3: "value3", field6: "value6" } }]],
-        ["users/user123/document1/messages/message1", [{ action: "merge", priority: "normal", dstPath: "users/user123/document1/messages/message1", doc: { field4: "value4" } }]],
-        ["users/user123/document1/images/image1", [{ action: "merge", priority: "low", dstPath: "users/user123/document1/images/image1", doc: { field7: "value7" } }]],
-        ["othercollection/document2", [{ action: "merge", priority: "normal", dstPath: "othercollection/document2", doc: { field5: "value5" } }]],
-        ["othercollection/document3", [{ action: "delete", priority: "normal", dstPath: "othercollection/document3" }]],
-    ]);
-    it("should group documents by docPath", () => {
-        const docPath = "users/user123/document1";
-        const expectedResults = {
-            docsByDocPath: new Map([
-                ["users/user123/document1", [{ action: "merge", priority: "normal", dstPath: "users/user123/document1", doc: { field1: "value1", field2: "value2" } }]],
-                ["users/user123/document1/threads/thread1", [{ action: "merge", priority: "normal", dstPath: "users/user123/document1/threads/thread1", doc: { field3: "value3", field6: "value6" } }]],
-                ["users/user123/document1/messages/message1", [{ action: "merge", priority: "normal", dstPath: "users/user123/document1/messages/message1", doc: { field4: "value4" } }]],
-                ["users/user123/document1/images/image1", [{ action: "merge", priority: "low", dstPath: "users/user123/document1/images/image1", doc: { field7: "value7" } }]],
-            ]),
-            otherDocsByDocPath: new Map([
-                ["othercollection/document2", [{ action: "merge", priority: "normal", dstPath: "othercollection/document2", doc: { field5: "value5" } }]],
-                ["othercollection/document3", [{ action: "delete", priority: "normal", dstPath: "othercollection/document3" }]],
-            ]),
-        };
-        const results = indexUtils.groupDocsByTargetDocPath(docsByDstPath, docPath);
-        expect(results).toEqual(expectedResults);
-    });
-});
 describe("expandConsolidateAndGroupByDstPath", () => {
     let dbSpy;
     let consoleWarnSpy;
@@ -1209,15 +1424,40 @@ describe("expandConsolidateAndGroupByDstPath", () => {
 });
 describe("runViewLogics", () => {
     // Define mock functions
-    const viewLogicFn1 = jest.fn();
     const viewLogicFn2 = jest.fn();
+    const viewLogicFn1V2Point5 = jest.fn();
     const customViewLogicsConfig = [
         {
             name: "logic 1",
             entity: "user",
             actionTypes: ["merge", "delete"],
             modifiedFields: ["sampleField1", "sampleField2"],
-            viewLogicFn: viewLogicFn1,
+            viewLogicFn: jest.fn(),
+            version: "1.0.0",
+        },
+        {
+            name: "logic 1",
+            entity: "user",
+            actionTypes: ["merge", "delete"],
+            modifiedFields: ["sampleField1", "sampleField2"],
+            viewLogicFn: jest.fn(),
+            version: "2.0.0",
+        },
+        {
+            name: "logic 1",
+            entity: "user",
+            actionTypes: ["merge", "delete"],
+            modifiedFields: ["sampleField1", "sampleField2"],
+            viewLogicFn: viewLogicFn1V2Point5,
+            version: "2.5.0",
+        },
+        {
+            name: "logic 1",
+            entity: "user",
+            actionTypes: ["merge", "delete"],
+            modifiedFields: ["sampleField1", "sampleField2"],
+            viewLogicFn: jest.fn(),
+            version: "3.0.0",
         },
         {
             name: "logic 2",
@@ -1225,12 +1465,14 @@ describe("runViewLogics", () => {
             actionTypes: ["merge", "delete"],
             modifiedFields: ["sampleField3"],
             viewLogicFn: viewLogicFn2,
+            version: "1.0.0",
         },
     ];
     beforeEach(() => {
-        jest.spyOn(indexUtils._mockable, "getViewLogicsConfig").mockReturnValue(customViewLogicsConfig);
+        jest.spyOn(indexUtils._mockable, "getViewLogicConfigs").mockReturnValue(customViewLogicsConfig);
     });
     it("should run view logics properly", async () => {
+        const targetVersion = "2.9.0";
         const logicResult1 = {
             action: "merge",
             priority: "normal",
@@ -1242,14 +1484,14 @@ describe("runViewLogics", () => {
             priority: "normal",
             dstPath: "users/user124",
         };
-        viewLogicFn1.mockResolvedValue({});
+        viewLogicFn1V2Point5.mockResolvedValue({});
         viewLogicFn2.mockResolvedValue({});
-        const results1 = await indexUtils.runViewLogics(logicResult1);
-        const results2 = await indexUtils.runViewLogics(logicResult2);
+        const results1 = await indexUtils.runViewLogics(logicResult1, targetVersion);
+        const results2 = await indexUtils.runViewLogics(logicResult2, targetVersion);
         const results = [...results1, ...results2];
-        expect(viewLogicFn1).toHaveBeenCalledTimes(2);
-        expect(viewLogicFn1.mock.calls[0][0]).toBe(logicResult1);
-        expect(viewLogicFn1.mock.calls[1][0]).toBe(logicResult2);
+        expect(viewLogicFn1V2Point5).toHaveBeenCalledTimes(2);
+        expect(viewLogicFn1V2Point5.mock.calls[0][0]).toBe(logicResult1);
+        expect(viewLogicFn1V2Point5.mock.calls[1][0]).toBe(logicResult2);
         expect(viewLogicFn2).toHaveBeenCalledTimes(1);
         expect(viewLogicFn2).toHaveBeenCalledWith(logicResult2);
         expect(results).toHaveLength(3);