@thoughtspot/visual-embed-sdk 1.48.0 → 1.49.0

package/src/embed/ts-embed.spec.ts

@@ -133,9 +133,9 @@ const getMockAppInitPayload = (data: any) => {
         customVariablesForThirdPartyTools,
         interceptTimeout: undefined,
         interceptUrls: [],
-        shouldBypassPayloadValidation:undefined,
-        useHostEventsV2:undefined,
-        embedExpiryInAuthToken:true
+        shouldBypassPayloadValidation: undefined,
+        useHostEventsV2: undefined,
+        embedExpiryInAuthToken: true,
     };
     return {
         type: EmbedEvent.APP_INIT,
@@ -144,7 +144,7 @@ const getMockAppInitPayload = (data: any) => {
             ...data,
         },
     };
-}
+};
 
 describe('Unit test case for ts embed', () => {
     const mockMixPanelEvent = jest.spyOn(mixpanelInstance, 'uploadMixpanelEvent');
@@ -223,9 +223,7 @@ describe('Unit test case for ts embed', () => {
             liveboardEmbed.render();
             mockProcessTrigger.mockResolvedValue({ session: 'test' });
             await executeAfterWait(async () => {
-                await liveboardEmbed.trigger(
-                    HostEvent.Save,
-                );
+                await liveboardEmbed.trigger(HostEvent.Save);
                 expect(mockProcessTrigger).toHaveBeenCalledWith(
                     getIFrameEl(),
                     HostEvent.Save,
@@ -244,10 +242,7 @@ describe('Unit test case for ts embed', () => {
             liveboardEmbed.render();
             mockProcessTrigger.mockResolvedValue({ session: 'test' });
             await executeAfterWait(async () => {
-                await liveboardEmbed.trigger(
-                    HostEvent.Save,
-                    false,
-                );
+                await liveboardEmbed.trigger(HostEvent.Save, false);
                 expect(mockProcessTrigger).toHaveBeenCalledWith(
                     getIFrameEl(),
                     HostEvent.Save,
@@ -398,9 +393,11 @@ describe('Unit test case for ts embed', () => {
                 postMessageToParent(iframe.contentWindow, mockEmbedEventPayload, mockPort);
             });
             await executeAfterWait(() => {
-                expect(mockPort.postMessage).toHaveBeenCalledWith(getMockAppInitPayload({
-                    customisations: customisationsView,
-                }));
+                expect(mockPort.postMessage).toHaveBeenCalledWith(
+                    getMockAppInitPayload({
+                        customisations: customisationsView,
+                    }),
+                );
             });
         });
 
@@ -427,9 +424,11 @@ describe('Unit test case for ts embed', () => {
                 postMessageToParent(iframe.contentWindow, mockEmbedEventPayload, mockPort);
             });
             await executeAfterWait(() => {
-                expect(mockPort.postMessage).toHaveBeenCalledWith(getMockAppInitPayload({
-                    hiddenHomepageModules: [HomepageModule.MyLibrary, HomepageModule.Learning],
-                }));
+                expect(mockPort.postMessage).toHaveBeenCalledWith(
+                    getMockAppInitPayload({
+                        hiddenHomepageModules: [HomepageModule.MyLibrary, HomepageModule.Learning],
+                    }),
+                );
             });
         });
 
@@ -480,10 +479,14 @@ describe('Unit test case for ts embed', () => {
                 postMessageToParent(iframe.contentWindow, mockEmbedEventPayload, mockPort);
             });
             await executeAfterWait(() => {
-                expect(mockPort.postMessage).toHaveBeenCalledWith(getMockAppInitPayload({
-                    reorderedHomepageModules:
-                        [HomepageModule.MyLibrary, HomepageModule.Watchlist],
-                }));
+                expect(mockPort.postMessage).toHaveBeenCalledWith(
+                    getMockAppInitPayload({
+                        reorderedHomepageModules: [
+                            HomepageModule.MyLibrary,
+                            HomepageModule.Watchlist,
+                        ],
+                    }),
+                );
             });
         });
 
@@ -513,9 +516,11 @@ describe('Unit test case for ts embed', () => {
                 postMessageToParent(iframe.contentWindow, mockEmbedEventPayload, mockPort);
             });
             await executeAfterWait(() => {
-                expect(mockPort.postMessage).toHaveBeenCalledWith(getMockAppInitPayload({
-                    runtimeParameterParams: 'param1=color&paramVal1=blue',
-                }));
+                expect(mockPort.postMessage).toHaveBeenCalledWith(
+                    getMockAppInitPayload({
+                        runtimeParameterParams: 'param1=color&paramVal1=blue',
+                    }),
+                );
             });
         });
 
@@ -546,9 +551,11 @@ describe('Unit test case for ts embed', () => {
                 postMessageToParent(iframe.contentWindow, mockEmbedEventPayload, mockPort);
             });
             await executeAfterWait(() => {
-                expect(mockPort.postMessage).toHaveBeenCalledWith(getMockAppInitPayload({
-                    runtimeFilterParams: 'col1=color&op1=EQ&val1=blue',
-                }));
+                expect(mockPort.postMessage).toHaveBeenCalledWith(
+                    getMockAppInitPayload({
+                        runtimeFilterParams: 'col1=color&op1=EQ&val1=blue',
+                    }),
+                );
             });
         });
 
@@ -638,10 +645,14 @@ describe('Unit test case for ts embed', () => {
             });
 
             await executeAfterWait(() => {
-                expect(mockPort.postMessage).toHaveBeenCalledWith(getMockAppInitPayload({
-                    hiddenHomeLeftNavItems:
-                        [HomeLeftNavItem.Home, HomeLeftNavItem.MonitorSubscription],
-                }));
+                expect(mockPort.postMessage).toHaveBeenCalledWith(
+                    getMockAppInitPayload({
+                        hiddenHomeLeftNavItems: [
+                            HomeLeftNavItem.Home,
+                            HomeLeftNavItem.MonitorSubscription,
+                        ],
+                    }),
+                );
             });
         });
 
@@ -824,10 +835,12 @@ describe('Unit test case for ts embed', () => {
                 postMessageToParent(iframe.contentWindow, mockEmbedEventPayload, mockPort);
             });
             await executeAfterWait(() => {
-                expect(mockPort.postMessage).toHaveBeenCalledWith(getMockAppInitPayload({
-                    authToken: 'test_auth_token1',
-                    customVariablesForThirdPartyTools: {},
-                }));
+                expect(mockPort.postMessage).toHaveBeenCalledWith(
+                    getMockAppInitPayload({
+                        authToken: 'test_auth_token1',
+                        customVariablesForThirdPartyTools: {},
+                    }),
+                );
             });
 
             jest.spyOn(authService, 'verifyTokenService').mockClear();
@@ -878,37 +891,40 @@ describe('Unit test case for ts embed', () => {
 
             await executeAfterWait(() => {
                 const iframe = getIFrameEl();
-                expect(iframe.src).toContain('overrideStringIDsUrl=https://sample-string-ids-url.com');
+                expect(iframe.src).toContain(
+                    'overrideStringIDsUrl=https://sample-string-ids-url.com',
+                );
                 postMessageToParent(iframe.contentWindow, mockEmbedEventPayload, mockPort);
             });
 
             await executeAfterWait(() => {
-                expect(mockPort.postMessage).toHaveBeenCalledWith(getMockAppInitPayload({
-                    customisations: {
-                        content: {
-                            strings: {
-                                Liveboard: 'Dashboard',
+                expect(mockPort.postMessage).toHaveBeenCalledWith(
+                    getMockAppInitPayload({
+                        customisations: {
+                            content: {
+                                strings: {
+                                    Liveboard: 'Dashboard',
+                                },
+                                stringIDsUrl: 'https://sample-string-ids-url.com',
+                                stringIDs: {
+                                    'liveboard.header.title': 'Dashboard name',
+                                },
                             },
-                            stringIDsUrl: 'https://sample-string-ids-url.com',
-                            stringIDs: {
-                                'liveboard.header.title': 'Dashboard name',
+                            style: {
+                                customCSS: {},
+                                customCSSUrl: undefined,
                             },
                         },
-                        style: {
-                            customCSS: {},
-                            customCSSUrl: undefined,
-                        },
-                    },
-                    authToken: 'test_auth_token1',
-                    customVariablesForThirdPartyTools: {},
-                }));
-                const customisationContent = mockPort.postMessage.mock.calls[0][0].data.customisations.content;
-                expect(customisationContent.stringIDsUrl)
-                    .toBe('https://sample-string-ids-url.com');
-                expect(customisationContent.stringIDs)
-                    .toEqual({
-                        'liveboard.header.title': 'Dashboard name',
-                    });
+                        authToken: 'test_auth_token1',
+                        customVariablesForThirdPartyTools: {},
+                    }),
+                );
+                const customisationContent =
+                    mockPort.postMessage.mock.calls[0][0].data.customisations.content;
+                expect(customisationContent.stringIDsUrl).toBe('https://sample-string-ids-url.com');
+                expect(customisationContent.stringIDs).toEqual({
+                    'liveboard.header.title': 'Dashboard name',
+                });
             });
         });
 
@@ -917,7 +933,10 @@ describe('Unit test case for ts embed', () => {
                 type: EmbedEvent.APP_INIT,
                 data: {},
             };
-            const searchEmbed = new SearchEmbed(getRootEl(), { ...defaultViewConfig, exposeTranslationIDs: true });
+            const searchEmbed = new SearchEmbed(getRootEl(), {
+                ...defaultViewConfig,
+                exposeTranslationIDs: true,
+            });
             searchEmbed.render();
             const mockPort: any = {
                 postMessage: jest.fn(),
@@ -963,16 +982,16 @@ describe('Unit test case for ts embed', () => {
                         name: 'Valid Action',
                         target: CustomActionTarget.LIVEBOARD,
                         position: CustomActionsPosition.PRIMARY,
-                        metadataIds: { liveboardIds: ['lb123'] }
+                        metadataIds: { liveboardIds: ['lb123'] },
                     },
                     {
                         id: 'action2',
                         name: 'Another Valid Action',
                         target: CustomActionTarget.VIZ,
                         position: CustomActionsPosition.MENU,
-                        metadataIds: { vizIds: ['viz456'] }
-                    }
-                ]
+                        metadataIds: { vizIds: ['viz456'] },
+                    },
+                ],
             });
 
             searchEmbed.render();
@@ -986,33 +1005,35 @@ describe('Unit test case for ts embed', () => {
             });
 
             await executeAfterWait(() => {
-                expect(mockPort.postMessage).toHaveBeenCalledWith(getMockAppInitPayload({
-                    customisations: {
-                        content: {},
-                        style: {
-                            customCSS: {},
-                            customCSSUrl: undefined,
-                        },
-                    },
-                    authToken: 'test_auth_token1',
-                    customActions: [
-                        {
-                            id: 'action2',
-                            name: 'Another Valid Action',
-                            target: CustomActionTarget.VIZ,
-                            position: CustomActionsPosition.MENU,
-                            metadataIds: { vizIds: ['viz456'] }
+                expect(mockPort.postMessage).toHaveBeenCalledWith(
+                    getMockAppInitPayload({
+                        customisations: {
+                            content: {},
+                            style: {
+                                customCSS: {},
+                                customCSSUrl: undefined,
+                            },
                         },
-                        {
-                            id: 'action1',
-                            name: 'Valid Action',
-                            target: CustomActionTarget.LIVEBOARD,
-                            position: CustomActionsPosition.PRIMARY,
-                            metadataIds: { liveboardIds: ['lb123'] }
-                        }
-                    ], // Actions should be sorted by name
-                    customVariablesForThirdPartyTools: {},
-                }));
+                        authToken: 'test_auth_token1',
+                        customActions: [
+                            {
+                                id: 'action2',
+                                name: 'Another Valid Action',
+                                target: CustomActionTarget.VIZ,
+                                position: CustomActionsPosition.MENU,
+                                metadataIds: { vizIds: ['viz456'] },
+                            },
+                            {
+                                id: 'action1',
+                                name: 'Valid Action',
+                                target: CustomActionTarget.LIVEBOARD,
+                                position: CustomActionsPosition.PRIMARY,
+                                metadataIds: { liveboardIds: ['lb123'] },
+                            },
+                        ], // Actions should be sorted by name
+                        customVariablesForThirdPartyTools: {},
+                    }),
+                );
 
                 // Verify that CustomActionsValidationResult structure is
                 // correct
@@ -1024,15 +1045,15 @@ describe('Unit test case for ts embed', () => {
                             id: 'action1',
                             name: 'Valid Action',
                             target: CustomActionTarget.LIVEBOARD,
-                            position: CustomActionsPosition.PRIMARY
+                            position: CustomActionsPosition.PRIMARY,
                         }),
                         expect.objectContaining({
                             id: 'action2',
                             name: 'Another Valid Action',
                             target: CustomActionTarget.VIZ,
-                            position: CustomActionsPosition.MENU
-                        })
-                    ])
+                            position: CustomActionsPosition.MENU,
+                        }),
+                    ]),
                 );
 
                 // Verify actions are sorted by name (alphabetically)
@@ -1094,7 +1115,10 @@ describe('Unit test case for ts embed', () => {
                 type: EmbedEvent.APP_INIT,
                 data: {},
             };
-            const searchEmbed = new SearchEmbed(getRootEl(), { ...defaultViewConfig, preRenderId: 'test' });
+            const searchEmbed = new SearchEmbed(getRootEl(), {
+                ...defaultViewConfig,
+                preRenderId: 'test',
+            });
             searchEmbed.preRender();
             const mockPort: any = {
                 postMessage: jest.fn(),
@@ -1153,7 +1177,10 @@ describe('Unit test case for ts embed', () => {
                 type: EmbedEvent.AuthExpire,
                 data: {},
             };
-            const searchEmbed = new SearchEmbed(getRootEl(), { ...defaultViewConfig, preRenderId: 'test' });
+            const searchEmbed = new SearchEmbed(getRootEl(), {
+                ...defaultViewConfig,
+                preRenderId: 'test',
+            });
             jest.spyOn(baseInstance, 'notifyAuthFailure');
             searchEmbed.preRender();
             const loggerSpy = jest.spyOn(logger, 'error').mockImplementation(() => {});
@@ -1318,23 +1345,27 @@ describe('Unit test case for ts embed', () => {
             let mockGetPreauthInfo = null;
 
             if (overrideOrgId) {
-                mockGetPreauthInfo = jest.spyOn(sessionInfoService, 'getPreauthInfo').mockImplementation(jest.fn());
+                mockGetPreauthInfo = jest
+                    .spyOn(sessionInfoService, 'getPreauthInfo')
+                    .mockImplementation(jest.fn());
             }
 
-            const mockPreauthInfoFetch = jest.spyOn(authService, 'fetchPreauthInfoService').mockResolvedValueOnce({
-                ok: true,
-                headers: new Headers({ 'content-type': 'application/json' }), // Mock headers correctly
-                json: async () => ({
-                    info: {
-                        configInfo: {
-                            mixpanelConfig: {
-                                devSdkKey: 'devSdkKey',
+            const mockPreauthInfoFetch = jest
+                .spyOn(authService, 'fetchPreauthInfoService')
+                .mockResolvedValueOnce({
+                    ok: true,
+                    headers: new Headers({ 'content-type': 'application/json' }), // Mock headers correctly
+                    json: async () => ({
+                        info: {
+                            configInfo: {
+                                mixpanelConfig: {
+                                    devSdkKey: 'devSdkKey',
+                                },
                             },
+                            userGUID: 'userGUID',
                         },
-                        userGUID: 'userGUID',
-                    },
-                }), // Mock JSON response
-            });
+                    }), // Mock JSON response
+                });
             const iFrame: any = document.createElement('div');
             jest.spyOn(baseInstance, 'getAuthPromise').mockResolvedValueOnce(isLoggedIn);
             const tsEmbed = new SearchEmbed(getRootEl(), {
@@ -1360,10 +1391,7 @@ describe('Unit test case for ts embed', () => {
         };
 
         test('should call InfoSuccess Event on preauth call success', async () => {
-            const {
-                mockPreauthInfoFetch,
-                iFrame,
-            } = await setup(true);
+            const { mockPreauthInfoFetch, iFrame } = await setup(true);
             expect(mockPreauthInfoFetch).toHaveBeenCalledTimes(1);
 
             await executeAfterWait(() => {
@@ -1378,9 +1406,7 @@ describe('Unit test case for ts embed', () => {
         });
 
         test('should not call InfoSuccess Event if overrideOrgId is true', async () => {
-            const {
-                mockGetPreauthInfo,
-            } = await setup(true, 123);
+            const { mockGetPreauthInfo } = await setup(true, 123);
             expect(mockGetPreauthInfo).toHaveBeenCalledTimes(0);
         });
     });
@@ -1405,7 +1431,7 @@ describe('Unit test case for ts embed', () => {
             embedType: 'AppEmbed' | 'SearchEmbed',
             showPrimaryNavbar?: boolean,
             overrideOrgId?: number,
-            disablePreauthCache?: boolean
+            disablePreauthCache?: boolean,
         ) => {
             jest.spyOn(window, 'addEventListener').mockImplementationOnce(
                 (event, handler, options) => {
@@ -1422,18 +1448,23 @@ describe('Unit test case for ts embed', () => {
             let mockGetPreauthInfo = null;
 
             // Determine if preauth cache should be enabled
-            const isAppEmbedWithPrimaryNavbar = embedType === 'AppEmbed' && showPrimaryNavbar === true;
-            const shouldDisableCache = overrideOrgId || disablePreauthCache || isAppEmbedWithPrimaryNavbar;
+            const isAppEmbedWithPrimaryNavbar =
+                embedType === 'AppEmbed' && showPrimaryNavbar === true;
+            const shouldDisableCache =
+                overrideOrgId || disablePreauthCache || isAppEmbedWithPrimaryNavbar;
 
             if (shouldDisableCache) {
-                mockGetPreauthInfo = jest.spyOn(sessionInfoService, 'getPreauthInfo')
+                mockGetPreauthInfo = jest
+                    .spyOn(sessionInfoService, 'getPreauthInfo')
                     .mockImplementation(jest.fn());
             } else {
-                mockGetPreauthInfo = jest.spyOn(sessionInfoService, 'getPreauthInfo')
-                 .mockResolvedValue({ info: { test: 'data' } } as any);
+                mockGetPreauthInfo = jest
+                    .spyOn(sessionInfoService, 'getPreauthInfo')
+                    .mockResolvedValue({ info: { test: 'data' } } as any);
             }
 
-            const mockPreauthInfoFetch = jest.spyOn(authService, 'fetchPreauthInfoService')
+            const mockPreauthInfoFetch = jest
+                .spyOn(authService, 'fetchPreauthInfoService')
                 .mockResolvedValueOnce({
                     ok: true,
                     headers: new Headers({ 'content-type': 'application/json' }),
@@ -1568,7 +1599,12 @@ describe('Unit test case for ts embed', () => {
         });
 
         test('should disable preauth cache for FullAppEmbed with disablePreauthCache = true', async () => {
-            const { mockGetPreauthInfo } = await setupPreauthTest('AppEmbed', false, undefined, true);
+            const { mockGetPreauthInfo } = await setupPreauthTest(
+                'AppEmbed',
+                false,
+                undefined,
+                true,
+            );
 
             await executeAfterWait(() => {
                 expect(mockGetPreauthInfo).toHaveBeenCalledTimes(0);
@@ -1634,9 +1670,12 @@ describe('Unit test case for ts embed', () => {
 
         test('mixpanel should call with VISUAL_SDK_RENDER_FAILED', () => {
             expect(mockMixPanelEvent).toHaveBeenCalledWith(MIXPANEL_EVENT.VISUAL_SDK_RENDER_START);
-            expect(mockMixPanelEvent).toHaveBeenCalledWith(MIXPANEL_EVENT.VISUAL_SDK_RENDER_FAILED, {
-                error: 'false',
-            });
+            expect(mockMixPanelEvent).toHaveBeenCalledWith(
+                MIXPANEL_EVENT.VISUAL_SDK_RENDER_FAILED,
+                {
+                    error: 'false',
+                },
+            );
         });
     });
 
@@ -2003,8 +2042,8 @@ describe('Unit test case for ts embed', () => {
             await appEmbed.render();
             expectUrlMatchesWithParams(
                 getIFrameSrc(),
-                `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&${defaultParamsForPinboardEmbed}`
-                + `&foo=bar&baz=1&bool=true${defaultParamsPost}#/home`,
+                `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&${defaultParamsForPinboardEmbed}` +
+                    `&foo=bar&baz=1&bool=true${defaultParamsPost}#/home`,
             );
         });
 
@@ -2032,8 +2071,8 @@ describe('Unit test case for ts embed', () => {
             await appEmbed.render();
             expectUrlMatchesWithParams(
                 getIFrameSrc(),
-                `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&${defaultParamsForPinboardEmbed}`
-                + `&foo=bar&foo2=bar2&foo3=false&baz=1&bool=true${defaultParamsPost}#/home`,
+                `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&${defaultParamsForPinboardEmbed}` +
+                    `&foo=bar&foo2=bar2&foo3=false&baz=1&bool=true${defaultParamsPost}#/home`,
             );
         });
 
@@ -2048,8 +2087,8 @@ describe('Unit test case for ts embed', () => {
             await appEmbed.render();
             expectUrlMatchesWithParams(
                 getIFrameSrc(),
-                `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&${defaultParamsForPinboardEmbed}`
-                + `&showAlerts=true${defaultParamsPost}#/home`,
+                `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&${defaultParamsForPinboardEmbed}` +
+                    `&showAlerts=true${defaultParamsPost}#/home`,
             );
         });
         it('Sets the locale param', async () => {
@@ -2063,8 +2102,8 @@ describe('Unit test case for ts embed', () => {
             await appEmbed.render();
             expectUrlMatchesWithParams(
                 getIFrameSrc(),
-                `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&${defaultParamsForPinboardEmbed}`
-                + `&locale=ja-JP${defaultParamsPost}#/home`,
+                `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&${defaultParamsForPinboardEmbed}` +
+                    `&locale=ja-JP${defaultParamsPost}#/home`,
             );
         });
         it('Sets both params when enableLinkOverridesV2 is set', async () => {
@@ -2082,7 +2121,7 @@ describe('Unit test case for ts embed', () => {
                 `http://${thoughtSpotHost}/?embedApp=true&${defaultParamsForPinboardEmbed}&enableLinkOverridesV2=true&linkOverride=true${defaultParamsPost}#/embed/viz/test-lb`,
             );
         });
-        it('Sets only linkOverride when enableLinkOverridesV2 is not set', async () => {
+        it('Auto-upgrades V1 linkOverride to V2 (sends both flags)', async () => {
             const liveboardEmbed = new LiveboardEmbed(getRootEl(), {
                 frameParams: {
                     width: '100%',
@@ -2092,10 +2131,40 @@ describe('Unit test case for ts embed', () => {
                 linkOverride: true,
             });
             await liveboardEmbed.render();
-            expectUrlMatchesWithParams(
-                getIFrameSrc(),
-                `http://${thoughtSpotHost}/?embedApp=true&${defaultParamsForPinboardEmbed}&linkOverride=true${defaultParamsPost}#/embed/viz/test-lb`,
-            );
+            const src = getIFrameSrc();
+            expect(src).toContain('linkOverride=true');
+            expect(src).toContain('enableLinkOverridesV2=true');
+        });
+        it('Auto-disables V2 link overrides when disableRedirectionLinksInNewTab is true', async () => {
+            const liveboardEmbed = new LiveboardEmbed(getRootEl(), {
+                frameParams: {
+                    width: '100%',
+                    height: '100%',
+                },
+                liveboardId: 'test-lb',
+                enableLinkOverridesV2: true,
+                disableRedirectionLinksInNewTab: true,
+            });
+            await liveboardEmbed.render();
+            const src = getIFrameSrc();
+            expect(src).not.toContain('enableLinkOverridesV2=true');
+            expect(src).not.toContain('linkOverride=true');
+            expect(src).toContain('disableRedirectionLinksInNewTab=true');
+        });
+        it('Auto-disables V1 link override when disableRedirectionLinksInNewTab is true', async () => {
+            const liveboardEmbed = new LiveboardEmbed(getRootEl(), {
+                frameParams: {
+                    width: '100%',
+                    height: '100%',
+                },
+                liveboardId: 'test-lb',
+                linkOverride: true,
+                disableRedirectionLinksInNewTab: true,
+            });
+            await liveboardEmbed.render();
+            const src = getIFrameSrc();
+            expect(src).not.toContain('linkOverride=true');
+            expect(src).toContain('disableRedirectionLinksInNewTab=true');
         });
         it('Sets the iconSprite url', async () => {
             const appEmbed = new AppEmbed(getRootEl(), {
@@ -2110,8 +2179,8 @@ describe('Unit test case for ts embed', () => {
             await appEmbed.render();
             expectUrlMatchesWithParams(
                 getIFrameSrc(),
-                `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&${defaultParamsForPinboardEmbed}`
-                + `&iconSprite=iconSprite.com${defaultParamsPost}#/home`,
+                `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&${defaultParamsForPinboardEmbed}` +
+                    `&iconSprite=iconSprite.com${defaultParamsPost}#/home`,
             );
         });
 
@@ -2391,8 +2460,9 @@ describe('Unit test case for ts embed', () => {
             tsEmbedDiv.style.height = '100px';
 
             let resizeObserverCb: any;
-            (window as any).ResizeObserver = window.ResizeObserver
-                || jest.fn().mockImplementation((resizeObserverCbParam) => {
+            (window as any).ResizeObserver =
+                window.ResizeObserver ||
+                jest.fn().mockImplementation((resizeObserverCbParam) => {
                     resizeObserverCb = resizeObserverCbParam;
                     return {
                         disconnect: jest.fn(),
@@ -2406,9 +2476,12 @@ describe('Unit test case for ts embed', () => {
             libEmbed.showPreRender();
             expect(warnSpy).toHaveBeenCalledTimes(1);
 
+            // The ResizeObserver now tracks the placeholder inside this.el,
+            // not this.el itself, so pass it as the target.
+            const preRenderPlaceholder = tsEmbedDiv.firstElementChild as HTMLElement;
             resizeObserverCb([
                 {
-                    target: tsEmbedDiv,
+                    target: preRenderPlaceholder,
                     contentRect: { height: 297, width: 987 },
                 },
             ]);
@@ -2473,8 +2546,9 @@ describe('Unit test case for ts embed', () => {
         it('should set overflow:hidden when hidePreRender and remove when showPreRender', async () => {
             createRootEleForEmbed();
 
-            (window as any).ResizeObserver = window.ResizeObserver
-                || jest.fn().mockImplementation(() => ({
+            (window as any).ResizeObserver =
+                window.ResizeObserver ||
+                jest.fn().mockImplementation(() => ({
                     disconnect: jest.fn(),
                     observe: jest.fn(),
                     unobserve: jest.fn(),
@@ -2511,8 +2585,9 @@ describe('Unit test case for ts embed', () => {
         it('it should connect with another object', async () => {
             createRootEleForEmbed();
             mockMessageChannel();
-            (window as any).ResizeObserver = window.ResizeObserver
-                || jest.fn().mockImplementation(() => ({
+            (window as any).ResizeObserver =
+                window.ResizeObserver ||
+                jest.fn().mockImplementation(() => ({
                     disconnect: jest.fn(),
                     observe: jest.fn(),
                     unobserve: jest.fn(),
@@ -2524,7 +2599,6 @@ describe('Unit test case for ts embed', () => {
 
             libEmbed.preRender();
             await waitFor(() => !!getIFrameEl());
-            const warnSpy = jest.spyOn(logger, 'warn');
             const newEmbed = new LiveboardEmbed('#tsEmbedDiv', {
                 preRenderId: 'i-am-preRendered',
                 liveboardId: 'awdawda',
@@ -2532,9 +2606,10 @@ describe('Unit test case for ts embed', () => {
                 frameParams: { height: 90 },
             });
 
-            newEmbed.showPreRender();
+            await newEmbed.showPreRender();
 
-            expect(warnSpy).toHaveBeenCalledTimes(2);
+            // Verify newEmbed successfully connected to the existing preRender
+            expect((newEmbed as any).isPreRenderConnected()).toBe(true);
         });
         it('showPreRender should not preRender if not available', async () => {
             createRootEleForEmbed();
@@ -2711,7 +2786,10 @@ describe('Unit test case for ts embed', () => {
             document.body.innerHTML = getDocumentBody();
         });
         test('Pre-render should wait for init to complete', async () => {
-            const lib = new LiveboardEmbed(getRootEl(), { preRenderId: 'test', liveboardId: 'test' });
+            const lib = new LiveboardEmbed(getRootEl(), {
+                preRenderId: 'test',
+                liveboardId: 'test',
+            });
             lib.preRender();
             await executeAfterWait(() => {
                 expect(errorSpy).toHaveBeenCalledWith(ERROR_MESSAGE.RENDER_CALLED_BEFORE_INIT);
@@ -2786,10 +2864,12 @@ describe('Unit test case for ts embed', () => {
             jest.clearAllMocks();
             document.body.innerHTML = getDocumentBody();
             mockPort.postMessage.mockClear();
-            jest.spyOn(authToken, 'getAuthenticationToken').mockResolvedValue('mock-test-token-placeholder');
+            jest.spyOn(authToken, 'getAuthenticationToken').mockResolvedValue(
+                'mock-test-token-placeholder',
+            );
 
             jest.spyOn(baseInstance, 'handleAuth').mockImplementation(() => Promise.resolve(true));
-            jest.spyOn(baseInstance, 'notifyAuthFailure').mockImplementation(() => { });
+            jest.spyOn(baseInstance, 'notifyAuthFailure').mockImplementation(() => {});
         });
 
         const renderAndTriggerAuthExpire = async () => {
@@ -2902,7 +2982,7 @@ describe('Unit test case for ts embed', () => {
 
         afterEach(() => {
             expect(baseInstance.notifyAuthFailure).toHaveBeenCalledWith(
-                authInstance.AuthFailureType.EXPIRY
+                authInstance.AuthFailureType.EXPIRY,
             );
         });
     });
@@ -2915,8 +2995,10 @@ describe('Unit test case for ts embed', () => {
             jest.clearAllMocks();
             document.body.innerHTML = getDocumentBody();
             mockPort.postMessage.mockClear();
-            jest.spyOn(authToken, 'getAuthenticationToken').mockResolvedValue('mock-test-token-placeholder');
-            jest.spyOn(processData, 'processAuthFailure').mockImplementation(() => ({} as any));
+            jest.spyOn(authToken, 'getAuthenticationToken').mockResolvedValue(
+                'mock-test-token-placeholder',
+            );
+            jest.spyOn(processData, 'processAuthFailure').mockImplementation(() => ({}) as any);
             jest.spyOn(logger, 'error').mockImplementation(() => {});
         });
 
@@ -2946,7 +3028,7 @@ describe('Unit test case for ts embed', () => {
             await executeAfterWait(() => {
                 expect(authToken.getAuthenticationToken).toHaveBeenCalledWith(
                     expect.any(Object),
-                    true
+                    true,
                 );
             });
         });
@@ -2963,7 +3045,7 @@ describe('Unit test case for ts embed', () => {
             await executeAfterWait(() => {
                 expect(authToken.getAuthenticationToken).toHaveBeenCalledWith(
                     expect.any(Object),
-                    true
+                    true,
                 );
             });
         });
@@ -2998,17 +3080,19 @@ describe('Unit test case for ts embed', () => {
             await executeAfterWait(() => {
                 expect(authToken.getAuthenticationToken).toHaveBeenCalledWith(
                     expect.any(Object),
-                    true
+                    true,
                 );
                 // Check that logger.error was called with the token refresh
                 // error
                 const errorCalls = (logger.error as jest.Mock).mock.calls.filter(
-                    (call) => call[0]?.includes(ERROR_MESSAGE.INVALID_TOKEN_ERROR) && call[0]?.includes('Token fetch failed')
+                    (call) =>
+                        call[0]?.includes(ERROR_MESSAGE.INVALID_TOKEN_ERROR) &&
+                        call[0]?.includes('Token fetch failed'),
                 );
                 expect(errorCalls.length).toBeGreaterThan(0);
                 expect(processData.processAuthFailure).toHaveBeenCalledWith(
                     error,
-                    expect.any(Element)
+                    expect.any(Element),
                 );
                 expect(mockPort.postMessage).not.toHaveBeenCalled();
             });
@@ -3197,7 +3281,9 @@ describe('Unit test case for ts embed', () => {
                 isEmbedContainerLoaded: true,
             };
 
-            jest.spyOn(searchEmbed as any, 'getPreRenderObj').mockReturnValue(mockPreRenderObj as any);
+            jest.spyOn(searchEmbed as any, 'getPreRenderObj').mockReturnValue(
+                mockPreRenderObj as any,
+            );
 
             const result = searchEmbed['checkEmbedContainerLoaded']();
 
@@ -3209,8 +3295,8 @@ describe('Unit test case for ts embed', () => {
             const searchEmbed = new SearchEmbed(getRootEl(), defaultViewConfig);
             const loggerSpy = jest.spyOn(logger, 'info');
 
-            // Mock insertedDomEl to have the embed object
-            (searchEmbed as any).insertedDomEl = {
+            // getPreRenderObj reads the embed reference from preRenderWrapper
+            (searchEmbed as any).preRenderWrapper = {
                 [searchEmbed['embedNodeKey']]: searchEmbed,
             };
 
@@ -3273,7 +3359,8 @@ describe('Unit test case for ts embed', () => {
                 const searchEmbed = new SearchEmbed(getRootEl(), defaultViewConfig);
                 searchEmbed.isEmbedContainerLoaded = true;
 
-                const triggerSpy = jest.spyOn(searchEmbed, 'trigger')
+                const triggerSpy = jest
+                    .spyOn(searchEmbed, 'trigger')
                     .mockResolvedValue(mockContext);
 
                 const context = await searchEmbed.getCurrentContext();
@@ -3286,7 +3373,8 @@ describe('Unit test case for ts embed', () => {
                 const searchEmbed = new SearchEmbed(getRootEl(), defaultViewConfig);
                 searchEmbed.isEmbedContainerLoaded = false;
 
-                const triggerSpy = jest.spyOn(searchEmbed, 'trigger')
+                const triggerSpy = jest
+                    .spyOn(searchEmbed, 'trigger')
                     .mockResolvedValue(mockContext);
 
                 const contextPromise = searchEmbed.getCurrentContext();
@@ -3338,7 +3426,9 @@ describe('Unit test case for ts embed', () => {
         test('should handle handleEmbedContainerLoaded with EmbedListenerReady source', () => {
             const searchEmbed = new SearchEmbed(getRootEl(), defaultViewConfig);
 
-            const handler = searchEmbed['createEmbedContainerHandler'](EmbedEvent.EmbedListenerReady);
+            const handler = searchEmbed['createEmbedContainerHandler'](
+                EmbedEvent.EmbedListenerReady,
+            );
 
             expect(searchEmbed.isEmbedContainerLoaded).toBe(false);
 
@@ -3408,7 +3498,7 @@ describe('Unit test case for ts embed', () => {
                 new Error('Auth failed'),
             );
             const searchEmbed = new SearchEmbed(getRootEl(), defaultViewConfig);
-            const errorSpy = jest.spyOn(console, 'error').mockImplementation(() => { });
+            const errorSpy = jest.spyOn(console, 'error').mockImplementation(() => {});
             await searchEmbed.render();
             await executeAfterWait(() => {
                 expect(getRootEl().innerHTML).toContain('Not logged in');
@@ -3530,11 +3620,13 @@ describe('Unit test case for ts embed', () => {
             await appEmbed.render();
 
             jest.spyOn(appEmbed, 'trigger').mockRejectedValue(new Error('trigger failed'));
-            const removeChildSpy = jest.spyOn(Node.prototype, 'removeChild').mockImplementation(() => getRootEl());
+            const removeChildSpy = jest
+                .spyOn(Node.prototype, 'removeChild')
+                .mockImplementation(() => getRootEl());
 
             appEmbed.destroy();
 
-            await new Promise(resolve => setTimeout(resolve, 50));
+            await new Promise((resolve) => setTimeout(resolve, 50));
 
             expect(removeChildSpy).toHaveBeenCalled();
 
@@ -3567,7 +3659,9 @@ describe('Unit test case for ts embed', () => {
                 await appEmbed.render();
 
                 const triggerSpy = jest.spyOn(appEmbed, 'trigger').mockResolvedValue(null);
-                const removeChildSpy = jest.spyOn(Node.prototype, 'removeChild').mockImplementation(() => getRootEl());
+                const removeChildSpy = jest
+                    .spyOn(Node.prototype, 'removeChild')
+                    .mockImplementation(() => getRootEl());
 
                 appEmbed.destroy();
 
@@ -3591,7 +3685,9 @@ describe('Unit test case for ts embed', () => {
                 await appEmbed.render();
 
                 const triggerSpy = jest.spyOn(appEmbed, 'trigger').mockResolvedValue(null);
-                const removeChildSpy = jest.spyOn(Node.prototype, 'removeChild').mockImplementation(() => getRootEl());
+                const removeChildSpy = jest
+                    .spyOn(Node.prototype, 'removeChild')
+                    .mockImplementation(() => getRootEl());
 
                 appEmbed.destroy();
 
@@ -3599,7 +3695,7 @@ describe('Unit test case for ts embed', () => {
                 expect(triggerSpy).toHaveBeenCalledWith(HostEvent.DestroyEmbed);
 
                 // Wait for the timeout to complete
-                await new Promise(resolve => setTimeout(resolve, 1100));
+                await new Promise((resolve) => setTimeout(resolve, 1100));
 
                 expect(removeChildSpy).toHaveBeenCalled();
             });
@@ -3620,15 +3716,19 @@ describe('Unit test case for ts embed', () => {
                 await appEmbed.render();
 
                 // Mock trigger to resolve quickly (before timeout)
-                const triggerSpy = jest.spyOn(appEmbed, 'trigger').mockImplementation(() =>
-                    new Promise(resolve => setTimeout(() => resolve(null), 100))
-                );
-                const removeChildSpy = jest.spyOn(Node.prototype, 'removeChild').mockImplementation(() => getRootEl());
+                const triggerSpy = jest
+                    .spyOn(appEmbed, 'trigger')
+                    .mockImplementation(
+                        () => new Promise((resolve) => setTimeout(() => resolve(null), 100)),
+                    );
+                const removeChildSpy = jest
+                    .spyOn(Node.prototype, 'removeChild')
+                    .mockImplementation(() => getRootEl());
 
                 appEmbed.destroy();
 
                 // Wait for the trigger to complete
-                await new Promise(resolve => setTimeout(resolve, 200));
+                await new Promise((resolve) => setTimeout(resolve, 200));
 
                 expect(triggerSpy).toHaveBeenCalledWith(HostEvent.DestroyEmbed);
                 expect(removeChildSpy).toHaveBeenCalled();
@@ -3650,15 +3750,19 @@ describe('Unit test case for ts embed', () => {
                 await appEmbed.render();
 
                 // Mock trigger to take longer than timeout
-                const triggerSpy = jest.spyOn(appEmbed, 'trigger').mockImplementation(() =>
-                    new Promise(resolve => setTimeout(() => resolve(null), 500))
-                );
-                const removeChildSpy = jest.spyOn(Node.prototype, 'removeChild').mockImplementation(() => getRootEl());
+                const triggerSpy = jest
+                    .spyOn(appEmbed, 'trigger')
+                    .mockImplementation(
+                        () => new Promise((resolve) => setTimeout(() => resolve(null), 500)),
+                    );
+                const removeChildSpy = jest
+                    .spyOn(Node.prototype, 'removeChild')
+                    .mockImplementation(() => getRootEl());
 
                 appEmbed.destroy();
 
                 // Wait for the timeout to complete
-                await new Promise(resolve => setTimeout(resolve, 200));
+                await new Promise((resolve) => setTimeout(resolve, 200));
 
                 expect(triggerSpy).toHaveBeenCalledWith(HostEvent.DestroyEmbed);
                 expect(removeChildSpy).toHaveBeenCalled();
@@ -3690,11 +3794,11 @@ describe('Unit test case for ts embed', () => {
                         body: JSON.stringify({
                             variables: {
                                 session: { sessionId: 'session-123' },
-                                contextBookId: 'viz-456'
-                            }
-                        })
-                    }
-                })
+                                contextBookId: 'viz-456',
+                            },
+                        }),
+                    },
+                }),
             };
 
             const mockPort: any = {
@@ -3731,8 +3835,8 @@ describe('Unit test case for ts embed', () => {
                 type: EmbedEvent.ApiIntercept,
                 data: JSON.stringify({
                     input: '/prism/?op=GetChartWithData',
-                    init: {}
-                })
+                    init: {},
+                }),
             };
 
             const mockPort: any = {
@@ -3773,8 +3877,8 @@ describe('Unit test case for ts embed', () => {
                 type: EmbedEvent.ApiIntercept,
                 data: JSON.stringify({
                     input: '/prism/?op=GetChartWithData',
-                    init: {}
-                })
+                    init: {},
+                }),
             };
 
             const mockPort: any = {
@@ -3796,7 +3900,7 @@ describe('Unit test case for ts embed', () => {
                 // handleInterceptEvent
                 const result = await capturedGetUnsavedAnswerTml({
                     sessionId: 'session-123',
-                    vizId: 'viz-456'
+                    vizId: 'viz-456',
                 });
 
                 expect(mockProcessTrigger).toHaveBeenCalled();
@@ -3807,8 +3911,8 @@ describe('Unit test case for ts embed', () => {
                     type: 'getUnsavedAnswerTML',
                     parameters: {
                         sessionId: 'session-123',
-                        vizId: 'viz-456'
-                    }
+                        vizId: 'viz-456',
+                    },
                 });
                 expect(result).toEqual(mockTmlResponse);
             });
@@ -3828,8 +3932,8 @@ describe('Unit test case for ts embed', () => {
                 type: EmbedEvent.ApiIntercept,
                 data: JSON.stringify({
                     input: '/api/test',
-                    init: {}
-                })
+                    init: {},
+                }),
             };
 
             const mockPort: any = {
@@ -3858,8 +3962,8 @@ describe('Unit test case for ts embed', () => {
                 type: EmbedEvent.ApiIntercept,
                 data: JSON.stringify({
                     input: '/prism/?op=GetChartWithData',
-                    init: {}
-                })
+                    init: {},
+                }),
             };
 
             const mockPort: any = {
@@ -3911,16 +4015,16 @@ describe('Unit test case for ts embed', () => {
                 type: EmbedEvent.ApiIntercept,
                 data: JSON.stringify({
                     input: '/prism/?op=GetChartWithData',
-                    init: {}
-                })
+                    init: {},
+                }),
             };
 
             const mockEventData2 = {
                 type: EmbedEvent.ApiIntercept,
                 data: JSON.stringify({
                     input: '/prism/?op=LoadContextBook',
-                    init: {}
-                })
+                    init: {},
+                }),
             };
 
             const mockPort: any = {
@@ -3956,8 +4060,8 @@ describe('Unit test case for ts embed', () => {
                 type: EmbedEvent.ApiIntercept,
                 data: JSON.stringify({
                     input: '/prism/?op=GetChartWithData',
-                    init: {}
-                })
+                    init: {},
+                }),
             };
 
             const mockPort: any = {
@@ -3997,8 +4101,8 @@ describe('Unit test case for ts embed', () => {
                 type: EmbedEvent.ApiIntercept,
                 data: JSON.stringify({
                     input: '/prism/?op=GetChartWithData',
-                    init: {}
-                })
+                    init: {},
+                }),
             };
 
             const mockPort: any = {
@@ -4015,7 +4119,7 @@ describe('Unit test case for ts embed', () => {
 
                 const result = await capturedGetUnsavedAnswerTml({
                     sessionId: 'session-123',
-                    vizId: 'viz-456'
+                    vizId: 'viz-456',
                 });
 
                 expect(result).toBeUndefined();
@@ -4033,8 +4137,8 @@ describe('Unit test case for ts embed', () => {
                 type: EmbedEvent.ApiIntercept,
                 data: JSON.stringify({
                     input: '/prism/?op=LoadContextBook',
-                    init: {}
-                })
+                    init: {},
+                }),
             };
 
             const mockPort: any = {
@@ -4051,14 +4155,13 @@ describe('Unit test case for ts embed', () => {
                 expect(mockHandleInterceptEvent).toHaveBeenCalledWith(
                     expect.objectContaining({
                         eventData: mockEventData,
-                    })
+                    }),
                 );
             });
         });
     });
 });
 
-
 describe('Additional Coverage Tests', () => {
     beforeAll(() => {
         init({
@@ -4163,9 +4266,7 @@ describe('Trigger method edge cases', () => {
     });
 
     test('should return null when trigger is called before iframe is ready', async () => {
-        jest.spyOn(baseInstance, 'getAuthPromise').mockRejectedValueOnce(
-            new Error('Auth failed'),
-        );
+        jest.spyOn(baseInstance, 'getAuthPromise').mockRejectedValueOnce(new Error('Auth failed'));
         const searchEmbed = new SearchEmbed(getRootEl(), defaultViewConfig);
         jest.spyOn(logger, 'debug');
         await searchEmbed.render();
@@ -4333,12 +4434,16 @@ describe('Fullscreen change handler behavior', () => {
 });
 
 describe('ShowPreRender with UpdateEmbedParams', () => {
-    const setupPreRenderTest = async (preRenderId: string, initialConfig: Partial<LiveboardViewConfig>) => {
+    const setupPreRenderTest = async (
+        preRenderId: string,
+        initialConfig: Partial<LiveboardViewConfig>,
+    ) => {
         createRootEleForEmbed();
         mockMessageChannel();
 
-        (window as any).ResizeObserver = window.ResizeObserver
-            || jest.fn().mockImplementation(() => ({
+        (window as any).ResizeObserver =
+            window.ResizeObserver ||
+            jest.fn().mockImplementation(() => ({
                 disconnect: jest.fn(),
                 observe: jest.fn(),
                 unobserve: jest.fn(),
@@ -4548,7 +4653,8 @@ describe('ShowPreRender with UpdateEmbedParams', () => {
             });
         });
 
-        // Matches the structure produced by createValidationError / embedErrorDetails
+        // Matches the structure produced by createValidationError /
+        // embedErrorDetails
         const makeNestedValidationData = (message = 'invalid payload') => ({
             type: EmbedEvent.Error,
             data: {
@@ -4559,7 +4665,8 @@ describe('ShowPreRender with UpdateEmbedParams', () => {
             },
         });
 
-        // Matches the flat structure where errorType sits at the top level of data
+        // Matches the flat structure where errorType sits at the top level of
+        // data
         const makeFlatValidationData = (message = 'invalid payload') => ({
             errorType: EmbedErrorCodes.HOST_EVENT_VALIDATION,
             message,
@@ -4594,12 +4701,13 @@ describe('ShowPreRender with UpdateEmbedParams', () => {
             const embed = makeEmbed({ useHostEventsV2: true, shouldBypassPayloadValidation: true });
             embed.on(EmbedEvent.Error, errorHandler);
 
-            (embed as any).executeCallbacks(EmbedEvent.Error, makeNestedValidationData('nested error'));
+            (embed as any).executeCallbacks(
+                EmbedEvent.Error,
+                makeNestedValidationData('nested error'),
+            );
 
             expect(errorHandler).not.toHaveBeenCalled();
-            expect(logger.warn).toHaveBeenCalledWith(
-                'Host Event Validation failed: nested error',
-            );
+            expect(logger.warn).toHaveBeenCalledWith('Host Event Validation failed: nested error');
         });
 
         test('skips Error event when errorType is resolved from data.errorType (flat format)', () => {
@@ -4615,7 +4723,10 @@ describe('ShowPreRender with UpdateEmbedParams', () => {
 
         test('delivers Error event to handler when useHostEventsV2 is true and shouldBypassPayloadValidation is undefined', () => {
             const errorHandler = jest.fn();
-            const embed = makeEmbed({ useHostEventsV2: true, shouldBypassPayloadValidation: undefined });
+            const embed = makeEmbed({
+                useHostEventsV2: true,
+                shouldBypassPayloadValidation: undefined,
+            });
             embed.on(EmbedEvent.Error, errorHandler);
 
             (embed as any).executeCallbacks(EmbedEvent.Error, makeNestedValidationData());
@@ -4625,7 +4736,10 @@ describe('ShowPreRender with UpdateEmbedParams', () => {
 
         test('delivers Error event to handler when useHostEventsV2 is true and shouldBypassPayloadValidation is false', () => {
             const errorHandler = jest.fn();
-            const embed = makeEmbed({ useHostEventsV2: true, shouldBypassPayloadValidation: false });
+            const embed = makeEmbed({
+                useHostEventsV2: true,
+                shouldBypassPayloadValidation: false,
+            });
             embed.on(EmbedEvent.Error, errorHandler);
 
             (embed as any).executeCallbacks(EmbedEvent.Error, makeNestedValidationData());
@@ -4636,7 +4750,10 @@ describe('ShowPreRender with UpdateEmbedParams', () => {
         test('skips Error event when useHostEventsV2 is false regardless of shouldBypassPayloadValidation', () => {
             jest.spyOn(logger, 'warn');
             const errorHandler = jest.fn();
-            const embed = makeEmbed({ useHostEventsV2: false, shouldBypassPayloadValidation: undefined });
+            const embed = makeEmbed({
+                useHostEventsV2: false,
+                shouldBypassPayloadValidation: undefined,
+            });
             embed.on(EmbedEvent.Error, errorHandler);
 
             (embed as any).executeCallbacks(EmbedEvent.Error, makeNestedValidationData());
@@ -4660,13 +4777,18 @@ describe('ShowPreRender with UpdateEmbedParams', () => {
         test('skips Error event when useHostEventsV2 is false and shouldBypassPayloadValidation is true', () => {
             jest.spyOn(logger, 'warn');
             const errorHandler = jest.fn();
-            const embed = makeEmbed({ useHostEventsV2: false, shouldBypassPayloadValidation: true });
+            const embed = makeEmbed({
+                useHostEventsV2: false,
+                shouldBypassPayloadValidation: true,
+            });
             embed.on(EmbedEvent.Error, errorHandler);
 
             (embed as any).executeCallbacks(EmbedEvent.Error, makeNestedValidationData());
 
             expect(errorHandler).not.toHaveBeenCalled();
-            expect(logger.warn).toHaveBeenCalledWith('Host Event Validation failed: invalid payload');
+            expect(logger.warn).toHaveBeenCalledWith(
+                'Host Event Validation failed: invalid payload',
+            );
         });
 
         test('skips via handleError when shouldBypassPayloadValidation is true', () => {
@@ -4691,7 +4813,10 @@ describe('ShowPreRender with UpdateEmbedParams', () => {
 
         test('delivers Error event to EmbedEvent.ALL handler when not skipped', () => {
             const allHandler = jest.fn();
-            const embed = makeEmbed({ useHostEventsV2: true, shouldBypassPayloadValidation: false });
+            const embed = makeEmbed({
+                useHostEventsV2: true,
+                shouldBypassPayloadValidation: false,
+            });
             embed.on(EmbedEvent.ALL, allHandler);
 
             (embed as any).executeCallbacks(EmbedEvent.Error, makeNestedValidationData());
@@ -4701,7 +4826,10 @@ describe('ShowPreRender with UpdateEmbedParams', () => {
 
         test('does not skip non-Error events even with HOST_EVENT_VALIDATION error code', () => {
             const customActionHandler = jest.fn();
-            const embed = makeEmbed({ useHostEventsV2: true, shouldBypassPayloadValidation: false });
+            const embed = makeEmbed({
+                useHostEventsV2: true,
+                shouldBypassPayloadValidation: false,
+            });
             embed.on(EmbedEvent.CustomAction, customActionHandler);
 
             (embed as any).executeCallbacks(EmbedEvent.CustomAction, {
@@ -4713,7 +4841,10 @@ describe('ShowPreRender with UpdateEmbedParams', () => {
 
         test('does not skip Error events with unrelated error codes', () => {
             const errorHandler = jest.fn();
-            const embed = makeEmbed({ useHostEventsV2: true, shouldBypassPayloadValidation: false });
+            const embed = makeEmbed({
+                useHostEventsV2: true,
+                shouldBypassPayloadValidation: false,
+            });
             embed.on(EmbedEvent.Error, errorHandler);
 
             (embed as any).executeCallbacks(EmbedEvent.Error, {
@@ -4724,4 +4855,171 @@ describe('ShowPreRender with UpdateEmbedParams', () => {
             expect(errorHandler).toHaveBeenCalled();
         });
     });
+
+    describe('constructor init-path branching (SCAL-315058)', () => {
+        beforeEach(() => {
+            init({
+                thoughtSpotHost: 'tshost',
+                authType: AuthType.None,
+            });
+        });
+
+        it('sets hostElement from domSelector', () => {
+            const embed = new SearchEmbed(getRootEl(), defaultViewConfig);
+            expect((embed as any).hostElement).toBe(getRootEl());
+        });
+
+        it('skips isReadyForRenderPromise when init already completed', () => {
+            jest.spyOn(baseInstance, 'getIsInitCompleted').mockReturnValue(true);
+            const embed = new SearchEmbed(getRootEl(), defaultViewConfig);
+            expect((embed as any).shouldWaitForRenderPromise).toBe(false);
+            expect((embed as any).isReadyForRenderPromise).toBeUndefined();
+        });
+
+        it('sets isReadyForRenderPromise when init not yet completed', () => {
+            jest.spyOn(baseInstance, 'getIsInitCompleted').mockReturnValue(false);
+            const embed = new SearchEmbed(getRootEl(), defaultViewConfig);
+            expect((embed as any).shouldWaitForRenderPromise).toBe(true);
+            expect((embed as any).isReadyForRenderPromise).toBeInstanceOf(Promise);
+        });
+
+        it('afterInit runs synchronously when init already completed', () => {
+            jest.spyOn(baseInstance, 'getIsInitCompleted').mockReturnValue(true);
+            const embed = new SearchEmbed(getRootEl(), defaultViewConfig);
+            // thoughtSpotHost is set by afterInit;
+            // must be non-empty after constructor
+            expect((embed as any).thoughtSpotHost).toBeTruthy();
+        });
+
+        it('does not set isReadyForRenderPromise when shouldWaitForRenderPromise is false', () => {
+            jest.spyOn(baseInstance, 'getIsInitCompleted').mockReturnValue(true);
+            const embed = new SearchEmbed(getRootEl(), defaultViewConfig);
+            expect((embed as any).shouldWaitForRenderPromise).toBe(false);
+            // isReadyForRenderPromise must be undefined
+            //  so no unnecessary await occurs
+            expect((embed as any).isReadyForRenderPromise).toBeUndefined();
+        });
+
+        it('shouldWaitForRenderPromise flips to false after promise settles', async () => {
+            jest.spyOn(baseInstance, 'getIsInitCompleted').mockReturnValue(false);
+            const embed = new SearchEmbed(getRootEl(), defaultViewConfig);
+            expect((embed as any).shouldWaitForRenderPromise).toBe(true);
+            await (embed as any).isReadyForRenderPromise;
+            expect((embed as any).shouldWaitForRenderPromise).toBe(false);
+        });
+
+        it('calls throwInitError when getInitPromise rejects', async () => {
+            jest.spyOn(baseInstance, 'getIsInitCompleted').mockReturnValue(false);
+            jest.spyOn(baseInstance, 'getInitPromise').mockReturnValue(
+                Promise.reject(new Error('init failed')),
+            );
+            const embed = new SearchEmbed(getRootEl(), defaultViewConfig);
+            const throwInitErrorSpy = jest.spyOn(embed as any, 'throwInitError');
+            await (embed as any).isReadyForRenderPromise;
+            expect(throwInitErrorSpy).toHaveBeenCalled();
+        });
+
+        it('shouldWaitForRenderPromise flips to false even when getInitPromise rejects', async () => {
+            jest.spyOn(baseInstance, 'getIsInitCompleted').mockReturnValue(false);
+            jest.spyOn(baseInstance, 'getInitPromise').mockReturnValue(
+                Promise.reject(new Error('init failed')),
+            );
+            const embed = new SearchEmbed(getRootEl(), defaultViewConfig);
+            await (embed as any).isReadyForRenderPromise;
+            expect((embed as any).shouldWaitForRenderPromise).toBe(false);
+        });
+    });
+
+    describe('preRender ID object includes placeHolder (SCAL-315058)', () => {
+        beforeAll(() => {
+            init({
+                thoughtSpotHost: 'tshost',
+                authType: AuthType.None,
+            });
+        });
+
+        it('getPreRenderIds returns placeHolder key', () => {
+            createRootEleForEmbed();
+            const embed = new LiveboardEmbed('#tsEmbedDiv', {
+                preRenderId: 'ph-test',
+                liveboardId: 'lb-id',
+            });
+            const ids = embed.getPreRenderIds();
+            expect(ids.placeHolder).toBe('tsEmbed-pre-render-placeholder-ph-test');
+        });
+    });
+
+    describe('isPreRenderConnected logic (SCAL-315058)', () => {
+        beforeAll(() => {
+            init({
+                thoughtSpotHost: 'tshost',
+                authType: AuthType.None,
+            });
+        });
+
+        it('returns false when preRenderWrapper is absent', () => {
+            createRootEleForEmbed();
+            const embed = new LiveboardEmbed('#tsEmbedDiv', {
+                preRenderId: 'conn-test',
+                liveboardId: 'lb-id',
+            });
+            expect((embed as any).isPreRenderConnected()).toBe(false);
+        });
+
+        it('returns true once preRenderWrapper and preRenderChild are set', () => {
+            createRootEleForEmbed();
+            const embed = new LiveboardEmbed('#tsEmbedDiv', {
+                preRenderId: 'conn-test-2',
+                liveboardId: 'lb-id',
+            });
+            (embed as any).preRenderWrapper = document.createElement('div');
+            (embed as any).preRenderChild = document.createElement('div');
+            expect((embed as any).isPreRenderConnected()).toBe(true);
+        });
+    });
+
+    describe('showPreRender inserts placeholder into hostElement (SCAL-315058)', () => {
+        beforeAll(() => {
+            // Clear spy implementations that may have leaked from prior
+            // describe blocks (e.g. getIsInitCompleted/getInitPromise mocks set
+            // in 'constructor init-path branching' persist across describes
+            // because clearAllMocks only clears call history, not
+            // implementations).
+            jest.restoreAllMocks();
+            jest.spyOn(authInstance, 'postLoginService').mockResolvedValue(undefined);
+            init({
+                thoughtSpotHost: 'tshost',
+                authType: AuthType.None,
+            });
+            (window as any).ResizeObserver = jest.fn().mockImplementation(() => ({
+                disconnect: jest.fn(),
+                observe: jest.fn(),
+                unobserve: jest.fn(),
+            }));
+        });
+
+        it('showPreRender creates a placeholder element with the correct id', async () => {
+            createRootEleForEmbed();
+            const embed = new LiveboardEmbed('#tsEmbedDiv', {
+                preRenderId: 'ph-lifecycle-test',
+                liveboardId: 'lb-id',
+            });
+            await embed.preRender();
+            await waitFor(
+                () => !!document.querySelector('#tsEmbed-pre-render-child-ph-lifecycle-test'),
+            );
+
+            await embed.showPreRender();
+
+            const placeholderEle = (
+                embed as any
+            ).getPreRenderPlaceHolderElement() as HTMLDivElement;
+            expect(placeholderEle).not.toBeNull();
+            expect(placeholderEle.id).toBe('tsEmbed-pre-render-placeholder-ph-lifecycle-test');
+            // placeholder is attached to a parent element (inside hostElement)
+            expect(placeholderEle.parentElement).not.toBeNull();
+
+            embed.destroy();
+        });
+    });
 });