fix type errors in integrations tests

Author: lobsterkatie

packages/integrations/test/captureconsole.test.ts

@@ -1,4 +1,5 @@
-import { Event, Integration } from '@sentry/types';
+/* eslint-disable @typescript-eslint/unbound-method */
+import { Event, Hub, Integration } from '@sentry/types';
 
 import { CaptureConsole } from '../src/captureconsole';
 
@@ -25,10 +26,11 @@ const mockConsole = {
   info: jest.fn(),
 };
 
-const getMockHubWithIntegration = (integration: Integration) => ({
-  ...mockHub,
-  getIntegration: jest.fn(() => integration),
-});
+const getMockHubWithIntegration = (integration: Integration) =>
+  ({
+    ...mockHub,
+    getIntegration: jest.fn(() => integration),
+  } as unknown as Hub);
 
 // We're using this to un-monkey patch the console after each test.
 const originalConsole = Object.assign({}, global.console);
@@ -59,7 +61,7 @@ describe('CaptureConsole setup', () => {
       const captureConsoleIntegration = new CaptureConsole({ levels: ['log', 'warn'] });
       captureConsoleIntegration.setupOnce(
         () => undefined,
-        () => getMockHubWithIntegration(captureConsoleIntegration) as any,
+        () => getMockHubWithIntegration(captureConsoleIntegration),
       );
 
       expect(global.console.error).toBe(originalConsole.error); // not monkey patched
@@ -71,7 +73,7 @@ describe('CaptureConsole setup', () => {
       const captureConsoleIntegration = new CaptureConsole();
       captureConsoleIntegration.setupOnce(
         () => undefined,
-        () => getMockHubWithIntegration(captureConsoleIntegration) as any,
+        () => getMockHubWithIntegration(captureConsoleIntegration),
       );
 
       // expect a set of defined console levels to have been monkey patched
@@ -91,7 +93,7 @@ describe('CaptureConsole setup', () => {
       const captureConsoleIntegration = new CaptureConsole({ levels: [] });
       captureConsoleIntegration.setupOnce(
         () => undefined,
-        () => getMockHubWithIntegration(captureConsoleIntegration) as any,
+        () => getMockHubWithIntegration(captureConsoleIntegration),
       );
 
       // expect the default set of console levels not to have been monkey patched
@@ -120,7 +122,7 @@ describe('CaptureConsole setup', () => {
       const captureConsoleIntegration = new CaptureConsole();
       captureConsoleIntegration.setupOnce(
         () => undefined,
-        () => getMockHubWithIntegration(captureConsoleIntegration) as any,
+        () => getMockHubWithIntegration(captureConsoleIntegration),
       );
     }).not.toThrow();
 
@@ -132,7 +134,7 @@ describe('CaptureConsole setup', () => {
     const captureConsoleIntegration = new CaptureConsole({ levels: ['error'] });
     captureConsoleIntegration.setupOnce(
       () => undefined,
-      () => getMockHubWithIntegration(captureConsoleIntegration) as any,
+      () => getMockHubWithIntegration(captureConsoleIntegration),
     );
 
     // call a wrapped function
@@ -146,7 +148,7 @@ describe('CaptureConsole setup', () => {
     const captureConsoleIntegration = new CaptureConsole({ levels: ['log'] });
     captureConsoleIntegration.setupOnce(
       () => undefined,
-      () => getMockHubWithIntegration(captureConsoleIntegration) as any,
+      () => getMockHubWithIntegration(captureConsoleIntegration),
     );
 
     // call a wrapped function
@@ -162,7 +164,7 @@ describe('CaptureConsole setup', () => {
     const captureConsoleIntegration = new CaptureConsole({ levels: ['log'] });
     captureConsoleIntegration.setupOnce(
       () => undefined,
-      () => getMockHubWithIntegration(captureConsoleIntegration) as any,
+      () => getMockHubWithIntegration(captureConsoleIntegration),
     );
 
     // call a wrapped function
@@ -181,7 +183,7 @@ describe('CaptureConsole setup', () => {
     const captureConsoleIntegration = new CaptureConsole({ levels: ['assert'] });
     captureConsoleIntegration.setupOnce(
       () => undefined,
-      () => getMockHubWithIntegration(captureConsoleIntegration) as any,
+      () => getMockHubWithIntegration(captureConsoleIntegration),
     );
 
     global.console.assert(1 + 1 === 3);
@@ -195,7 +197,7 @@ describe('CaptureConsole setup', () => {
     const captureConsoleIntegration = new CaptureConsole({ levels: ['assert'] });
     captureConsoleIntegration.setupOnce(
       () => undefined,
-      () => getMockHubWithIntegration(captureConsoleIntegration) as any,
+      () => getMockHubWithIntegration(captureConsoleIntegration),
     );
 
     global.console.assert(1 + 1 === 3, 'expression is false');
@@ -209,7 +211,7 @@ describe('CaptureConsole setup', () => {
     const captureConsoleIntegration = new CaptureConsole({ levels: ['assert'] });
     captureConsoleIntegration.setupOnce(
       () => undefined,
-      () => getMockHubWithIntegration(captureConsoleIntegration) as any,
+      () => getMockHubWithIntegration(captureConsoleIntegration),
     );
 
     global.console.assert(1 + 1 === 2);
@@ -219,7 +221,7 @@ describe('CaptureConsole setup', () => {
     const captureConsoleIntegration = new CaptureConsole({ levels: ['error'] });
     captureConsoleIntegration.setupOnce(
       () => undefined,
-      () => getMockHubWithIntegration(captureConsoleIntegration) as any,
+      () => getMockHubWithIntegration(captureConsoleIntegration),
     );
 
     const someError = new Error('some error');
@@ -233,7 +235,7 @@ describe('CaptureConsole setup', () => {
     const captureConsoleIntegration = new CaptureConsole();
     captureConsoleIntegration.setupOnce(
       () => undefined,
-      () => getMockHubWithIntegration(captureConsoleIntegration) as any,
+      () => getMockHubWithIntegration(captureConsoleIntegration),
     );
 
     const someError = new Error('some error');
@@ -247,7 +249,7 @@ describe('CaptureConsole setup', () => {
     const captureConsoleIntegration = new CaptureConsole();
     captureConsoleIntegration.setupOnce(
       () => undefined,
-      () => getMockHubWithIntegration(captureConsoleIntegration) as any,
+      () => getMockHubWithIntegration(captureConsoleIntegration),
     );
 
     global.console.error('some message');
@@ -260,7 +262,7 @@ describe('CaptureConsole setup', () => {
     const captureConsoleIntegration = new CaptureConsole({ levels: ['error'] });
     captureConsoleIntegration.setupOnce(
       () => undefined,
-      () => getMockHubWithIntegration(captureConsoleIntegration) as any,
+      () => getMockHubWithIntegration(captureConsoleIntegration),
     );
 
     global.console.error('some non-error message');
@@ -274,7 +276,7 @@ describe('CaptureConsole setup', () => {
     const captureConsoleIntegration = new CaptureConsole({ levels: ['info'] });
     captureConsoleIntegration.setupOnce(
       () => undefined,
-      () => getMockHubWithIntegration(captureConsoleIntegration) as any,
+      () => getMockHubWithIntegration(captureConsoleIntegration),
     );
 
     global.console.info('some message');
@@ -292,7 +294,7 @@ describe('CaptureConsole setup', () => {
     const captureConsoleIntegration = new CaptureConsole({ levels: ['log'] });
     captureConsoleIntegration.setupOnce(
       () => undefined,
-      () => getMockHubWithIntegration(captureConsoleIntegration) as any,
+      () => getMockHubWithIntegration(captureConsoleIntegration),
     );
 
     global.console.log('some message 1', 'some message 2');
@@ -308,11 +310,11 @@ describe('CaptureConsole setup', () => {
     const captureConsoleIntegration = new CaptureConsole({ levels: ['log', 'someNonExistingLevel', 'error'] });
     captureConsoleIntegration.setupOnce(
       () => undefined,
-      () => getMockHubWithIntegration(captureConsoleIntegration) as any,
+      () => getMockHubWithIntegration(captureConsoleIntegration),
     );
 
     // The provided level should not be created
-    expect(global.console['someNonExistingLevel']).toBeUndefined();
+    expect((global.console as any)['someNonExistingLevel']).toBeUndefined();
 
     // Ohter levels should be wrapped as expected
     expect(global.console.log).not.toBe(originalConsole.log);
@@ -323,7 +325,7 @@ describe('CaptureConsole setup', () => {
     const captureConsoleIntegration = new CaptureConsole({ levels: ['log', 'error'] });
     captureConsoleIntegration.setupOnce(
       () => undefined,
-      () => getMockHubWithIntegration(null) as any, // simulate not having the integration registered
+      () => getMockHubWithIntegration(null as any), // simulate not having the integration registered
     );
 
     // Console should be wrapped
@@ -337,12 +339,12 @@ describe('CaptureConsole setup', () => {
 
   it("should not crash when the original console methods don't exist at time of invocation", () => {
     const originalConsoleLog = global.console.log;
-    global.console.log = undefined; // don't `delete` here, otherwise `fill` won't wrap the function
+    global.console.log = undefined as any; // don't `delete` here, otherwise `fill` won't wrap the function
 
     const captureConsoleIntegration = new CaptureConsole({ levels: ['log'] });
     captureConsoleIntegration.setupOnce(
       () => undefined,
-      () => getMockHubWithIntegration(captureConsoleIntegration) as any,
+      () => getMockHubWithIntegration(captureConsoleIntegration),
     );
 
     expect(() => {

packages/integrations/test/extraerrordata.test.ts

@@ -71,7 +71,7 @@ describe('ExtraErrorData()', () => {
     event = {
       // @ts-ignore Allow contexts on event
       contexts: {
-        foo: 42,
+        foo: { bar: 42 },
       },
     };
     const error = new TypeError('foo') as ExtendedError;
@@ -85,7 +85,7 @@ describe('ExtraErrorData()', () => {
       TypeError: {
         baz: 42,
       },
-      foo: 42,
+      foo: { bar: 42 },
     });
   });
 

packages/integrations/test/offline.test.ts

@@ -169,7 +169,7 @@ function initIntegration(options: { maxStoredEvents?: number } = {}): void {
   jest.spyOn(utils, 'getGlobalObject').mockImplementation(
     () =>
       ({
-        addEventListener: (_windowEvent, callback) => {
+        addEventListener: (_windowEvent: any, callback: any) => {
           eventListeners.push(callback);
         },
         navigator: {