From 4f184f504d4bd6a20a4c397e751c91cb3bc56cee Mon Sep 17 00:00:00 2001 From: "Timo K." Date: Thu, 5 Nov 2026 15:30:25 +0100 Subject: [PATCH] Expect the join to fetch no model, as it adopts the lobby's pipeline The e2e check held back the model download at the join and required one, since the pipeline used to be built again for the call's track. The join now takes the lobby's running pipeline over, so no download happens, and the check says so. Co-Authored-By: Claude Fable 5.1 --- playwright/background-effects.spec.ts | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/playwright/background-effects.spec.ts b/playwright/background-effects.spec.ts index 4be24c1a5..c77ea5faf 100644 --- a/playwright/background-effects.spec.ts +++ b/playwright/background-effects.spec.ts @@ -183,8 +183,8 @@ test.describe("joining with a background chosen", () => { timeout: 60_000, }) .toBeLessThan(distance(camera, picture) / 2); - // A pipeline built again for the call's track now takes as long as it - // does on the slower route. Routing bypasses the cache, so it waits. + // Were the pipeline built again for the call's track, it would take as + // long as on the slower route. Routing bypasses the cache, so it waits. let heldBack = 0; await guest.route("**/*.tflite*", async (route) => { heldBack += 1; @@ -212,8 +212,8 @@ test.describe("joining with a background chosen", () => { }) .toBeDefined(); await expectNoFrameOfTheRoom(host, trackId!, 0, camera, picture); - // Otherwise nothing was waited for. - expect(heldBack).toBeGreaterThan(0); + // The join adopts the lobby's pipeline, so the model is never fetched again. + expect(heldBack).toBe(0); await hostContext.close(); await guestContext.close();