From e01c0d6b4818c815897fd13b6ec319721a226b73 Mon Sep 17 00:00:00 2001
From: SaelixCode <dev@saelix.com>
Date: Sun, 22 Mar 2026 01:27:33 -0400
Subject: [PATCH] fix(e2e): use #node-name locator instead of getByLabel in
 nodes tests

The NodeManager form labels the field 'Name' (not 'Node Name'), so
getByLabel(/node name/i) never matched and timed out after 30s.
Switch to the stable #node-name id which is bound via htmlFor.
---
 e2e/nodes.spec.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/e2e/nodes.spec.ts b/e2e/nodes.spec.ts
index d2ae2411..011b404c 100644
--- a/e2e/nodes.spec.ts
+++ b/e2e/nodes.spec.ts
@@ -23,7 +23,7 @@ test.describe('Node management', () => {
     }
     await addBtn.click();
 
-    await page.getByLabel(/node name/i).fill('bad-node');
+    await page.locator('#node-name').fill('bad-node');
     // Select "remote" type if there's a type selector
     const typeSelect = page.getByLabel(/type/i);
     if (await typeSelect.isVisible()) await typeSelect.selectOption('remote');