test(e2e-install): navigate to clawbox.tech, not the old domain - #154
KrasimirKralev wants to merge 1 commit into
Conversation
openclawhardware.dev now 308-redirects to clawbox.tech, so the browser
test's `expect(launched.url).toContain("openclawhardware.dev")` failed
on the post-redirect URL. Navigate directly to the canonical clawbox.tech
host and assert on it. Test intent (browser launches + navigates + returns
url/title/screenshot) is unchanged.
|
No actionable comments were generated in the recent review. π βΉοΈ Recent review infoβοΈ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: π Files selected for processing (1)
π WalkthroughWalkthroughThe browser CDP end-to-end test is updated to navigate to the new canonical domain ChangesBrowser e2e test update
Estimated code review effortπ― 1 (Trivial) | β±οΈ ~3 minutes Poem
π₯ Pre-merge checks | β 5β Passed checks (5 passed)
βοΈ Tip: You can configure your own custom pre-merge checks in the settings. β¨ Finishing Touchesπ Generate docstrings
π§ͺ Generate unit tests (beta)
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. Comment |
|
Actionable comments posted: 0 |
Why
openclawhardware.devnow 308-redirects toclawbox.tech(permanent). The browser e2e test launcheshttps://openclawhardware.dev/and assertslaunched.urlcontains the old domain β but the browser follows the redirect, so the final URL ishttps://clawbox.tech/and the assertion fails. This is the only test that breaks on the redirect (the clawkeep/app-store specs use mocks or survive the GET redirect), and it's currently the sole blocker on the betaβmain promotion #153 (the codex-pin fix).Change
e2e-install/70-browser.spec.tsβ navigate directly to the canonicalclawbox.techand assert on it. Test intent (browser launches β navigates β returns url/title/screenshot) is unchanged. One file, 7 lines.Scope note
This only fixes the test. The broader
openclawhardware.dev β clawbox.techmigration touches ~30 files including production code (src/app/setup-api/ai-models/clawai/poll/route.ts, portal/OAuth URLs innext.config.ts, etc.). That's deliberately out of scope here and needs its own migration PR + verification that ClawBox AI activation works against the new domain (the 308 may not survive POST/auth). Flagging separately.Test plan
Summary by CodeRabbit
Release Notes