Skip to content

Commit d15b5ec

Browse files
committed
fix tests
1 parent 9c8d916 commit d15b5ec

File tree

3 files changed

+8
-6
lines changed

3 files changed

+8
-6
lines changed

packages/replay/test/unit/index-errorSampleRate.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,9 +60,9 @@ describe('Replay (errorSampleRate)', () => {
6060
expect(replay).toHaveSentReplay({
6161
recordingPayloadHeader: { segment_id: 0 },
6262
replayEventPayload: expect.objectContaining({
63+
replay_type: 'error',
6364
tags: expect.objectContaining({
6465
errorSampleRate: 1,
65-
replayType: 'error',
6666
sessionSampleRate: 0,
6767
}),
6868
}),
@@ -90,9 +90,9 @@ describe('Replay (errorSampleRate)', () => {
9090
expect(replay).toHaveLastSentReplay({
9191
recordingPayloadHeader: { segment_id: 1 },
9292
replayEventPayload: expect.objectContaining({
93+
replay_type: 'error',
9394
tags: expect.objectContaining({
9495
errorSampleRate: 1,
95-
replayType: 'error',
9696
sessionSampleRate: 0,
9797
}),
9898
}),

packages/replay/test/unit/index.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -789,9 +789,9 @@ describe('Replay', () => {
789789
replayEventPayload: expect.objectContaining({
790790
replay_start_timestamp: (BASE_TIMESTAMP - 10000) / 1000,
791791
urls: ['http://localhost/'], // this doesn't truly test if we are capturing the right URL as we don't change URLs, but good enough
792+
replay_type: 'session',
792793
tags: expect.objectContaining({
793794
errorSampleRate: 0,
794-
replayType: 'session',
795795
sessionSampleRate: 1,
796796
}),
797797
}),

packages/replay/test/unit/util/createReplayEnvelope.test.ts

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,10 @@ describe('createReplayEnvelope', () => {
2323
name: 'sentry.javascript.browser',
2424
version: '7.25.0',
2525
},
26+
replay_type: 'error',
2627
tags: {
2728
sessionSampleRate: 1,
2829
errorSampleRate: 0,
29-
replayType: 'error',
3030
},
3131
};
3232

@@ -59,9 +59,10 @@ describe('createReplayEnvelope', () => {
5959
event_id: REPLAY_ID,
6060
platform: 'javascript',
6161
replay_id: REPLAY_ID,
62+
replay_type: 'error',
6263
sdk: { integrations: ['BrowserTracing', 'Replay'], name: 'sentry.javascript.browser', version: '7.25.0' },
6364
segment_id: 3,
64-
tags: { errorSampleRate: 0, replayType: 'error', sessionSampleRate: 1 },
65+
tags: { errorSampleRate: 0, sessionSampleRate: 1 },
6566
timestamp: 1670837008.634,
6667
trace_ids: ['traceId'],
6768
type: 'replay_event',
@@ -94,7 +95,8 @@ describe('createReplayEnvelope', () => {
9495
replay_id: REPLAY_ID,
9596
sdk: { integrations: ['BrowserTracing', 'Replay'], name: 'sentry.javascript.browser', version: '7.25.0' },
9697
segment_id: 3,
97-
tags: { errorSampleRate: 0, replayType: 'error', sessionSampleRate: 1 },
98+
replay_type: 'error',
99+
tags: { errorSampleRate: 0, sessionSampleRate: 1 },
98100
timestamp: 1670837008.634,
99101
trace_ids: ['traceId'],
100102
type: 'replay_event',

0 commit comments

Comments
 (0)