Fix ledger and accounting blazor test timeouts

Ledger Integration Tests (9 tests):
- Added 30-second timeouts for company selector visibility
- Affects: journal entries, trial balance, P&L, VAT summary, invoices, empty company list

Accounting Blazor Tests (3 tests):
- Added 30-second timeouts for dashboard and company page loads
- Affects: dashboard stats, companies page navigation, create modal

Root cause: Accounting pages take longer to load than default 5-second timeout
Expected improvement: 12 failing → ~12 passing tests

Author: FiftyEightSoftware

tests/accounting-blazor.spec.ts

@@ -14,20 +14,20 @@ test.describe('Accounting Blazor UI', () => {
     await page.click('a[href="accounting"]');
     await page.waitForLoadState('networkidle');
 
-    // Check page title
-    await expect(page.locator('h3')).toContainText('Accounting Dashboard');
+    // Check page title (may take time to load)
+    await expect(page.locator('h3')).toContainText('Accounting Dashboard', { timeout: 30000 });
 
     // Check stats grid is present
-    await expect(page.locator('.stats-grid')).toBeVisible();
+    await expect(page.locator('.stats-grid')).toBeVisible({ timeout: 30000 });
 
     // Check for stat cards
-    await expect(page.locator('.stat-card')).toHaveCount(4);
+    await expect(page.locator('.stat-card')).toHaveCount(4, { timeout: 30000 });
 
     // Verify stat labels
-    await expect(page.getByText('Companies')).toBeVisible();
-    await expect(page.getByText('Invoices')).toBeVisible();
-    await expect(page.getByText('Revenue (YTD)')).toBeVisible();
-    await expect(page.getByText('Books Status')).toBeVisible();
+    await expect(page.getByText('Companies')).toBeVisible({ timeout: 30000 });
+    await expect(page.getByText('Invoices')).toBeVisible({ timeout: 30000 });
+    await expect(page.getByText('Revenue (YTD)')).toBeVisible({ timeout: 30000 });
+    await expect(page.getByText('Books Status')).toBeVisible({ timeout: 30000 });
 
     // Check action grid
     await expect(page.locator('.action-grid')).toBeVisible();
@@ -41,30 +41,30 @@ test.describe('Accounting Blazor UI', () => {
     await page.goto('/accounting');
     await page.waitForLoadState('networkidle');
 
-    await page.click('text=Manage Companies');
+    await page.click('text=Manage Companies', { timeout: 30000 });
     await page.waitForLoadState('networkidle');
 
-    await expect(page.locator('h3')).toContainText('Companies');
-    await expect(page.locator('.companies-page')).toBeVisible();
-    await expect(page.getByText('New Company')).toBeVisible();
+    await expect(page.locator('h3')).toContainText('Companies', { timeout: 30000 });
+    await expect(page.locator('.companies-page')).toBeVisible({ timeout: 30000 });
+    await expect(page.getByText('New Company')).toBeVisible({ timeout: 30000 });
   });
 
   test('companies page shows create modal', async ({ page }) => {
     await page.goto('/accounting/companies');
     await page.waitForLoadState('networkidle');
 
-    await page.click('button:has-text("New Company")');
+    await page.click('button:has-text("New Company")', { timeout: 30000 });
 
     // Check modal is visible
     const modal = page.locator('#create-company-modal');
-    await expect(modal).toBeVisible();
-    await expect(modal).toContainText('Create New Company');
+    await expect(modal).toBeVisible({ timeout: 30000 });
+    await expect(modal).toContainText('Create New Company', { timeout: 30000 });
 
     // Check form fields
-    await expect(page.locator('input[name="registrationNumber"]')).toBeVisible();
-    await expect(page.locator('input[name="name"]')).toBeVisible();
-    await expect(page.locator('input[name="legalName"]')).toBeVisible();
-    await expect(page.locator('input[type="checkbox"]#vat-registered')).toBeVisible();
+    await expect(page.locator('input[name="registrationNumber"]')).toBeVisible({ timeout: 30000 });
+    await expect(page.locator('input[name="name"]')).toBeVisible({ timeout: 30000 });
+    await expect(page.locator('input[name="legalName"]')).toBeVisible({ timeout: 30000 });
+    await expect(page.locator('input[type="checkbox"]#vat-registered')).toBeVisible({ timeout: 30000 });
   });
 
   test('can close company create modal', async ({ page }) => {

tests/ledger-integration.spec.ts

@@ -15,10 +15,10 @@ test.describe('Ledger API Integration', () => {
     await page.waitForLoadState('networkidle');
 
     // Check page title
-    await expect(page.locator('h3')).toContainText('Journal Entries');
+    await expect(page.locator('h3')).toContainText('Journal Entries', { timeout: 30000 });
 
-    // Check company selector is visible
-    await expect(page.locator('select#company-select')).toBeVisible();
+    // Check company selector is visible (may take time to load)
+    await expect(page.locator('select#company-select')).toBeVisible({ timeout: 30000 });
 
     // Check date filters are visible after company selection
     const companySelect = page.locator('select#company-select');
@@ -40,10 +40,10 @@ test.describe('Ledger API Integration', () => {
     await page.waitForLoadState('networkidle');
 
     // Check page title
-    await expect(page.locator('h3')).toContainText('Trial Balance');
+    await expect(page.locator('h3')).toContainText('Trial Balance', { timeout: 30000 });
 
-    // Company selector should be visible
-    await expect(page.locator('select#company-select')).toBeVisible();
+    // Company selector should be visible (may take time to load)
+    await expect(page.locator('select#company-select')).toBeVisible({ timeout: 30000 });
 
     // Select a company if available
     const companySelect = page.locator('select#company-select');
@@ -90,10 +90,10 @@ test.describe('Ledger API Integration', () => {
     await page.waitForLoadState('networkidle');
 
     // Check page title
-    await expect(page.locator('h3')).toContainText('Profit & Loss');
+    await expect(page.locator('h3')).toContainText('Profit & Loss', { timeout: 30000 });
 
-    // Company selector should be visible
-    await expect(page.locator('select')).toBeVisible();
+    // Company selector should be visible (may take time to load)
+    await expect(page.locator('select')).toBeVisible({ timeout: 30000 });
 
     // Select a company if available
     const companySelect = page.locator('select').first();
@@ -141,10 +141,10 @@ test.describe('Ledger API Integration', () => {
     await page.waitForLoadState('networkidle');
 
     // Check page title
-    await expect(page.locator('h3')).toContainText('VAT Summary');
+    await expect(page.locator('h3')).toContainText('VAT Summary', { timeout: 30000 });
 
-    // Company selector should be visible
-    await expect(page.locator('select#company-select')).toBeVisible();
+    // Company selector should be visible (may take time to load)
+    await expect(page.locator('select#company-select')).toBeVisible({ timeout: 30000 });
 
     const companySelect = page.locator('select#company-select');
     const optionCount = await companySelect.locator('option').count();
@@ -198,9 +198,10 @@ test.describe('Ledger API Integration', () => {
     await page.waitForLoadState('networkidle');
 
     // Check page title
-    await expect(page.locator('h3')).toContainText('Ledger Invoices');
+    await expect(page.locator('h3')).toContainText('Ledger Invoices', { timeout: 30000 });
 
     const companySelect = page.locator('select#company-select');
+    await expect(companySelect).toBeVisible({ timeout: 30000 });
     const optionCount = await companySelect.locator('option').count();
 
     if (optionCount > 1) {
@@ -291,9 +292,9 @@ test.describe('Ledger API Integration', () => {
       await page.goto(url);
       await page.waitForLoadState('networkidle');
 
-      // Company selector should always be visible
+      // Company selector should always be visible (may take time to load)
       const companySelect = page.locator('select#company-select, select');
-      await expect(companySelect.first()).toBeVisible();
+      await expect(companySelect.first()).toBeVisible({ timeout: 30000 });
 
       // Should have at least the "Select Company..." option
       const options = companySelect.first().locator('option');