Skip to content

feat(FR-2594): add E2E test for SSO sToken login flow#6758

Open
nowgnuesLee wants to merge 1 commit intomainfrom
04-16-feat_fr-2594_add_e2e_test_for_sso_stoken_login_flow
Open

feat(FR-2594): add E2E test for SSO sToken login flow#6758
nowgnuesLee wants to merge 1 commit intomainfrom
04-16-feat_fr-2594_add_e2e_test_for_sso_stoken_login_flow

Conversation

@nowgnuesLee
Copy link
Copy Markdown
Contributor

@nowgnuesLee nowgnuesLee commented Apr 16, 2026

Resolves #6757 (FR-2594)

Summary

  • Add Playwright E2E test for SSO sToken-based login flow (e2e/auth/sso-stoken-login.spec.ts)
  • Verifies that users can authenticate via an SSO sToken URL and land on the correct page
  • Covers the scenario originally fixed in FR-2574

Test plan

  • Run pnpm run test:e2e against a live Backend.AI cluster with SSO configured
  • Verify test passes for valid sToken login
  • Verify test covers error/invalid token cases

Copilot AI review requested due to automatic review settings April 16, 2026 09:29
@github-actions github-actions Bot added the size:L 100~500 LoC label Apr 16, 2026
Copy link
Copy Markdown
Contributor Author


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • flow:merge-queue - adds this PR to the back of the merge queue
  • flow:hotfix - for urgent changes, fast-track this PR to the front of the merge queue

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new Playwright E2E spec to validate the root-URL /?sToken=... SSO login flow and its session persistence behavior, expanding Auth-related regression coverage in the WebUI E2E suite.

Changes:

  • Adds an E2E test verifying sToken auto-login redirects to /start and removes sToken from the URL.
  • Adds an E2E test verifying the authenticated session persists across a page refresh.
  • Preconfigures config.toml during the test to ensure apiEndpoint is present for the orchestration flow.

Comment thread e2e/auth/sso-stoken-login.spec.ts
Comment thread e2e/auth/sso-stoken-login.spec.ts Outdated
Comment thread e2e/auth/sso-stoken-login.spec.ts Outdated
Comment thread e2e/auth/sso-stoken-login.spec.ts
Comment thread e2e/auth/sso-stoken-login.spec.ts
@nowgnuesLee nowgnuesLee force-pushed the 04-16-feat_fr-2594_add_e2e_test_for_sso_stoken_login_flow branch from cec1994 to 2982139 Compare April 16, 2026 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L 100~500 LoC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(FR-2594): Add E2E test for SSO sToken login flow

2 participants