test(browser): Add simulated mfe integration test#19768
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
| sentryTest.skip(); | ||
| } | ||
|
|
||
| await page.route('http://sentry-test-site.example/*', route => route.fulfill({ body: '{}' })); |
There was a problem hiding this comment.
Route glob pattern won't match multi-segment URL paths
Medium Severity
The page.route glob pattern 'http://sentry-test-site.example/*' uses a single * which only matches a single path segment in Playwright's URL glob matching. The fetched URLs (/api/todos/1, /api/todos/2, /api/todos/3, /api/shell-config) all have multiple path segments and won't be intercepted. The pattern needs **/* (as used in dsc-txn-name-update/test.ts) to match across path separators. Without this fix, the fetches hit the real (non-existent) host instead of being mocked, risking test flakiness or timeouts.


Adds a very simple integration test for propagating MFE names via scopes
Closes #19769 (added automatically)
ref #19470