remove extra/unneeded changes

billyvg · billyvg · commit c0d4cfa8e1c2 · 2024-05-05T12:51:44.000-04:00
diff --git a/packages/replay-internal/test/integration/session.test.ts b/packages/replay-internal/test/integration/session.test.ts
@@ -357,7 +357,7 @@ describe('Integration | session', () => {
 
     await vi.runAllTimersAsync();
 
-    expect(replay.session.id).not.toBe(initialSession.id);
+    expect(replay).not.toHaveSameSession(initialSession);
     expect(replay).not.toHaveLastSentReplay();
     expect(replay['_stopRecording']).toBeDefined();
 
diff --git a/packages/replay-internal/test/mocks/mockSdk.ts b/packages/replay-internal/test/mocks/mockSdk.ts
@@ -5,7 +5,6 @@ import type { Replay as ReplayIntegration } from '../../src/integration';
 import type { ReplayContainer } from '../../src/replay';
 import type { ReplayConfiguration } from '../../src/types';
 import type { TestClientOptions } from '../utils/TestClient';
-import { BASE_TIMESTAMP } from './../index';
 import { getDefaultClientOptions, init } from '../utils/TestClient';
 
 export interface MockSdkParams {
@@ -52,7 +51,6 @@ export async function mockSdk({ replayOptions, sentryOptions, autoStart = true }
 }> {
   const { Replay } = await import('../../src/integration');
 
-  vi.setSystemTime(new Date(BASE_TIMESTAMP));
   // Scope this to the test, instead of the module
   let _initialized = false;
   class TestReplayIntegration extends Replay {
diff --git a/packages/replay-internal/test/mocks/resetSdkMock.ts b/packages/replay-internal/test/mocks/resetSdkMock.ts
@@ -39,8 +39,6 @@ export async function resetSdkMock({ replayOptions, sentryOptions, autoStart }:
 
   // XXX: This is needed to ensure `domHandler` is set
   await vi.runAllTimersAsync();
-  // vi.runAllTimers();
-  // await new Promise(process.nextTick);
   vi.setSystemTime(new Date(BASE_TIMESTAMP));
 
   return {
diff --git a/packages/replay-internal/test/unit/coreHandlers/util/xhrUtils.test.ts b/packages/replay-internal/test/unit/coreHandlers/util/xhrUtils.test.ts
@@ -9,7 +9,9 @@ describe('Unit | coreHandlers | util | xhrUtils', () => {
 
     it('works with a Document', () => {
       const doc = document.implementation.createHTMLDocument();
-      doc.body.innerHTML = '<div>abc</div>';
+      const bodyEl = document.createElement('body');
+      bodyEl.innerHTML = '<div>abc</div>';
+      doc.body = bodyEl;
 
       const actual = _parseXhrResponse(doc, '');
       expect(actual).toEqual(['<body><div>abc</div></body>']);
diff --git a/packages/replay-internal/test/unit/util/handleRecordingEmit.test.ts b/packages/replay-internal/test/unit/util/handleRecordingEmit.test.ts
@@ -1,3 +1,6 @@
+import { vi } from 'vitest';
+import type { MockInstance } from 'vitest';
+
 import { EventType } from '@sentry-internal/rrweb';
 
 import { BASE_TIMESTAMP } from '../..';
