Skip to content

Commit f4f016f

Browse files
committed
fix flake
1 parent d464260 commit f4f016f

File tree

1 file changed

+9
-1
lines changed
  • dev-packages/browser-integration-tests/suites/replay/bufferStalledRequests

1 file changed

+9
-1
lines changed

dev-packages/browser-integration-tests/suites/replay/bufferStalledRequests/test.ts

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import {
55
getReplaySnapshot,
66
isReplayEvent,
77
shouldSkipReplayTest,
8+
waitForReplayRequest,
89
waitForReplayRunning,
910
} from '../../../utils/replayHelpers';
1011

@@ -284,6 +285,8 @@ sentryTest(
284285
});
285286
});
286287

288+
const replayRequestPromise = waitForReplayRequest(page, 0);
289+
287290
const url = await getLocalTestUrl({ testDir: __dirname, skipDsnRouteHandler: true });
288291
await page.goto(url);
289292

@@ -310,7 +313,6 @@ sentryTest(
310313
// After error is sent, verify state is still correct
311314
const afterError = await getReplaySnapshot(page);
312315
expect(afterError.session?.sampled).toBe('buffer');
313-
expect(afterError.recordingMode).toBe('session');
314316
expect(afterError.session?.dirty).toBe(false);
315317

316318
// Verify the session was persisted to sessionStorage (if sticky sessions enabled)
@@ -323,5 +325,11 @@ sentryTest(
323325
expect(sessionData).toBeDefined();
324326
expect(sessionData.sampled).toBe('buffer');
325327
expect(sessionData.dirty).toBe(false);
328+
329+
// Need to wait for replay request before checking `recordingMode`, otherwise it will be flakey
330+
await replayRequestPromise;
331+
const afterReplay = await getReplaySnapshot(page);
332+
expect(afterReplay.recordingMode).toBe('session');
333+
326334
},
327335
);

0 commit comments

Comments
 (0)