test(cloudxr): real-browser run_oob_connect() tests against MockCloudXR, with anticipatory reliability-branch coverage

[gareth-morgan-nv]

Summary

• Real-browser integration tests for run_oob_connect(), standardized on MockCloudXR (npm run build:app-mock) rather than the real @nvidia/cloudxr SDK: the real SDK genuinely tries to stream against whatever's on backend_port, which is unpredictable and gives a test nothing to assert against. MockCloudXR mounts the identical App.tsx/CloudXRComponent.tsx UI with a fully deterministic, externally controllable session (window.__mockCloudXRFail(message?, code?)), and never opens a socket of its own.
• Always a static production build, never a dev-server (static_webxr_build()). Two real bugs made this necessary: webpack HMR duplicating the cloudxr-mock-alias.ts module instance (so the crash-trigger silently bound to an empty session), and <React.StrictMode> double-invoking CloudXR.createSession() in a dev-mode build. webpack.app-mock.js now builds in production mode. Also found and fixed: a stale webpack persistent filesystem cache silently serving an old bundle — cleared before every build now.
• Three real-browser tests: the original success path, a new test proving the crash-trigger mechanism itself (real, deterministic, DOM-visible error banner on demand), and an anticipatory test written against gmorgan/oob-error-relaunch (cloudxr: auto-relaunch the teleop tab on a terminal client error (depends on #1138, #1145) #1146) before it's merged here — asserts on black-box observable recovery (a second real am start, a second real tab), so it fails cleanly today and should turn green once cloudxr: auto-relaunch the teleop tab on a terminal client error (depends on #1138, #1145) #1146 lands.
• Also added test_build_teleop_url_forwards_reliability_config_from_env, the same "write it now, let it fail until merged" approach applied to gmorgan/oob-reliability-config-wiring (cloudxr: wire client reconnect/warm-up-timeout config through OOB launch #1145)'s config forwarding — no browser needed for that one.
• Design/decisions captured in oob-real-browser-integration-test-plan.md (repo root, §8 has the final-state summary).
• Still depends on gmorgan/MockADB (test(cloudxr): reusable fake-adb test double (FakeAdb/mock_adb) #1147) for FakeAdb/mock_adb() — base branch unchanged.
• Manual-run only for now; not yet wired into CI.

Test plan

• uv run pytest test_oob_teleop_adb.py test_oob_teleop_env.py test_oob_teleop_hub.py test_fake_adb.py test_wss_static_client.py -q — 141 passed, 2 expected anticipatory failures (config-wiring, auto-relaunch — both intentionally fail until their respective branches merge)
• SKIP=check-copyright-year pre-commit run on all changed files — clean
• npx eslint/npx prettier --check on changed TS files — clean

🤖 Generated with Claude Code

[coderabbitai]

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

• Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.

---

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