From dd97daa16ba8514df980ec48d573b6b75b352a7c Mon Sep 17 00:00:00 2001
From: Peter Hanssens <phanssens1@gmail.com>
Date: Tue, 23 Sep 2025 21:21:06 +1000
Subject: [PATCH] fix: Use correct sidebar locator based on actual HTML
 structure

- Change from generic nav/sidebar selectors to specific div.w-64.bg-card.border-r.border-border
- This matches the actual sidebar structure in the demo source code
- Should fix the remaining sidebar toggle test failure
---
 tests/e2e/comprehensive-demo.spec.ts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/e2e/comprehensive-demo.spec.ts b/tests/e2e/comprehensive-demo.spec.ts
index 4d90687..7e20be0 100644
--- a/tests/e2e/comprehensive-demo.spec.ts
+++ b/tests/e2e/comprehensive-demo.spec.ts
@@ -33,7 +33,7 @@ test.describe('Comprehensive Dashboard Demo E2E Tests', () => {
     });
 
     test('should have proper sidebar navigation', async () => {
-      const sidebar = page.locator('nav, .sidebar, [role="navigation"]').first();
+      const sidebar = page.locator('div.w-64.bg-card.border-r.border-border');
       await expect(sidebar).toBeVisible();
       
       // Check navigation links
@@ -214,7 +214,7 @@ test.describe('Comprehensive Dashboard Demo E2E Tests', () => {
       await sidebarToggle.click();
       
       // Check that sidebar is hidden/shown
-      const sidebar = page.locator('nav, .sidebar, [role="navigation"]').first();
+      const sidebar = page.locator('div.w-64.bg-card.border-r.border-border');
       await expect(sidebar).toBeVisible();
     });
   });