build: update jest, ts-jest to latest versions

Author: anonrig

jest/jest.config.js

@@ -3,16 +3,23 @@ module.exports = {
   rootDir: process.cwd(),
   collectCoverage: true,
   transform: {
-    '^.+\\.ts$': 'ts-jest',
-    '^.+\\.tsx$': 'ts-jest',
+    '^.+\\.ts$': [
+      'ts-jest',
+      {
+        tsconfig: '<rootDir>/tsconfig.test.json',
+      },
+    ],
+    '^.+\\.tsx$': [
+      'ts-jest',
+      {
+        tsconfig: '<rootDir>/tsconfig.test.json',
+      },
+    ],
   },
   coverageDirectory: '<rootDir>/coverage',
   moduleFileExtensions: ['js', 'ts', 'tsx'],
   testMatch: ['<rootDir>/**/*.test.ts', '<rootDir>/**/*.test.tsx'],
   globals: {
-    'ts-jest': {
-      tsconfig: '<rootDir>/tsconfig.test.json',
-    },
     __DEBUG_BUILD__: true,
   },
   testPathIgnorePatterns: ['<rootDir>/build/', '<rootDir>/node_modules/'],

package.json

@@ -92,7 +92,7 @@
     "@size-limit/webpack": "~9.0.0",
     "@strictsoftware/typedoc-plugin-monorepo": "^0.3.1",
     "@types/chai": "^4.1.3",
-    "@types/jest": "^27.4.1",
+    "@types/jest": "^29.5.10",
     "@types/jsdom": "^16.2.3",
     "@types/mocha": "^5.2.0",
     "@types/node": "~10.17.0",
@@ -106,8 +106,9 @@
     "downlevel-dts": "~0.11.0",
     "es-check": "7.1.0",
     "eslint": "7.32.0",
-    "jest": "^27.5.1",
-    "jest-environment-node": "^27.5.1",
+    "jest": "^29.7.0",
+    "jest-environment-jsdom": "^29.7.0",
+    "jest-environment-node": "^29.7.0",
     "jsdom": "^19.0.0",
     "karma-browserstack-launcher": "^1.5.1",
     "karma-firefox-launcher": "^1.1.0",
@@ -125,7 +126,7 @@
     "rollup-plugin-terser": "^7.0.2",
     "sinon": "^7.3.2",
     "size-limit": "~9.0.0",
-    "ts-jest": "^27.1.4",
+    "ts-jest": "^29.1.1",
     "ts-node": "10.9.1",
     "typedoc": "^0.18.0",
     "typescript": "4.9.5",

packages/angular/test/errorhandler.test.ts

@@ -516,7 +516,7 @@ describe('SentryErrorHandler', () => {
         // this simulates the afterSend hook being called
         client.cb({});
 
-        expect(showReportDialogSpy).toBeCalledTimes(1);
+        expect(showReportDialogSpy).toHaveBeenCalledTimes(1);
       });
 
       it('by just calling `showReportDialog` if hooks are not available', () => {
@@ -525,7 +525,7 @@ describe('SentryErrorHandler', () => {
         const errorHandler = createErrorHandler({ showDialog: true });
         errorHandler.handleError(new Error('test'));
 
-        expect(showReportDialogSpy).toBeCalledTimes(1);
+        expect(showReportDialogSpy).toHaveBeenCalledTimes(1);
       });
     });
   });

packages/angular/test/sdk.test.ts

@@ -1,4 +1,6 @@
 import * as SentryBrowser from '@sentry/browser';
+import type { BrowserTransportOptions } from '@sentry/browser/src/transports/types';
+import type { Options } from '@sentry/types';
 
 import { defaultIntegrations, init } from '../src/index';
 
@@ -26,19 +28,19 @@ describe('init', () => {
 
       expect(browserInitSpy).toHaveBeenCalledTimes(1);
 
-      const options = browserInitSpy.mock.calls[0][0] || {};
-      expect(options.defaultIntegrations).not.toContainEqual(expect.objectContaining({ name: 'TryCatch' }));
+      const options = browserInitSpy.mock.calls[0][0] as Options<BrowserTransportOptions> | undefined;
+      expect(options?.defaultIntegrations).not.toContainEqual(expect.objectContaining({ name: 'TryCatch' }));
     });
 
     it.each([false as const, defaultIntegrations])(
       "doesn't filter if `defaultIntegrations` is set to %s",
-      defaultIntegrations => {
+      (defaultIntegrations: Options<BrowserTransportOptions>['defaultIntegrations']) => {
         init({ defaultIntegrations });
 
         expect(browserInitSpy).toHaveBeenCalledTimes(1);
 
-        const options = browserInitSpy.mock.calls[0][0] || {};
-        expect(options.defaultIntegrations).toEqual(defaultIntegrations);
+        const options = browserInitSpy.mock.calls[0][0] as Options<BrowserTransportOptions> | undefined;
+        expect(options?.defaultIntegrations).toEqual(defaultIntegrations);
       },
     );
   });

packages/angular/test/tracing.test.ts

@@ -1,5 +1,5 @@
 import { Component } from '@angular/core';
-import type { ActivatedRouteSnapshot, CanActivate, RouterStateSnapshot } from '@angular/router';
+import type { ActivatedRouteSnapshot, CanActivate, RouterStateSnapshot, Routes } from '@angular/router';
 import type { Hub } from '@sentry/types';
 
 import { TraceClassDecorator, TraceDirective, TraceMethodDecorator, instrumentAngularRouting } from '../src';
@@ -99,10 +99,8 @@ describe('Angular Tracing', () => {
         },
         '/users/:id/',
       ],
-    ])('%s', (_, routeSnapshot, expectedParams) => {
-      expect(getParameterizedRouteFromSnapshot(routeSnapshot as unknown as ActivatedRouteSnapshot)).toEqual(
-        expectedParams,
-      );
+    ])('%s', (_: unknown, routeSnapshot: ActivatedRouteSnapshot, expectedParams: unknown) => {
+      expect(getParameterizedRouteFromSnapshot(routeSnapshot)).toEqual(expectedParams);
     });
   });
 
@@ -316,7 +314,7 @@ describe('Angular Tracing', () => {
             },
           ],
         ],
-      ])('%s and sets the source to `route`', async (_, url, result, routes) => {
+      ])('%s and sets the source to `route`', async (_: unknown, url: string, result: string, routes: Routes) => {
         const customStartTransaction = jest.fn(defaultStartTransaction);
         const env = await TestEnv.setup({
           customStartTransaction,

packages/browser/test/unit/index.test.ts

@@ -357,7 +357,7 @@ describe('SentryBrowser initialization', () => {
       const sdkData = getClient()?.getOptions()._metadata?.sdk || {};
 
       expect(sdkData.packages?.[0].name).toBe('cdn:@sentry/browser');
-      expect(utils.getSDKSource).toBeCalledTimes(1);
+      expect(utils.getSDKSource).toHaveBeenCalledTimes(1);
       spy.mockRestore();
     });
 

packages/core/test/lib/base.test.ts

@@ -996,8 +996,8 @@ describe('BaseClient', () => {
       expect(TestClient.instance!.event).toBeUndefined();
       // This proves that the reason the event didn't send/didn't get set on the test client is not because there was an
       // error, but because `beforeSend` returned `null`
-      expect(captureExceptionSpy).not.toBeCalled();
-      expect(loggerWarnSpy).toBeCalledWith('before send for type `error` returned `null`, will not send event.');
+      expect(captureExceptionSpy).not.toHaveBeenCalled();
+      expect(loggerWarnSpy).toHaveBeenCalledWith('before send for type `error` returned `null`, will not send event.');
     });
 
     test('calls `beforeSendTransaction` and discards the event', () => {
@@ -1015,8 +1015,10 @@ describe('BaseClient', () => {
       expect(TestClient.instance!.event).toBeUndefined();
       // This proves that the reason the event didn't send/didn't get set on the test client is not because there was an
       // error, but because `beforeSendTransaction` returned `null`
-      expect(captureExceptionSpy).not.toBeCalled();
-      expect(loggerWarnSpy).toBeCalledWith('before send for type `transaction` returned `null`, will not send event.');
+      expect(captureExceptionSpy).not.toHaveBeenCalled();
+      expect(loggerWarnSpy).toHaveBeenCalledWith(
+        'before send for type `transaction` returned `null`, will not send event.',
+      );
     });
 
     test('calls `beforeSend` and logs info about invalid return value', () => {
@@ -1034,7 +1036,7 @@ describe('BaseClient', () => {
 
         expect(beforeSend).toHaveBeenCalled();
         expect(TestClient.instance!.event).toBeUndefined();
-        expect(loggerWarnSpy).toBeCalledWith(
+        expect(loggerWarnSpy).toHaveBeenCalledWith(
           new SentryError('before send for type `error` must return `null` or a valid event.'),
         );
       }
@@ -1055,7 +1057,7 @@ describe('BaseClient', () => {
 
         expect(beforeSendTransaction).toHaveBeenCalled();
         expect(TestClient.instance!.event).toBeUndefined();
-        expect(loggerWarnSpy).toBeCalledWith(
+        expect(loggerWarnSpy).toHaveBeenCalledWith(
           new SentryError('before send for type `transaction` must return `null` or a valid event.'),
         );
       }
@@ -1317,8 +1319,8 @@ describe('BaseClient', () => {
       expect(TestClient.instance!.event).toBeUndefined();
       // This proves that the reason the event didn't send/didn't get set on the test client is not because there was an
       // error, but because the event processor returned `null`
-      expect(captureExceptionSpy).not.toBeCalled();
-      expect(loggerLogSpy).toBeCalledWith('An event processor returned `null`, will not send event.');
+      expect(captureExceptionSpy).not.toHaveBeenCalled();
+      expect(loggerLogSpy).toHaveBeenCalledWith('An event processor returned `null`, will not send event.');
     });
 
     test('event processor drops transaction event when it returns `null`', () => {
@@ -1335,8 +1337,8 @@ describe('BaseClient', () => {
       expect(TestClient.instance!.event).toBeUndefined();
       // This proves that the reason the event didn't send/didn't get set on the test client is not because there was an
       // error, but because the event processor returned `null`
-      expect(captureExceptionSpy).not.toBeCalled();
-      expect(loggerLogSpy).toBeCalledWith('An event processor returned `null`, will not send event.');
+      expect(captureExceptionSpy).not.toHaveBeenCalled();
+      expect(loggerLogSpy).toHaveBeenCalledWith('An event processor returned `null`, will not send event.');
     });
 
     test('event processor records dropped error events', () => {
@@ -1440,13 +1442,13 @@ describe('BaseClient', () => {
       client.captureEvent({ message: 'hello' }, {}, scope);
 
       expect(TestClient.instance!.event!.exception!.values![0]).toStrictEqual({ type: 'Error', value: 'sorry' });
-      expect(captureExceptionSpy).toBeCalledWith(exception, {
+      expect(captureExceptionSpy).toHaveBeenCalledWith(exception, {
         data: {
           __sentry__: true,
         },
         originalException: exception,
       });
-      expect(loggerWarnSpy).toBeCalledWith(
+      expect(loggerWarnSpy).toHaveBeenCalledWith(
         new SentryError(
           `Event processing pipeline threw an error, original event will not be sent. Details have been sent as a new event.\nReason: ${exception}`,
         ),
@@ -1670,9 +1672,9 @@ describe('BaseClient', () => {
       await undefined;
       await undefined;
 
-      expect(mockSend).toBeCalledTimes(1);
-      expect(callback).toBeCalledTimes(1);
-      expect(callback).toBeCalledWith(errorEvent, {});
+      expect(mockSend).toHaveBeenCalledTimes(1);
+      expect(callback).toHaveBeenCalledTimes(1);
+      expect(callback).toHaveBeenCalledWith(errorEvent, {});
     });
 
     it('emits `afterSendEvent` when sending a transaction', async () => {
@@ -1698,9 +1700,9 @@ describe('BaseClient', () => {
       await undefined;
       await undefined;
 
-      expect(mockSend).toBeCalledTimes(1);
-      expect(callback).toBeCalledTimes(1);
-      expect(callback).toBeCalledWith(transactionEvent, {});
+      expect(mockSend).toHaveBeenCalledTimes(1);
+      expect(callback).toHaveBeenCalledTimes(1);
+      expect(callback).toHaveBeenCalledWith(transactionEvent, {});
     });
 
     it('still triggers `afterSendEvent` when transport.send rejects', async () => {
@@ -1730,9 +1732,9 @@ describe('BaseClient', () => {
       await undefined;
       await undefined;
 
-      expect(mockSend).toBeCalledTimes(1);
-      expect(callback).toBeCalledTimes(1);
-      expect(callback).toBeCalledWith(errorEvent, undefined);
+      expect(mockSend).toHaveBeenCalledTimes(1);
+      expect(callback).toHaveBeenCalledTimes(1);
+      expect(callback).toHaveBeenCalledWith(errorEvent, undefined);
     });
 
     it('passes the response to the hook', async () => {
@@ -1762,9 +1764,9 @@ describe('BaseClient', () => {
       await undefined;
       await undefined;
 
-      expect(mockSend).toBeCalledTimes(1);
-      expect(callback).toBeCalledTimes(1);
-      expect(callback).toBeCalledWith(errorEvent, { statusCode: 200 });
+      expect(mockSend).toHaveBeenCalledTimes(1);
+      expect(callback).toHaveBeenCalledTimes(1);
+      expect(callback).toHaveBeenCalledWith(errorEvent, { statusCode: 200 });
     });
   });
 

packages/deno/jest.config.js

@@ -46,35 +46,35 @@ snapshot[`captureException 1`] = `
               filename: "ext:cli/40_testing.js",
               function: "outerWrapped",
               in_app: false,
-              lineno: 488,
+              lineno: 472,
             },
             {
               colno: 33,
               filename: "ext:cli/40_testing.js",
               function: "exitSanitizer",
               in_app: false,
-              lineno: 474,
+              lineno: 458,
             },
             {
               colno: 31,
               filename: "ext:cli/40_testing.js",
               function: "resourceSanitizer",
               in_app: false,
-              lineno: 425,
+              lineno: 410,
             },
             {
               colno: 33,
               filename: "ext:cli/40_testing.js",
               function: "asyncOpSanitizer",
               in_app: false,
-              lineno: 192,
+              lineno: 177,
             },
             {
               colno: 11,
               filename: "ext:cli/40_testing.js",
               function: "innerWrapped",
               in_app: false,
-              lineno: 543,
+              lineno: 526,
             },
             {
               colno: 24,

packages/deno/test/__snapshots__/mod.test.ts.snap

@@ -2,10 +2,21 @@ const baseConfig = require('../../jest/jest.config.js');
 
 module.exports = {
   ...baseConfig,
-  setupFiles: ['<rootDir>/test/setEnvVars.ts'],
-  testEnvironment: 'jsdom',
+  preset: 'ts-jest/presets/js-with-ts',
   transform: {
-    '^.+\\.js$': 'ts-jest',
-    ...baseConfig.transform,
+    '^.+\\.ts$': [
+      'ts-jest',
+      {
+        tsconfig: '<rootDir>/tsconfig.test.json',
+      },
+    ],
+    '^.+\\.js$': [
+      'ts-jest',
+      {
+        tsconfig: '<rootDir>/tsconfig.test.json',
+      },
+    ],
   },
+  setupFiles: ['<rootDir>/test/setEnvVars.ts'],
+  testEnvironment: 'jsdom',
 };