@thoughtspot/visual-embed-sdk 1.20.1 → 1.21.0-alpha.0

package/src/embed/search.ts

@@ -15,11 +15,7 @@ import {
     ViewConfig,
     RuntimeFilter,
 } from '../types';
-import {
-    getQueryParamString,
-    checkReleaseVersionInBeta,
-    getFilterQuery,
-} from '../utils';
+import { getQueryParamString, checkReleaseVersionInBeta, getFilterQuery } from '../utils';
 import { TsEmbed } from './ts-embed';
 import { version } from '../../package.json';
 import { ERROR_MESSAGE } from '../errors';
@@ -37,7 +33,8 @@ export interface SearchOptions {
     /**
      * Boolean to determine if the search should be executed or not.
      * if it is executed, put the focus on the results.
-     * if it’s not executed, put the focus in the search bar - at the end of the tokens
+     * if it’s not executed, put the focus in the search bar - at the end of
+     * the tokens
      */
     executeSearch?: boolean;
 }
@@ -63,12 +60,9 @@ export interface SearchViewConfig extends ViewConfig {
      * using raw answer data.
      */
     hideResults?: boolean;
-    /**
-     * If set to true, expands all the data sources panel.
-     */
-    expandAllDataSource?: boolean;
     /**
      * If set to true, the Search Assist feature is enabled.
+     *
      * @version SDK: 1.13.0 | ThoughtSpot: 8.5.0.cl, 8.8.1-sw
      */
     enableSearchAssist?: boolean;
@@ -80,16 +74,19 @@ export interface SearchViewConfig extends ViewConfig {
     /**
      * The array of data source GUIDs to set on load.
      * Only a single dataSource supported currently.
+     *
      * @deprecated Use dataSource instead
      */
     dataSources?: string[];
     /**
      * The array of data source GUIDs to set on load.
+     *
      * @version: SDK: 1.19.0
      */
     dataSource?: string;
     /**
      * The initial search query to load the answer with.
+     *
      * @deprecated Use {@link searchOptions} instead
      */
     searchQuery?: string;
@@ -104,6 +101,7 @@ export interface SearchViewConfig extends ViewConfig {
     /**
      * If set to true, search page will render without the Search Bar
      * The chart/table should still be visible.
+     *
      * @version SDK: 1.21.0 | ThoughtSpot: 9.2.0.cl
      */
     hideSearchBar?: boolean;
@@ -149,23 +147,16 @@ export class SearchEmbed extends TsEmbed {
         return dataSourceMode;
     }
 
-    /**
-     * Construct the URL of the embedded ThoughtSpot search to be
-     * loaded in the iframe
-     * @param answerId The GUID of a saved answer
-     * @param dataSources A list of data source GUIDs
-     */
-    private getIFrameSrc(answerId: string, dataSources?: string[]) {
+    protected getEmbedParams(): string {
         const {
             hideResults,
-            expandAllDataSource,
             enableSearchAssist,
             forceTable,
             searchOptions,
             runtimeFilters,
             dataSource,
+            dataSources,
         } = this.viewConfig;
-        const answerPath = answerId ? `saved-answer/${answerId}` : 'answer';
         const queryParams = this.getBaseQueryParams();
 
         queryParams[Param.HideActions] = [
@@ -210,11 +201,21 @@ export class SearchEmbed extends TsEmbed {
         if (filterQuery) {
             query += `&${filterQuery}`;
         }
+        return query;
+    }
+
+    /**
+     * Construct the URL of the embedded ThoughtSpot search to be
+     * loaded in the iframe
+     *
+     * @param answerId The GUID of a saved answer
+     * @param dataSources A list of data source GUIDs
+     */
+    private getIFrameSrc(answerId: string) {
+        const answerPath = answerId ? `saved-answer/${answerId}` : 'answer';
         const tsPostHashParams = this.getThoughtSpotPostUrlParams();
 
-        return `${this.getEmbedBasePath(
-            query,
-        )}/${answerPath}${tsPostHashParams}`;
+        return `${this.getRootIframeSrc()}/embed/${answerPath}${tsPostHashParams}`;
     }
 
     /**
@@ -222,10 +223,10 @@ export class SearchEmbed extends TsEmbed {
      */
     public render(): SearchEmbed {
         super.render();
-        const { answerId, dataSources } = this.viewConfig;
+        const { answerId } = this.viewConfig;
 
-        const src = this.getIFrameSrc(answerId, dataSources);
-        this.renderIFrame(src, this.viewConfig.frameParams);
+        const src = this.getIFrameSrc(answerId);
+        this.renderIFrame(src);
         getAuthPromise().then(() => {
             if (
                 checkReleaseVersionInBeta(

package/src/embed/searchEmbed-basic-auth.spec.ts

@@ -19,27 +19,27 @@ init({
 });
 
 describe('Search embed tests when authType is Basic', () => {
+    /**
+     *
+     * @param version
+     */
     function setupVersion(version: string) {
-        jest.spyOn(window, 'addEventListener').mockImplementationOnce(
-            (event, handler, options) => {
-                handler({
-                    data: {
-                        type: 'xyz',
-                    },
-                    ports: [3000],
-                    source: null,
-                });
-            },
-        );
-        jest.spyOn(authService, 'fetchSessionInfoService').mockImplementation(
-            async () => ({
-                json: () => ({
-                    ...mockSessionInfo,
-                    releaseVersion: version,
-                }),
-                status: 200,
+        jest.spyOn(window, 'addEventListener').mockImplementationOnce((event, handler, options) => {
+            handler({
+                data: {
+                    type: 'xyz',
+                },
+                ports: [3000],
+                source: null,
+            });
+        });
+        jest.spyOn(authService, 'fetchSessionInfoService').mockImplementation(async () => ({
+            json: () => ({
+                ...mockSessionInfo,
+                releaseVersion: version,
             }),
-        );
+            status: 200,
+        }));
         jest.spyOn(authInstance, 'getReleaseVersion').mockReturnValue(version);
     }
     beforeEach(() => {
@@ -76,9 +76,7 @@ describe('Search embed tests when authType is Basic', () => {
 
     test('releaseVersion is above 8.4.0.sw', async () => {
         setupVersion('8.8.0.sw');
-        jest.spyOn(config, 'getThoughtSpotHost').mockImplementation(
-            () => 'http://tshost',
-        );
+        jest.spyOn(config, 'getThoughtSpotHost').mockImplementation(() => 'http://tshost');
         const mockAlert = jest.spyOn(window, 'alert');
         const searchEmbed = new SearchEmbed(getRootEl(), {});
         await searchEmbed.render();
@@ -87,9 +85,7 @@ describe('Search embed tests when authType is Basic', () => {
 
     test('Alert should not appear when suppressSearchEmbedBetaWarning is true and releaseVersion is ts7.dec.cl', async () => {
         setupVersion('ts7.dec.cl');
-        jest.spyOn(config, 'getThoughtSpotHost').mockImplementation(
-            () => 'http://tshost',
-        );
+        jest.spyOn(config, 'getThoughtSpotHost').mockImplementation(() => 'http://tshost');
         jest.spyOn(baseInstance, 'getEmbedConfig').mockReturnValue({
             suppressSearchEmbedBetaWarning: true,
         });
@@ -101,9 +97,7 @@ describe('Search embed tests when authType is Basic', () => {
 
     test('Alert should not appear when suppressSearchEmbedBetaWarning is true and releaseVersion is 8.4.0.sw', async () => {
         setupVersion('8.4.0.sw');
-        jest.spyOn(config, 'getThoughtSpotHost').mockImplementation(
-            () => 'http://tshost',
-        );
+        jest.spyOn(config, 'getThoughtSpotHost').mockImplementation(() => 'http://tshost');
         jest.spyOn(baseInstance, 'getEmbedConfig').mockReturnValue({
             suppressSearchEmbedBetaWarning: true,
         });