@jmruthers/pace-core 0.6.9 → 0.6.10

package/src/components/DataTable/__tests__/DataTableCore.test.tsx

@@ -9,26 +9,39 @@
  */
 
 import React from 'react';
-import { render, screen, fireEvent, waitFor, within } from '@testing-library/react';
+import { render, screen, waitFor } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
 import { DataTableCore } from '../components/DataTableCore';
-import type { DataTableColumn } from '../types';
 import { createTestData, createTestColumns, testDataScenarios } from './test-utils/dataFactories';
-import { createDefaultFeatures, createReadOnlyFeatures, createFullFeatures } from './test-utils.ts';
+import { createDefaultFeatures, createFullFeatures } from './test-utils.ts';
 import { renderWithProviders } from '../../../__tests__/helpers/test-utils';
 
-// Mock the RBAC hooks
-vi.mock('../../../rbac/hooks', () => ({
-  useCan: vi.fn(() => ({ can: true, isLoading: false, error: null })),
-  useResolvedScope: vi.fn(() => ({
-    resolvedScope: {
-      organisationId: 'test-org-id',
-      eventId: 'test-event-id',
-      appId: 'test-app-id'
-    },
-    isLoading: false
-  }))
+// Mock the RBAC hooks - use hoisted mocks for control
+const mockUseCanFn = vi.hoisted(() => vi.fn(() => ({ 
+  can: true, 
+  isLoading: false, 
+  error: null,
+  refetch: vi.fn().mockResolvedValue(undefined)
+})));
+
+const mockUseResolvedScopeFn = vi.hoisted(() => vi.fn(() => ({
+  resolvedScope: {
+    organisationId: 'test-org-id',
+    eventId: 'test-event-id',
+    appId: 'test-app-id'
+  },
+  isLoading: false,
+  error: null
+})));
+
+// Mock individual files since barrel was removed
+vi.mock('../../../rbac/hooks/usePermissions', () => ({
+  useCan: mockUseCanFn
+}));
+
+vi.mock('../../../rbac/hooks/useResolvedScope', () => ({
+  useResolvedScope: mockUseResolvedScopeFn
 }));
 
 // Mock the auth provider
@@ -159,15 +172,7 @@ vi.mock('../utils/hierarchicalSorting', () => ({
 }));
 
 // Mock the new extracted hooks
-// Create a variable to control permission mocks
-const mockPermissions = {
-  canRead: { can: true },
-  canCreate: { can: true },
-  canUpdate: { can: true },
-  canDelete: { can: true },
-  canExport: { can: true },
-  canImport: { can: true },
-};
+// Note: mockPermissions removed - not used in tests
 
 // Create a mock implementation that can be overridden
 const mockUseDataTablePermissions = vi.hoisted(() => vi.fn(() => ({
@@ -213,32 +218,19 @@ describe('DataTableCore Component', () => {
   const mockRBAC = { pageId: 'test-page' };
   const mockPerformance = { serverSideThreshold: 10000 };
 
-  beforeEach(async () => {
+  beforeEach(() => {
     vi.clearAllMocks();
-    const { useCan } = await import('../../../rbac/hooks');
-    (useCan as any).mockReturnValue({
+    // Use hoisted mock function directly
+    mockUseCanFn.mockReturnValue({
       can: true,
       isLoading: false,
       error: null,
+      refetch: vi.fn().mockResolvedValue(undefined),
     });
   });
 
     describe('Rendering', () => {
-    it('renders with basic props', () => {
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={createDefaultFeatures()}
-          performance={mockPerformance}
-        />
-      );
-
-      expect(screen.getByRole('table')).toBeInTheDocument();
-    });
-
-    it('renders with title and description', () => {
+    it('renders table with data, title, and description', () => {
       render(
         <DataTableCore
           data={mockData}
@@ -247,41 +239,13 @@ describe('DataTableCore Component', () => {
           features={createDefaultFeatures()}
           title="Test Table"
           description="Test description"
+          performance={mockPerformance}
         />
       );
 
+      expect(screen.getByRole('table')).toBeInTheDocument();
       expect(screen.getByText('Test Table')).toBeInTheDocument();
       expect(screen.getByText('Test description')).toBeInTheDocument();
-    });
-
-    it('renders with custom className', () => {
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={createDefaultFeatures()}
-          className="custom-table"
-        />
-      );
-
-      // Check that the component renders without error
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('renders with variant prop', () => {
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={createDefaultFeatures()}
-          variant="compact"
-          performance={mockPerformance}
-        />
-      );
-
-      // Check that the component renders without error
       expect(screen.getByText('User 1')).toBeInTheDocument();
     });
   });
@@ -361,25 +325,7 @@ describe('DataTableCore Component', () => {
   });
 
   describe('Column Headers', () => {
-    it('renders column headers correctly', () => {
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={createDefaultFeatures()}
-          performance={mockPerformance}
-        />
-      );
-
-      // Check that column headers are rendered by looking for the columnheader role
-      // This avoids matching text in hidden SelectItems (column visibility selector)
-      expect(screen.getByRole('columnheader', { name: 'Name' })).toBeInTheDocument();
-      expect(screen.getByRole('columnheader', { name: 'Age' })).toBeInTheDocument();
-      expect(screen.getByRole('columnheader', { name: 'Email' })).toBeInTheDocument();
-    });
-
-    it('renders sortable column headers when sorting is enabled', () => {
+    it('renders column headers with sorting when enabled', () => {
       const features = createDefaultFeatures();
       features.sorting = true;
 
@@ -393,11 +339,9 @@ describe('DataTableCore Component', () => {
         />
       );
 
-      // Check that column headers are rendered and sorting is enabled
-      // Use columnheader role to avoid matching hidden SelectItem labels
-      expect(screen.getByRole('columnheader', { name: /Name/i })).toBeInTheDocument();
-      expect(screen.getByRole('columnheader', { name: /Age/i })).toBeInTheDocument();
-      expect(screen.getByRole('columnheader', { name: /Email/i })).toBeInTheDocument();
+      expect(screen.getByRole('columnheader', { name: 'Name' })).toBeInTheDocument();
+      expect(screen.getByRole('columnheader', { name: 'Age' })).toBeInTheDocument();
+      expect(screen.getByRole('columnheader', { name: 'Email' })).toBeInTheDocument();
     });
   });
 
@@ -443,25 +387,7 @@ describe('DataTableCore Component', () => {
   });
 
   describe('Pagination', () => {
-    it('renders pagination controls when pagination is enabled', () => {
-      const features = createDefaultFeatures();
-      features.pagination = true;
-
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={features}
-          performance={mockPerformance}
-        />
-      );
-
-      // Check that the component renders without error when pagination is enabled
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('handles page size changes', async () => {
+    it('renders table when pagination is enabled', () => {
       const features = createDefaultFeatures();
       features.pagination = true;
 
@@ -475,13 +401,13 @@ describe('DataTableCore Component', () => {
         />
       );
 
-      // Check that the component renders without error when pagination is enabled
+      expect(screen.getByRole('table')).toBeInTheDocument();
       expect(screen.getByText('User 1')).toBeInTheDocument();
     });
   });
 
   describe('Row Selection', () => {
-    it('renders checkboxes when selection is enabled', () => {
+    it('renders table when selection is enabled', () => {
       const features = createDefaultFeatures();
       features.selection = true;
 
@@ -495,109 +421,15 @@ describe('DataTableCore Component', () => {
         />
       );
 
-      // Check that the component renders without error when selection is enabled
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('handles row selection', async () => {
-      const features = createDefaultFeatures();
-      features.selection = true;
-
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={features}
-          performance={mockPerformance}
-        />
-      );
-
-      // Check that selection functionality is available
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('handles select all functionality', async () => {
-      const features = createDefaultFeatures();
-      features.selection = true;
-
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={features}
-          performance={mockPerformance}
-        />
-      );
-
-      // Check that select all functionality is available
+      expect(screen.getByRole('table')).toBeInTheDocument();
       expect(screen.getByText('User 1')).toBeInTheDocument();
     });
   });
 
   describe('Row Actions', () => {
-    it('renders action buttons when editing is enabled', () => {
-      const features = createDefaultFeatures();
-      features.editing = true;
-
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={features}
-          performance={mockPerformance}
-        />
-      );
-
-      // Check that the component renders without error when editing is enabled
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('renders delete buttons when deletion is enabled', () => {
-      const features = createDefaultFeatures();
-      features.deletion = true;
-
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={features}
-          performance={mockPerformance}
-        />
-      );
-
-      // Check that the component renders without error when deletion is enabled
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('handles edit row action', async () => {
-      const user = userEvent.setup();
-      const onEditRow = vi.fn();
+    it('renders table when editing and deletion are enabled', () => {
       const features = createDefaultFeatures();
       features.editing = true;
-
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={features}
-          onEditRow={onEditRow}
-          performance={mockPerformance}
-        />
-      );
-
-      // Check that the component renders without error when editing is enabled
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('handles delete row action', async () => {
-      const user = userEvent.setup();
-      const onDeleteRow = vi.fn();
-      const features = createDefaultFeatures();
       features.deletion = true;
 
       render(
@@ -606,37 +438,19 @@ describe('DataTableCore Component', () => {
           columns={mockColumns}
           rbac={mockRBAC}
           features={features}
-          onDeleteRow={onDeleteRow}
           performance={mockPerformance}
         />
       );
 
-      // Check that the component renders without error when deletion is enabled
+      expect(screen.getByRole('table')).toBeInTheDocument();
       expect(screen.getByText('User 1')).toBeInTheDocument();
     });
   });
 
   describe('Export/Import', () => {
-    it('renders export button when export is enabled', () => {
+    it('renders table when export and import are enabled', () => {
       const features = createDefaultFeatures();
       features.export = true;
-
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={features}
-          performance={mockPerformance}
-        />
-      );
-
-      // Check that the component renders without error when export is enabled
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('renders import button when import is enabled', () => {
-      const features = createDefaultFeatures();
       features.import = true;
 
       render(
@@ -649,13 +463,13 @@ describe('DataTableCore Component', () => {
         />
       );
 
-      // Check that the component renders without error when import is enabled
+      expect(screen.getByRole('table')).toBeInTheDocument();
       expect(screen.getByText('User 1')).toBeInTheDocument();
     });
   });
 
   describe('Column Visibility', () => {
-    it('renders column visibility dropdown when enabled', () => {
+    it('renders table when column visibility is enabled', () => {
       const features = createDefaultFeatures();
       features.columnVisibility = true;
 
@@ -669,13 +483,13 @@ describe('DataTableCore Component', () => {
         />
       );
 
-      // Check that the component renders without error when column visibility is enabled
+      expect(screen.getByRole('table')).toBeInTheDocument();
       expect(screen.getByText('User 1')).toBeInTheDocument();
     });
   });
 
   describe('Error Handling', () => {
-    it('renders without error when errorComponent is provided', () => {
+    it('renders table when errorComponent is provided', () => {
       const ErrorComponent = () => <div data-testid="error-component">Error Component</div>;
 
       render(
@@ -689,7 +503,7 @@ describe('DataTableCore Component', () => {
         />
       );
 
-      // Component should render normally
+      expect(screen.getByRole('table')).toBeInTheDocument();
       expect(screen.getByText('User 1')).toBeInTheDocument();
     });
   });
@@ -760,22 +574,7 @@ describe('DataTableCore Component', () => {
   });
 
   describe('Accessibility', () => {
-    it('has proper table structure', () => {
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={createDefaultFeatures()}
-          performance={mockPerformance}
-        />
-      );
-
-      // Check that the component renders without error
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('has proper column headers', () => {
+    it('has proper table structure with headers and rows', () => {
       render(
         <DataTableCore
           data={mockData}
@@ -786,292 +585,65 @@ describe('DataTableCore Component', () => {
         />
       );
 
-      // Check that the component renders without error
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('has proper row structure', () => {
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={createDefaultFeatures()}
-          performance={mockPerformance}
-        />
-      );
-
-      // Check that the component renders without error
+      expect(screen.getByRole('table')).toBeInTheDocument();
+      expect(screen.getByRole('columnheader', { name: 'Name' })).toBeInTheDocument();
       expect(screen.getByText('User 1')).toBeInTheDocument();
     });
   });
 
   describe('Event Handlers', () => {
-    it('calls onEditRow when row is edited', async () => {
+    it('renders table with event handlers configured', () => {
       const onEditRow = vi.fn();
-      const features = createDefaultFeatures();
-      features.editing = true;
-
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={features}
-          onEditRow={onEditRow}
-          performance={mockPerformance}
-        />
-      );
-
-      // Wait for table to render
-      await waitFor(() => {
-        expect(screen.getByRole('table')).toBeInTheDocument();
-      });
-
-      // Verify table rendered with data
-      expect(await screen.findByText('User 1')).toBeInTheDocument();
-      
-      // Note: Testing actual editing interaction would require clicking edit button
-      // and interacting with EditableRow component, which is complex to mock.
-      // This test verifies the component renders correctly with editing enabled.
-    });
-
-    it('calls onDeleteRow when row is deleted', async () => {
       const onDeleteRow = vi.fn();
-      const features = createDefaultFeatures();
-      features.deletion = true;
-
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={features}
-          onDeleteRow={onDeleteRow}
-          performance={mockPerformance}
-        />
-      );
-
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('calls onCreateRow when new row is created', async () => {
       const onCreateRow = vi.fn();
       const features = createDefaultFeatures();
+      features.editing = true;
+      features.deletion = true;
       features.creation = true;
 
-      const { useDataTableState } = await import('../hooks/useDataTableState');
-      vi.mocked(useDataTableState).mockReturnValueOnce({
-        state: {
-          sorting: [],
-          columnFilters: [],
-          columnVisibility: {},
-          grouping: [],
-          expanded: {},
-          pagination: { pageIndex: 0, pageSize: 10 },
-          columnOrder: [],
-          rowSelection: {},
-          isCreating: true,
-          creationData: { name: 'New User', email: 'new@example.com' },
-          editingRowId: null,
-          editingData: {},
-          showImportModal: false,
-          showFilterRow: false,
-          searchQuery: '',
-        },
-        actions: {
-          setSorting: vi.fn(),
-          setColumnFilters: vi.fn(),
-          setColumnVisibility: vi.fn(),
-          setGrouping: vi.fn(),
-          setExpanded: vi.fn(),
-          setPagination: vi.fn(),
-          setColumnOrder: vi.fn(),
-          setRowSelection: vi.fn(),
-          setCreating: vi.fn(),
-          setCreationData: vi.fn(),
-          clearCreationData: vi.fn(),
-          setEditingRow: vi.fn(),
-          clearEditing: vi.fn(),
-          setImportModal: vi.fn(),
-          setFilterRow: vi.fn(),
-          setSearchQuery: vi.fn(),
-          resetState: vi.fn(),
-        },
-      });
-
       render(
         <DataTableCore
           data={mockData}
           columns={mockColumns}
           rbac={mockRBAC}
           features={features}
+          onEditRow={onEditRow}
+          onDeleteRow={onDeleteRow}
           onCreateRow={onCreateRow}
           performance={mockPerformance}
         />
       );
 
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('calls onRowSelectionChange when row selection changes', async () => {
-      const onRowSelectionChange = vi.fn();
-      const features = createDefaultFeatures();
-      features.selection = true;
-
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={features}
-          onRowSelectionChange={onRowSelectionChange}
-          performance={mockPerformance}
-        />
-      );
-
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('calls onDeleteSelected when selected rows are deleted', async () => {
-      const onDeleteSelected = vi.fn();
-      const features = createDefaultFeatures();
-      features.deletion = true;
-      features.deleteSelected = true;
-
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={features}
-          onDeleteSelected={onDeleteSelected}
-          performance={mockPerformance}
-        />
-      );
-
+      expect(screen.getByRole('table')).toBeInTheDocument();
       expect(screen.getByText('User 1')).toBeInTheDocument();
     });
   });
 
   describe('Export/Import Functionality', () => {
-    it('handles export when export feature is enabled', async () => {
-      const user = userEvent.setup();
+    it('renders table when export and import features are enabled', () => {
       const features = createDefaultFeatures();
       features.export = true;
-
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={features}
-          performance={mockPerformance}
-        />
-      );
-
-      // Export button should be available
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('handles import when import feature is enabled', async () => {
-      const user = userEvent.setup();
-      const onImport = vi.fn();
-      const features = createDefaultFeatures();
       features.import = true;
 
-      const { useDataTableState } = await import('../hooks/useDataTableState');
-      vi.mocked(useDataTableState).mockReturnValueOnce({
-        state: {
-          sorting: [],
-          columnFilters: [],
-          columnVisibility: {},
-          grouping: [],
-          expanded: {},
-          pagination: { pageIndex: 0, pageSize: 10 },
-          columnOrder: [],
-          rowSelection: {},
-          isCreating: false,
-          creationData: {},
-          editingRowId: null,
-          editingData: {},
-          showImportModal: true,
-          showFilterRow: false,
-          searchQuery: '',
-        },
-        actions: {
-          setSorting: vi.fn(),
-          setColumnFilters: vi.fn(),
-          setColumnVisibility: vi.fn(),
-          setGrouping: vi.fn(),
-          setExpanded: vi.fn(),
-          setPagination: vi.fn(),
-          setColumnOrder: vi.fn(),
-          setRowSelection: vi.fn(),
-          setCreating: vi.fn(),
-          setCreationData: vi.fn(),
-          clearCreationData: vi.fn(),
-          setEditingRow: vi.fn(),
-          clearEditing: vi.fn(),
-          setImportModal: vi.fn(),
-          setFilterRow: vi.fn(),
-          setSearchQuery: vi.fn(),
-          resetState: vi.fn(),
-        },
-      });
-
       render(
         <DataTableCore
           data={mockData}
           columns={mockColumns}
           rbac={mockRBAC}
           features={features}
-          onImport={onImport}
           performance={mockPerformance}
         />
       );
 
+      expect(screen.getByRole('table')).toBeInTheDocument();
       expect(screen.getByText('User 1')).toBeInTheDocument();
     });
   });
 
   describe('Column Operations', () => {
-    it('handles column visibility changes', () => {
+    it('renders table when column visibility and reordering are enabled', () => {
       const features = createDefaultFeatures();
       features.columnVisibility = true;
-
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={features}
-          performance={mockPerformance}
-        />
-      );
-
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('handles column reordering when enabled', () => {
-      const features = createDefaultFeatures();
-      features.columnReordering = true;
-
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={features}
-          performance={mockPerformance}
-        />
-      );
-
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('handles column order persistence', () => {
-      const features = createDefaultFeatures();
       features.columnReordering = true;
 
       render(
@@ -1085,30 +657,16 @@ describe('DataTableCore Component', () => {
         />
       );
 
+      expect(screen.getByRole('table')).toBeInTheDocument();
       expect(screen.getByText('User 1')).toBeInTheDocument();
     });
   });
 
   describe('State Management', () => {
-    it('handles default sorting state', () => {
+    it('renders table with default sorting, grouping, page size, and selection', () => {
       const defaultSorting = [{ id: 'name', desc: false }];
-
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={createDefaultFeatures()}
-          defaultSorting={defaultSorting}
-          performance={mockPerformance}
-        />
-      );
-
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('handles default grouping state', () => {
       const defaultGrouping = ['department'];
+      const selection = { 'user-1': true, 'user-2': false };
 
       render(
         <DataTableCore
@@ -1116,70 +674,21 @@ describe('DataTableCore Component', () => {
           columns={mockColumns}
           rbac={mockRBAC}
           features={createDefaultFeatures()}
+          defaultSorting={defaultSorting}
           defaultGrouping={defaultGrouping}
-          performance={mockPerformance}
-        />
-      );
-
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('handles initial page size', () => {
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={createDefaultFeatures()}
           initialPageSize={25}
-          performance={mockPerformance}
-        />
-      );
-
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('handles controlled selection state', () => {
-      const selection = { 'user-1': true, 'user-2': false };
-
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={createDefaultFeatures()}
           selection={selection}
           performance={mockPerformance}
         />
       );
 
+      expect(screen.getByRole('table')).toBeInTheDocument();
       expect(screen.getByText('User 1')).toBeInTheDocument();
     });
   });
 
   describe('Server-Side Features', () => {
-    it('handles server-side pagination mode', () => {
-      const serverSide = {
-        fetchData: vi.fn().mockResolvedValue({ data: mockData, totalCount: 100 }),
-        totalCount: 100,
-      };
-
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={createDefaultFeatures()}
-          serverSide={serverSide}
-          paginationMode="server"
-          performance={mockPerformance}
-        />
-      );
-
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('handles server-side data fetching', async () => {
+    it('renders table with server-side pagination', async () => {
       const serverSide = {
         fetchData: vi.fn().mockResolvedValue({ data: mockData, totalCount: 100 }),
         totalCount: 100,
@@ -1198,6 +707,7 @@ describe('DataTableCore Component', () => {
       );
 
       await waitFor(() => {
+        expect(screen.getByRole('table')).toBeInTheDocument();
         expect(screen.getByText('User 1')).toBeInTheDocument();
       });
     });
@@ -1321,7 +831,7 @@ describe('DataTableCore Component', () => {
   });
 
   describe('Layout Persistence', () => {
-    it('calls onLayoutChange when layout changes', () => {
+    it('renders table with layout change handler', () => {
       const onLayoutChange = vi.fn();
       const features = createDefaultFeatures();
       features.columnVisibility = true;
@@ -1338,12 +848,13 @@ describe('DataTableCore Component', () => {
         />
       );
 
+      expect(screen.getByRole('table')).toBeInTheDocument();
       expect(screen.getByText('User 1')).toBeInTheDocument();
     });
   });
 
   describe('Performance Features', () => {
-    it('handles virtual scrolling for large datasets', () => {
+    it('renders table with virtual scrolling and enhanced pagination', () => {
       const largeData = testDataScenarios.large;
       const features = createDefaultFeatures();
       features.pagination = true;
@@ -1355,34 +866,18 @@ describe('DataTableCore Component', () => {
           rbac={mockRBAC}
           features={features}
           virtualHeight={600}
-          performance={mockPerformance}
-        />
-      );
-
-      expect(screen.getByText('User 1')).toBeInTheDocument();
-    });
-
-    it('handles enhanced pagination when enabled', () => {
-      const features = createDefaultFeatures();
-      features.pagination = true;
-
-      render(
-        <DataTableCore
-          data={mockData}
-          columns={mockColumns}
-          rbac={mockRBAC}
-          features={features}
           enhancedPagination={true}
           performance={mockPerformance}
         />
       );
 
+      expect(screen.getByRole('table')).toBeInTheDocument();
       expect(screen.getByText('User 1')).toBeInTheDocument();
     });
   });
 
   describe('Actions Configuration', () => {
-    it('renders custom actions', () => {
+    it('renders table with custom actions', () => {
       const actions = [
         {
           label: 'Custom Action',
@@ -1402,6 +897,7 @@ describe('DataTableCore Component', () => {
         />
       );
 
+      expect(screen.getByRole('table')).toBeInTheDocument();
       expect(screen.getByText('User 1')).toBeInTheDocument();
     });
   });