Skip to content

test(browser): Fix flaky browser integration test for profiles - #20587

Merged
mydea merged 2 commits into
developfrom
fn/fix-flaky-browser-test
May 4, 2026
Merged

test(browser): Fix flaky browser integration test for profiles#20587
mydea merged 2 commits into
developfrom
fn/fix-flaky-browser-test

Conversation

@mydea

@mydeamydea commented Apr 29, 2026

Copy link
Copy Markdown
Member

This updates a browser integration test to make it less flaky. Instead of first calling await page.goto(...) we now combine this with getMultipleSentryEnvelopeRequests to avoid possible race conditions.

Fixes#20585
Fixes#20524
Fixes#20612

@mydea
mydea requested a review from a teamApril 29, 2026 08:34
@mydeamydea self-assigned this Apr 29, 2026

@cursorcursorBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit eaf147e. Configure here.

@github-actions

github-actionsBot commented Apr 29, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

PathSize% ChangeChange
@sentry/browser26.16 kB--
@sentry/browser - with treeshaking flags24.63 kB--
@sentry/browser (incl. Tracing)44.11 kB--
@sentry/browser (incl. Tracing + Span Streaming)46.16 kB--
@sentry/browser (incl. Tracing, Profiling)49.06 kB--
@sentry/browser (incl. Tracing, Replay)83.46 kB--
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags72.94 kB--
@sentry/browser (incl. Tracing, Replay with Canvas)88.14 kB--
@sentry/browser (incl. Tracing, Replay, Feedback)100.79 kB--
@sentry/browser (incl. Feedback)43.4 kB--
@sentry/browser (incl. sendFeedback)30.96 kB--
@sentry/browser (incl. FeedbackAsync)36.14 kB--
@sentry/browser (incl. Metrics)27.44 kB--
@sentry/browser (incl. Logs)27.59 kB--
@sentry/browser (incl. Metrics & Logs)28.28 kB--
@sentry/react27.9 kB--
@sentry/react (incl. Tracing)46.35 kB--
@sentry/vue31.03 kB--
@sentry/vue (incl. Tracing)45.95 kB--
@sentry/svelte26.18 kB--
CDN Bundle28.84 kB--
CDN Bundle (incl. Tracing)46.71 kB--
CDN Bundle (incl. Logs, Metrics)30.25 kB--
CDN Bundle (incl. Tracing, Logs, Metrics)47.8 kB--
CDN Bundle (incl. Replay, Logs, Metrics)69.34 kB--
CDN Bundle (incl. Tracing, Replay)83.88 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)84.94 kB--
CDN Bundle (incl. Tracing, Replay, Feedback)89.69 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)90.78 kB--
CDN Bundle - uncompressed84.55 kB--
CDN Bundle (incl. Tracing) - uncompressed139.68 kB--
CDN Bundle (incl. Logs, Metrics) - uncompressed88.75 kB--
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed143.14 kB--
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed212.71 kB--
CDN Bundle (incl. Tracing, Replay) - uncompressed257.49 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed260.93 kB--
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed271.18 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed274.62 kB--
@sentry/nextjs (client)48.84 kB--
@sentry/sveltekit (client)44.56 kB--
@sentry/node-core58.96 kB+0.02%+8 B 🔺
@sentry/node170.25 kB+0.01%+8 B 🔺
@sentry/node - without tracing96.82 kB+0.01%+8 B 🔺
@sentry/aws-serverless113.69 kB+0.03%+33 B 🔺
@sentry/cloudflare (withSentry) - minified164.13 kB--
@sentry/cloudflare (withSentry)415.2 kB--

View base workflow run

@mydeamydea changed the title test(browser): Fix flaky browser integration testtest(browser): Fix flaky browser integration test for profilesApr 29, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment