test(platform): read the rollout where the handoff reads it

waitForRollout reads the surface API, whose route bundle has its own AppConfig
cache. The layout's bundle still held the rollback for one more read, so a single
navigation right after the API agreed stayed on /o. The proxy's zone verdict was
never negative in that phase.

Author: waleedlatif1

apps/sim/platform/checks/live/rollout/per-organization.spec.ts

@@ -6,7 +6,11 @@ import {
   signIn,
   signInAsAdmin,
 } from '@/platform/checks/live/session'
-import { ROLLOUT_TEST_TIMEOUT_MS, waitForRollout } from '@/platform/checks/live/support/rollout'
+import {
+  expectHandoff,
+  ROLLOUT_TEST_TIMEOUT_MS,
+  waitForRollout,
+} from '@/platform/checks/live/support/rollout'
 import {
   expectMainAppSurface,
   expectPlatformSurface,
@@ -24,8 +28,7 @@ test('a targeted organization is handed to the Platform', async ({ page }) => {
   const route = `/o/${organizationId()}/home`
   await signInAsAdmin(page, route)
   await waitForRollout(page, true)
-  await page.goto(route)
-  await expect(page).toHaveURL(/\/org\/home/)
+  await expectHandoff(page, route, /\/org\/home/)
   await expectPlatformSurface(page)
 })
 

apps/sim/platform/checks/live/support/rollout.ts

@@ -3,7 +3,9 @@ import { requireEnv } from '@/platform/checks/live/session'
 
 /**
  * Longer than a poll of AppConfig, whose in-process answer holds for 30s and is served once more
- * while it refreshes, so a change reaches the app at some point in this window.
+ * while it refreshes, so a change reaches the app at some point in this window. Next bundles the
+ * module into each route, so each holds its own answer on its own clock: a route agreeing says
+ * nothing about another, and a page has to be read until it does.
  */
 const ROLLOUT_PROPAGATION_MS = 100_000
 
@@ -17,7 +19,7 @@ export async function targetOrganizations(request: APIRequestContext, orgIds: re
   expect(response.status()).toBe(204)
 }
 
-/** Waits until the main app answers `expected` for the signed-in viewer's organization. */
+/** Waits until the surface API answers `expected` for the signed-in viewer's organization. */
 export async function waitForRollout(page: Page, expected: boolean) {
   await expect
     .poll(
@@ -30,3 +32,11 @@ export async function waitForRollout(page: Page, expected: boolean) {
     )
     .toBe(expected)
 }
+
+/** Opens `route` until this page's own read of the rollout hands it to the Platform. */
+export async function expectHandoff(page: Page, route: string, destination: RegExp) {
+  await expect(async () => {
+    await page.goto(route)
+    await expect(page).toHaveURL(destination, { timeout: 2_000 })
+  }).toPass({ timeout: ROLLOUT_PROPAGATION_MS })
+}