@opencorestack/opengridx 2.0.4 → 3.0.0

package/lib/components/DataGrid/DataGrid.v2.test.tsx

@@ -1,127 +0,0 @@
-import React from 'react';
-import { describe, it, expect, vi } from 'vitest';
-import { render, screen, fireEvent } from '@testing-library/react';
-import { DataGrid } from './DataGrid';
-import type { GridColDef } from '../../types';
-
-const COLUMNS: GridColDef[] = [
-    { field: 'id', headerName: 'ID', width: 80 },
-    { field: 'name', headerName: 'Name', width: 150 },
-    { field: 'dept', headerName: 'Dept', width: 120 },
-];
-
-const ROWS = [
-    { id: 1, name: 'Alice', dept: 'Eng' },
-    { id: 2, name: 'Bob', dept: 'HR' },
-];
-
-// ── density ──────────────────────────────────────────────────────────────────
-
-describe('DataGrid — density prop', () => {
-    it('renders without throwing for compact density', () => {
-        expect(() =>
-            render(<DataGrid rows={ROWS} columns={COLUMNS} density="compact" />)
-        ).not.toThrow();
-    });
-
-    it('renders without throwing for comfortable density', () => {
-        expect(() =>
-            render(<DataGrid rows={ROWS} columns={COLUMNS} density="comfortable" />)
-        ).not.toThrow();
-    });
-
-    it('applies compact row height via CSS variable', () => {
-        const { container } = render(
-            <DataGrid rows={ROWS} columns={COLUMNS} density="compact" />
-        );
-        const root = container.firstElementChild as HTMLElement;
-        expect(root.style.getPropertyValue('--ogx-row-height')).toBe('32px');
-    });
-
-    it('applies comfortable row height via CSS variable', () => {
-        const { container } = render(
-            <DataGrid rows={ROWS} columns={COLUMNS} density="comfortable" />
-        );
-        const root = container.firstElementChild as HTMLElement;
-        expect(root.style.getPropertyValue('--ogx-row-height')).toBe('72px');
-    });
-
-    it('uses rowHeight as default when density is standard', () => {
-        const { container } = render(
-            <DataGrid rows={ROWS} columns={COLUMNS} density="standard" rowHeight={52} />
-        );
-        const root = container.firstElementChild as HTMLElement;
-        expect(root.style.getPropertyValue('--ogx-row-height')).toBe('52px');
-    });
-});
-
-// ── disableRowSelectionOnClick ────────────────────────────────────────────────
-
-describe('DataGrid — disableRowSelectionOnClick', () => {
-    it('clicking a row changes selection when prop is false (default)', () => {
-        const onChange = vi.fn();
-        render(
-            <DataGrid
-                rows={ROWS}
-                columns={COLUMNS}
-                onRowSelectionModelChange={onChange}
-            />
-        );
-        const rows = screen.getAllByRole('row');
-        // rows[0] is header, rows[1] is Alice
-        fireEvent.click(rows[1]);
-        expect(onChange).toHaveBeenCalled();
-    });
-
-    it('clicking a row does NOT change selection when disableRowSelectionOnClick=true', () => {
-        const onChange = vi.fn();
-        render(
-            <DataGrid
-                rows={ROWS}
-                columns={COLUMNS}
-                disableRowSelectionOnClick
-                onRowSelectionModelChange={onChange}
-            />
-        );
-        const rows = screen.getAllByRole('row');
-        fireEvent.click(rows[1]);
-        expect(onChange).not.toHaveBeenCalled();
-    });
-});
-
-// ── disableMultipleRowSelection ───────────────────────────────────────────────
-
-describe('DataGrid — disableMultipleRowSelection', () => {
-    it('selecting a second row replaces the first when disableMultipleRowSelection=true', () => {
-        const selections: unknown[][] = [];
-        render(
-            <DataGrid
-                rows={ROWS}
-                columns={COLUMNS}
-                disableMultipleRowSelection
-                onRowSelectionModelChange={(model) => selections.push(model)}
-            />
-        );
-        const rows = screen.getAllByRole('row');
-        fireEvent.click(rows[1]); // select Alice (id=1)
-        fireEvent.click(rows[2]); // select Bob (id=2) — should replace
-        expect(selections.at(-1)).toEqual([2]);
-        expect(selections.at(-1)?.length).toBe(1);
-    });
-
-    it('clicking already-selected row deselects it when disableMultipleRowSelection=true', () => {
-        const selections: unknown[][] = [];
-        render(
-            <DataGrid
-                rows={ROWS}
-                columns={COLUMNS}
-                disableMultipleRowSelection
-                onRowSelectionModelChange={(model) => selections.push(model)}
-            />
-        );
-        const rows = screen.getAllByRole('row');
-        fireEvent.click(rows[1]); // select
-        fireEvent.click(rows[1]); // deselect
-        expect(selections.at(-1)).toEqual([]);
-    });
-});

package/lib/components/Header/Header.test.tsx

@@ -1,135 +0,0 @@
-import React from 'react';
-import { describe, it, expect, vi } from 'vitest';
-import { render, screen, fireEvent } from '@testing-library/react';
-import { Header } from './Header';
-import type { GridColDef } from '../../types';
-
-const COLUMNS: GridColDef[] = [
-    { field: 'name', headerName: 'Name', width: 150 },
-    { field: 'age', headerName: 'Age', width: 100 },
-    { field: 'dept', headerName: 'Dept', width: 120 },
-];
-
-describe('Header — multi-sort shift-click', () => {
-    it('plain click calls onSort with single field (replaces model)', () => {
-        const onSort = vi.fn();
-        const onSortAdd = vi.fn();
-        render(
-            <Header
-                columns={COLUMNS}
-                sortModel={[]}
-                onSort={onSort}
-                onSortAdd={onSortAdd}
-            />
-        );
-        fireEvent.click(screen.getByText('Name'));
-        expect(onSort).toHaveBeenCalledWith('name', 'asc');
-        expect(onSortAdd).not.toHaveBeenCalled();
-    });
-
-    it('shift-click calls onSortAdd (accumulates)', () => {
-        const onSort = vi.fn();
-        const onSortAdd = vi.fn();
-        render(
-            <Header
-                columns={COLUMNS}
-                sortModel={[{ field: 'name', sort: 'asc' }]}
-                onSort={onSort}
-                onSortAdd={onSortAdd}
-            />
-        );
-        fireEvent.click(screen.getByText('Age'), { shiftKey: true });
-        expect(onSortAdd).toHaveBeenCalledWith('age', 'asc');
-        expect(onSort).not.toHaveBeenCalled();
-    });
-
-    it('shift-click on already-sorted field cycles direction without replacing others', () => {
-        const onSort = vi.fn();
-        const onSortAdd = vi.fn();
-        render(
-            <Header
-                columns={COLUMNS}
-                sortModel={[
-                    { field: 'name', sort: 'asc' },
-                    { field: 'age', sort: 'asc' },
-                ]}
-                onSort={onSort}
-                onSortAdd={onSortAdd}
-            />
-        );
-        fireEvent.click(screen.getByText('Age'), { shiftKey: true });
-        expect(onSortAdd).toHaveBeenCalledWith('age', 'desc');
-        expect(onSort).not.toHaveBeenCalled();
-    });
-
-    it('shift-click on desc-sorted field removes it from model', () => {
-        const onSort = vi.fn();
-        const onSortAdd = vi.fn();
-        render(
-            <Header
-                columns={COLUMNS}
-                sortModel={[
-                    { field: 'name', sort: 'asc' },
-                    { field: 'age', sort: 'desc' },
-                ]}
-                onSort={onSort}
-                onSortAdd={onSortAdd}
-            />
-        );
-        fireEvent.click(screen.getByText('Age'), { shiftKey: true });
-        expect(onSortAdd).toHaveBeenCalledWith('age', null);
-        expect(onSort).not.toHaveBeenCalled();
-    });
-
-    it('plain click on one column while another is sorted resets to single-key', () => {
-        const onSort = vi.fn();
-        const onSortAdd = vi.fn();
-        render(
-            <Header
-                columns={COLUMNS}
-                sortModel={[{ field: 'name', sort: 'asc' }]}
-                onSort={onSort}
-                onSortAdd={onSortAdd}
-            />
-        );
-        fireEvent.click(screen.getByText('Age'));
-        expect(onSort).toHaveBeenCalledWith('age', 'asc');
-        expect(onSortAdd).not.toHaveBeenCalled();
-    });
-
-    it('does not call onSort/onSortAdd for sortable:false columns', () => {
-        const onSort = vi.fn();
-        const onSortAdd = vi.fn();
-        const cols: GridColDef[] = [
-            { field: 'id', headerName: 'ID', sortable: false },
-        ];
-        render(<Header columns={cols} sortModel={[]} onSort={onSort} onSortAdd={onSortAdd} />);
-        fireEvent.click(screen.getByText('ID'));
-        expect(onSort).not.toHaveBeenCalled();
-        expect(onSortAdd).not.toHaveBeenCalled();
-    });
-});
-
-describe('Header — description tooltip', () => {
-    it('renders title attribute from column description', () => {
-        const cols: GridColDef[] = [
-            { field: 'salary', headerName: 'Salary', description: 'Annual salary in USD' },
-        ];
-        const { container } = render(
-            <Header columns={cols} sortModel={[]} />
-        );
-        const headerCell = container.querySelector('[data-field="salary"]');
-        expect(headerCell?.getAttribute('title')).toBe('Annual salary in USD');
-    });
-
-    it('has no title attribute when description is absent', () => {
-        const cols: GridColDef[] = [
-            { field: 'name', headerName: 'Name' },
-        ];
-        const { container } = render(
-            <Header columns={cols} sortModel={[]} />
-        );
-        const headerCell = container.querySelector('[data-field="name"]');
-        expect(headerCell?.getAttribute('title')).toBeNull();
-    });
-});

package/lib/components/Pagination/Pagination.test.tsx

@@ -1,64 +0,0 @@
-import { describe, it, expect, vi } from 'vitest';
-import { render, screen } from '@testing-library/react';
-import { Pagination } from './Pagination';
-
-const NOOP = () => {};
-
-describe('Pagination', () => {
-    it('renders "0 of 0" when rowCount is 0', () => {
-        render(
-            <Pagination
-                page={0}
-                pageSize={10}
-                rowCount={0}
-                onPageChange={NOOP}
-                onPageSizeChange={NOOP}
-            />
-        );
-        expect(screen.getByText('0 of 0')).toBeTruthy();
-    });
-
-    it('calls localeText.paginationOf with (0, 0, 0) when rowCount is 0', () => {
-        const paginationOf = vi.fn().mockReturnValue('custom empty');
-        render(
-            <Pagination
-                page={0}
-                pageSize={10}
-                rowCount={0}
-                onPageChange={NOOP}
-                onPageSizeChange={NOOP}
-                localeText={{ paginationOf }}
-            />
-        );
-        expect(paginationOf).toHaveBeenCalledWith(0, 0, 0);
-        expect(screen.getByText('custom empty')).toBeTruthy();
-    });
-
-    it('renders correct row range label for non-empty data', () => {
-        render(
-            <Pagination
-                page={0}
-                pageSize={10}
-                rowCount={25}
-                onPageChange={NOOP}
-                onPageSizeChange={NOOP}
-            />
-        );
-        expect(screen.getByText('1–10 of 25')).toBeTruthy();
-    });
-
-    it('calls localeText.paginationOf with correct args for non-empty data', () => {
-        const paginationOf = vi.fn().mockReturnValue('custom range');
-        render(
-            <Pagination
-                page={1}
-                pageSize={10}
-                rowCount={25}
-                onPageChange={NOOP}
-                onPageSizeChange={NOOP}
-                localeText={{ paginationOf }}
-            />
-        );
-        expect(paginationOf).toHaveBeenCalledWith(11, 20, 25);
-    });
-});

package/lib/hooks/core/useDataGrid.regression.test.ts

@@ -1,66 +0,0 @@
-/**
- * Regression tests for bugs fixed in the useDataGrid reducer.
- *
- * Each describe block names the bug and the commit that fixed it, so a
- * future failure tells you exactly what regressed and where to look.
- */
-
-import { describe, it, expect } from 'vitest';
-import { renderHook, act } from '@testing-library/react';
-import { useDataGrid } from './useDataGrid';
-import type { GridColDef } from '../../types';
-
-const COLS: GridColDef[] = [
-    { field: 'id',   headerName: 'ID',   width: 80 },
-    { field: 'name', headerName: 'Name', width: 150 },
-    { field: 'age',  headerName: 'Age',  width: 100 },
-];
-
-const ROWS = [
-    { id: 1, name: 'Alice', age: 30 },
-    { id: 2, name: 'Bob',   age: 25 },
-];
-
-// ─────────────────────────────────────────────────────────────────────────────
-// Bug: SET_COLUMNS reducer was hardcoding columnVisibilityModel: {} on every
-// column update, silently wiping any hidden columns.
-// ─────────────────────────────────────────────────────────────────────────────
-describe('SET_COLUMNS preserves columnVisibilityModel', () => {
-    it('keeps hidden columns hidden after setColumns is called', () => {
-        const { result } = renderHook(() =>
-            useDataGrid({
-                rows: ROWS,
-                columns: COLS,
-                columnVisibilityModel: { age: false },
-            })
-        );
-
-        // Confirm the initial visibility model was applied.
-        expect(result.current.state.columns.columnVisibilityModel).toEqual({ age: false });
-
-        // Simulate a column update (e.g. a width change coming from a resize handler).
-        const updatedCols: GridColDef[] = COLS.map(c =>
-            c.field === 'name' ? { ...c, width: 200 } : c
-        );
-
-        act(() => {
-            result.current.setColumns(updatedCols);
-        });
-
-        // The visibility model must survive the column update — not reset to {}.
-        expect(result.current.state.columns.columnVisibilityModel).toEqual({ age: false });
-    });
-
-    it('does not gain a visibility entry for columns that were already visible', () => {
-        const { result } = renderHook(() =>
-            useDataGrid({ rows: ROWS, columns: COLS })
-        );
-
-        act(() => {
-            result.current.setColumns(COLS);
-        });
-
-        // No spurious entries: an empty model means all columns are visible.
-        expect(result.current.state.columns.columnVisibilityModel).toEqual({});
-    });
-});

package/lib/hooks/core/useGridControlledState.test.ts

@@ -1,86 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { renderHook, act } from '@testing-library/react';
-import { useGridControlledState } from './useGridControlledState';
-
-const BASE_PARAMS = {
-    sortModel: undefined,
-    paginationModel: undefined,
-    rowSelectionModel: undefined,
-};
-
-describe('useGridControlledState — sort', () => {
-    it('uncontrolled: setInternalSortModel updates sortModel', () => {
-        const { result } = renderHook(() => useGridControlledState(BASE_PARAMS));
-        expect(result.current.sortModel).toEqual([]);
-        act(() => {
-            result.current.setInternalSortModel([{ field: 'name', sort: 'asc' }]);
-        });
-        expect(result.current.sortModel).toEqual([{ field: 'name', sort: 'asc' }]);
-    });
-
-    it('controlled: external sortModel prop always wins', () => {
-        const externalSort = [{ field: 'age', sort: 'desc' as const }];
-        const { result } = renderHook(() =>
-            useGridControlledState({ ...BASE_PARAMS, sortModel: externalSort })
-        );
-        expect(result.current.isSortControlled).toBe(true);
-        expect(result.current.sortModel).toEqual(externalSort);
-        act(() => {
-            result.current.setInternalSortModel([{ field: 'name', sort: 'asc' }]);
-        });
-        // External prop still wins
-        expect(result.current.sortModel).toEqual(externalSort);
-    });
-
-    it('initialState.sorting seeds the internal sort model', () => {
-        const { result } = renderHook(() =>
-            useGridControlledState({
-                ...BASE_PARAMS,
-                initialState: { sorting: { sortModel: [{ field: 'name', sort: 'asc' }] } },
-            })
-        );
-        expect(result.current.sortModel).toEqual([{ field: 'name', sort: 'asc' }]);
-    });
-});
-
-describe('useGridControlledState — pagination', () => {
-    it('uncontrolled: handlePaginationModelChange updates effectivePaginationModel', () => {
-        const { result } = renderHook(() => useGridControlledState(BASE_PARAMS));
-        act(() => {
-            result.current.handlePaginationModelChange({ page: 2, pageSize: 25 });
-        });
-        expect(result.current.effectivePaginationModel).toEqual({ page: 2, pageSize: 25 });
-    });
-
-    it('controlled: external paginationModel always wins', () => {
-        const externalPagination = { page: 3, pageSize: 50 };
-        const { result } = renderHook(() =>
-            useGridControlledState({ ...BASE_PARAMS, paginationModel: externalPagination })
-        );
-        expect(result.current.effectivePaginationModel).toEqual(externalPagination);
-        act(() => {
-            result.current.handlePaginationModelChange({ page: 0, pageSize: 10 });
-        });
-        expect(result.current.effectivePaginationModel).toEqual(externalPagination);
-    });
-
-    it('initialState.pagination seeds page and pageSize', () => {
-        const { result } = renderHook(() =>
-            useGridControlledState({
-                ...BASE_PARAMS,
-                initialState: { pagination: { paginationModel: { page: 1, pageSize: 25 } } },
-            })
-        );
-        expect(result.current.effectivePaginationModel).toEqual({ page: 1, pageSize: 25 });
-    });
-});
-
-describe('useGridControlledState — selection', () => {
-    it('uncontrolled: setInternalRowSelectionModel updates selectedRowIds', () => {
-        const { result } = renderHook(() => useGridControlledState(BASE_PARAMS));
-        act(() => {
-            result.current.setInternalRowSelectionModel([1, 2, 3]);
-        });
-        expect(result.current.selectedRowIds).toEqual(new Set([1, 2, 3]));
-    });
-});

package/lib/hooks/core/useGridKeyboardNavigation.test.ts

@@ -1,176 +0,0 @@
-import { describe, it, expect, vi } from 'vitest';
-import { renderHook, act } from '@testing-library/react';
-import React from 'react';
-import { useGridKeyboardNavigation } from './useGridKeyboardNavigation';
-
-// Helper: build a minimal React.KeyboardEvent mock
-function keyEvent(key: string): React.KeyboardEvent {
-    return {
-        key,
-        preventDefault: vi.fn(),
-        stopPropagation: vi.fn(),
-        nativeEvent: new KeyboardEvent('keydown', { key }),
-        isDefaultPrevented: () => false,
-        isPropagationStopped: () => false,
-        persist: vi.fn(),
-        bubbles: true,
-        cancelable: true,
-        defaultPrevented: false,
-        eventPhase: 0,
-        isTrusted: false,
-        timeStamp: 0,
-        type: 'keydown',
-        currentTarget: null,
-        target: null,
-        shiftKey: false,
-        ctrlKey: false,
-        metaKey: false,
-        altKey: false,
-    } as unknown as React.KeyboardEvent;
-}
-
-const ROWS = [
-    { id: 1, name: 'Alice' },
-    { id: 2, name: 'Bob' },
-    { id: 3, name: 'Carol' },
-];
-
-const NAV_COLS = [
-    { field: 'name' },
-    { field: 'age' },
-];
-
-const MOCK_EDITING = {
-    editingCell: null,
-    startCellEdit: vi.fn(),
-    stopCellEdit: vi.fn(),
-};
-
-const MOCK_VIRTUALIZATION = {
-    cumulativeHeights: [52, 104, 156],
-    pinnedTopHeight: 0,
-    columnMetrics: null,
-};
-
-const BASE_PARAMS = {
-    allRenderableRows: ROWS,
-    navigationColumns: NAV_COLS,
-    checkboxSelection: false,
-    selectedRowIds: new Set<number | string>(),
-    handleSelectionChange: vi.fn(),
-    handleDetailPanelToggle: vi.fn(),
-    editingHandlers: MOCK_EDITING,
-    setKeyboardMode: vi.fn(),
-    sortModel: [],
-    handleSort: vi.fn(),
-    isCellEditable: undefined,
-    pagination: false,
-    pageSize: 100,
-    virtualization: MOCK_VIRTUALIZATION,
-    viewportRef: { current: null } as React.RefObject<HTMLDivElement | null>,
-};
-
-describe('useGridKeyboardNavigation — arrow keys', () => {
-    it('ArrowDown moves focusedCell to the next row', () => {
-        const { result } = renderHook(() => useGridKeyboardNavigation(BASE_PARAMS));
-        act(() => { result.current.setFocusedCell({ id: 1, field: 'name' }); });
-        act(() => { result.current.handleKeyDown(keyEvent('ArrowDown')); });
-        expect(result.current.focusedCell).toEqual({ id: 2, field: 'name' });
-    });
-
-    it('ArrowUp at header row does not move above it', () => {
-        const { result } = renderHook(() => useGridKeyboardNavigation(BASE_PARAMS));
-        // HEADER is the topmost position; ArrowUp from there should not move
-        act(() => { result.current.setFocusedCell({ id: 'HEADER', field: 'name' }); });
-        act(() => { result.current.handleKeyDown(keyEvent('ArrowUp')); });
-        expect(result.current.focusedCell?.id).toBe('HEADER');
-    });
-
-    it('ArrowDown at last row — no change', () => {
-        const { result } = renderHook(() => useGridKeyboardNavigation(BASE_PARAMS));
-        // ROWS has 3 entries; last row id is 3
-        const lastRow = ROWS[ROWS.length - 1];
-        act(() => { result.current.setFocusedCell({ id: lastRow.id, field: 'name' }); });
-        act(() => { result.current.handleKeyDown(keyEvent('ArrowDown')); });
-        expect(result.current.focusedCell?.id).toBe(lastRow.id);
-    });
-
-    it('ArrowRight moves focusedCell to next column', () => {
-        const { result } = renderHook(() => useGridKeyboardNavigation(BASE_PARAMS));
-        act(() => { result.current.setFocusedCell({ id: 1, field: 'name' }); });
-        act(() => { result.current.handleKeyDown(keyEvent('ArrowRight')); });
-        expect(result.current.focusedCell?.field).toBe('age');
-    });
-
-    it('ArrowLeft at header first column does not move left', () => {
-        const { result } = renderHook(() => useGridKeyboardNavigation(BASE_PARAMS));
-        // HEADER + first col is the leftmost position; wrap-left would require r=-2 which is out of bounds
-        act(() => { result.current.setFocusedCell({ id: 'HEADER', field: 'name' }); });
-        act(() => { result.current.handleKeyDown(keyEvent('ArrowLeft')); });
-        expect(result.current.focusedCell?.field).toBe('name');
-    });
-});
-
-describe('useGridKeyboardNavigation — Home / End', () => {
-    it('Home moves to first column', () => {
-        const { result } = renderHook(() => useGridKeyboardNavigation(BASE_PARAMS));
-        act(() => { result.current.setFocusedCell({ id: 1, field: 'age' }); });
-        act(() => { result.current.handleKeyDown(keyEvent('Home')); });
-        expect(result.current.focusedCell?.field).toBe('name');
-    });
-
-    it('End moves to last column', () => {
-        const { result } = renderHook(() => useGridKeyboardNavigation(BASE_PARAMS));
-        act(() => { result.current.setFocusedCell({ id: 1, field: 'name' }); });
-        act(() => { result.current.handleKeyDown(keyEvent('End')); });
-        expect(result.current.focusedCell?.field).toBe('age');
-    });
-});
-
-describe('useGridKeyboardNavigation — editing', () => {
-    it('Enter calls startCellEdit on an editable cell', () => {
-        const startCellEdit = vi.fn();
-        const params = {
-            ...BASE_PARAMS,
-            editingHandlers: { ...MOCK_EDITING, startCellEdit },
-            navigationColumns: [{ field: 'name', editable: true }],
-        };
-        const { result } = renderHook(() => useGridKeyboardNavigation(params));
-        act(() => { result.current.setFocusedCell({ id: 1, field: 'name' }); });
-        act(() => { result.current.handleKeyDown(keyEvent('Enter')); });
-        expect(startCellEdit).toHaveBeenCalledWith(
-            expect.objectContaining({ id: 1, field: 'name' })
-        );
-    });
-
-    it('Escape calls stopCellEdit with cancel=true when editing', () => {
-        const stopCellEdit = vi.fn();
-        const params = {
-            ...BASE_PARAMS,
-            editingHandlers: {
-                editingCell: { id: 1, field: 'name', value: 'Alice' },
-                startCellEdit: vi.fn(),
-                stopCellEdit,
-            },
-        };
-        const { result } = renderHook(() => useGridKeyboardNavigation(params));
-        act(() => { result.current.setFocusedCell({ id: 1, field: 'name' }); });
-        act(() => { result.current.handleKeyDown(keyEvent('Escape')); });
-        expect(stopCellEdit).toHaveBeenCalledWith({ cancel: true });
-    });
-});
-
-describe('useGridKeyboardNavigation — selection', () => {
-    it('Space toggles checkbox selection on focused row', () => {
-        const handleSelectionChange = vi.fn();
-        const { result } = renderHook(() => useGridKeyboardNavigation({
-            ...BASE_PARAMS,
-            checkboxSelection: true,
-            handleSelectionChange,
-        }));
-        // Space only fires handleSelectionChange when field === '__checkbox_col__'
-        act(() => { result.current.setFocusedCell({ id: 2, field: '__checkbox_col__' }); });
-        act(() => { result.current.handleKeyDown(keyEvent(' ')); });
-        expect(handleSelectionChange).toHaveBeenCalledWith(2, true);
-    });
-});