Fix auth cookie name (arm_token) + separate Run All from suite count
Some checks failed
CI/CD / detect-changes (pull_request) Successful in 4s
CI/CD / test-frontend (pull_request) Successful in 5s
CI/CD / test-runners (pull_request) Failing after 2m34s
CI/CD / test-sqlite (pull_request) Successful in 3m8s
CI/CD / test-go-pg (pull_request) Successful in 3m16s
CI/CD / build-and-deploy (pull_request) Has been skipped
Some checks failed
CI/CD / detect-changes (pull_request) Successful in 4s
CI/CD / test-frontend (pull_request) Successful in 5s
CI/CD / test-runners (pull_request) Failing after 2m34s
CI/CD / test-sqlite (pull_request) Successful in 3m8s
CI/CD / test-go-pg (pull_request) Successful in 3m16s
CI/CD / build-and-deploy (pull_request) Has been skipped
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) <noreply@anthropic.com>
This commit is contained in:
@@ -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: '/',
|
||||
|
||||
@@ -326,7 +326,7 @@
|
||||
<h2>Test Runners</h2>
|
||||
${props.running
|
||||
? html`<span class="ext-test-runners-running"><span class="ext-test-runners-spinner"></span> Running${props.runningSuite === 'all' ? ' all' : ''}...</span>`
|
||||
: html`<button class="sw-btn sw-btn--primary" onClick=${props.onRunAll}>Run All (${props.suiteCount} suites)</button>`
|
||||
: html`<button class="sw-btn sw-btn--primary" onClick=${props.onRunAll}>Run All</button> <span class="ext-test-runners-suite-count">${props.suiteCount} suites</span>`
|
||||
}
|
||||
${s ? html`
|
||||
<div class="ext-test-runners-summary">
|
||||
|
||||
Reference in New Issue
Block a user