[App Search] Synonyms set up (#97187)

* Add barebones Synonyms view

* Update Synonyms route+nav link

* Add server API routes

- will be used in upcoming PRs, may change

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

x-pack/plugins/enterprise_search/public/applications/app_search/components/engine/engine_nav.tsx

@@ -211,8 +211,7 @@ export const EngineNav: React.FC = () => {
       )}
       {canManageEngineSynonyms && (
         <SideNavLink
-          isExternal
-          to={getAppSearchUrl(generateEnginePath(ENGINE_SYNONYMS_PATH))}
+          to={generateEnginePath(ENGINE_SYNONYMS_PATH)}
           data-test-subj="EngineSynonymsLink"
         >
           {SYNONYMS_TITLE}

x-pack/plugins/enterprise_search/public/applications/app_search/components/engine/engine_router.test.tsx

@@ -22,6 +22,7 @@ import { CurationsRouter } from '../curations';
 import { EngineOverview } from '../engine_overview';
 import { RelevanceTuning } from '../relevance_tuning';
 import { ResultSettings } from '../result_settings';
+import { Synonyms } from '../synonyms';
 
 import { EngineRouter } from './engine_router';
 
@@ -100,6 +101,13 @@ describe('EngineRouter', () => {
     expect(wrapper.find(AnalyticsRouter)).toHaveLength(1);
   });
 
+  it('renders a synonyms view', () => {
+    setMockValues({ ...values, myRole: { canManageEngineSynonyms: true } });
+    const wrapper = shallow(<EngineRouter />);
+
+    expect(wrapper.find(Synonyms)).toHaveLength(1);
+  });
+
   it('renders a curations view', () => {
     setMockValues({ ...values, myRole: { canManageEngineCurations: true } });
     const wrapper = shallow(<EngineRouter />);

x-pack/plugins/enterprise_search/public/applications/app_search/components/engine/engine_router.tsx

@@ -27,7 +27,7 @@ import {
   // ENGINE_CRAWLER_PATH,
   // META_ENGINE_SOURCE_ENGINES_PATH,
   ENGINE_RELEVANCE_TUNING_PATH,
-  // ENGINE_SYNONYMS_PATH,
+  ENGINE_SYNONYMS_PATH,
   ENGINE_CURATIONS_PATH,
   ENGINE_RESULT_SETTINGS_PATH,
   // ENGINE_SEARCH_UI_PATH,
@@ -39,8 +39,8 @@ import { CurationsRouter } from '../curations';
 import { DocumentDetail, Documents } from '../documents';
 import { EngineOverview } from '../engine_overview';
 import { RelevanceTuning } from '../relevance_tuning';
-
 import { ResultSettings } from '../result_settings';
+import { Synonyms } from '../synonyms';
 
 import { EngineLogic, getEngineBreadcrumbs } from './';
 
@@ -53,7 +53,7 @@ export const EngineRouter: React.FC = () => {
       // canViewEngineCrawler,
       // canViewMetaEngineSourceEngines,
       canManageEngineRelevanceTuning,
-      // canManageEngineSynonyms,
+      canManageEngineSynonyms,
       canManageEngineCurations,
       canManageEngineResultSettings,
       // canManageEngineSearchUi,
@@ -107,6 +107,11 @@ export const EngineRouter: React.FC = () => {
           <RelevanceTuning />
         </Route>
       )}
+      {canManageEngineSynonyms && (
+        <Route path={ENGINE_SYNONYMS_PATH}>
+          <Synonyms />
+        </Route>
+      )}
       {canManageEngineResultSettings && (
         <Route path={ENGINE_RESULT_SETTINGS_PATH}>
           <ResultSettings />

x-pack/plugins/enterprise_search/public/applications/app_search/components/synonyms/index.ts

@@ -6,3 +6,4 @@
  */
 
 export { SYNONYMS_TITLE } from './constants';
+export { Synonyms } from './synonyms';

x-pack/plugins/enterprise_search/public/applications/app_search/components/synonyms/synonyms.test.tsx

@@ -0,0 +1,21 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License
+ * 2.0; you may not use this file except in compliance with the Elastic License
+ * 2.0.
+ */
+
+import '../../__mocks__/engine_logic.mock';
+
+import React from 'react';
+
+import { shallow } from 'enzyme';
+
+import { Synonyms } from './';
+
+describe('Synonyms', () => {
+  it('renders', () => {
+    shallow(<Synonyms />);
+    // TODO: Check for Synonym cards, Synonym modal
+  });
+});

x-pack/plugins/enterprise_search/public/applications/app_search/components/synonyms/synonyms.tsx

@@ -0,0 +1,27 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License
+ * 2.0; you may not use this file except in compliance with the Elastic License
+ * 2.0.
+ */
+
+import React from 'react';
+
+import { EuiPageHeader, EuiPageContentBody } from '@elastic/eui';
+
+import { FlashMessages } from '../../../shared/flash_messages';
+import { SetAppSearchChrome as SetPageChrome } from '../../../shared/kibana_chrome';
+import { getEngineBreadcrumbs } from '../engine';
+
+import { SYNONYMS_TITLE } from './constants';
+
+export const Synonyms: React.FC = () => {
+  return (
+    <>
+      <SetPageChrome trail={getEngineBreadcrumbs([SYNONYMS_TITLE])} />
+      <EuiPageHeader pageTitle={SYNONYMS_TITLE} />
+      <FlashMessages />
+      <EuiPageContentBody>TODO</EuiPageContentBody>
+    </>
+  );
+};

x-pack/plugins/enterprise_search/server/routes/app_search/synonyms.test.ts

@@ -0,0 +1,207 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License
+ * 2.0; you may not use this file except in compliance with the Elastic License
+ * 2.0.
+ */
+
+import { MockRouter, mockRequestHandler, mockDependencies } from '../../__mocks__';
+
+import { registerSynonymsRoutes } from './synonyms';
+
+describe('synonyms routes', () => {
+  describe('GET /api/app_search/engines/{engineName}/synonyms', () => {
+    let mockRouter: MockRouter;
+
+    beforeEach(() => {
+      jest.clearAllMocks();
+      mockRouter = new MockRouter({
+        method: 'get',
+        path: '/api/app_search/engines/{engineName}/synonyms',
+      });
+
+      registerSynonymsRoutes({
+        ...mockDependencies,
+        router: mockRouter.router,
+      });
+    });
+
+    it('creates a request handler', () => {
+      expect(mockRequestHandler.createRequest).toHaveBeenCalledWith({
+        path: '/as/engines/:engineName/synonyms/collection',
+      });
+    });
+
+    describe('validates', () => {
+      it('with pagination query params', () => {
+        const request = {
+          query: {
+            'page[current]': 1,
+            'page[size]': 10,
+          },
+        };
+        mockRouter.shouldValidate(request);
+      });
+
+      it('missing query params', () => {
+        const request = { query: {} };
+        mockRouter.shouldThrow(request);
+      });
+    });
+  });
+
+  describe('POST /api/app_search/engines/{engineName}/synonyms', () => {
+    let mockRouter: MockRouter;
+
+    beforeEach(() => {
+      jest.clearAllMocks();
+      mockRouter = new MockRouter({
+        method: 'post',
+        path: '/api/app_search/engines/{engineName}/synonyms',
+      });
+
+      registerSynonymsRoutes({
+        ...mockDependencies,
+        router: mockRouter.router,
+      });
+    });
+
+    it('creates a request handler', () => {
+      expect(mockRequestHandler.createRequest).toHaveBeenCalledWith({
+        path: '/as/engines/:engineName/synonyms/collection',
+      });
+    });
+
+    describe('validates', () => {
+      it('with synonyms', () => {
+        const request = {
+          body: {
+            synonyms: ['a', 'b', 'c'],
+          },
+        };
+        mockRouter.shouldValidate(request);
+      });
+
+      it('empty synonyms array', () => {
+        const request = {
+          body: {
+            queries: [],
+          },
+        };
+        mockRouter.shouldThrow(request);
+      });
+
+      it('only one synonym', () => {
+        const request = {
+          body: {
+            queries: ['a'],
+          },
+        };
+        mockRouter.shouldThrow(request);
+      });
+
+      it('empty synonym strings', () => {
+        const request = {
+          body: {
+            queries: ['', '', ''],
+          },
+        };
+        mockRouter.shouldThrow(request);
+      });
+
+      it('missing synonyms', () => {
+        const request = { body: {} };
+        mockRouter.shouldThrow(request);
+      });
+    });
+  });
+
+  describe('PUT /api/app_search/engines/{engineName}/synonyms/{synonymId}', () => {
+    let mockRouter: MockRouter;
+
+    beforeEach(() => {
+      jest.clearAllMocks();
+      mockRouter = new MockRouter({
+        method: 'put',
+        path: '/api/app_search/engines/{engineName}/synonyms/{synonymId}',
+      });
+
+      registerSynonymsRoutes({
+        ...mockDependencies,
+        router: mockRouter.router,
+      });
+    });
+
+    it('creates a request handler', () => {
+      expect(mockRequestHandler.createRequest).toHaveBeenCalledWith({
+        path: '/as/engines/:engineName/synonyms/:synonymId',
+      });
+    });
+
+    describe('validates', () => {
+      it('with synonyms', () => {
+        const request = {
+          body: {
+            synonyms: ['a', 'b', 'c'],
+          },
+        };
+        mockRouter.shouldValidate(request);
+      });
+
+      it('empty synonyms array', () => {
+        const request = {
+          body: {
+            queries: [],
+          },
+        };
+        mockRouter.shouldThrow(request);
+      });
+
+      it('only one synonym', () => {
+        const request = {
+          body: {
+            queries: ['a'],
+          },
+        };
+        mockRouter.shouldThrow(request);
+      });
+
+      it('empty synonym strings', () => {
+        const request = {
+          body: {
+            queries: ['', '', ''],
+          },
+        };
+        mockRouter.shouldThrow(request);
+      });
+
+      it('missing synonyms', () => {
+        const request = { body: {} };
+        mockRouter.shouldThrow(request);
+      });
+    });
+  });
+
+  describe('DELETE /api/app_search/engines/{engineName}/synonyms/{synonymId}', () => {
+    let mockRouter: MockRouter;
+
+    beforeEach(() => {
+      jest.clearAllMocks();
+      mockRouter = new MockRouter({
+        method: 'delete',
+        path: '/api/app_search/engines/{engineName}/synonyms/{synonymId}',
+      });
+
+      registerSynonymsRoutes({
+        ...mockDependencies,
+        router: mockRouter.router,
+      });
+    });
+
+    it('creates a request handler', () => {
+      expect(mockRequestHandler.createRequest).toHaveBeenCalledWith({
+        path: '/as/engines/:engineName/synonyms/:synonymId',
+      });
+    });
+  });
+});