@thoughtspot/visual-embed-sdk 1.47.3 → 1.49.0

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

@@ -97,6 +97,7 @@ beforeAll(() => {
 const customisations = {
     style: {
         customCSS: {},
+        customCSSUrl: undefined as string | undefined,
     },
     content: {},
 };
@@ -132,6 +133,9 @@ const getMockAppInitPayload = (data: any) => {
         customVariablesForThirdPartyTools,
         interceptTimeout: undefined,
         interceptUrls: [],
+        shouldBypassPayloadValidation: undefined,
+        useHostEventsV2: undefined,
+        embedExpiryInAuthToken: true,
     };
     return {
         type: EmbedEvent.APP_INIT,
@@ -140,7 +144,7 @@ const getMockAppInitPayload = (data: any) => {
             ...data,
         },
     };
-}
+};
 
 describe('Unit test case for ts embed', () => {
     const mockMixPanelEvent = jest.spyOn(mixpanelInstance, 'uploadMixpanelEvent');
@@ -219,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,
@@ -240,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,
@@ -394,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,
+                    }),
+                );
             });
         });
 
@@ -423,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],
+                    }),
+                );
             });
         });
 
@@ -476,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,
+                        ],
+                    }),
+                );
             });
         });
 
@@ -509,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',
+                    }),
+                );
             });
         });
 
@@ -542,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',
+                    }),
+                );
             });
         });
 
@@ -634,13 +645,48 @@ 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,
+                        ],
+                    }),
+                );
             });
         });
 
+        test.each([
+            ['not set', undefined, true],
+            ['false', false, false],
+            ['true', true, true],
+        ] as [string, boolean | undefined, boolean][])(
+            'embedExpiryInAuthToken is %s when refreshAuthTokenOnNearExpiry is %s',
+            async (_label, refreshAuthTokenOnNearExpiry, expectedEmbedExpiry) => {
+                const mockEmbedEventPayload = {
+                    type: EmbedEvent.APP_INIT,
+                    data: {},
+                };
+                const searchEmbed = new AppEmbed(getRootEl(), {
+                    ...defaultViewConfig,
+                    refreshAuthTokenOnNearExpiry,
+                });
+                searchEmbed.render();
+                const mockPort: any = {
+                    postMessage: jest.fn(),
+                };
+                await executeAfterWait(() => {
+                    const iframe = getIFrameEl();
+                    postMessageToParent(iframe.contentWindow, mockEmbedEventPayload, mockPort);
+                });
+                await executeAfterWait(() => {
+                    expect(mockPort.postMessage).toHaveBeenCalledWith(
+                        getMockAppInitPayload({ embedExpiryInAuthToken: expectedEmbedExpiry }),
+                    );
+                });
+            },
+        );
+
         test('when Embed event status have start status', (done) => {
             const mockEmbedEventPayload = {
                 type: EmbedEvent.Save,
@@ -789,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();
@@ -843,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',
+                });
             });
         });
 
@@ -882,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(),
@@ -928,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();
@@ -951,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
@@ -989,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)
@@ -1059,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(),
@@ -1118,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(() => {});
@@ -1283,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(), {
@@ -1325,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(() => {
@@ -1343,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);
         });
     });
@@ -1370,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) => {
@@ -1387,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' }),
@@ -1533,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);
@@ -1599,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',
+                },
+            );
         });
     });
 
@@ -1968,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`,
             );
         });
 
@@ -1997,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`,
             );
         });
 
@@ -2013,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 () => {
@@ -2028,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 () => {
@@ -2047,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%',
@@ -2057,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(), {
@@ -2075,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`,
             );
         });
 
@@ -2356,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(),
@@ -2371,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 },
                 },
             ]);
@@ -2438,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(),
@@ -2476,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(),
@@ -2489,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',
@@ -2497,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();
@@ -2676,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);
@@ -2751,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 () => {
@@ -2867,7 +2982,7 @@ describe('Unit test case for ts embed', () => {
 
         afterEach(() => {
             expect(baseInstance.notifyAuthFailure).toHaveBeenCalledWith(
-                authInstance.AuthFailureType.EXPIRY
+                authInstance.AuthFailureType.EXPIRY,
             );
         });
     });
@@ -2880,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(() => {});
         });
 
@@ -2911,7 +3028,7 @@ describe('Unit test case for ts embed', () => {
             await executeAfterWait(() => {
                 expect(authToken.getAuthenticationToken).toHaveBeenCalledWith(
                     expect.any(Object),
-                    true
+                    true,
                 );
             });
         });
@@ -2928,7 +3045,7 @@ describe('Unit test case for ts embed', () => {
             await executeAfterWait(() => {
                 expect(authToken.getAuthenticationToken).toHaveBeenCalledWith(
                     expect.any(Object),
-                    true
+                    true,
                 );
             });
         });
@@ -2963,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();
             });
@@ -3162,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']();
 
@@ -3174,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,
             };
 
@@ -3238,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();
@@ -3251,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();
@@ -3303,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);
 
@@ -3373,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');
@@ -3495,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();
 
@@ -3532,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();
 
@@ -3556,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();
 
@@ -3564,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();
             });
@@ -3585,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();
@@ -3615,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();
@@ -3655,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 = {
@@ -3696,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 = {
@@ -3738,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 = {
@@ -3761,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();
@@ -3772,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);
             });
@@ -3793,8 +3932,8 @@ describe('Unit test case for ts embed', () => {
                 type: EmbedEvent.ApiIntercept,
                 data: JSON.stringify({
                     input: '/api/test',
-                    init: {}
-                })
+                    init: {},
+                }),
             };
 
             const mockPort: any = {
@@ -3823,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 = {
@@ -3876,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 = {
@@ -3921,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 = {
@@ -3962,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 = {
@@ -3980,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();
@@ -3998,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 = {
@@ -4016,14 +4155,13 @@ describe('Unit test case for ts embed', () => {
                 expect(mockHandleInterceptEvent).toHaveBeenCalledWith(
                     expect.objectContaining({
                         eventData: mockEventData,
-                    })
+                    }),
                 );
             });
         });
     });
 });
 
-
 describe('Additional Coverage Tests', () => {
     beforeAll(() => {
         init({
@@ -4128,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();
@@ -4298,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(),
@@ -4513,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: {
@@ -4524,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,
@@ -4559,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)', () => {
@@ -4580,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());
@@ -4590,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());
@@ -4601,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());
@@ -4625,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', () => {
@@ -4656,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());
@@ -4666,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, {
@@ -4678,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, {
@@ -4689,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();
+        });
+    });
 });