@thoughtspot/visual-embed-sdk 1.44.1 → 1.44.2

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

@@ -44,7 +44,7 @@ exports.defaultParams = `&${exports.defaultParamsWithoutHiddenActions}&hideActio
 const hideBydefault = `&hideAction=${(0, test_utils_1.fixedEncodeURI)(JSON.stringify([types_1.Action.ReportError, ...search_1.HiddenActionItemByDefaultForSearchEmbed]))}`;
 const defaultParamsWithHiddenActions = exports.defaultParamsWithoutHiddenActions + hideBydefault;
 beforeAll(() => {
-    spyOn(window, 'alert');
+    jest.spyOn(window, 'alert').mockImplementation(() => { });
 });
 const customisations = {
     style: {
@@ -100,7 +100,7 @@ describe('Unit test case for ts embed', () => {
         (0, authToken_1.resetCachedAuthToken)();
     });
     beforeAll(() => {
-        jest.spyOn(authInstance, 'postLoginService').mockResolvedValue(true);
+        jest.spyOn(authInstance, 'postLoginService').mockResolvedValue(undefined);
     });
     describe('Vaidate iframe properties', () => {
         beforeAll(() => {
@@ -208,7 +208,7 @@ describe('Unit test case for ts embed', () => {
                 (0, test_utils_1.postMessageToParent)(iframe.contentWindow, mockEmbedEventPayload, mockPort);
             });
             await (0, test_utils_1.executeAfterWait)(() => {
-                expect(baseInstance.notifyAuthFailure).toBeCalledWith(authInstance.AuthFailureType.EXPIRY);
+                expect(baseInstance.notifyAuthFailure).toHaveBeenCalledWith(authInstance.AuthFailureType.EXPIRY);
                 expect(baseInstance.handleAuth).not.toHaveBeenCalled();
                 expect(mockPort.postMessage).toHaveBeenCalledWith({
                     type: index_1.EmbedEvent.AuthExpire,
@@ -240,7 +240,7 @@ describe('Unit test case for ts embed', () => {
                 (0, test_utils_1.postMessageToParent)(iframe.contentWindow, mockEmbedEventPayload, mockPort);
             });
             await (0, test_utils_1.executeAfterWait)(() => {
-                expect(baseInstance.notifyAuthFailure).toBeCalledWith(authInstance.AuthFailureType.EXPIRY);
+                expect(baseInstance.notifyAuthFailure).toHaveBeenCalledWith(authInstance.AuthFailureType.EXPIRY);
                 expect(mockPort.postMessage).not.toHaveBeenCalledWith({
                     type: index_1.EmbedEvent.AuthExpire,
                     data: { authToken: 'test_auth_token2' },
@@ -592,7 +592,7 @@ describe('Unit test case for ts embed', () => {
                 expect(iframe.contentWindow.postMessage).toHaveBeenCalledTimes(0);
             }, 1000);
         });
-        test('should remove event listener when called off method', async (done) => {
+        test('should remove event listener when called off method', async () => {
             const mockEmbedEventPayload = {
                 type: index_1.EmbedEvent.Save,
                 data: { answerId: '123' },
@@ -610,10 +610,7 @@ describe('Unit test case for ts embed', () => {
                 const iframe = (0, test_utils_1.getIFrameEl)();
                 (0, test_utils_1.postMessageToParent)(iframe.contentWindow, mockEmbedEventPayload);
             });
-            await (0, test_utils_1.executeAfterWait)(() => {
-                expect(mockFn).toHaveBeenCalledTimes(1);
-                done();
-            }, 100);
+            expect(mockFn).toHaveBeenCalledTimes(1);
         });
     });
     describe('Appinit embedEvent in cookieless authentication authType', () => {
@@ -854,7 +851,7 @@ describe('Unit test case for ts embed', () => {
                 authType: index_1.AuthType.TrustedAuthTokenCookieless,
                 getAuthToken: () => Promise.reject(),
             });
-            jest.spyOn(logger_1.logger, 'error').mockResolvedValue(true);
+            jest.spyOn(logger_1.logger, 'error').mockImplementation(() => { });
         });
         afterEach(() => {
             jest.clearAllMocks();
@@ -921,14 +918,14 @@ describe('Unit test case for ts embed', () => {
             const mockPort = {
                 postMessage: jest.fn(),
             };
-            const loggerSpy = jest.spyOn(logger_1.logger, 'error').mockResolvedValueOnce(true);
+            const loggerSpy = jest.spyOn(logger_1.logger, 'error').mockImplementation(() => { });
             await (0, test_utils_1.executeAfterWait)(() => {
                 const iframe = (0, test_utils_1.getIFrameEl)();
                 (0, test_utils_1.postMessageToParent)(iframe.contentWindow, mockEmbedEventPayload, mockPort);
             });
             await (0, test_utils_1.executeAfterWait)(() => {
                 expect((0, test_utils_1.getRootEl)().innerHTML).toContain('Not logged in');
-                expect(baseInstance.notifyAuthFailure).toBeCalledWith(authInstance.AuthFailureType.EXPIRY);
+                expect(baseInstance.notifyAuthFailure).toHaveBeenCalledWith(authInstance.AuthFailureType.EXPIRY);
                 expect(loggerSpy).toHaveBeenCalledTimes(1);
             });
             jest.spyOn(authService, 'verifyTokenService').mockClear();
@@ -945,7 +942,7 @@ describe('Unit test case for ts embed', () => {
             const searchEmbed = new index_1.SearchEmbed((0, test_utils_1.getRootEl)(), { ...defaultViewConfig, preRenderId: 'test' });
             jest.spyOn(baseInstance, 'notifyAuthFailure');
             searchEmbed.preRender();
-            const loggerSpy = jest.spyOn(logger_1.logger, 'error').mockResolvedValueOnce(true);
+            const loggerSpy = jest.spyOn(logger_1.logger, 'error').mockImplementation(() => { });
             const mockPort = {
                 postMessage: jest.fn(),
             };
@@ -956,7 +953,7 @@ describe('Unit test case for ts embed', () => {
             const preRenderWrapper = document.getElementById('tsEmbed-pre-render-wrapper-test');
             await (0, test_utils_1.executeAfterWait)(() => {
                 expect(preRenderWrapper.innerHTML).toContain('Not logged in');
-                expect(baseInstance.notifyAuthFailure).toBeCalledWith(authInstance.AuthFailureType.EXPIRY);
+                expect(baseInstance.notifyAuthFailure).toHaveBeenCalledWith(authInstance.AuthFailureType.EXPIRY);
                 expect(loggerSpy).toHaveBeenCalledTimes(1);
             });
             jest.spyOn(authService, 'verifyTokenService').mockClear();
@@ -986,7 +983,7 @@ describe('Unit test case for ts embed', () => {
                 (0, test_utils_1.postMessageToParent)(iframe.contentWindow, mockEmbedEventPayload);
             });
             await (0, test_utils_1.executeAfterWait)(() => {
-                expect(baseInstance.notifyAuthFailure).toBeCalledWith(authInstance.AuthFailureType.EXPIRY);
+                expect(baseInstance.notifyAuthFailure).toHaveBeenCalledWith(authInstance.AuthFailureType.EXPIRY);
                 expect(baseInstance.handleAuth).toHaveBeenCalled();
             });
         });
@@ -1011,7 +1008,7 @@ describe('Unit test case for ts embed', () => {
                 (0, test_utils_1.postMessageToParent)(iframe.contentWindow, mockEmbedEventPayload);
             });
             await (0, test_utils_1.executeAfterWait)(() => {
-                expect(baseInstance.notifyAuthFailure).toBeCalledWith(authInstance.AuthFailureType.EXPIRY);
+                expect(baseInstance.notifyAuthFailure).toHaveBeenCalledWith(authInstance.AuthFailureType.EXPIRY);
                 expect(baseInstance.handleAuth).not.toHaveBeenCalled();
             });
         });
@@ -1027,9 +1024,7 @@ describe('Unit test case for ts embed', () => {
         const setup = async (isLoggedIn = false) => {
             jest.spyOn(window, 'addEventListener').mockImplementationOnce((event, handler, options) => {
                 handler({
-                    data: {
-                        type: 'xyz',
-                    },
+                    data: { type: 'xyz' },
                     ports: [3000],
                     source: null,
                 });
@@ -1047,8 +1042,8 @@ describe('Unit test case for ts embed', () => {
         };
         test('mixpanel should call with VISUAL_SDK_RENDER_COMPLETE', async () => {
             await setup(true);
-            expect(mockMixPanelEvent).toBeCalledWith(mixpanel_service_1.MIXPANEL_EVENT.VISUAL_SDK_RENDER_START);
-            expect(mockMixPanelEvent).toBeCalledWith(mixpanel_service_1.MIXPANEL_EVENT.VISUAL_SDK_RENDER_COMPLETE, expect.objectContaining({
+            expect(mockMixPanelEvent).toHaveBeenCalledWith(mixpanel_service_1.MIXPANEL_EVENT.VISUAL_SDK_RENDER_START);
+            expect(mockMixPanelEvent).toHaveBeenCalledWith(mixpanel_service_1.MIXPANEL_EVENT.VISUAL_SDK_RENDER_COMPLETE, expect.objectContaining({
                 elWidth: 0,
                 elHeight: 0,
             }));
@@ -1058,12 +1053,9 @@ describe('Unit test case for ts embed', () => {
             const prefetchIframe = document.querySelectorAll('.prefetchIframe');
             expect(prefetchIframe.length).toBe(0);
         });
-        test('Should render failure when login fails', async (done) => {
-            setup(false);
-            (0, test_utils_1.executeAfterWait)(() => {
-                expect((0, test_utils_1.getRootEl)().innerHTML).toContain('Failed to Login');
-                done();
-            });
+        test('Should render failure when login fails', async () => {
+            await setup(false);
+            expect((0, test_utils_1.getRootEl)().innerHTML).toContain('Failed to Login');
         });
     });
     describe('Trigger infoSuccess event on iframe load', () => {
@@ -1078,9 +1070,7 @@ describe('Unit test case for ts embed', () => {
         const setup = async (isLoggedIn = false, overrideOrgId = undefined) => {
             jest.spyOn(window, 'addEventListener').mockImplementationOnce((event, handler, options) => {
                 handler({
-                    data: {
-                        type: 'xyz',
-                    },
+                    data: { type: 'xyz' },
                     ports: [3000],
                     source: null,
                 });
@@ -1313,19 +1303,19 @@ describe('Unit test case for ts embed', () => {
             const iFrame = document.createElement('div');
             iFrame.contentWindow = null;
             jest.spyOn(document, 'createElement').mockReturnValueOnce(iFrame);
-            spyOn(logger_1.logger, 'error');
+            jest.spyOn(logger_1.logger, 'error');
             await tsEmbed.render();
         });
         test('mixpanel should call with VISUAL_SDK_RENDER_FAILED', () => {
-            expect(mockMixPanelEvent).toBeCalledWith(mixpanel_service_1.MIXPANEL_EVENT.VISUAL_SDK_RENDER_START);
-            expect(mockMixPanelEvent).toBeCalledWith(mixpanel_service_1.MIXPANEL_EVENT.VISUAL_SDK_RENDER_FAILED, {
+            expect(mockMixPanelEvent).toHaveBeenCalledWith(mixpanel_service_1.MIXPANEL_EVENT.VISUAL_SDK_RENDER_START);
+            expect(mockMixPanelEvent).toHaveBeenCalledWith(mixpanel_service_1.MIXPANEL_EVENT.VISUAL_SDK_RENDER_FAILED, {
                 error: 'false',
             });
         });
     });
     describe('when visible actions are set', () => {
         test('should throw error when there are both visible and hidden actions - pinboard', async () => {
-            spyOn(logger_1.logger, 'error');
+            jest.spyOn(logger_1.logger, 'error');
             const pinboardEmbed = new index_1.PinboardEmbed((0, test_utils_1.getRootEl)(), {
                 hiddenActions: [types_1.Action.DownloadAsCsv],
                 visibleActions: [types_1.Action.DownloadAsCsv],
@@ -1334,7 +1324,12 @@ describe('Unit test case for ts embed', () => {
             });
             await pinboardEmbed.render();
             expect(pinboardEmbed['isError']).toBe(true);
-            expect(logger_1.logger.error).toHaveBeenCalledWith('You cannot have both hidden actions and visible actions');
+            expect(logger_1.logger.error).toHaveBeenCalledWith({
+                errorType: types_1.ErrorDetailsTypes.VALIDATION_ERROR,
+                message: errors_1.ERROR_MESSAGE.CONFLICTING_ACTIONS_CONFIG,
+                code: types_1.EmbedErrorCodes.CONFLICTING_ACTIONS_CONFIG,
+                error: errors_1.ERROR_MESSAGE.CONFLICTING_ACTIONS_CONFIG,
+            });
         });
         test('should not throw error when there are only visible or hidden actions - pinboard', async () => {
             const pinboardEmbed = new index_1.PinboardEmbed((0, test_utils_1.getRootEl)(), {
@@ -1351,7 +1346,7 @@ describe('Unit test case for ts embed', () => {
          * @param visibleActions
          */
         async function testActionsForLiveboards(hiddenActions, visibleActions) {
-            spyOn(logger_1.logger, 'error');
+            jest.spyOn(logger_1.logger, 'error');
             const liveboardEmbed = new index_1.LiveboardEmbed((0, test_utils_1.getRootEl)(), {
                 hiddenActions,
                 visibleActions,
@@ -1360,7 +1355,12 @@ describe('Unit test case for ts embed', () => {
             });
             await liveboardEmbed.render();
             expect(liveboardEmbed['isError']).toBe(true);
-            expect(logger_1.logger.error).toHaveBeenCalledWith('You cannot have both hidden actions and visible actions');
+            expect(logger_1.logger.error).toHaveBeenCalledWith({
+                errorType: types_1.ErrorDetailsTypes.VALIDATION_ERROR,
+                message: errors_1.ERROR_MESSAGE.CONFLICTING_ACTIONS_CONFIG,
+                code: types_1.EmbedErrorCodes.CONFLICTING_ACTIONS_CONFIG,
+                error: errors_1.ERROR_MESSAGE.CONFLICTING_ACTIONS_CONFIG,
+            });
         }
         test('should throw error when there are both visible and hidden action arrays', async () => {
             await testActionsForLiveboards([types_1.Action.DownloadAsCsv], [types_1.Action.DownloadAsCsv]);
@@ -1392,7 +1392,7 @@ describe('Unit test case for ts embed', () => {
     });
     describe('when visible Tabs are set', () => {
         test('should throw error when there are both visible and hidden Tabs - pinboard', async () => {
-            spyOn(logger_1.logger, 'error');
+            jest.spyOn(logger_1.logger, 'error');
             const pinboardEmbed = new index_1.PinboardEmbed((0, test_utils_1.getRootEl)(), {
                 visibleTabs: [tabId1],
                 hiddenTabs: [tabId2],
@@ -1401,7 +1401,12 @@ describe('Unit test case for ts embed', () => {
             });
             await pinboardEmbed.render();
             expect(pinboardEmbed['isError']).toBe(true);
-            expect(logger_1.logger.error).toHaveBeenCalledWith('You cannot have both hidden Tabs and visible Tabs');
+            expect(logger_1.logger.error).toHaveBeenCalledWith({
+                errorType: types_1.ErrorDetailsTypes.VALIDATION_ERROR,
+                message: errors_1.ERROR_MESSAGE.CONFLICTING_TABS_CONFIG,
+                code: types_1.EmbedErrorCodes.CONFLICTING_TABS_CONFIG,
+                error: errors_1.ERROR_MESSAGE.CONFLICTING_TABS_CONFIG,
+            });
         });
         test('should not throw error when there are only visible or hidden Tabs - pinboard', async () => {
             const pinboardEmbed = new index_1.PinboardEmbed((0, test_utils_1.getRootEl)(), {
@@ -1418,7 +1423,7 @@ describe('Unit test case for ts embed', () => {
          * @param visibleTabs
          */
         async function testTabsForLiveboards(hiddenTabs, visibleTabs) {
-            spyOn(logger_1.logger, 'error');
+            jest.spyOn(logger_1.logger, 'error');
             const liveboardEmbed = new index_1.LiveboardEmbed((0, test_utils_1.getRootEl)(), {
                 hiddenTabs,
                 visibleTabs,
@@ -1427,7 +1432,12 @@ describe('Unit test case for ts embed', () => {
             });
             await liveboardEmbed.render();
             expect(liveboardEmbed['isError']).toBe(true);
-            expect(logger_1.logger.error).toHaveBeenCalledWith('You cannot have both hidden Tabs and visible Tabs');
+            expect(logger_1.logger.error).toHaveBeenCalledWith({
+                errorType: types_1.ErrorDetailsTypes.VALIDATION_ERROR,
+                message: errors_1.ERROR_MESSAGE.CONFLICTING_TABS_CONFIG,
+                code: types_1.EmbedErrorCodes.CONFLICTING_TABS_CONFIG,
+                error: errors_1.ERROR_MESSAGE.CONFLICTING_TABS_CONFIG,
+            });
         }
         test('should throw error when there are both visible and hidden Tab arrays', async () => {
             await testTabsForLiveboards([tabId1], [tabId2]);
@@ -1466,11 +1476,16 @@ describe('Unit test case for ts embed', () => {
             });
         });
         test('Error should be true', async () => {
-            spyOn(logger_1.logger, 'error');
+            jest.spyOn(logger_1.logger, 'error');
             const tsEmbed = new index_1.SearchEmbed((0, test_utils_1.getRootEl)(), {});
             await tsEmbed.render();
             expect(tsEmbed['isError']).toBe(true);
-            expect(logger_1.logger.error).toHaveBeenCalledWith('You need to init the ThoughtSpot SDK module first');
+            expect(logger_1.logger.error).toHaveBeenCalledWith({
+                errorType: types_1.ErrorDetailsTypes.VALIDATION_ERROR,
+                message: errors_1.ERROR_MESSAGE.INIT_SDK_REQUIRED,
+                code: types_1.EmbedErrorCodes.INIT_ERROR,
+                error: errors_1.ERROR_MESSAGE.INIT_SDK_REQUIRED,
+            });
         });
     });
     describe('V1Embed ', () => {
@@ -1478,7 +1493,7 @@ describe('Unit test case for ts embed', () => {
             jest.spyOn(config, 'getThoughtSpotHost').mockImplementation(() => 'http://tshost');
         });
         test('when isRendered is true than isError will be true', async () => {
-            spyOn(logger_1.logger, 'warn');
+            jest.spyOn(logger_1.logger, 'warn');
             const viEmbedIns = new tsEmbedInstance.V1Embed((0, test_utils_1.getRootEl)(), defaultViewConfig);
             expect(viEmbedIns['isError']).toBe(false);
             await viEmbedIns.render();
@@ -1522,7 +1537,7 @@ describe('Unit test case for ts embed', () => {
             (0, test_utils_1.expectUrlMatchesWithParams)((0, test_utils_1.getIFrameSrc)(), `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&${test_utils_1.defaultParamsForPinboardEmbed}${defaultParamsPost}#/${path}`);
         });
         test('navigateToPage function use before render', async () => {
-            spyOn(logger_1.logger, 'log');
+            jest.spyOn(logger_1.logger, 'log');
             const appEmbed = new index_1.AppEmbed((0, test_utils_1.getRootEl)(), {
                 frameParams: {
                     width: '100%',
@@ -1726,7 +1741,7 @@ describe('Unit test case for ts embed', () => {
             await appEmbed.render();
             (0, test_utils_1.expectUrlMatchesWithParams)((0, test_utils_1.getIFrameSrc)(), `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&additionalPendoKey=1234${defaultParamsPost}#/home`);
         });
-        xit('Sets the forceSAMLAutoRedirect param', async (done) => {
+        xit('Sets the forceSAMLAutoRedirect param', async () => {
             jest.spyOn(baseInstance, 'getAuthPromise').mockResolvedValue(true);
             (0, index_1.init)({
                 thoughtSpotHost: 'tshost',
@@ -1739,11 +1754,8 @@ describe('Unit test case for ts embed', () => {
                 },
             });
             appEmbed.render();
-            await (0, test_utils_1.waitFor)(() => !!(0, test_utils_1.getIFrameEl)()).then(() => {
-                expect((0, test_utils_1.getIFrameSrc)()).toContain('authType=EmbeddedSSO');
-                expect((0, test_utils_1.getIFrameSrc)()).toContain('forceSAMLAutoRedirect=true');
-                done();
-            });
+            expect((0, test_utils_1.getIFrameSrc)()).toContain('authType=EmbeddedSSO');
+            expect((0, test_utils_1.getIFrameSrc)()).toContain('forceSAMLAutoRedirect=true');
         });
         it('Should set the override locale for number/date and currency format', async () => {
             jest.spyOn(baseInstance, 'getAuthPromise').mockResolvedValue(true);
@@ -1893,8 +1905,7 @@ describe('Unit test case for ts embed', () => {
         beforeAll(() => {
             delete window.location;
             window.location = {
-                hash: '',
-                search: '',
+                assign: jest.fn(),
             };
         });
         beforeEach(() => {
@@ -1964,7 +1975,7 @@ describe('Unit test case for ts embed', () => {
         });
         afterAll(() => {
             const rootEle = document.getElementById('myRoot');
-            rootEle.remove();
+            rootEle === null || rootEle === void 0 ? void 0 : rootEle.remove();
             jest.clearAllMocks();
         });
         it('should preRender and hide the iframe', async () => {
@@ -1998,9 +2009,9 @@ describe('Unit test case for ts embed', () => {
                     };
                 });
             // show preRender
-            const warnSpy = spyOn(logger_1.logger, 'warn');
+            const warnSpy = jest.spyOn(logger_1.logger, 'warn');
             libEmbed.showPreRender();
-            expect(warnSpy).toHaveBeenCalledTimes(0);
+            expect(warnSpy).toHaveBeenCalledTimes(1);
             resizeObserverCb([
                 {
                     target: tsEmbedDiv,
@@ -2021,7 +2032,7 @@ describe('Unit test case for ts embed', () => {
         });
         it('preRender called without preRenderId should log error ', () => {
             (0, test_utils_1.createRootEleForEmbed)();
-            spyOn(logger_1.logger, 'error');
+            jest.spyOn(logger_1.logger, 'error');
             const libEmbed = new index_1.LiveboardEmbed('#tsEmbedDiv', {
                 liveboardId: 'myLiveboardId',
             });
@@ -2048,7 +2059,7 @@ describe('Unit test case for ts embed', () => {
                 preRenderId: 'i-am-preRendered',
                 liveboardId: 'myLiveboardId',
             });
-            spyOn(libEmbed, 'preRender');
+            jest.spyOn(libEmbed, 'preRender');
             libEmbed.hidePreRender();
             expect(libEmbed.preRender).toHaveBeenCalledTimes(0);
         });
@@ -2082,8 +2093,8 @@ describe('Unit test case for ts embed', () => {
             const libEmbed = new index_1.LiveboardEmbed('#tsEmbedDiv', {
                 liveboardId: 'myLiveboardId',
             });
-            spyOn(libEmbed, 'preRender');
-            spyOn(logger_1.logger, 'error');
+            jest.spyOn(libEmbed, 'preRender');
+            jest.spyOn(logger_1.logger, 'error');
             libEmbed.showPreRender();
             expect(libEmbed.preRender).toHaveBeenCalledTimes(0);
             expect(logger_1.logger.error).toHaveBeenCalledTimes(1);
@@ -2113,7 +2124,7 @@ describe('Unit test case for ts embed', () => {
                 preRenderId: 'test',
             });
             await libEmbed.syncPreRenderStyle();
-            expect(logger_1.logger.error).toBeCalledWith('PreRender should be called before using syncPreRenderStyle');
+            expect(logger_1.logger.error).toHaveBeenCalledWith('PreRender should be called before using syncPreRenderStyle');
             logger_1.logger.error.mockClear();
         });
     });
@@ -2146,7 +2157,7 @@ describe('Unit test case for ts embed', () => {
                 (0, test_utils_1.postMessageToParent)(iframe.contentWindow, mockEmbedEventPayload, mockPort);
             });
             await (0, test_utils_1.executeAfterWait)(() => {
-                expect(baseInstance.notifyAuthFailure).toBeCalledWith(authInstance.AuthFailureType.IDLE_SESSION_TIMEOUT);
+                expect(baseInstance.notifyAuthFailure).toHaveBeenCalledWith(authInstance.AuthFailureType.IDLE_SESSION_TIMEOUT);
                 expect(baseInstance.handleAuth).toHaveBeenCalled();
                 expect(mockPort.postMessage).toHaveBeenCalledWith({
                     type: index_1.EmbedEvent.IdleSessionTimeout,
@@ -2172,14 +2183,14 @@ describe('Unit test case for ts embed', () => {
             const mockPort = {
                 postMessage: jest.fn(),
             };
-            const loggerSpy = jest.spyOn(logger_1.logger, 'error').mockResolvedValueOnce(true);
+            const loggerSpy = jest.spyOn(logger_1.logger, 'error').mockImplementation(() => { });
             await (0, test_utils_1.executeAfterWait)(() => {
                 const iframe = (0, test_utils_1.getIFrameEl)();
                 (0, test_utils_1.postMessageToParent)(iframe.contentWindow, mockEmbedEventPayload, mockPort);
             });
             await (0, test_utils_1.executeAfterWait)(() => {
                 expect((0, test_utils_1.getRootEl)().innerHTML).toContain('Not logged in');
-                expect(baseInstance.notifyAuthFailure).toBeCalledWith(authInstance.AuthFailureType.IDLE_SESSION_TIMEOUT);
+                expect(baseInstance.notifyAuthFailure).toHaveBeenCalledWith(authInstance.AuthFailureType.IDLE_SESSION_TIMEOUT);
                 expect(loggerSpy).toHaveBeenCalledTimes(1);
             });
             jest.spyOn(authService, 'verifyTokenService').mockClear();
@@ -2204,13 +2215,13 @@ describe('Unit test case for ts embed', () => {
             const mockPort = {
                 postMessage: jest.fn(),
             };
-            const loggerSpy = jest.spyOn(logger_1.logger, 'error').mockResolvedValueOnce(true);
+            const loggerSpy = jest.spyOn(logger_1.logger, 'error').mockImplementation(() => { });
             await (0, test_utils_1.executeAfterWait)(() => {
                 const iframe = (0, test_utils_1.getIFrameEl)();
                 (0, test_utils_1.postMessageToParent)(iframe.contentWindow, mockEmbedEventPayload, mockPort);
             });
             await (0, test_utils_1.executeAfterWait)(() => {
-                expect(baseInstance.notifyAuthFailure).toBeCalledWith(authInstance.AuthFailureType.IDLE_SESSION_TIMEOUT);
+                expect(baseInstance.notifyAuthFailure).toHaveBeenCalledWith(authInstance.AuthFailureType.IDLE_SESSION_TIMEOUT);
                 expect(loggerSpy).toHaveBeenCalledTimes(0);
             });
             jest.spyOn(authService, 'verifyTokenService').mockClear();
@@ -2218,7 +2229,7 @@ describe('Unit test case for ts embed', () => {
         });
     });
     describe('Renders should wait for init to completed', () => {
-        const errorSpy = jest.spyOn(logger_1.logger, 'error').mockResolvedValue(true);
+        const errorSpy = jest.spyOn(logger_1.logger, 'error').mockImplementation(() => { });
         beforeEach(() => {
             errorSpy.mockClear();
             (0, utils_1.resetValueFromWindow)('initFlagKey');
@@ -2862,7 +2873,7 @@ describe('Unit test case for ts embed', () => {
         });
         test('should execute callbacks through executeEvent function', async () => {
             let capturedExecuteEvent;
-            mockHandleInterceptEvent.mockImplementation((params) => {
+            mockHandleInterceptEvent.mockImplementation(async (params) => {
                 capturedExecuteEvent = params.executeEvent;
             });
             const searchEmbed = new index_1.SearchEmbed((0, test_utils_1.getRootEl)(), defaultViewConfig);
@@ -2895,7 +2906,7 @@ describe('Unit test case for ts embed', () => {
         });
         test('should call triggerUIPassThrough through getUnsavedAnswerTml function', async () => {
             let capturedGetUnsavedAnswerTml;
-            mockHandleInterceptEvent.mockImplementation((params) => {
+            mockHandleInterceptEvent.mockImplementation(async (params) => {
                 capturedGetUnsavedAnswerTml = params.getUnsavedAnswerTml;
             });
             const mockTmlResponse = { tml: 'test-tml-content' };
@@ -3045,7 +3056,7 @@ describe('Unit test case for ts embed', () => {
         });
         test('should pass eventPort to executeCallbacks', async () => {
             let capturedExecuteEvent;
-            mockHandleInterceptEvent.mockImplementation((params) => {
+            mockHandleInterceptEvent.mockImplementation(async (params) => {
                 capturedExecuteEvent = params.executeEvent;
             });
             const searchEmbed = new index_1.SearchEmbed((0, test_utils_1.getRootEl)(), defaultViewConfig);
@@ -3078,7 +3089,7 @@ describe('Unit test case for ts embed', () => {
         });
         test('should handle getUnsavedAnswerTml with empty response', async () => {
             let capturedGetUnsavedAnswerTml;
-            mockHandleInterceptEvent.mockImplementation((params) => {
+            mockHandleInterceptEvent.mockImplementation(async (params) => {
                 capturedGetUnsavedAnswerTml = params.getUnsavedAnswerTml;
             });
             mockProcessTrigger.mockResolvedValue([]);
@@ -3136,4 +3147,257 @@ describe('Unit test case for ts embed', () => {
         });
     });
 });
+describe('Additional Coverage Tests', () => {
+    beforeAll(() => {
+        (0, index_1.init)({
+            thoughtSpotHost: 'tshost',
+            authType: index_1.AuthType.None,
+        });
+    });
+    test('should handle getAuthTokenForCookielessInit with non-cookieless auth', async () => {
+        const searchEmbed = new index_1.SearchEmbed((0, test_utils_1.getRootEl)(), defaultViewConfig);
+        const token = await searchEmbed['getAuthTokenForCookielessInit']();
+        expect(token).toBe('');
+    });
+    test('should call setIFrameHeight', async () => {
+        const searchEmbed = new index_1.SearchEmbed((0, test_utils_1.getRootEl)(), defaultViewConfig);
+        await searchEmbed.render();
+        await (0, test_utils_1.executeAfterWait)(() => {
+            searchEmbed['setIFrameHeight'](500);
+            expect((0, test_utils_1.getIFrameEl)().style.height).toBe('500px');
+        });
+    });
+    test('should test getIframeCenter calculation', async () => {
+        const searchEmbed = new index_1.SearchEmbed((0, test_utils_1.getRootEl)(), defaultViewConfig);
+        await searchEmbed.render();
+        await (0, test_utils_1.executeAfterWait)(() => {
+            const center = searchEmbed['getIframeCenter']();
+            expect(center).toHaveProperty('iframeCenter');
+            expect(center).toHaveProperty('iframeHeight');
+            expect(center).toHaveProperty('viewPortHeight');
+        });
+    });
+    test('should handle preRender with replaceExistingPreRender=true', async () => {
+        (0, test_utils_1.createRootEleForEmbed)();
+        const embed1 = new index_1.LiveboardEmbed('#tsEmbedDiv', {
+            preRenderId: 'test-replace',
+            liveboardId: 'lb1',
+        });
+        await embed1.preRender();
+        const embed2 = new index_1.LiveboardEmbed('#tsEmbedDiv', {
+            preRenderId: 'test-replace',
+            liveboardId: 'lb2',
+        });
+        await embed2.preRender(false, true);
+        expect(document.getElementById('tsEmbed-pre-render-wrapper-test-replace')).toBeTruthy();
+    });
+    test('should test getIframeSrc base implementation', () => {
+        const searchEmbed = new index_1.SearchEmbed((0, test_utils_1.getRootEl)(), defaultViewConfig);
+        expect(searchEmbed.getIframeSrc()).toBe('');
+    });
+    test('should handle createEmbedEventResponder with OnBeforeGetVizDataIntercept', async () => {
+        const searchEmbed = new index_1.SearchEmbed((0, test_utils_1.getRootEl)(), defaultViewConfig);
+        const mockPort = { postMessage: jest.fn() };
+        const responder = searchEmbed['createEmbedEventResponder'](mockPort, index_1.EmbedEvent.OnBeforeGetVizDataIntercept);
+        responder({ data: 'test' });
+        expect(mockPort.postMessage).toHaveBeenCalled();
+    });
+    test('should clean up message event listeners', async () => {
+        const removeEventListenerSpy = jest.spyOn(window, 'removeEventListener');
+        const searchEmbed = new index_1.SearchEmbed((0, test_utils_1.getRootEl)(), defaultViewConfig);
+        await searchEmbed.render();
+        searchEmbed['unsubscribeToMessageEvents']();
+        expect(removeEventListenerSpy).toHaveBeenCalledWith('message', expect.any(Function));
+    });
+});
+describe('Trigger method edge cases', () => {
+    beforeAll(() => {
+        (0, index_1.init)({
+            thoughtSpotHost: 'tshost',
+            authType: index_1.AuthType.None,
+        });
+    });
+    beforeEach(() => {
+        document.body.innerHTML = (0, test_utils_1.getDocumentBody)();
+    });
+    test('should handle error when trigger is called with undefined messageType', async () => {
+        const searchEmbed = new index_1.SearchEmbed((0, test_utils_1.getRootEl)(), defaultViewConfig);
+        jest.spyOn(logger_1.logger, 'error');
+        await searchEmbed.render();
+        await (0, test_utils_1.executeAfterWait)(async () => {
+            const result = await searchEmbed.trigger(undefined);
+            expect(result).toBeNull();
+            expect(logger_1.logger.error).toHaveBeenCalledWith(expect.objectContaining({
+                errorType: types_1.ErrorDetailsTypes.VALIDATION_ERROR,
+                code: types_1.EmbedErrorCodes.HOST_EVENT_TYPE_UNDEFINED,
+            }));
+        });
+    });
+    test('should return null when trigger is called before iframe is ready', async () => {
+        jest.spyOn(baseInstance, 'getAuthPromise').mockRejectedValueOnce(new Error('Auth failed'));
+        const searchEmbed = new index_1.SearchEmbed((0, test_utils_1.getRootEl)(), defaultViewConfig);
+        jest.spyOn(logger_1.logger, 'debug');
+        await searchEmbed.render();
+        await (0, test_utils_1.executeAfterWait)(async () => {
+            const result = await searchEmbed.trigger(types_1.HostEvent.Reload);
+            expect(result).toBeNull();
+        });
+    });
+});
+describe('PreRender replaceExistingPreRender scenarios', () => {
+    beforeAll(() => {
+        (0, index_1.init)({
+            thoughtSpotHost: 'tshost',
+            authType: index_1.AuthType.None,
+        });
+    });
+    afterEach(() => {
+        const rootEle = document.getElementById('myRoot');
+        rootEle === null || rootEle === void 0 ? void 0 : rootEle.remove();
+    });
+    test('should skip re-rendering when preRender already exists and replaceExistingPreRender is false', async () => {
+        (0, test_utils_1.createRootEleForEmbed)();
+        const embed1 = new index_1.LiveboardEmbed('#tsEmbedDiv', {
+            preRenderId: 'no-replace-test',
+            liveboardId: 'lb1',
+        });
+        await embed1.preRender();
+        await (0, test_utils_1.waitFor)(() => !!(0, test_utils_1.getIFrameEl)());
+        const embed2 = new index_1.LiveboardEmbed('#tsEmbedDiv', {
+            preRenderId: 'no-replace-test',
+            liveboardId: 'lb2',
+        });
+        const result = await embed2.preRender(false, false);
+        expect(result).toBe(embed2);
+        // The original iframe should still have lb1
+        const iframe = (0, test_utils_1.getIFrameEl)();
+        expect(iframe.src).toContain('lb1');
+    });
+});
+describe('Destroy error handling', () => {
+    beforeAll(() => {
+        (0, index_1.init)({
+            thoughtSpotHost: 'tshost',
+            authType: index_1.AuthType.None,
+        });
+    });
+    beforeEach(() => {
+        document.body.innerHTML = (0, test_utils_1.getDocumentBody)();
+    });
+    test('should handle error gracefully when destroy fails', async () => {
+        const appEmbed = new index_1.AppEmbed((0, test_utils_1.getRootEl)(), {
+            frameParams: { width: '100%', height: '100%' },
+        });
+        await appEmbed.render();
+        const logSpy = jest.spyOn(logger_1.logger, 'log').mockImplementation(() => { });
+        jest.spyOn(Node.prototype, 'removeChild').mockImplementationOnce(() => {
+            throw new Error('Remove failed');
+        });
+        expect(() => {
+            appEmbed.destroy();
+        }).not.toThrow();
+        expect(logSpy).toHaveBeenCalledWith('Error destroying TS Embed', expect.any(Error));
+        logSpy.mockRestore();
+    });
+});
+describe('Fullscreen change handler behavior', () => {
+    beforeAll(() => {
+        (0, index_1.init)({
+            thoughtSpotHost: 'tshost',
+            authType: index_1.AuthType.None,
+            disableFullscreenPresentation: false,
+        });
+    });
+    beforeEach(() => {
+        document.body.innerHTML = (0, test_utils_1.getDocumentBody)();
+    });
+    test('should trigger ExitPresentMode when exiting fullscreen', async () => {
+        const liveboardEmbed = new index_1.LiveboardEmbed((0, test_utils_1.getRootEl)(), {
+            ...defaultViewConfig,
+            liveboardId: 'test-lb',
+        });
+        await liveboardEmbed.render();
+        await (0, test_utils_1.executeAfterWait)(() => {
+            const iframe = (0, test_utils_1.getIFrameEl)();
+            expect(iframe).toBeTruthy();
+        });
+        mockProcessTrigger.mockResolvedValue({});
+        liveboardEmbed['setupFullscreenChangeHandler']();
+        Object.defineProperty(document, 'fullscreenElement', {
+            value: null,
+            writable: true,
+            configurable: true,
+        });
+        const event = new Event('fullscreenchange');
+        document.dispatchEvent(event);
+        await (0, test_utils_1.executeAfterWait)(() => {
+            expect(mockProcessTrigger).toHaveBeenCalledWith(expect.any(Object), types_1.HostEvent.ExitPresentMode, expect.any(String), expect.any(Object));
+        });
+    });
+    test('should not trigger ExitPresentMode when entering fullscreen', async () => {
+        const liveboardEmbed = new index_1.LiveboardEmbed((0, test_utils_1.getRootEl)(), {
+            ...defaultViewConfig,
+            liveboardId: 'test-lb-fullscreen',
+        });
+        await liveboardEmbed.render();
+        await (0, test_utils_1.executeAfterWait)(() => {
+            const iframe = (0, test_utils_1.getIFrameEl)();
+            expect(iframe).toBeTruthy();
+        });
+        mockProcessTrigger.mockClear();
+        mockProcessTrigger.mockResolvedValue({});
+        liveboardEmbed['setupFullscreenChangeHandler']();
+        Object.defineProperty(document, 'fullscreenElement', {
+            value: (0, test_utils_1.getIFrameEl)(),
+            writable: true,
+            configurable: true,
+        });
+        const event = new Event('fullscreenchange');
+        document.dispatchEvent(event);
+        await (0, test_utils_1.executeAfterWait)(() => {
+            expect(mockProcessTrigger).not.toHaveBeenCalledWith(expect.any(Object), types_1.HostEvent.ExitPresentMode, expect.any(String), expect.any(Object));
+        });
+    });
+});
+describe('ShowPreRender with UpdateEmbedParams', () => {
+    beforeAll(() => {
+        (0, index_1.init)({
+            thoughtSpotHost: 'tshost',
+            authType: index_1.AuthType.None,
+        });
+    });
+    afterEach(() => {
+        const rootEle = document.getElementById('myRoot');
+        rootEle === null || rootEle === void 0 ? void 0 : rootEle.remove();
+    });
+    test('should trigger UpdateEmbedParams when showPreRender connects to existing prerendered component', async () => {
+        (0, test_utils_1.createRootEleForEmbed)();
+        (0, test_utils_1.mockMessageChannel)();
+        window.ResizeObserver = window.ResizeObserver
+            || jest.fn().mockImplementation(() => ({
+                disconnect: jest.fn(),
+                observe: jest.fn(),
+                unobserve: jest.fn(),
+            }));
+        const embed1 = new index_1.LiveboardEmbed('#tsEmbedDiv', {
+            preRenderId: 'update-params-test',
+            liveboardId: 'original-lb',
+        });
+        await embed1.preRender();
+        await (0, test_utils_1.waitFor)(() => !!(0, test_utils_1.getIFrameEl)());
+        embed1.isEmbedContainerLoaded = true;
+        mockProcessTrigger.mockClear();
+        mockProcessTrigger.mockResolvedValue({});
+        const embed2 = new index_1.LiveboardEmbed('#tsEmbedDiv', {
+            preRenderId: 'update-params-test',
+            liveboardId: 'updated-lb',
+        });
+        embed2.showPreRender();
+        await (0, test_utils_1.executeAfterWait)(() => {
+            expect(mockProcessTrigger).toHaveBeenCalledWith(expect.any(Object), types_1.HostEvent.UpdateEmbedParams, expect.any(String), expect.objectContaining({
+                liveboardId: 'updated-lb',
+            }));
+        });
+    });
+});
 //# sourceMappingURL=ts-embed.spec.js.map