test: added remaining gap tests from VB-1504

Author: contentstackMridul

src/visualBuilder/eventManager/__test__/useCollab.test.ts

@@ -410,3 +410,51 @@ describe("cleanup", () => {
         });
     });
 });
+
+// Integration tests using real Config to document the intentional read-only isolation design.
+// Share recipients receive COLLAB_ENABLE with fromShare:true, which sets pauseFeedback and
+// isFeedbackMode but intentionally does NOT set collab.enable. As a result, share recipients
+// never enter the collab block in mouseClick.ts (guarded by `collab.enable === true`) and
+// bypass the full collab flow by design.
+describe("read-only isolation — fromShare integration", () => {
+    let realConfig: { get: () => any; set: (key: string, value: any) => void; reset: () => void };
+    let collabEnableHandler: (data: any) => void;
+    let collabDisableHandler: (data: any) => void;
+
+    beforeEach(async () => {
+        vi.clearAllMocks();
+        const realConfigModule = await vi.importActual<{ default: typeof realConfig }>(
+            "../../../configManager/configManager"
+        );
+        realConfig = realConfigModule.default;
+        realConfig.reset();
+        vi.mocked(Config.set).mockImplementation((key, value) => realConfig.set(key, value));
+        vi.mocked(Config.get).mockImplementation(() => realConfig.get());
+        useCollab();
+        collabEnableHandler = getHandler(0);
+        collabDisableHandler = getHandler(2);
+    });
+
+    afterEach(() => {
+        realConfig.reset();
+        vi.mocked(Config.get).mockReset();
+        vi.mocked(Config.set).mockReset();
+    });
+
+    it("should set pauseFeedback but not enable after COLLAB_ENABLE with fromShare:true", () => {
+        collabEnableHandler({
+            data: { collab: { fromShare: true, pauseFeedback: true, isFeedbackMode: true } },
+        });
+
+        expect(realConfig.get().collab.pauseFeedback).toBe(true);
+        expect(realConfig.get().collab.enable).not.toBe(true);
+    });
+
+    it("should set collab.pauseFeedback when COLLAB_DISABLE fires with fromShare:true", () => {
+        collabDisableHandler({
+            data: { collab: { fromShare: true, pauseFeedback: true } },
+        });
+
+        expect(realConfig.get().collab.pauseFeedback).toBe(true);
+    });
+});

src/visualBuilder/listeners/__test__/mouseClick.test.ts

@@ -73,11 +73,15 @@ vi.mock("../../components/FieldRevert/FieldRevertComponent", () => ({
 
 vi.mock("get-xpath", () => ({ default: vi.fn().mockReturnValue("/div") }));
 
-vi.mock("../../configManager/configManager", () => ({
+const hoistedConfigMocks = vi.hoisted(() => ({
+    configGet: vi.fn().mockReturnValue({
+        collab: { enable: false, isFeedbackMode: false, pauseFeedback: false },
+    }),
+}));
+
+vi.mock("../../../configManager/configManager", () => ({
     default: {
-        get: vi.fn().mockReturnValue({
-            collab: { enable: false, isFeedbackMode: false, pauseFeedback: false },
-        }),
+        get: hoistedConfigMocks.configGet,
         set: vi.fn(),
     },
 }));
@@ -110,6 +114,7 @@ const { getCsDataOfElement } = await import("../../utils/getCsDataOfElement");
 const { addFocusOverlay } = await import("../../generators/generateOverlay");
 const { handleIndividualFields } = await import("../../utils/handleIndividualFields");
 const { isCustomFieldMultipleInstance } = await import("../../utils/isCustomFieldMultipleInstance");
+const { generateThread, toggleCollabPopup } = await import("../../generators/generateThread");
 
 function makeEditableElement(): HTMLElement {
     const el = document.createElement("div");
@@ -210,3 +215,38 @@ describe("handleBuilderInteraction — custom field multiple instance suppressio
         expect(addFocusOverlay).toHaveBeenCalled();
     });
 });
+
+describe("handleBuilderInteraction — pauseFeedback guard", () => {
+    let editableElement: HTMLElement;
+
+    beforeEach(() => {
+        vi.clearAllMocks();
+        document.body.innerHTML = "";
+        editableElement = makeEditableElement();
+        VisualBuilder.VisualBuilderGlobalState.value.previousSelectedEditableDOM = null;
+        VisualBuilder.VisualBuilderGlobalState.value.isFocussed = false;
+    });
+
+    it("should return early without processing click when collab is enabled and pauseFeedback is true", async () => {
+        hoistedConfigMocks.configGet.mockReturnValue({
+            collab: { enable: true, pauseFeedback: true, isFeedbackMode: true },
+        });
+
+        await handleBuilderInteraction(makeParams(editableElement));
+
+        expect(getCsDataOfElement).not.toHaveBeenCalled();
+        expect(generateThread).not.toHaveBeenCalled();
+        expect(toggleCollabPopup).not.toHaveBeenCalled();
+        expect(addFocusOverlay).not.toHaveBeenCalled();
+    });
+
+    it("should call generateThread when collab is enabled and pauseFeedback is false", async () => {
+        hoistedConfigMocks.configGet.mockReturnValue({
+            collab: { enable: true, pauseFeedback: false, isFeedbackMode: true },
+        });
+
+        await handleBuilderInteraction(makeParams(editableElement));
+
+        expect(generateThread).toHaveBeenCalled();
+    });
+});