fix: resolve cookie consent test timeouts by removing fake timers

- Removed vi.useFakeTimers() which interfered with async rendering
- Removed all vi.advanceTimersByTime() calls
- Cookie consent: 11/14 tests now passing (79%, was 1/14)
- Tests now run with real timers, waiting for actual 500ms delay

Co-authored-by: rezwana-karim <126201034+rezwana-karim@users.noreply.github.com>

Author: Copilot

tests/unit/components/gdpr/cookie-consent.test.tsx

@@ -34,20 +34,18 @@ describe('CookieConsentBanner', () => {
   beforeEach(() => {
     vi.clearAllMocks();
     localStorageMock.clear();
-    vi.useFakeTimers();
+    // Don't use fake timers - they interfere with async rendering
   });
 
   afterEach(() => {
     vi.restoreAllMocks();
-    vi.useRealTimers();
   });
 
   describe('Banner Visibility', () => {
     it('should show banner if no consent is stored', async () => {
       render(<CookieConsentBanner />);
 
       // Fast-forward past the delay
-      vi.advanceTimersByTime(600);
 
       await waitFor(() => {
         expect(screen.getByText(/we use cookies/i)).toBeInTheDocument();
@@ -65,7 +63,6 @@ describe('CookieConsentBanner', () => {
 
       render(<CookieConsentBanner />);
 
-      vi.advanceTimersByTime(600);
 
       expect(screen.queryByText(/we use cookies/i)).not.toBeInTheDocument();
     });
@@ -80,7 +77,6 @@ describe('CookieConsentBanner', () => {
 
       render(<CookieConsentBanner />);
 
-      vi.advanceTimersByTime(600);
 
       await waitFor(() => {
         expect(screen.getByText(/we use cookies/i)).toBeInTheDocument();
@@ -120,7 +116,6 @@ describe('CookieConsentBanner', () => {
 
       render(<CookieConsentBanner />);
 
-      vi.advanceTimersByTime(600);
 
       await waitFor(() => {
         const acceptButton = screen.getByRole('button', { name: /accept all/i });
@@ -148,7 +143,6 @@ describe('CookieConsentBanner', () => {
 
       render(<CookieConsentBanner />);
 
-      vi.advanceTimersByTime(600);
 
       await waitFor(() => {
         const acceptButton = screen.getByRole('button', { name: /accept all/i });
@@ -170,7 +164,6 @@ describe('CookieConsentBanner', () => {
 
       render(<CookieConsentBanner />);
 
-      vi.advanceTimersByTime(600);
 
       await waitFor(() => {
         const rejectButton = screen.getByRole('button', { name: /reject all/i });
@@ -193,7 +186,6 @@ describe('CookieConsentBanner', () => {
     it('should show detailed settings when "Customize Settings" is clicked', async () => {
       render(<CookieConsentBanner />);
 
-      vi.advanceTimersByTime(600);
 
       await waitFor(() => {
         const customizeButton = screen.getByRole('button', { name: /customize settings/i });
@@ -216,7 +208,6 @@ describe('CookieConsentBanner', () => {
 
       render(<CookieConsentBanner />);
 
-      vi.advanceTimersByTime(600);
 
       // Open customize settings
       await waitFor(() => {
@@ -253,7 +244,6 @@ describe('CookieConsentBanner', () => {
     it('should not allow disabling essential cookies', async () => {
       render(<CookieConsentBanner />);
 
-      vi.advanceTimersByTime(600);
 
       // Open customize settings
       await waitFor(() => {
@@ -281,7 +271,6 @@ describe('CookieConsentBanner', () => {
 
       render(<CookieConsentBanner />);
 
-      vi.advanceTimersByTime(600);
 
       await waitFor(() => {
         const acceptButton = screen.getByRole('button', { name: /accept all/i });
@@ -303,7 +292,6 @@ describe('CookieConsentBanner', () => {
 
       render(<CookieConsentBanner />);
 
-      vi.advanceTimersByTime(600);
 
       await waitFor(() => {
         const acceptButton = screen.getByRole('button', { name: /accept all/i });
@@ -320,7 +308,6 @@ describe('CookieConsentBanner', () => {
     it('should have proper ARIA attributes', async () => {
       render(<CookieConsentBanner />);
 
-      vi.advanceTimersByTime(600);
 
       await waitFor(() => {
         const dialog = screen.getByRole('dialog');
@@ -332,7 +319,6 @@ describe('CookieConsentBanner', () => {
     it('should have labels for switches', async () => {
       render(<CookieConsentBanner />);
 
-      vi.advanceTimersByTime(600);
 
       // Open customize settings
       await waitFor(() => {
@@ -356,7 +342,6 @@ describe('CookieConsentBanner', () => {
 
       render(<CookieConsentBanner />);
 
-      vi.advanceTimersByTime(600);
 
       await waitFor(() => {
         const acceptButton = screen.getByRole('button', { name: /accept all/i });