Update jest to v24

package.json

@@ -281,7 +281,7 @@
     "@types/has-ansi": "^3.0.0",
     "@types/hoek": "^4.1.3",
     "@types/humps": "^1.1.2",
-    "@types/jest": "^23.3.1",
+    "@types/jest": "^24.0.6",
     "@types/joi": "^13.4.2",
     "@types/jquery": "^3.3.6",
     "@types/js-yaml": "^3.11.1",
@@ -359,8 +359,8 @@
     "intl-messageformat-parser": "^1.4.0",
     "is-path-inside": "^2.0.0",
     "istanbul-instrumenter-loader": "3.0.1",
-    "jest": "^23.6.0",
-    "jest-cli": "^23.6.0",
+    "jest": "^24.1.0",
+    "jest-cli": "^24.1.0",
     "jest-raw-loader": "^1.0.1",
     "jimp": "0.2.28",
     "json-stable-stringify": "^1.0.1",

packages/kbn-i18n/src/core/i18n.test.ts

@@ -885,7 +885,7 @@ describe('I18n engine', () => {
   });
 
   describe('load', () => {
-    let mockFetch: jest.Mock<unknown>;
+    let mockFetch: jest.Mock;
     beforeEach(() => {
       mockFetch = jest.spyOn(global as any, 'fetch').mockImplementation();
     });

packages/kbn-pm/package.json

@@ -20,7 +20,6 @@
     "@types/globby": "^6.1.0",
     "@types/has-ansi": "^3.0.0",
     "@types/indent-string": "^3.0.0",
-    "@types/jest": "^23.3.1",
     "@types/lodash.clonedeepwith": "^4.5.3",
     "@types/log-symbols": "^2.0.0",
     "@types/mkdirp": "^0.5.2",

src/core/public/chrome/chrome_service.test.ts

@@ -60,20 +60,12 @@ describe('start', () => {
     const startDeps = defaultStartDeps();
     startDeps.injectedMetadata.getCspConfig.mockReturnValue({ warnLegacyBrowsers: true });
     service.start(startDeps);
-    expect(startDeps.notifications.toasts.addWarning).toMatchInlineSnapshot(`
-[MockFunction] {
-  "calls": Array [
-    Array [
-      "Your browser does not meet the security requirements for Kibana.",
-    ],
-  ],
-  "results": Array [
-    Object {
-      "isThrow": false,
-      "value": undefined,
-    },
+    expect(startDeps.notifications.toasts.addWarning.mock.calls).toMatchInlineSnapshot(`
+Array [
+  Array [
+    "Your browser does not meet the security requirements for Kibana.",
   ],
-}
+]
 `);
   });
 

src/core/public/core_system.test.ts

@@ -27,92 +27,106 @@ import { LoadingCountService } from './loading_count';
 import { NotificationsService } from './notifications';
 import { UiSettingsService } from './ui_settings';
 
-const MockLegacyPlatformService = jest.fn<LegacyPlatformService>(
+const MockLegacyPlatformService = jest.fn<LegacyPlatformService, any>(
   function _MockLegacyPlatformService(this: any) {
     this.start = jest.fn();
     this.stop = jest.fn();
+    return this;
   }
 );
+
 jest.mock('./legacy_platform', () => ({
   LegacyPlatformService: MockLegacyPlatformService,
 }));
 
 const mockInjectedMetadataStartContract = {};
-const MockInjectedMetadataService = jest.fn<InjectedMetadataService>(
+const MockInjectedMetadataService = jest.fn<InjectedMetadataService, any>(
   function _MockInjectedMetadataService(this: any) {
     this.start = jest.fn().mockReturnValue(mockInjectedMetadataStartContract);
+    return this;
   }
 );
 jest.mock('./injected_metadata', () => ({
   InjectedMetadataService: MockInjectedMetadataService,
 }));
 
 const mockFatalErrorsStartContract = {};
-const MockFatalErrorsService = jest.fn<FatalErrorsService>(function _MockFatalErrorsService(
+const MockFatalErrorsService = jest.fn<FatalErrorsService, any>(function _MockFatalErrorsService(
   this: any
 ) {
   this.start = jest.fn().mockReturnValue(mockFatalErrorsStartContract);
   this.add = jest.fn();
+  return this;
 });
 jest.mock('./fatal_errors', () => ({
   FatalErrorsService: MockFatalErrorsService,
 }));
 
 const mockI18nStartContract = {};
-const MockI18nService = jest.fn<I18nService>(function _MockI18nService(this: any) {
+const MockI18nService = jest.fn<I18nService, any>(function _MockI18nService(this: any) {
   this.start = jest.fn().mockReturnValue(mockI18nStartContract);
   this.stop = jest.fn();
+  return this;
 });
 jest.mock('./i18n', () => ({
   I18nService: MockI18nService,
 }));
 
 const mockNotificationStartContract = {};
-const MockNotificationsService = jest.fn<NotificationsService>(function _MockNotificationsService(
-  this: any
-) {
-  this.start = jest.fn().mockReturnValue(mockNotificationStartContract);
-  this.add = jest.fn();
-  this.stop = jest.fn();
-});
+const MockNotificationsService = jest.fn<NotificationsService, any>(
+  function _MockNotificationsService(this: any) {
+    this.start = jest.fn().mockReturnValue(mockNotificationStartContract);
+    this.add = jest.fn();
+    this.stop = jest.fn();
+    return this;
+  }
+);
 jest.mock('./notifications', () => ({
   NotificationsService: MockNotificationsService,
 }));
 
 const mockLoadingCountContract = {};
-const MockLoadingCountService = jest.fn<LoadingCountService>(function _MockNotificationsService(
-  this: any
-) {
-  this.start = jest.fn().mockReturnValue(mockLoadingCountContract);
-  this.stop = jest.fn();
-});
+const MockLoadingCountService = jest.fn<LoadingCountService, any>(
+  function _MockNotificationsService(this: any) {
+    this.start = jest.fn().mockReturnValue(mockLoadingCountContract);
+    this.stop = jest.fn();
+    return this;
+  }
+);
 jest.mock('./loading_count', () => ({
   LoadingCountService: MockLoadingCountService,
 }));
 
 const mockBasePathStartContract = {};
-const MockBasePathService = jest.fn<BasePathService>(function _MockNotificationsService(this: any) {
+const MockBasePathService = jest.fn<BasePathService, any>(function _MockNotificationsService(
+  this: any
+) {
   this.start = jest.fn().mockReturnValue(mockBasePathStartContract);
+  return this;
 });
 jest.mock('./base_path', () => ({
   BasePathService: MockBasePathService,
 }));
 
 const mockUiSettingsContract = {};
-const MockUiSettingsService = jest.fn<UiSettingsService>(function _MockNotificationsService(
+const MockUiSettingsService = jest.fn<UiSettingsService, any>(function _MockNotificationsService(
   this: any
 ) {
   this.start = jest.fn().mockReturnValue(mockUiSettingsContract);
   this.stop = jest.fn();
+  return this;
 });
 jest.mock('./ui_settings', () => ({
   UiSettingsService: MockUiSettingsService,
 }));
 
 const mockChromeStartContract = {};
-const MockChromeService = jest.fn<ChromeService>(function _MockNotificationsService(this: any) {
+const MockChromeService = jest.fn<ChromeService, any>(function _MockNotificationsService(
+  this: any
+) {
   this.start = jest.fn().mockReturnValue(mockChromeStartContract);
   this.stop = jest.fn();
+  return this;
 });
 jest.mock('./chrome', () => ({
   ChromeService: MockChromeService,

src/core/public/fatal_errors/fatal_errors_screen.test.tsx

@@ -60,7 +60,7 @@ describe('reloading', () => {
 
     expect(locationReloadSpy).not.toHaveBeenCalled();
     const [, handler] = addEventListenerSpy.mock.calls[0];
-    handler();
+    (handler as jest.Mock)();
     expect(locationReloadSpy).toHaveBeenCalledTimes(1);
   });
 });

src/core/public/fatal_errors/fatal_errors_service.test.ts

@@ -100,9 +100,7 @@ describe('start.add()', () => {
   it('exposes a function that passes its two arguments to fatalErrors.add()', () => {
     const { fatalErrors, i18n } = setup();
 
-    jest.spyOn(fatalErrors, 'add').mockImplementation(() => {
-      /* noop */
-    });
+    jest.spyOn(fatalErrors, 'add').mockImplementation(() => undefined as never);
 
     expect(fatalErrors.add).not.toHaveBeenCalled();
     const { add } = fatalErrors.start({ i18n });

src/core/public/ui_settings/__snapshots__/ui_settings_service.test.ts.snap

@@ -12,7 +12,7 @@ exports[`#start constructs UiSettingsClient and UiSettingsApi: UiSettingsApi arg
   ],
   "results": Array [
     Object {
-      "isThrow": false,
+      "type": "return",
       "value": undefined,
     },
   ],
@@ -31,7 +31,7 @@ exports[`#start constructs UiSettingsClient and UiSettingsApi: UiSettingsClient
             ],
             "results": Array [
               Object {
-                "isThrow": false,
+                "type": "return",
                 "value": Object {
                   "loadingCountObservable": true,
                 },
@@ -52,7 +52,7 @@ exports[`#start constructs UiSettingsClient and UiSettingsApi: UiSettingsClient
   ],
   "results": Array [
     Object {
-      "isThrow": false,
+      "type": "return",
       "value": undefined,
     },
   ],
@@ -70,7 +70,7 @@ exports[`#start passes the uiSettings loading count to the loading count api: lo
   ],
   "results": Array [
     Object {
-      "isThrow": false,
+      "type": "return",
       "value": undefined,
     },
   ],

src/core/public/ui_settings/ui_settings_service.test.ts

@@ -22,7 +22,7 @@ function mockClass<T>(
   Class: { new (...args: any[]): T },
   setup: (instance: any, args: any[]) => void
 ) {
-  const MockClass = jest.fn<T>(function(this: any, ...args: any[]) {
+  const MockClass = jest.fn(function(this: any, ...args: any[]) {
     setup(this, args);
   });
 

src/core/server/legacy_compat/__snapshots__/legacy_service.test.ts.snap

@@ -37,7 +37,7 @@ Array [
           ],
           "results": Array [
             Object {
-              "isThrow": false,
+              "type": "return",
               "value": undefined,
             },
           ],

src/core/server/legacy_compat/legacy_platform_proxy.test.ts

@@ -46,8 +46,8 @@ test('correctly redirects server events.', () => {
       ([serverEventName]) => serverEventName === eventName
     )!;
 
-    serverListener(1, 2, 3, 4);
-    serverListener(5, 6, 7, 8);
+    (serverListener as jest.Mock)(1, 2, 3, 4);
+    (serverListener as jest.Mock)(5, 6, 7, 8);
 
     expect(listener).toHaveBeenCalledTimes(2);
     expect(listener).toHaveBeenCalledWith(1, 2, 3, 4);

src/core/server/logging/logger.test.ts

@@ -27,7 +27,7 @@ let appenderMocks: Appender[];
 let logger: BaseLogger;
 
 const timestamp = new Date(2012, 1, 1);
-jest.spyOn(global, 'Date').mockImplementation(() => timestamp);
+jest.spyOn<any, any>(global, 'Date').mockImplementation(() => timestamp);
 
 beforeEach(() => {
   appenderMocks = [{ append: jest.fn() }, { append: jest.fn() }];

src/core/server/logging/logging_service.test.ts

@@ -27,10 +27,10 @@ const timestamp = new Date(Date.UTC(2012, 1, 1));
 const mockConsoleLog = jest.spyOn(global.console, 'log').mockImplementation(() => {
   // noop
 });
-jest.spyOn(global, 'Date').mockImplementation(() => timestamp);
+jest.spyOn<any, any>(global, 'Date').mockImplementation(() => timestamp);
 
 import { createWriteStream } from 'fs';
-const mockCreateWriteStream = createWriteStream as jest.Mock<typeof createWriteStream>;
+const mockCreateWriteStream = (createWriteStream as unknown) as jest.Mock<typeof createWriteStream>;
 
 import { LoggingConfig, LoggingService } from '.';
 

src/core/server/root/index.test.ts

@@ -40,9 +40,9 @@ import { logger } from '../logging/__mocks__';
 const env = new Env('.', getEnvOptions());
 const config$ = new BehaviorSubject({} as Config);
 
-const mockProcessExit = jest.spyOn(global.process, 'exit').mockImplementation(() => {
-  // noop
-});
+const mockProcessExit = jest
+  .spyOn(global.process, 'exit')
+  .mockImplementation(() => undefined as never);
 
 const mockConsoleError = jest.spyOn(console, 'error').mockImplementation(() => {
   // noop
@@ -178,7 +178,7 @@ test('fails and stops services if initial logger upgrade fails', async () => {
 
 test('stops services if consequent logger upgrade fails', async () => {
   const onShutdown = new BehaviorSubject<string | null>(null);
-  const mockOnShutdown = jest.fn(() => {
+  const mockOnShutdown = jest.fn<any, any>(() => {
     onShutdown.next('completed');
     onShutdown.complete();
   });

src/legacy/core_plugins/kibana/public/management/sections/index_patterns/edit_index_pattern/source_filters_table/__jest__/__snapshots__/source_filters_table.test.js.snap

@@ -20,7 +20,7 @@ exports[`SourceFiltersTable should add a filter 1`] = `
           ],
           "results": Array [
             Object {
-              "isThrow": false,
+              "type": "return",
               "value": undefined,
             },
           ],
@@ -114,7 +114,7 @@ exports[`SourceFiltersTable should remove a filter 1`] = `
           ],
           "results": Array [
             Object {
-              "isThrow": false,
+              "type": "return",
               "value": undefined,
             },
           ],
@@ -319,7 +319,7 @@ exports[`SourceFiltersTable should update a filter 1`] = `
           ],
           "results": Array [
             Object {
-              "isThrow": false,
+              "type": "return",
               "value": undefined,
             },
           ],

src/legacy/core_plugins/kibana/server/lib/export/collect_references_deep.test.ts

@@ -17,7 +17,10 @@
  * under the License.
  */
 
-import { SavedObject } from '../../../../../server/saved_objects/service/saved_objects_client';
+import {
+  SavedObject,
+  SavedObjectsClient,
+} from '../../../../../server/saved_objects/service/saved_objects_client';
 import { collectReferencesDeep } from './collect_references_deep';
 
 const data = [
@@ -100,7 +103,7 @@ const data = [
 ];
 
 test('collects dashboard and all dependencies', async () => {
-  const savedObjectClient = {
+  const savedObjectClient = ({
     errors: {} as any,
     create: jest.fn(),
     bulkCreate: jest.fn(),
@@ -115,7 +118,7 @@ test('collects dashboard and all dependencies', async () => {
         ),
       };
     }),
-  };
+  } as unknown) as SavedObjectsClient;
   const objects = await collectReferencesDeep(savedObjectClient, [{ type: 'dashboard', id: '1' }]);
   expect(objects).toMatchInlineSnapshot(`
 Array [