@open-mercato/core 0.4.5-develop-2e9903a57a → 0.4.5-develop-eeccf7adf4

package/src/modules/configs/components/__tests__/CachePanel.test.tsx

@@ -1,134 +0,0 @@
-/**
- * @jest-environment jsdom
- */
-
-import '@testing-library/jest-dom'
-import * as React from 'react'
-import { fireEvent, screen, waitFor } from '@testing-library/react'
-import { CachePanel } from '../CachePanel'
-import { renderWithProviders } from '@open-mercato/shared/lib/testing/renderWithProviders'
-import { readApiResultOrThrow } from '@open-mercato/ui/backend/utils/apiCall'
-import { flash } from '@open-mercato/ui/backend/FlashMessages'
-
-jest.mock('@open-mercato/ui/backend/utils/apiCall', () => ({
-  readApiResultOrThrow: jest.fn(),
-}))
-
-jest.mock('@open-mercato/ui/backend/FlashMessages', () => ({
-  flash: jest.fn(),
-}))
-
-// Mock HTMLDialogElement methods for jsdom compatibility
-HTMLDialogElement.prototype.showModal = jest.fn(function(this: HTMLDialogElement) {
-  this.open = true
-  this.setAttribute('open', '')
-})
-HTMLDialogElement.prototype.close = jest.fn(function(this: HTMLDialogElement) {
-  this.open = false
-  this.removeAttribute('open')
-})
-
-const dict = {
-  'configs.cache.title': 'Cache overview',
-  'configs.cache.description': 'Inspect cache',
-  'configs.cache.loading': 'Loading cache statistics…',
-  'configs.cache.loadError': 'Failed to load cache statistics.',
-  'configs.cache.refresh': 'Refresh',
-  'configs.cache.purgeAll': 'Purge all cache',
-  'configs.cache.purgeSegment': 'Purge segment',
-  'configs.cache.purgeSegmentSuccess': 'Purged {{segment}} ({{count}})',
-  'configs.cache.purgeAllSuccess': 'All cache purged',
-  'configs.cache.table.segment': 'Segment',
-  'configs.cache.table.path': 'Path',
-  'configs.cache.table.method': 'Method',
-  'configs.cache.table.count': 'Cached keys',
-  'configs.cache.table.countValue': '{{count}} keys',
-  'configs.cache.table.actions': 'Actions',
-  'configs.cache.table.pathUnknown': 'n/a',
-  'configs.cache.generatedAt': 'Stats generated {{timestamp}}',
-  'configs.cache.totalEntries': '{{count}} cached entries',
-  'configs.cache.retry': 'Retry',
-  'configs.cache.purgeError': 'Failed to purge cache segment.',
-  'configs.cache.purgeAllError': 'Failed to purge',
-  'configs.cache.purgeSegmentConfirm': 'Confirm segment purge?',
-  'configs.cache.purgeAllConfirm': 'Confirm all purge?',
-  'configs.cache.inactive': 'Inactive',
-  'ui.dialogs.confirm.confirmText': 'Confirm',
-  'ui.dialogs.confirm.cancelText': 'Cancel',
-  'ui.dialog.close.ariaLabel': 'Close',
-}
-
-const statsPayload = {
-  generatedAt: '2025-01-01T00:00:00.000Z',
-  totalKeys: 5,
-  segments: [
-    {
-      segment: 'users.list',
-      resource: 'users',
-      method: 'GET',
-      path: '/api/users',
-      keyCount: 3,
-    },
-  ],
-}
-
-describe('CachePanel', () => {
-  beforeEach(() => {
-    jest.resetAllMocks()
-  })
-
-  it('renders cache statistics and management actions', async () => {
-    ;(readApiResultOrThrow as jest.Mock)
-      .mockResolvedValueOnce(statsPayload) // stats
-      .mockResolvedValueOnce({ ok: true, granted: ['configs.cache.manage'] }) // feature check
-
-    renderWithProviders(<CachePanel />, { dict })
-
-    await waitFor(() => {
-      expect(screen.getByText('users.list')).toBeInTheDocument()
-    })
-    expect(screen.getByRole('button', { name: 'Purge all cache' })).toBeInTheDocument()
-  })
-
-  it('purges a segment when user confirms', async () => {
-    ;(readApiResultOrThrow as jest.Mock)
-      .mockResolvedValueOnce(statsPayload)
-      .mockResolvedValueOnce({ ok: true, granted: ['configs.cache.manage'] })
-      .mockResolvedValueOnce({
-        stats: statsPayload,
-        deleted: 1,
-      })
-
-    renderWithProviders(<CachePanel />, { dict })
-
-    await waitFor(() => {
-      expect(screen.getByText('users.list')).toBeInTheDocument()
-    })
-
-    // Click purge segment button
-    fireEvent.click(screen.getByRole('button', { name: 'Purge segment' }))
-
-    // Wait for confirm dialog to appear (dialog title)
-    await waitFor(() => {
-      expect(screen.getByText('Confirm segment purge?')).toBeInTheDocument()
-    })
-
-    // Click confirm button by text (inside dialog)
-    const confirmButtons = screen.getAllByText('Confirm')
-    fireEvent.click(confirmButtons[confirmButtons.length - 1])
-
-    await waitFor(() => {
-      expect(flash).toHaveBeenCalledWith('Purged users.list (1)', 'success')
-    })
-  })
-
-  it('shows an error notice when stats cannot be loaded', async () => {
-    ;(readApiResultOrThrow as jest.Mock).mockRejectedValueOnce(new Error('boom'))
-
-    renderWithProviders(<CachePanel />, { dict })
-
-    await waitFor(() => {
-      expect(screen.getByText('boom')).toBeInTheDocument()
-    })
-  })
-})

package/src/modules/configs/components/__tests__/SystemStatusPanel.test.tsx

@@ -1,93 +0,0 @@
-/**
- * @jest-environment jsdom
- */
-
-import '@testing-library/jest-dom'
-import * as React from 'react'
-import { screen, waitFor } from '@testing-library/react'
-import { SystemStatusPanel } from '../SystemStatusPanel'
-import { renderWithProviders } from '@open-mercato/shared/lib/testing/renderWithProviders'
-import { readApiResultOrThrow } from '@open-mercato/ui/backend/utils/apiCall'
-
-jest.mock('@open-mercato/ui/backend/utils/apiCall', () => ({
-  readApiResultOrThrow: jest.fn(),
-}))
-
-const dict = {
-  'configs.systemStatus.title': 'System status',
-  'configs.systemStatus.description': 'Review system flags',
-  'configs.systemStatus.loading': 'Loading status snapshot…',
-  'configs.systemStatus.runtime.development': 'Development',
-  'configs.systemStatus.state.enabled': 'Enabled',
-  'configs.systemStatus.state.disabled': 'Disabled',
-  'configs.systemStatus.state.set': 'Set',
-  'configs.systemStatus.state.unset': 'Unset',
-  'configs.systemStatus.state.unknown': 'Unknown',
-  'configs.systemStatus.details.currentValue': 'Current value',
-  'configs.systemStatus.details.defaultValue': 'Default',
-  'configs.systemStatus.details.updateHint': 'Update {{key}}',
-  'configs.systemStatus.details.updateDocs': 'Docs',
-  'configs.systemStatus.viewDocs': 'Documentation',
-  'configs.systemStatus.error': 'Failed to load system status',
-  'configs.systemStatus.invalidResponse': 'Unexpected response when loading system status',
-  'configs.systemStatus.category.debug': 'Debug tools',
-  'configs.systemStatus.item.verbose': 'Verbose logging',
-  'configs.systemStatus.item.verbose.description': 'Verbose logging description',
-}
-
-const snapshot = {
-  generatedAt: '2025-01-01T00:00:00.000Z',
-  runtimeMode: 'development',
-  categories: [
-    {
-      key: 'debug',
-      labelKey: 'configs.systemStatus.category.debug',
-      descriptionKey: null,
-      items: [
-        {
-          key: 'DEBUG_LOGGING',
-          labelKey: 'configs.systemStatus.item.verbose',
-          descriptionKey: 'configs.systemStatus.item.verbose.description',
-          state: 'enabled',
-          value: '1',
-          defaultValue: '0',
-          docUrl: null,
-        },
-      ],
-    },
-  ],
-}
-
-describe('SystemStatusPanel', () => {
-  beforeEach(() => {
-    jest.resetAllMocks()
-  })
-
-  it('renders snapshot categories returned from the API', async () => {
-    ;(readApiResultOrThrow as jest.Mock).mockResolvedValueOnce(snapshot)
-    renderWithProviders(<SystemStatusPanel />, { dict })
-
-    await waitFor(() => {
-      expect(screen.getByText('Debug tools')).toBeInTheDocument()
-      expect(screen.getByText('Verbose logging')).toBeInTheDocument()
-    })
-  })
-
-  it('shows a friendly error if the payload is not a snapshot', async () => {
-    ;(readApiResultOrThrow as jest.Mock).mockResolvedValueOnce({ invalid: true })
-    renderWithProviders(<SystemStatusPanel />, { dict })
-
-    await waitFor(() => {
-      expect(screen.getByText('Unexpected response when loading system status')).toBeInTheDocument()
-    })
-  })
-
-  it('handles request failures', async () => {
-    ;(readApiResultOrThrow as jest.Mock).mockRejectedValueOnce(new Error('boom'))
-    renderWithProviders(<SystemStatusPanel />, { dict })
-
-    await waitFor(() => {
-      expect(screen.getByText('boom')).toBeInTheDocument()
-    })
-  })
-})

package/src/modules/configs/lib/__tests__/system-status.test.ts

@@ -1,55 +0,0 @@
-import { buildSystemStatusSnapshot } from '../system-status'
-
-function findItemValue(snapshot: ReturnType<typeof buildSystemStatusSnapshot>, key: string): string | null {
-  for (const category of snapshot.categories) {
-    const item = category.items.find((entry) => entry.key === key)
-    if (item) return item.value
-  }
-  return null
-}
-
-describe('buildSystemStatusSnapshot', () => {
-  it('masks credentials for DATABASE_URL', () => {
-    const snapshot = buildSystemStatusSnapshot({
-      DATABASE_URL: 'postgresql://app_user:secret@db.example.com:5432/mercato?sslmode=require',
-    })
-
-    const value = findItemValue(snapshot, 'DATABASE_URL')
-    expect(value).toBe('postgresql://db.example.com:5432/mercato?sslmode=require')
-    expect(value).not.toContain('app_user')
-    expect(value).not.toContain('secret')
-  })
-
-  it('keeps database url unchanged when no credentials are present', () => {
-    const snapshot = buildSystemStatusSnapshot({
-      DATABASE_URL: 'postgresql://db.example.com:5432/mercato',
-    })
-
-    expect(findItemValue(snapshot, 'DATABASE_URL')).toBe('postgresql://db.example.com:5432/mercato')
-  })
-
-  it('masks credentials for malformed DSNs using fallback redaction', () => {
-    const cases = [
-      {
-        input: 'postgresql://app_user:se/cret@db.example.com:5432/mercato',
-        expected: 'postgresql://db.example.com:5432/mercato',
-      },
-      {
-        input: 'postgresql://app_user:secret@/mercato',
-        expected: 'postgresql:///mercato',
-      },
-      {
-        input: 'postgresql://app_user:secret@db.example.com:5432/mercato?sslmode=require',
-        expected: 'postgresql://db.example.com:5432/mercato?sslmode=require',
-      },
-    ]
-
-    for (const entry of cases) {
-      const snapshot = buildSystemStatusSnapshot({ DATABASE_URL: entry.input })
-      const value = findItemValue(snapshot, 'DATABASE_URL')
-      expect(value).toBe(entry.expected)
-      expect(value).not.toContain('app_user')
-      expect(value).not.toContain('secret')
-    }
-  })
-})

package/src/modules/configs/lib/__tests__/upgrade-actions.test.ts

@@ -1,135 +0,0 @@
-jest.mock('@open-mercato/core/modules/catalog/lib/seeds', () => ({
-  installExampleCatalogData: jest.fn(),
-}))
-jest.mock('@open-mercato/cache', () => ({
-  runWithCacheTenant: jest.fn((_, fn) => fn()),
-}))
-jest.mock('@open-mercato/shared/lib/crud/cache-stats', () => ({
-  collectCrudCacheStats: jest.fn().mockResolvedValue({ segments: [] }),
-  purgeCrudCacheSegment: jest.fn(),
-}))
-jest.mock('@open-mercato/shared/lib/crud/cache', () => ({
-  isCrudCacheEnabled: jest.fn().mockReturnValue(false),
-  resolveCrudCache: jest.fn(),
-}))
-
-import { compareVersions, actionsUpToVersion, findUpgradeAction, upgradeActions } from '../upgrade-actions'
-
-describe('compareVersions', () => {
-  describe('basic comparisons', () => {
-    it('returns 0 for equal versions', () => {
-      expect(compareVersions('1.0.0', '1.0.0')).toBe(0)
-      expect(compareVersions('0.3.4', '0.3.4')).toBe(0)
-    })
-
-    it('returns negative when first version is less than second', () => {
-      expect(compareVersions('1.0.0', '2.0.0')).toBeLessThan(0)
-      expect(compareVersions('1.0.0', '1.1.0')).toBeLessThan(0)
-      expect(compareVersions('1.0.0', '1.0.1')).toBeLessThan(0)
-    })
-
-    it('returns positive when first version is greater than second', () => {
-      expect(compareVersions('2.0.0', '1.0.0')).toBeGreaterThan(0)
-      expect(compareVersions('1.1.0', '1.0.0')).toBeGreaterThan(0)
-      expect(compareVersions('1.0.1', '1.0.0')).toBeGreaterThan(0)
-    })
-  })
-
-  describe('edge cases - partial versions', () => {
-    it('handles versions with fewer than 3 parts via semver coercion', () => {
-      expect(compareVersions('1.2', '1.2.0')).toBe(0)
-      expect(compareVersions('1', '1.0.0')).toBe(0)
-      expect(compareVersions('2', '1.9.9')).toBeGreaterThan(0)
-    })
-  })
-
-  describe('edge cases - versions with extra parts', () => {
-    it('handles versions with more than 3 parts by coercing to first 3', () => {
-      expect(compareVersions('1.2.3.4', '1.2.3')).toBe(0)
-      expect(compareVersions('1.2.3.4.5', '1.2.3')).toBe(0)
-    })
-  })
-
-  describe('edge cases - prerelease versions', () => {
-    it('coerces prerelease versions to numeric form', () => {
-      expect(compareVersions('1.2.3-alpha', '1.2.3')).toBe(0)
-      expect(compareVersions('1.2.3-beta.1', '1.2.3')).toBe(0)
-    })
-  })
-
-  describe('edge cases - invalid versions', () => {
-    it('throws an error for completely invalid version strings', () => {
-      expect(() => compareVersions('invalid', '1.0.0')).toThrow(/Invalid version string/)
-      expect(() => compareVersions('1.0.0', 'invalid')).toThrow(/Invalid version string/)
-    })
-
-    it('throws an error for empty strings', () => {
-      expect(() => compareVersions('', '1.0.0')).toThrow(/Invalid version string/)
-      expect(() => compareVersions('1.0.0', '')).toThrow(/Invalid version string/)
-    })
-  })
-
-  describe('boundary cases', () => {
-    it('handles version 0.0.0', () => {
-      expect(compareVersions('0.0.0', '0.0.1')).toBeLessThan(0)
-      expect(compareVersions('0.0.1', '0.0.0')).toBeGreaterThan(0)
-    })
-
-    it('handles large version numbers', () => {
-      expect(compareVersions('100.200.300', '100.200.299')).toBeGreaterThan(0)
-      expect(compareVersions('999.999.999', '1000.0.0')).toBeLessThan(0)
-    })
-  })
-})
-
-describe('actionsUpToVersion', () => {
-  it('returns actions up to and including the specified version', () => {
-    const actions = actionsUpToVersion('0.3.4')
-    actions.forEach((action) => {
-      expect(compareVersions(action.version, '0.3.4')).toBeLessThanOrEqual(0)
-    })
-  })
-
-  it('returns empty array for versions before any action', () => {
-    const actions = actionsUpToVersion('0.0.1')
-    expect(actions).toEqual([])
-  })
-
-  it('returns actions sorted by version then by id', () => {
-    const actions = actionsUpToVersion('99.99.99')
-    for (let i = 1; i < actions.length; i++) {
-      const prevVersion = actions[i - 1].version
-      const currentVersion = actions[i].version
-      const versionComparison = compareVersions(prevVersion, currentVersion)
-      if (versionComparison === 0) {
-        expect(actions[i - 1].id.localeCompare(actions[i].id)).toBeLessThanOrEqual(0)
-      } else {
-        expect(versionComparison).toBeLessThan(0)
-      }
-    }
-  })
-})
-
-describe('findUpgradeAction', () => {
-  it('finds an action by id within the version range', () => {
-    const existingAction = upgradeActions[0]
-    if (existingAction) {
-      const found = findUpgradeAction(existingAction.id, existingAction.version)
-      expect(found).toBeDefined()
-      expect(found?.id).toBe(existingAction.id)
-    }
-  })
-
-  it('returns undefined for non-existent action id', () => {
-    const found = findUpgradeAction('non.existent.action', '99.99.99')
-    expect(found).toBeUndefined()
-  })
-
-  it('returns undefined when action version exceeds max version', () => {
-    const existingAction = upgradeActions[0]
-    if (existingAction) {
-      const found = findUpgradeAction(existingAction.id, '0.0.1')
-      expect(found).toBeUndefined()
-    }
-  })
-})

package/src/modules/core/__integration__/admin/TC-ADMIN-001.spec.ts

@@ -1,70 +0,0 @@
-import { test, expect } from '@playwright/test';
-import { login } from '@open-mercato/core/modules/core/__integration__/helpers/auth';
-import { getAuthToken, apiRequest } from '@open-mercato/core/modules/core/__integration__/helpers/api';
-
-/**
- * TC-ADMIN-001: Create API Key
- * Source: .ai/qa/scenarios/TC-ADMIN-001-api-key-creation.md
- *
- * Verifies that API keys can be created from the create form,
- * that the generated key is shown once, and that the key appears
- * in the API keys list.
- *
- * Navigation: Settings → Auth → API Keys → Create
- */
-test.describe('TC-ADMIN-001: Create API Key', () => {
-  test('should create an API key and show it in the list', async ({ page, request }) => {
-    const keyName = `QA TC-ADMIN-001 ${Date.now()}`;
-    let token: string | null = null;
-
-    try {
-      token = await getAuthToken(request);
-      await login(page, 'admin');
-
-      // Navigate to API Keys via Settings
-      await page.goto('/backend/api-keys');
-      await expect(page.getByRole('heading', { name: 'API Keys', level: 2 })).toBeVisible();
-
-      // Click Create
-      await page.getByRole('link', { name: 'Create' }).click();
-      await expect(page).toHaveURL(/\/backend\/api-keys\/create$/);
-      await expect(page.getByText('Create API Key')).toBeVisible();
-
-      // Fill in the name
-      const nameField = page.locator('form').getByRole('textbox').first();
-      await nameField.fill(keyName);
-
-      // Click Create button
-      await page.getByRole('button', { name: 'Create' }).last().click();
-
-      // After creation, a dialog shows the generated key (one-time view)
-      await expect(page.getByText('Keep this key safe')).toBeVisible({ timeout: 10_000 });
-      await expect(page.getByText(/Prefix: omk_/)).toBeVisible();
-
-      // Close the key dialog
-      await page.getByRole('button', { name: 'Close' }).click();
-
-      // Navigate back to list
-      await page.goto('/backend/api-keys');
-      await page.getByText('Loading data...').waitFor({ state: 'hidden', timeout: 10_000 }).catch(() => {});
-
-      // Search for the created key
-      await page.getByRole('textbox', { name: 'Search' }).fill(keyName);
-      await expect(page.getByText(keyName)).toBeVisible({ timeout: 5_000 });
-    } finally {
-      // Cleanup: try to delete the API key via API (best effort)
-      if (token) {
-        const listResponse = await apiRequest(request, 'GET', '/api/auth/api-keys', { token });
-        const listData = await listResponse.json().catch(() => null);
-        if (listData && Array.isArray(listData.items)) {
-          const keyToDelete = listData.items.find((item: Record<string, unknown>) =>
-            item.name === keyName,
-          );
-          if (keyToDelete && typeof keyToDelete.id === 'string') {
-            await apiRequest(request, 'DELETE', `/api/auth/api-keys?id=${keyToDelete.id}`, { token }).catch(() => {});
-          }
-        }
-      }
-    }
-  });
-});

package/src/modules/core/__integration__/admin/TC-ADMIN-002.spec.ts

@@ -1,83 +0,0 @@
-import { test, expect } from '@playwright/test';
-import { login } from '@open-mercato/core/modules/core/__integration__/helpers/auth';
-import { getAuthToken, apiRequest } from '@open-mercato/core/modules/core/__integration__/helpers/api';
-
-/**
- * TC-ADMIN-002: Revoke API Key
- * Source: .ai/qa/scenarios/TC-ADMIN-002-api-key-revocation.md
- *
- * Verifies that an existing API key can be revoked.
- * Creates a key via API, then revokes it through the UI.
- *
- * Navigation: Settings → Auth → API Keys
- */
-test.describe('TC-ADMIN-002: Revoke API Key', () => {
-  test('should revoke an existing API key', async ({ page, request }) => {
-    const keyName = `QA TC-ADMIN-002 ${Date.now()}`;
-    let token: string | null = null;
-    let keyId: string | null = null;
-
-    try {
-      token = await getAuthToken(request);
-
-      // Create an API key via the UI first
-      await login(page, 'admin');
-      await page.goto('/backend/api-keys/create');
-
-      const nameField = page.locator('form').getByRole('textbox').first();
-      await nameField.fill(keyName);
-      await page.getByRole('button', { name: 'Create' }).last().click();
-
-      // Wait for the key dialog
-      await expect(page.getByText('Keep this key safe')).toBeVisible({ timeout: 10_000 });
-      await page.getByRole('button', { name: 'Close' }).click();
-
-      // Navigate to API keys list
-      await page.goto('/backend/api-keys');
-      await page.getByText('Loading data...').waitFor({ state: 'hidden', timeout: 10_000 }).catch(() => {});
-
-      // Search for the key
-      await page.getByRole('textbox', { name: 'Search' }).fill(keyName);
-      await expect(page.getByText(keyName)).toBeVisible({ timeout: 5_000 });
-
-      // Find and click the row actions for this key — look for a revoke/delete button
-      const keyRow = page.locator('table tbody tr').filter({ hasText: keyName }).first();
-      await expect(keyRow).toBeVisible();
-
-      // Click the actions button on the row
-      const actionsButton = keyRow.getByRole('button').last();
-      await actionsButton.click();
-
-      // Look for a Revoke or Delete option in the dropdown menu
-      const revokeButton = page.getByRole('menuitem', { name: /revoke|delete/i }).first();
-      await expect(revokeButton).toBeVisible({ timeout: 3_000 });
-      await revokeButton.click();
-
-      // Handle confirmation dialog if present
-      const confirmButton = page.getByRole('button', { name: /confirm|revoke|delete|yes/i });
-      if (await confirmButton.isVisible().catch(() => false)) {
-        await confirmButton.click();
-      }
-
-      // Verify the key is revoked or removed
-      await page.waitForTimeout(1_000);
-      await page.getByRole('textbox', { name: 'Search' }).fill(keyName);
-      // Key should either show as revoked or be removed from list
-      // We verify at least that no error occurred
-    } finally {
-      // Cleanup via API
-      if (token) {
-        const listResponse = await apiRequest(request, 'GET', '/api/auth/api-keys', { token });
-        const listData = await listResponse.json().catch(() => null);
-        if (listData && Array.isArray(listData.items)) {
-          const keyToDelete = listData.items.find((item: Record<string, unknown>) =>
-            item.name === keyName,
-          );
-          if (keyToDelete && typeof keyToDelete.id === 'string') {
-            await apiRequest(request, 'DELETE', `/api/auth/api-keys?id=${keyToDelete.id}`, { token }).catch(() => {});
-          }
-        }
-      }
-    }
-  });
-});

package/src/modules/core/__integration__/admin/TC-ADMIN-003.spec.ts

@@ -1,50 +0,0 @@
-import { test, expect } from '@playwright/test';
-import { login } from '@open-mercato/core/modules/core/__integration__/helpers/auth';
-
-/**
- * TC-ADMIN-003: View and Filter Audit Logs
- * Source: .ai/qa/scenarios/TC-ADMIN-003-audit-log-viewing.md
- *
- * Verifies that audit logs are accessible via the Security sidebar section,
- * display action and access log tabs, and show log entries with details.
- *
- * Navigation: Main sidebar → Security → Audit Logs
- */
-test.describe('TC-ADMIN-003: View and Filter Audit Logs', () => {
-  test('should display audit log tabs with action log entries', async ({ page }) => {
-    await login(page, 'admin');
-    await page.goto('/backend/audit-logs');
-
-    // Verify Action Log tab is selected by default
-    const actionLogTab = page.getByRole('tab', { name: 'Action Log' });
-    await expect(actionLogTab).toBeVisible();
-    await expect(actionLogTab).toHaveAttribute('aria-selected', 'true');
-
-    // Verify Access Log tab exists
-    await expect(page.getByRole('tab', { name: 'Access Log' })).toBeVisible();
-
-    // Verify Action Log heading
-    await expect(page.getByRole('heading', { name: 'Action Log', level: 2 })).toBeVisible();
-
-    // Verify the undoable only filter
-    await expect(page.getByRole('checkbox', { name: 'Undoable only' })).toBeVisible();
-
-    // Wait for table to load
-    await page.getByText('Loading data...').waitFor({ state: 'hidden', timeout: 10_000 }).catch(() => {});
-
-    // Verify table columns
-    await expect(page.getByRole('columnheader', { name: 'Action' })).toBeVisible();
-    await expect(page.getByRole('columnheader', { name: 'Resource' })).toBeVisible();
-    await expect(page.getByRole('columnheader', { name: 'User' })).toBeVisible();
-    await expect(page.getByRole('columnheader', { name: 'When' })).toBeVisible();
-    await expect(page.getByRole('columnheader', { name: 'Status' })).toBeVisible();
-
-    // Verify audit log entries exist (since we just logged in, there should be entries)
-    const rows = page.locator('table tbody tr');
-    await expect(rows.first()).toBeVisible();
-
-    // Switch to Access Log tab
-    await page.getByRole('tab', { name: 'Access Log' }).click();
-    await expect(page.getByRole('heading', { name: 'Access Log', level: 2 })).toBeVisible();
-  });
-});