Skip to content

test(e2e): tolerate slow Keycloak redirect loads - #982

Closed
seonghobae wants to merge 1 commit into
feat/i18n-versioned-translation-ledgerfrom
codex/e2e-auth-redirect-20260908
Closed

test(e2e): tolerate slow Keycloak redirect loads#982
seonghobae wants to merge 1 commit into
feat/i18n-versioned-translation-ledgerfrom
codex/e2e-auth-redirect-20260908

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

The browser smoke test timed out while waiting for Keycloak's full document load even though the authorization URL had already been reached. Wait for navigation commit at that boundary, then retain the existing end-to-end assertion that the authorization-code callback reaches the authenticated destination.

Validation:

  • pnpm exec playwright test e2e/smoke.spec.ts --project=chromium --timeout=60000 (1 passed)
  • git diff --check

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 72088940-a7aa-47b5-8d62-c76055df564f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Superseded by PR #983, which carries the same two-file change directly against protected main so central review and required checks can run.

@seonghobae seonghobae closed this Sep 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant