From 4a3be4913f8106284490e3ddbd5376e4cf34827d Mon Sep 17 00:00:00 2001 From: Jeffrey Smith Date: Thu, 2 Apr 2026 11:26:59 +0000 Subject: [PATCH] Fix auth cookie name (arm_token) + separate Run All from suite count MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 1. The driver set cookie name 'token' but the server uses 'arm_token', so Playwright always saw the login page instead of the surface. 2. Split "Run All (N suites)" into a "Run All" button + separate suite count label — count changes over time and shouldn't be part of the button text. Co-Authored-By: Claude Opus 4.6 (1M context) --- ci/surface-test-driver.js | 4 ++-- packages/test-runners/js/main.js | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/ci/surface-test-driver.js b/ci/surface-test-driver.js index 9509c1d..abe6fcd 100644 --- a/ci/surface-test-driver.js +++ b/ci/surface-test-driver.js @@ -33,9 +33,9 @@ if (!TOKEN) { const browser = await chromium.launch({ headless: true }); const context = await browser.newContext(); - // Set auth cookie/token + // Set auth cookie — name must match server's SetCookie ("arm_token") await context.addCookies([{ - name: 'token', + name: 'arm_token', value: TOKEN, domain: new URL(SERVER).hostname, path: '/', diff --git a/packages/test-runners/js/main.js b/packages/test-runners/js/main.js index 5c697d9..e21a246 100644 --- a/packages/test-runners/js/main.js +++ b/packages/test-runners/js/main.js @@ -326,7 +326,7 @@

Test Runners

${props.running ? html` Running${props.runningSuite === 'all' ? ' all' : ''}...` - : html`` + : html` ${props.suiteCount} suites` } ${s ? html`