Spaces NP Migration - Moving server to the LegacyAPI model

src/core/server/http/http_service.mock.ts

@@ -23,6 +23,7 @@ import { OnPreAuthToolkit } from './lifecycle/on_pre_auth';
 import { AuthToolkit } from './lifecycle/auth';
 import { sessionStorageMock } from './cookie_session_storage.mocks';
 import { IRouter } from './router';
+import { OnPostAuthToolkit } from './lifecycle/on_post_auth';
 
 type ServiceSetupMockType = jest.Mocked<HttpServiceSetup> & {
   basePath: jest.Mocked<HttpServiceSetup['basePath']>;
@@ -89,6 +90,10 @@ const createOnPreAuthToolkitMock = (): jest.Mocked<OnPreAuthToolkit> => ({
   rewriteUrl: jest.fn(),
 });
 
+const createOnPostAuthToolkitMock = (): jest.Mocked<OnPostAuthToolkit> => ({
+  next: jest.fn(),
+});
+
 const createAuthToolkitMock = (): jest.Mocked<AuthToolkit> => ({
   authenticated: jest.fn(),
 });
@@ -98,6 +103,7 @@ export const httpServiceMock = {
   createBasePath: createBasePathMock,
   createSetupContract: createSetupContractMock,
   createOnPreAuthToolkit: createOnPreAuthToolkitMock,
+  createOnPostAuthToolkit: createOnPostAuthToolkitMock,
   createAuthToolkit: createAuthToolkitMock,
   createRouter: createRouterMock,
 };

x-pack/legacy/plugins/spaces/index.ts

@@ -7,6 +7,7 @@
 import * as Rx from 'rxjs';
 import { resolve } from 'path';
 import KbnServer, { Server } from 'src/legacy/server/kbn_server';
+import { CoreSetup, PluginInitializerContext } from 'src/core/server';
 import { createOptionalPlugin } from '../../server/lib/optional_plugin';
 // @ts-ignore
 import { AuditLogger } from '../../server/lib/audit_logger';
@@ -16,14 +17,9 @@ import { getActiveSpace } from './server/lib/get_active_space';
 import { getSpaceSelectorUrl } from './server/lib/get_space_selector_url';
 import { migrateToKibana660 } from './server/lib/migrations';
 import { plugin } from './server/new_platform';
-import {
-  SpacesInitializerContext,
-  SpacesCoreSetup,
-  SpacesHttpServiceSetup,
-} from './server/new_platform/plugin';
-import { initSpacesRequestInterceptors } from './server/lib/request_interceptors';
 import { SecurityPlugin } from '../security';
 import { SpacesServiceSetup } from './server/new_platform/spaces_service/spaces_service';
+import { initSpaceSelectorView } from './server/routes/views';
 
 export interface SpacesPlugin {
   getSpaceId: SpacesServiceSetup['getSpaceId'];
@@ -122,8 +118,7 @@ export const spaces = (kibana: Record<string, any>) =>
 
     async init(server: Server) {
       const kbnServer = (server as unknown) as KbnServer;
-      const initializerContext = ({
-        legacyConfig: server.config(),
+      const initializerContext = {
         config: {
           create: () => {
             return Rx.of({
@@ -140,29 +135,15 @@ export const spaces = (kibana: Record<string, any>) =>
             );
           },
         },
-      } as unknown) as SpacesInitializerContext;
-
-      const spacesHttpService: SpacesHttpServiceSetup = {
-        ...kbnServer.newPlatform.setup.core.http,
-        route: server.route.bind(server),
-      };
+      } as PluginInitializerContext;
 
-      const core: SpacesCoreSetup = {
-        http: spacesHttpService,
-        elasticsearch: kbnServer.newPlatform.setup.core.elasticsearch,
-        savedObjects: server.savedObjects,
-        usage: server.usage,
-        tutorial: {
-          addScopedTutorialContextFactory: server.addScopedTutorialContextFactory,
+      const core: CoreSetup = ({
+        ...kbnServer.newPlatform.setup.core,
+        http: {
+          ...kbnServer.newPlatform.setup.core.http,
+          route: server.route,
         },
-        capabilities: {
-          registerCapabilitiesModifier: server.registerCapabilitiesModifier,
-        },
-        auditLogger: {
-          create: (pluginId: string) =>
-            new AuditLogger(server, pluginId, server.config(), server.plugins.xpack_main.info),
-        },
-      };
+      } as unknown) as CoreSetup;
 
       const plugins = {
         xpackMain: server.plugins.xpack_main,
@@ -177,20 +158,31 @@ export const spaces = (kibana: Record<string, any>) =>
         spaces: this,
       };
 
-      const { spacesService, log } = await plugin(initializerContext).setup(core, plugins);
+      const legacyRouter = server.route.bind(server);
 
-      initSpacesRequestInterceptors({
-        config: initializerContext.legacyConfig,
-        http: core.http,
-        getHiddenUiAppById: server.getHiddenUiAppById,
-        onPostAuth: handler => {
-          server.ext('onPostAuth', handler);
+      const { spacesService, registerLegacyAPI } = await plugin(
+        initializerContext,
+        legacyRouter
+      ).setup(core, plugins);
+
+      registerLegacyAPI({
+        legacyConfig: server.config(),
+        savedObjects: server.savedObjects,
+        usage: server.usage,
+        tutorial: {
+          addScopedTutorialContextFactory: server.addScopedTutorialContextFactory,
+        },
+        capabilities: {
+          registerCapabilitiesModifier: server.registerCapabilitiesModifier,
+        },
+        auditLogger: {
+          create: (pluginId: string) =>
+            new AuditLogger(server, pluginId, server.config(), server.plugins.xpack_main.info),
         },
-        log,
-        spacesService,
-        xpackMain: plugins.xpackMain,
       });
 
+      initSpaceSelectorView(server);
+
       server.expose('getSpaceId', (request: any) => spacesService.getSpaceId(request));
       server.expose('spaceIdToNamespace', spacesService.spaceIdToNamespace);
       server.expose('namespaceToSpaceId', spacesService.namespaceToSpaceId);

x-pack/legacy/plugins/spaces/server/lib/create_default_space.ts

@@ -7,11 +7,11 @@
 import { i18n } from '@kbn/i18n';
 
 import { first } from 'rxjs/operators';
-import { ElasticsearchServiceSetup, SavedObjectsService } from 'src/core/server';
+import { SavedObjectsService, CoreSetup } from 'src/core/server';
 import { DEFAULT_SPACE_ID } from '../../common/constants';
 
 interface Deps {
-  elasticsearch: ElasticsearchServiceSetup;
+  elasticsearch: CoreSetup['elasticsearch'];
   savedObjects: SavedObjectsService;
 }
 

x-pack/legacy/plugins/spaces/server/lib/get_space_selector_url.test.ts

@@ -6,25 +6,14 @@
 
 import { getSpaceSelectorUrl } from './get_space_selector_url';
 
-const buildServerConfig = (serverBasePath?: string) => {
-  return {
-    get: (key: string) => {
-      if (key === 'server.basePath') {
-        return serverBasePath;
-      }
-      throw new Error(`unexpected config request: ${key}`);
-    },
-  };
-};
-
 describe('getSpaceSelectorUrl', () => {
   it('returns / when no server base path is defined', () => {
-    const serverConfig = buildServerConfig();
-    expect(getSpaceSelectorUrl(serverConfig)).toEqual('/');
+    expect(getSpaceSelectorUrl('')).toEqual('/spaces/space_selector');
   });
 
   it('returns the server base path when defined', () => {
-    const serverConfig = buildServerConfig('/my/server/base/path');
-    expect(getSpaceSelectorUrl(serverConfig)).toEqual('/my/server/base/path');
+    expect(getSpaceSelectorUrl('/my/server/base/path')).toEqual(
+      '/my/server/base/path/spaces/space_selector'
+    );
   });
 });

x-pack/legacy/plugins/spaces/server/lib/get_space_selector_url.ts

@@ -4,10 +4,6 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 
-interface Config {
-  get(key: string): string | boolean | number | null | undefined;
-}
-
-export function getSpaceSelectorUrl(serverConfig: Config) {
-  return serverConfig.get('server.basePath') || '/';
+export function getSpaceSelectorUrl(serverBasePath: string = '') {
+  return `${serverBasePath}/spaces/space_selector`;
 }