Update mocks/dependencies so other tests pass type checks

cee-chen · cee-chen · commit ffb659f2a55a · 2020-08-27T08:25:07.000-07:00
diff --git a/x-pack/plugins/enterprise_search/server/__mocks__/index.ts b/x-pack/plugins/enterprise_search/server/__mocks__/index.ts
@@ -5,4 +5,9 @@
  */
 
 export { MockRouter } from './router.mock';
-export { mockConfig, mockLogger, mockDependencies } from './routerDependencies.mock';
+export {
+  mockConfig,
+  mockLogger,
+  mockRequestHandler,
+  mockDependencies,
+} from './routerDependencies.mock';
diff --git a/x-pack/plugins/enterprise_search/server/__mocks__/routerDependencies.mock.ts b/x-pack/plugins/enterprise_search/server/__mocks__/routerDependencies.mock.ts
@@ -9,6 +9,10 @@ import { ConfigType } from '../';
 
 export const mockLogger = loggingSystemMock.createLogger().get();
 
+export const mockRequestHandler = {
+  createRequest: jest.fn(),
+};
+
 export const mockConfig = {
   enabled: true,
   host: 'http://localhost:3002',
@@ -24,4 +28,5 @@ export const mockDependencies = {
   // Mock router should be handled on a per-test basis
   config: mockConfig,
   log: mockLogger,
+  enterpriseSearchRequestHandler: mockRequestHandler,
 };
diff --git a/x-pack/plugins/enterprise_search/server/routes/enterprise_search/telemetry.test.ts b/x-pack/plugins/enterprise_search/server/routes/enterprise_search/telemetry.test.ts
@@ -5,7 +5,7 @@
  */
 
 import { loggingSystemMock, savedObjectsServiceMock } from 'src/core/server/mocks';
-import { MockRouter, mockConfig, mockLogger } from '../../__mocks__';
+import { MockRouter, mockLogger, mockDependencies } from '../../__mocks__';
 
 jest.mock('../../collectors/lib/telemetry', () => ({
   incrementUICounter: jest.fn(),
@@ -28,10 +28,10 @@ describe('Enterprise Search Telemetry API', () => {
     mockRouter = new MockRouter({ method: 'put', payload: 'body' });
 
     registerTelemetryRoute({
+      ...mockDependencies,
       router: mockRouter.router,
       getSavedObjectsService: () => savedObjectsServiceMock.createStartContract(),
       log: mockLogger,
-      config: mockConfig,
     });
   });
 
