Skip to content

Commit 5e4f40c

Browse files
mydeamydea
committed
test(replay): Properly split tests between integration & unit tests
1 parent 318b750 commit 5e4f40c

11 files changed

+35
-42
lines changed

‎packages/replay/test/unit/index-handleGlobalEvent.test.ts renamed to ‎packages/replay/test/integration/coreHandlers/handleGlobalEvent.test.ts

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,17 @@
11
import { getCurrentHub } from '@sentry/core';
22
import { Event } from '@sentry/types';
33

4-
import { REPLAY_EVENT_NAME } from '../../src/constants';
5-
import { handleGlobalEventListener } from '../../src/coreHandlers/handleGlobalEvent';
6-
import { overwriteRecordDroppedEvent, restoreRecordDroppedEvent } from '../../src/util/monkeyPatchRecordDroppedEvent';
7-
import { ReplayContainer } from './../../src/replay';
8-
import { Error } from './../fixtures/error';
9-
import { Transaction } from './../fixtures/transaction';
10-
import { resetSdkMock } from './../mocks/resetSdkMock';
11-
import { useFakeTimers } from './../utils/use-fake-timers';
4+
import { REPLAY_EVENT_NAME } from '../../../src/constants';
5+
import { handleGlobalEventListener } from '../../../src/coreHandlers/handleGlobalEvent';
6+
import { ReplayContainer } from '../../../src/replay';
7+
import {
8+
overwriteRecordDroppedEvent,
9+
restoreRecordDroppedEvent,
10+
} from '../../../src/util/monkeyPatchRecordDroppedEvent';
11+
import { Error } from '../../fixtures/error';
12+
import { Transaction } from '../../fixtures/transaction';
13+
import { resetSdkMock } from '../../mocks/resetSdkMock';
14+
import { useFakeTimers } from '../../utils/use-fake-timers';
1215

1316
useFakeTimers();
1417
let replay: ReplayContainer;

‎packages/replay/test/unit/coreHandlers/handleScope-unit.test.ts renamed to ‎packages/replay/test/integration/coreHandlers/handleScope.test.ts

File renamed without changes.

‎packages/replay/test/unit/index-errorSampleRate.test.ts renamed to ‎packages/replay/test/integration/errorSampleRate.test.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
11
import { captureException } from '@sentry/core';
22

33
import { DEFAULT_FLUSH_MIN_DELAY, REPLAY_SESSION_KEY, VISIBILITY_CHANGE_TIMEOUT, WINDOW } from '../../src/constants';
4+
import { ReplayContainer } from '../../src/replay';
45
import { addEvent } from '../../src/util/addEvent';
6+
import { PerformanceEntryResource } from '../fixtures/performanceEntry/resource';
7+
import { BASE_TIMESTAMP, RecordMock } from '../index';
8+
import { resetSdkMock } from '../mocks/resetSdkMock';
9+
import { DomHandler } from '../types';
510
import { clearSession } from '../utils/clearSession';
6-
import { ReplayContainer } from './../../src/replay';
7-
import { PerformanceEntryResource } from './../fixtures/performanceEntry/resource';
8-
import { BASE_TIMESTAMP, RecordMock } from './../index';
9-
import { resetSdkMock } from './../mocks/resetSdkMock';
10-
import { DomHandler } from './../types';
11-
import { useFakeTimers } from './../utils/use-fake-timers';
11+
import { useFakeTimers } from '../utils/use-fake-timers';
1212

1313
useFakeTimers();
1414

‎packages/replay/test/unit/flush.test.ts renamed to ‎packages/replay/test/integration/flush.test.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
import * as SentryUtils from '@sentry/utils';
22

33
import { DEFAULT_FLUSH_MIN_DELAY, SESSION_IDLE_DURATION, WINDOW } from '../../src/constants';
4+
import { createPerformanceEntries } from '../../src/createPerformanceEntry';
5+
import { ReplayContainer } from '../../src/replay';
46
import * as AddMemoryEntry from '../../src/util/addMemoryEntry';
57
import { createPerformanceSpans } from '../../src/util/createPerformanceSpans';
8+
import { BASE_TIMESTAMP, mockRrweb, mockSdk } from '../index';
69
import { clearSession } from '../utils/clearSession';
7-
import { createPerformanceEntries } from './../../src/createPerformanceEntry';
8-
import { ReplayContainer } from './../../src/replay';
9-
import { useFakeTimers } from './../../test/utils/use-fake-timers';
10-
import { BASE_TIMESTAMP, mockRrweb, mockSdk } from './../index';
10+
import { useFakeTimers } from '../utils/use-fake-timers';
1111

1212
useFakeTimers();
1313

‎packages/replay/test/unit/index-integrationSettings.test.ts renamed to ‎packages/replay/test/integration/integrationSettings.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { MASK_ALL_TEXT_SELECTOR } from '../../src/constants';
2-
import { mockSdk } from './../index';
2+
import { mockSdk } from '../index';
33

44
describe('integration settings', () => {
55
beforeEach(() => {

‎packages/replay/test/unit/index-noSticky.test.ts renamed to ‎packages/replay/test/integration/noSticky.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,11 @@ import { Transport } from '@sentry/types';
33
import * as SentryUtils from '@sentry/utils';
44

55
import { DEFAULT_FLUSH_MIN_DELAY, SESSION_IDLE_DURATION, VISIBILITY_CHANGE_TIMEOUT } from '../../src/constants';
6+
import { ReplayContainer } from '../../src/replay';
67
import { addEvent } from '../../src/util/addEvent';
8+
import { BASE_TIMESTAMP, mockRrweb, mockSdk } from '../index';
79
import { clearSession } from '../utils/clearSession';
8-
import { ReplayContainer } from './../../src/replay';
9-
import { BASE_TIMESTAMP, mockRrweb, mockSdk } from './../index';
10-
import { useFakeTimers } from './../utils/use-fake-timers';
10+
import { useFakeTimers } from '../utils/use-fake-timers';
1111

1212
useFakeTimers();
1313

‎packages/replay/test/unit/index.test.ts renamed to ‎packages/replay/test/integration/replay.test.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,12 @@ import { ReplayContainer } from '../../src/replay';
1414
import type { RecordingEvent } from '../../src/types';
1515
import { addEvent } from '../../src/util/addEvent';
1616
import { createPerformanceSpans } from '../../src/util/createPerformanceSpans';
17+
import { PerformanceEntryResource } from '../fixtures/performanceEntry/resource';
18+
import { BASE_TIMESTAMP, RecordMock } from '../index';
19+
import { resetSdkMock } from '../mocks/resetSdkMock';
20+
import { DomHandler } from '../types';
1721
import { clearSession } from '../utils/clearSession';
1822
import { useFakeTimers } from '../utils/use-fake-timers';
19-
import { PerformanceEntryResource } from './../fixtures/performanceEntry/resource';
20-
import { BASE_TIMESTAMP, RecordMock } from './../index';
21-
import { resetSdkMock } from './../mocks/resetSdkMock';
22-
import { DomHandler } from './../types';
2323

2424
useFakeTimers();
2525

‎packages/replay/test/unit/index-sampling.test.ts renamed to ‎packages/replay/test/integration/sampling.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
// mock functions need to be imported first
2-
import { mockRrweb, mockSdk } from './../index';
3-
import { useFakeTimers } from './../utils/use-fake-timers';
2+
import { mockRrweb, mockSdk } from '../index';
3+
import { useFakeTimers } from '../utils/use-fake-timers';
44

55
useFakeTimers();
66

‎packages/replay/test/unit/stop.test.ts renamed to ‎packages/replay/test/integration/stop.test.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
import * as SentryUtils from '@sentry/utils';
22

3+
import { Replay } from '../../src';
34
import { SESSION_IDLE_DURATION, WINDOW } from '../../src/constants';
45
import { ReplayContainer } from '../../src/replay';
56
import { addEvent } from '../../src/util/addEvent';
6-
import { clearSession } from '../utils/clearSession';
7-
import { Replay } from './../../src';
87
// mock functions need to be imported first
9-
import { BASE_TIMESTAMP, mockRrweb, mockSdk } from './../index';
10-
import { useFakeTimers } from './../utils/use-fake-timers';
8+
import { BASE_TIMESTAMP, mockRrweb, mockSdk } from '../index';
9+
import { clearSession } from '../utils/clearSession';
10+
import { useFakeTimers } from '../utils/use-fake-timers';
1111

1212
useFakeTimers();
1313

‎packages/replay/test/unit/coreHandlers/handleFetch.test.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,4 @@
11
import { handleFetch } from '../../../src/coreHandlers/handleFetch';
2-
import { mockSdk } from './../../index';
3-
4-
beforeAll(function () {
5-
mockSdk();
6-
});
72

83
const DEFAULT_DATA = {
94
args: ['/api/0/organizations/sentry/', { method: 'GET', headers: {}, credentials: 'include' }] as Parameters<

‎packages/replay/test/unit/createPerformanceEntry.test.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,4 @@
11
import { createPerformanceEntries } from '../../src/createPerformanceEntry';
2-
import { mockSdk } from './../index';
3-
4-
beforeAll(function () {
5-
mockSdk();
6-
});
72

83
it('ignores sdks own requests', function () {
94
const data = {

0 commit comments

Comments
 (0)