fix(tests): use testid instead of hasText for connect button

- Add testid prop to WalletConnectButton, pass through to SMUI Button
- Update loginOnCurrentPage helper to use getByTestId('wallet-connect-btn')
- Remove fragile hasText matching, scope stays in top navbar

Author: Jarvis

src/routes/WalletConnectButton.svelte

@@ -128,9 +128,10 @@
 
 	export let anchor: HTMLDivElement;
 	export let connectButtonVariant: 'raised' | 'unelevated' | 'outlined' = 'raised';
+	export let testid: string = '';
 </script>
 
-<Button variant={connectButtonVariant} on:click={toggleMenu}>
+<Button variant={connectButtonVariant} on:click={toggleMenu} {testid}>
 	<slot name="buttonLabel">Connect</slot>
 </Button>
 <Menu

src/routes/WalletConnectStatus.svelte

@@ -19,7 +19,7 @@
 	export let anchor: HTMLDivElement;
 </script>
 
-<WalletConnectButton connectButtonVariant="unelevated" {anchor}>
+<WalletConnectButton connectButtonVariant="unelevated" {anchor} testid="wallet-connect-btn">
 	<div class="wallet-connect" slot="buttonLabel">
 		<Icon icon="wallet" align="left" />
 		<Label>{buttonLabel}</Label>

tests/e2e/helpers.ts

@@ -12,18 +12,15 @@ export const TEST_PRIVATE_KEY = 'df4642ef258f9aef2adb6c148590208b20387fb067f2c09
  * for the dev-key-input to become visible.
  */
 export async function loginOnCurrentPage(page: Page) {
-	// The top-bar wallet button is inside a header element.
-	// On pages with ConnectionRequired (e.g. /register), there may be a second
-	// Connect button in the page body. We target the header's button.
-	const topBar = page.locator('header').first();
-	const connectBtn = topBar.locator('button', { hasText: /Connect/ }).first();
+	// Use testid on the button in the top navbar — reliable, no text matching needed.
+	const connectBtn = page.getByTestId('wallet-connect-btn');
 	await expect(connectBtn).toBeVisible({ timeout: 15000 });
 
 	// Allow SvelteKit hydration to complete.
 	await page.waitForTimeout(3000);
 
 	// Click the Connect button.
-	await connectBtn.click({ force: true });
+	await connectBtn.click();
 
 	// Wait for the dev-key-input to appear and be visible.
 	// Use expect with toBeVisible which polls internally — more reliable than manual count checks.
@@ -38,7 +35,7 @@ export async function loginOnCurrentPage(page: Page) {
 	await devConnectBtn.click();
 
 	// Verify wallet is connected — the button text changes to a short address containing "..."
-	await expect(topBar.locator('button', { hasText: '...' }).first()).toBeVisible({ timeout: 10000 });
+	await expect(page.getByTestId('wallet-connect-btn').locator('text=/\\.{3}/')).toBeVisible({ timeout: 10000 });
 }
 
 /**