[Bug][Workspace]fix: revert new home page ui setting for workspace de…

…fault route (opensearch-project#7858) * revert new home page ui setting for workspace default route Signed-off-by: yubonluo <yubonluo@amazon.com> * Changeset file for PR opensearch-project#7858 created/updated --------- Signed-off-by: yubonluo <yubonluo@amazon.com> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
RyanL1997 · Aug 29, 2024 · a0365d8 · a0365d8
1 parent 07504f6
commit a0365d8
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 51 deletions.
diff --git a/changelogs/fragments/7858.yml b/changelogs/fragments/7858.yml
@@ -0,0 +1,2 @@
+fix:
+- [Workspace] Revert new home page ui setting for workspace default route ([#7858](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/7858))
diff --git a/src/plugins/workspace/server/plugin.test.ts b/src/plugins/workspace/server/plugin.test.ts
@@ -183,23 +183,6 @@ describe('Workspace server plugin', () => {
   describe('#setUpRedirectPage', () => {
     const setupMock = coreMock.createSetup();
     const uiSettingsMock = uiSettingsServiceMock.createClient();
-    setupMock.getStartServices.mockResolvedValue([
-      {
-        ...coreMock.createStart(),
-        uiSettings: {
-          asScopedToClient: () => ({
-            ...uiSettingsMock,
-            get: jest.fn().mockImplementation((key) => {
-              if (key === 'home:useNewHomePage') {
-                return Promise.resolve(true);
-              }
-            }),
-          }),
-        },
-      },
-      {},
-      {},
-    ]);
     const initializerContextConfigMock = coreMock.createPluginInitializerContext({
       enabled: true,
       permission: {
@@ -304,8 +287,6 @@ describe('Workspace server plugin', () => {
               get: jest.fn().mockImplementation((key) => {
                 if (key === 'defaultWorkspace') {
                   return Promise.resolve('defaultWorkspace');
-                } else if (key === 'home:useNewHomePage') {
-                  return Promise.resolve('true');
                 }
               }),
             }),
@@ -337,29 +318,6 @@ describe('Workspace server plugin', () => {
         },
       });
     });
-
-    it('with / request path and home:useNewHomePage is false', async () => {
-      const request = httpServerMock.createOpenSearchDashboardsRequest({
-        path: '/',
-      });
-      setupMock.getStartServices.mockResolvedValue([
-        {
-          ...coreMock.createStart(),
-          uiSettings: {
-            asScopedToClient: () => ({
-              ...uiSettingsMock,
-              get: jest.fn().mockResolvedValue(false),
-            }),
-          },
-        },
-        {},
-        {},
-      ]);
-      const toolKitMock = httpServerMock.createToolkit();
-
-      await registerOnPostAuthFn(request, response, toolKitMock);
-      expect(toolKitMock.next).toBeCalledTimes(1);
-    });
   });
 
   it('#start', async () => {

diff --git a/src/plugins/workspace/server/plugin.ts b/src/plugins/workspace/server/plugin.ts
@@ -114,15 +114,6 @@ export class WorkspacePlugin implements Plugin<WorkspacePluginSetup, WorkspacePl
     core.http.registerOnPostAuth(async (request, response, toolkit) => {
       const path = request.url.pathname;
       if (path === '/') {
-        const [coreStart] = await core.getStartServices();
-        const uiSettings = coreStart.uiSettings.asScopedToClient(
-          coreStart.savedObjects.getScopedClient(request)
-        );
-        const useNewHomePage = await uiSettings.get('home:useNewHomePage');
-        if (!useNewHomePage) {
-          return toolkit.next();
-        }
-
         const workspaceListResponse = await this.client?.list(
           { request, logger: this.logger },
           { page: 1, perPage: 100 }
@@ -139,6 +130,10 @@ export class WorkspacePlugin implements Plugin<WorkspacePluginSetup, WorkspacePl
               },
             });
           }
+          const [coreStart] = await core.getStartServices();
+          const uiSettings = coreStart.uiSettings.asScopedToClient(
+            coreStart.savedObjects.getScopedClient(request)
+          );
           // Temporarily use defaultWorkspace as a placeholder
           const defaultWorkspaceId = await uiSettings.get('defaultWorkspace');
           const defaultWorkspace = workspaceList.find(