Only expose list method

x-pack/plugins/index_management/server/lib/enrich_policies.ts

@@ -46,11 +46,3 @@ export const fetchAll = async (client: IScopedClusterClient) => {
 
   return serializeEnrichmentPolicies(res.policies);
 };
-
-export const execute = (client: IScopedClusterClient, policyName: string) => {
-  return client.asCurrentUser.enrich.executePolicy({ name: policyName });
-};
-
-export const remove = (client: IScopedClusterClient, policyName: string) => {
-  return client.asCurrentUser.enrich.deletePolicy({ name: policyName });
-};

x-pack/plugins/index_management/server/routes/api/enrich_policies/enrich_policies.test.ts

@@ -22,12 +22,7 @@ const mockedPolicy = {
   },
 };
 
-/**
- * Since these route callbacks are so thin, these serve simply as integration tests
- * to ensure they're wired up to the lib functions correctly. Business logic is tested
- * more thoroughly in the es_deprecation_logging_apis test.
- */
-describe('deprecation logging API', () => {
+describe('Enrich policies API', () => {
   const router = new RouterMock();
 
   beforeEach(() => {
@@ -71,78 +66,4 @@ describe('deprecation logging API', () => {
       await expect(router.runRequest(mockRequest)).rejects.toThrowError(error);
     });
   });
-
-  describe('Execute policy - PUT /api/index_management/enrich_policies/{policy}', () => {
-    const executeEnrichPolicy = router.getMockESApiFn('enrich.executePolicy');
-
-    it('correctly executes a policy', async () => {
-      const mockRequest: RequestMock = {
-        method: 'put',
-        path: addBasePath('/enrich_policies/{name}'),
-        params: {
-          name: 'my-policy',
-        },
-      };
-
-      executeEnrichPolicy.mockResolvedValue({ status: { phase: 'COMPLETE' } });
-
-      const res = await router.runRequest(mockRequest);
-
-      expect(res).toEqual({
-        body: { status: { phase: 'COMPLETE' } },
-      });
-    });
-
-    it('should return an error if it fails', async () => {
-      const mockRequest: RequestMock = {
-        method: 'put',
-        path: addBasePath('/enrich_policies/{name}'),
-        params: {
-          name: 'my-policy',
-        },
-      };
-
-      const error = new Error('Oh no!');
-      executeEnrichPolicy.mockRejectedValue(error);
-
-      await expect(router.runRequest(mockRequest)).rejects.toThrowError(error);
-    });
-  });
-
-  describe('Delete policy - DELETE /api/index_management/enrich_policies/{policy}', () => {
-    const deleteEnrichPolicy = router.getMockESApiFn('enrich.deletePolicy');
-
-    it('correctly deletes a policy', async () => {
-      const mockRequest: RequestMock = {
-        method: 'delete',
-        path: addBasePath('/enrich_policies/{name}'),
-        params: {
-          name: 'my-policy',
-        },
-      };
-
-      deleteEnrichPolicy.mockResolvedValue({ status: { phase: 'COMPLETE' } });
-
-      const res = await router.runRequest(mockRequest);
-
-      expect(res).toEqual({
-        body: { status: { phase: 'COMPLETE' } },
-      });
-    });
-
-    it('should return an error if it fails', async () => {
-      const mockRequest: RequestMock = {
-        method: 'delete',
-        path: addBasePath('/enrich_policies/{name}'),
-        params: {
-          name: 'my-policy',
-        },
-      };
-
-      const error = new Error('Oh no!');
-      deleteEnrichPolicy.mockRejectedValue(error);
-
-      await expect(router.runRequest(mockRequest)).rejects.toThrowError(error);
-    });
-  });
 });

x-pack/plugins/index_management/server/routes/api/enrich_policies/register_enrich_policies_routes.ts

@@ -8,11 +8,7 @@
 import { RouteDependencies } from '../../../types';
 
 import { registerListRoute } from './register_list_route';
-import { registerDeleteRoute } from './register_delete_route';
-import { registerExecuteRoute } from './register_execute_route';
 
 export function registerEnrichPoliciesRoute(dependencies: RouteDependencies) {
   registerListRoute(dependencies);
-  registerDeleteRoute(dependencies);
-  registerExecuteRoute(dependencies);
 }