Skip to content

Commit dfd9775

Browse files
committed
reduce flush mindelay, increase maxdelay
1 parent 6b346ee commit dfd9775

File tree

2 files changed

+20
-14
lines changed

2 files changed

+20
-14
lines changed

packages/integration-tests/suites/replay/requests/init.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@ import * as Sentry from '@sentry/browser';
22

33
window.Sentry = Sentry;
44
window.Replay = new Sentry.Replay({
5-
flushMinDelay: 500,
6-
flushMaxDelay: 500,
7-
useCompression: true,
5+
flushMinDelay: 200,
6+
flushMaxDelay: 1000,
7+
useCompression: false,
88
});
99

1010
Sentry.init({

packages/integration-tests/suites/replay/requests/test.ts

Lines changed: 17 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import { expect } from '@playwright/test';
22

33
import { sentryTest } from '../../../utils/fixtures';
4+
import { envelopeRequestParser } from '../../../utils/helpers';
45
import { expectedFetchPerformanceSpan, expectedXHRPerformanceSpan } from '../../../utils/replayEventTemplates';
56
import { getReplayRecordingContent, shouldSkipReplayTest, waitForReplayRequest } from '../../../utils/replayHelpers';
67

@@ -14,6 +15,9 @@ for (let i = 0; i < 25; i++) {
1415
const reqPromise1 = waitForReplayRequest(page, 1);
1516

1617
await page.route('https://dsn.ingest.sentry.io/**/*', route => {
18+
// eslint-disable-next-line no-console
19+
console.log('>>>got request', envelopeRequestParser(route.request())?.type);
20+
1721
return route.fulfill({
1822
status: 200,
1923
contentType: 'application/json',
@@ -22,6 +26,8 @@ for (let i = 0; i < 25; i++) {
2226
});
2327

2428
await page.route('https://example.com', route => {
29+
// eslint-disable-next-line no-console
30+
console.log('>>>got request call');
2531
return route.fulfill({
2632
status: 200,
2733
contentType: 'application/json',
@@ -52,6 +58,8 @@ for (let i = 0; i < 25; i++) {
5258
const reqPromise1 = waitForReplayRequest(page, 1);
5359

5460
await page.route('https://dsn.ingest.sentry.io/**/*', route => {
61+
// eslint-disable-next-line no-console
62+
console.log('>>>got request', envelopeRequestParser(route.request())?.type);
5563
return route.fulfill({
5664
status: 200,
5765
contentType: 'application/json',
@@ -60,6 +68,8 @@ for (let i = 0; i < 25; i++) {
6068
});
6169

6270
await page.route('https://example.com', route => {
71+
// eslint-disable-next-line no-console
72+
console.log('>>>got request call');
6373
return route.fulfill({
6474
status: 200,
6575
contentType: 'application/json',
@@ -69,17 +79,13 @@ for (let i = 0; i < 25; i++) {
6979

7080
const url = await getLocalTestPath({ testDir: __dirname });
7181

72-
const [, , { performanceSpans: spans0 }] = await Promise.all([
73-
page.goto(url),
74-
page.click('#go-background'),
75-
getReplayRecordingContent(await reqPromise0),
76-
]);
77-
78-
const [, , { performanceSpans: spans1 }] = await Promise.all([
79-
page.click('#xhr'),
80-
page.click('#go-background'),
81-
getReplayRecordingContent(await reqPromise1),
82-
]);
82+
await page.goto(url);
83+
await page.click('#go-background');
84+
const { performanceSpans: spans0 } = getReplayRecordingContent(await reqPromise0);
85+
86+
await page.click('#xhr');
87+
await page.click('#go-background');
88+
const { performanceSpans: spans1 } = getReplayRecordingContent(await reqPromise1);
8389

8490
const performanceSpans = [...spans0, ...spans1];
8591

0 commit comments

Comments
 (0)