lint

Author: billyvg

packages/replay-internal/test.setup.ts

@@ -1,5 +1,5 @@
 import { vi } from 'vitest';
-import type { Assertion, AsymmetricMatchersContaining, Mocked, MockedFunction } from 'vitest';
+import type { Mocked, MockedFunction } from 'vitest';
 import { printDiffOrStringify } from 'jest-matcher-utils';
 
 /* eslint-disable @typescript-eslint/no-unsafe-member-access */
@@ -101,6 +101,7 @@ function checkCallForSentReplay(
     : (expected as SentReplayExpected);
 
   if (isObjectContaining) {
+    // eslint-disable-next-line no-console
     console.warn('`expect.objectContaining` is unnecessary when using the `toHaveSentReplay` matcher');
   }
 
@@ -242,18 +243,13 @@ expect.extend({
   toHaveLastSentReplay,
 });
 
-declare global {
-  // eslint-disable-next-line @typescript-eslint/no-namespace
-  namespace vi {
-    interface AsymmetricMatchers {
-      toHaveSentReplay(expected?: SentReplayExpected): void;
-      toHaveLastSentReplay(expected?: SentReplayExpected): void;
-      toHaveSameSession(expected: undefined | Session): void;
-    }
-    interface Matchers<R> {
-      toHaveSentReplay(expected?: SentReplayExpected): R;
-      toHaveLastSentReplay(expected?: SentReplayExpected): R;
-      toHaveSameSession(expected: undefined | Session): R;
-    }
-  }
+interface CustomMatchers<R = unknown> {
+  toHaveSentReplay(expected?: SentReplayExpected): R;
+  toHaveLastSentReplay(expected?: SentReplayExpected): R;
+  toHaveSameSession(expected: undefined | Session): R;
+}
+
+declare module 'vitest' {
+  type Assertion<T = any> = CustomMatchers<T>;
+  type AsymmetricMatchersContaining = CustomMatchers;
 }

packages/replay-internal/test/integration/errorSampleRate.test.ts

@@ -640,7 +640,6 @@ describe('Integration | errorSampleRate', () => {
 
     it('has correct timestamps when error occurs much later than initial pageload/checkout', async () => {
       const ELAPSED = BUFFER_CHECKOUT_TIME;
-      const TICK = 20;
       const TEST_EVENT = getTestEventIncremental({ timestamp: BASE_TIMESTAMP });
       mockRecord._emitter(TEST_EVENT);
 

packages/replay-internal/test/integration/flush.test.ts

@@ -1,5 +1,5 @@
 import { vi } from 'vitest';
-import type { MockedFunction, MockInstance } from 'vitest';
+import type { MockedFunction } from 'vitest';
 
 import * as SentryBrowserUtils from '@sentry-internal/browser-utils';
 import * as SentryUtils from '@sentry/utils';
@@ -8,7 +8,6 @@ import { DEFAULT_FLUSH_MIN_DELAY, MAX_REPLAY_DURATION, WINDOW } from '../../src/
 import type { ReplayContainer } from '../../src/replay';
 import { clearSession } from '../../src/session/clearSession';
 import type { EventBuffer } from '../../src/types';
-import * as AddMemoryEntry from '../../src/util/addMemoryEntry';
 import { createPerformanceEntries } from '../../src/util/createPerformanceEntries';
 import { createPerformanceSpans } from '../../src/util/createPerformanceSpans';
 import * as SendReplay from '../../src/util/sendReplay';
@@ -21,7 +20,6 @@ useFakeTimers();
 
 type MockSendReplay = MockedFunction<any>;
 type MockAddPerformanceEntries = MockedFunction<ReplayContainer['_addPerformanceEntries']>;
-type MockAddMemoryEntry = MockInstance;
 type MockEventBufferFinish = MockedFunction<EventBuffer['finish']>;
 type MockFlush = MockedFunction<ReplayContainer['_flush']>;
 type MockRunFlush = MockedFunction<ReplayContainer['_runFlush']>;
@@ -39,7 +37,6 @@ describe('Integration | flush', () => {
   let mockFlush: MockFlush;
   let mockRunFlush: MockRunFlush;
   let mockEventBufferFinish: MockEventBufferFinish;
-  let mockAddMemoryEntry: MockAddMemoryEntry;
   let mockAddPerformanceEntries: MockAddPerformanceEntries;
 
   beforeAll(async () => {
@@ -68,8 +65,6 @@ describe('Integration | flush', () => {
     mockAddPerformanceEntries.mockImplementation(async () => {
       return [];
     });
-
-    mockAddMemoryEntry = vi.spyOn(AddMemoryEntry, 'addMemoryEntry');
   });
 
   beforeEach(async () => {

packages/replay-internal/test/utils/TestClient.ts

@@ -20,10 +20,8 @@ export class TestClient extends BaseClient<TestClientOptions> {
       exception: {
         values: [
           {
-            /* eslint-disable @typescript-eslint/no-unsafe-member-access */
             type: exception.name,
             value: exception.message,
-            /* eslint-enable @typescript-eslint/no-unsafe-member-access */
           },
         ],
       },

packages/replay-internal/tsconfig.json

@@ -3,5 +3,5 @@
   "compilerOptions": {
     "module": "esnext"
   },
-  "include": ["src/**/*.ts"]
+  "include": ["src/**/*.ts", "./test.setup.ts", "./vitest.config.ts"]
 }