Skip to content

test(ui): scrub stale return-url cookie from e2e storageState #30317

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ryan-crabbe-berri merged 2 commits into from
Jun 23, 2026
Merged

test(ui): scrub stale return-url cookie from e2e storageState #30317

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
9fc5314
test(ui): scrub stale return-url cookie from e2e storageState
ryan-crabbe-berri Jun 12, 2026
0d0c654
Merge remote-tracking branch 'origin/litellm_internal_staging' into l…
ryan-crabbe-berri Jun 23, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions ui/litellm-dashboard/e2e_tests/globalSetup.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,11 @@ async function globalSetup() {
if (await dismiss.isVisible({ timeout: 1_500 }).catch(() => false)) {
await dismiss.click();
}
// The login flow stores a post-login return URL in the litellm_return_url
// cookie. If the snapshot captures it before the app consumes it, every
// test inheriting this storageState gets yanked to that stale URL the
// first time it mounts a page (the e2e suite's main flake source).
await page.context().clearCookies({ name: "litellm_return_url" });
await page.context().storageState({ path: storagePath });
} catch (e) {
fs.mkdirSync("test-results", { recursive: true });
Expand Down
Loading