@object-ui/fields 17.5.0 → 17.7.0

package/src/widgets/GridField.test.tsx

@@ -1,632 +0,0 @@
-import { describe, it, expect, vi } from 'vitest';
-import { render, screen, fireEvent, waitFor } from '@testing-library/react';
-import React from 'react';
-import { UploadProvider } from '@object-ui/providers';
-import { GridField, LineItemsField, sumColumn, lookupAutofillPatch } from './GridField';
-
-const columns = [
-  { field: 'description', label: 'Description', type: 'text' as const },
-  { field: 'amount', label: 'Amount', type: 'currency' as const },
-];
-
-const field = { columns, total_field: 'amount' } as any;
-
-describe('GridField / LineItemsField — editable line items', () => {
-  it('is exported under both names', () => {
-    expect(LineItemsField).toBe(GridField);
-  });
-
-  it('renders a column header per config and an empty hint', () => {
-    render(<GridField value={[]} onChange={() => {}} field={field} />);
-    expect(screen.getByText('Description')).toBeTruthy();
-    expect(screen.getByText('Amount')).toBeTruthy();
-    expect(screen.getByRole('button', { name: /Add line/i })).toBeTruthy();
-  });
-
-  it('Add line appends a blank row keyed by columns', () => {
-    const onChange = vi.fn();
-    render(<GridField value={[]} onChange={onChange} field={field} />);
-    fireEvent.click(screen.getByRole('button', { name: /Add line/i }));
-    expect(onChange).toHaveBeenCalledWith([{ description: null, amount: null }]);
-  });
-
-  describe('column chooser (defaultHidden columns)', () => {
-    const withHidden = {
-      columns: [
-        { field: 'title', label: 'Title', type: 'text' as const, required: true },
-        { field: 'amount', label: 'Amount', type: 'currency' as const },
-        { field: 'notes', label: 'Notes', type: 'text' as const, defaultHidden: true },
-      ],
-    } as any;
-
-    it('hides defaultHidden columns by default but keeps required/visible ones', () => {
-      render(<GridField value={[]} onChange={() => {}} field={withHidden} />);
-      expect(screen.getByText('Title')).toBeTruthy();
-      expect(screen.getByText('Amount')).toBeTruthy();
-      expect(screen.queryByText('Notes')).toBeNull(); // collapsed into the chooser
-      expect(screen.getByTestId('line-items-columns')).toBeTruthy();
-    });
-
-    it('reveals an optional column when toggled in the chooser', () => {
-      render(<GridField value={[]} onChange={() => {}} field={withHidden} />);
-      fireEvent.click(screen.getByTestId('line-items-columns'));
-      fireEvent.click(screen.getByLabelText('Notes'));
-      expect(screen.getAllByText('Notes').length).toBeGreaterThan(0);
-    });
-
-    it('shows no chooser when there are no optional columns', () => {
-      render(<GridField value={[]} onChange={() => {}} field={field} />);
-      expect(screen.queryByTestId('line-items-columns')).toBeNull();
-    });
-  });
-
-  describe('list mode (displayMode="list" — form-factor for fat children)', () => {
-    const listField = {
-      columns: [
-        { field: 'title', label: 'Title', type: 'text' as const, required: true },
-        { field: 'status', label: 'Status', type: 'select' as const, options: [{ label: 'To Do', value: 'todo' }] },
-      ],
-    } as any;
-
-    it('renders rows read-only (no cell inputs) and an Add button', () => {
-      const onAdd = vi.fn();
-      render(
-        <GridField
-          value={[{ title: 'Ship it', status: 'todo' }]}
-          onChange={() => {}}
-          field={listField}
-          displayMode="list"
-          onRowExpand={() => {}}
-          onAdd={onAdd}
-        />,
-      );
-      // Read-only display: the status renders its option label, not a combobox.
-      expect(screen.getByText('To Do')).toBeTruthy();
-      expect(screen.queryByLabelText('Title')).toBeNull(); // no editable input
-      expect(screen.getByRole('button', { name: /Open row/i })).toBeTruthy(); // per-row edit
-    });
-
-    it('Add calls onAdd (host opens the full form) instead of inserting a blank row', () => {
-      const onAdd = vi.fn();
-      const onChange = vi.fn();
-      render(
-        <GridField value={[]} onChange={onChange} field={listField} displayMode="list" onRowExpand={() => {}} onAdd={onAdd} />,
-      );
-      fireEvent.click(screen.getByTestId('line-items-add'));
-      expect(onAdd).toHaveBeenCalledTimes(1);
-      expect(onChange).not.toHaveBeenCalled(); // did NOT insert a blank inline row
-    });
-  });
-
-  it('editing a text cell emits the raw string', () => {
-    const onChange = vi.fn();
-    render(<GridField value={[{ description: '', amount: null }]} onChange={onChange} field={field} />);
-    // [0] = the data row ([1] would be the always-present trailing ghost row).
-    fireEvent.change(screen.getAllByLabelText('Description')[0], { target: { value: 'Taxi' } });
-    expect(onChange).toHaveBeenCalledWith([{ description: 'Taxi', amount: null }]);
-  });
-
-  it('editing a currency cell coerces to a number', () => {
-    const onChange = vi.fn();
-    render(<GridField value={[{ description: 'Taxi', amount: null }]} onChange={onChange} field={field} />);
-    fireEvent.change(screen.getAllByLabelText('Amount')[0], { target: { value: '42.5' } });
-    expect(onChange).toHaveBeenCalledWith([{ description: 'Taxi', amount: 42.5 }]);
-  });
-
-  /**
-   * objectui#3566 — the sub-grid face of #3127. `<input type="date">` accepts
-   * exactly `YYYY-MM-DD` and rejects every other shape SILENTLY: the attribute
-   * still lands in the DOM, so the value looks present in the markup, while
-   * `.value` reads back `''` and the cell paints its empty placeholder. Since
-   * the API hands back `2026-06-17T00:00:00.000Z` for a `date` field, every
-   * date cell in an inline child grid rendered blank on a row that has a value.
-   * Asserting `.value` (never `getAttribute('value')`) is the only way to see
-   * this bug at all.
-   */
-  describe('date columns echo the stored value (#3566)', () => {
-    const dateField = {
-      columns: [
-        { field: 'description', label: 'Description', type: 'text' as const },
-        { field: 'incurred_on', label: 'Incurred On', type: 'date' as const },
-      ],
-    } as any;
-
-    // Both spellings the API may hand back for a `date` field. The bare form
-    // must survive VERBATIM — re-parsing it as UTC midnight and reading local
-    // calendar components back out would move it to the 16th anywhere west of
-    // Greenwich, which is the off-by-one this fix must not introduce.
-    it.each(['2026-06-17', '2026-06-17T00:00:00.000Z'])(
-      'shows the stored calendar day for %s instead of an empty cell',
-      (stored) => {
-        render(
-          <GridField value={[{ description: 'Taxi', incurred_on: stored }]} onChange={() => {}} field={dateField} />,
-        );
-        const cell = screen.getAllByLabelText('Incurred On')[0] as HTMLInputElement;
-        expect(cell.type).toBe('date');
-        expect(cell.value).toBe('2026-06-17');
-      },
-    );
-
-    it('leaves an empty date cell empty', () => {
-      render(<GridField value={[{ description: 'Taxi', incurred_on: null }]} onChange={() => {}} field={dateField} />);
-      expect((screen.getAllByLabelText('Incurred On')[0] as HTMLInputElement).value).toBe('');
-    });
-
-    it('writes back the control\'s own YYYY-MM-DD, unchanged by the read fix', () => {
-      // No paired conversion on the write side: `onChange` hands over exactly
-      // what `<input type="date">` produces, matching DateField's contract. A
-      // conversion here would put read and write on different bases.
-      const onChange = vi.fn();
-      render(
-        <GridField
-          value={[{ description: 'Taxi', incurred_on: '2026-06-17T00:00:00.000Z' }]}
-          onChange={onChange}
-          field={dateField}
-        />,
-      );
-      fireEvent.change(screen.getAllByLabelText('Incurred On')[0], { target: { value: '2026-06-18' } });
-      expect(onChange).toHaveBeenCalledWith([{ description: 'Taxi', incurred_on: '2026-06-18' }]);
-    });
-  });
-
-  /**
-   * objectui#3569 — the type collapse behind #3566's symptom. `datetime` and
-   * `time` used to be mapped onto the ONE `date` column type, so a datetime
-   * cell rendered `<input type="date">`. That is not merely an under-render:
-   * the control emits a bare `YYYY-MM-DD`, so a user who only re-picked the
-   * DAY silently wrote the record's time component out of existence. The
-   * falsifiable claim of these tests is therefore the TIME COMPONENT, not the
-   * control's markup.
-   *
-   * Local-wall-clock expectations are computed here from `Date` rather than
-   * hard-coded, so the suite asserts the same thing in every timezone (and the
-   * day-shift arithmetic is done in LOCAL parts, so it is DST-safe too).
-   */
-  describe('datetime / time columns get their own control (#3569)', () => {
-    const pad2 = (n: number) => String(n).padStart(2, '0');
-    /** The local wall clock an `<input type="datetime-local">` must show for `iso`. */
-    const localInputValue = (iso: string | Date) => {
-      const d = iso instanceof Date ? iso : new Date(iso);
-      return (
-        `${d.getFullYear()}-${pad2(d.getMonth() + 1)}-${pad2(d.getDate())}` +
-        `T${pad2(d.getHours())}:${pad2(d.getMinutes())}`
-      );
-    };
-
-    const temporalField = {
-      columns: [
-        { field: 'merchant', label: 'Merchant', type: 'text' as const },
-        { field: 'incurred_on', label: 'Incurred On', type: 'date' as const },
-        { field: 'incurred_at', label: 'Incurred At', type: 'datetime' as const },
-        { field: 'started_at', label: 'Started At', type: 'time' as const },
-      ],
-    } as any;
-
-    const STORED_ISO = '2026-06-17T14:30:00.000Z';
-
-    it('renders a datetime column as datetime-local, echoing the stored instant', () => {
-      render(
-        <GridField
-          value={[{ merchant: 'Chipotle', incurred_at: STORED_ISO }]}
-          onChange={() => {}}
-          field={temporalField}
-        />,
-      );
-      const cell = screen.getAllByLabelText('Incurred At')[0] as HTMLInputElement;
-      expect(cell.type).toBe('datetime-local');
-      expect(cell.value).toBe(localInputValue(STORED_ISO));
-    });
-
-    it('does NOT collapse the datetime column onto the date control', () => {
-      render(
-        <GridField
-          value={[{ merchant: 'Chipotle', incurred_on: '2026-06-17', incurred_at: STORED_ISO }]}
-          onChange={() => {}}
-          field={temporalField}
-        />,
-      );
-      // The sibling `date` column keeps its own control — the two are distinct
-      // now, which is exactly what was lost before.
-      expect((screen.getAllByLabelText('Incurred On')[0] as HTMLInputElement).type).toBe('date');
-      expect((screen.getAllByLabelText('Incurred At')[0] as HTMLInputElement).type).toBe('datetime-local');
-    });
-
-    /** THE regression: editing only the day must not destroy the time. */
-    it('keeps the time component when only the day is edited', () => {
-      const onChange = vi.fn();
-      render(
-        <GridField
-          value={[{ merchant: 'Chipotle', incurred_at: STORED_ISO }]}
-          onChange={onChange}
-          field={temporalField}
-        />,
-      );
-      const cell = screen.getAllByLabelText('Incurred At')[0] as HTMLInputElement;
-
-      // What a user does to correct a date: move the day forward one, leave
-      // the wall clock alone.
-      const before = new Date(STORED_ISO);
-      const nextDay = new Date(before);
-      nextDay.setDate(nextDay.getDate() + 1);
-      fireEvent.change(cell, { target: { value: localInputValue(nextDay) } });
-
-      const emitted = onChange.mock.calls[0][0][0].incurred_at;
-      const after = new Date(emitted);
-      expect(after.getHours()).toBe(before.getHours());
-      expect(after.getMinutes()).toBe(before.getMinutes());
-      // Round-tripped as ISO-8601 (`fromDateTimeInputValue`), NOT the control's
-      // naive zone-less string — read and write share one basis.
-      expect(emitted).toBe(nextDay.toISOString());
-    });
-
-    it('leaves an empty datetime cell empty', () => {
-      render(
-        <GridField value={[{ merchant: 'Chipotle', incurred_at: null }]} onChange={() => {}} field={temporalField} />,
-      );
-      expect((screen.getAllByLabelText('Incurred At')[0] as HTMLInputElement).value).toBe('');
-    });
-
-    it('renders a time column as a time control and round-trips HH:mm verbatim', () => {
-      const onChange = vi.fn();
-      render(
-        <GridField
-          value={[{ merchant: 'Chipotle', started_at: '14:30' }]}
-          onChange={onChange}
-          field={temporalField}
-        />,
-      );
-      const cell = screen.getAllByLabelText('Started At')[0] as HTMLInputElement;
-      expect(cell.type).toBe('time');
-      expect(cell.value).toBe('14:30');
-      // A `time` value is a zone-less wall clock — stored exactly as typed, the
-      // same contract `TimeField` follows. No conversion in either direction.
-      fireEvent.change(cell, { target: { value: '09:15' } });
-      expect(onChange).toHaveBeenCalledWith([{ merchant: 'Chipotle', started_at: '09:15' }]);
-    });
-
-    /**
-     * The read-only faces the issue calls out: `displayText()` and the
-     * read-only table both fell through to `String(value)`, printing the raw
-     * `2026-06-17T00:00:00.000Z` on screen. Neither could be fixed before the
-     * collapse was undone — with one column type the renderer had no way to
-     * know whether to show a day or a day plus a time.
-     */
-    describe('read-only display formats each temporal type as itself', () => {
-      const row = { merchant: 'Chipotle', incurred_on: '2026-06-17T00:00:00.000Z', incurred_at: STORED_ISO, started_at: '14:30' };
-      // Spelled `'en'` rather than bare `toLocaleDateString()` (objectui#4468):
-      // the bare call reads the MACHINE's locale, so these pins agreed with the
-      // widget only by the accident of a CI runner set to en-US — and they
-      // would have kept agreeing with it after the widget started following the
-      // session locale, which is the bug this suite has to be able to see.
-      // `'en'` is what `useDisplayLocale()` resolves to with no provider.
-      const expectedDay = new Date(2026, 5, 17).toLocaleDateString('en');
-      const dt = new Date(STORED_ISO);
-      const expectedInstant = `${dt.toLocaleDateString('en')} ${dt.toLocaleTimeString('en')}`;
-
-      it('formats date / datetime / time in the read-only table', () => {
-        render(<GridField value={[row]} onChange={() => {}} field={temporalField} readonly />);
-        const table = screen.getByTestId('line-items-readonly');
-        expect(table.textContent).toContain(expectedDay);
-        expect(table.textContent).toContain(expectedInstant);
-        expect(table.textContent).toContain('14:30');
-        // The raw stored ISO must never reach the screen.
-        expect(table.textContent).not.toContain('2026-06-17T00:00:00.000Z');
-        expect(table.textContent).not.toContain(STORED_ISO);
-      });
-
-      it('formats them the same way in list display mode', () => {
-        const { container } = render(
-          <GridField value={[row]} onChange={() => {}} field={temporalField} displayMode="list" onRowExpand={() => {}} />,
-        );
-        expect(container.textContent).toContain(expectedDay);
-        expect(container.textContent).toContain(expectedInstant);
-        expect(container.textContent).not.toContain(STORED_ISO);
-      });
-
-      it('shows a value it cannot parse rather than "Invalid Date"', () => {
-        render(
-          <GridField value={[{ merchant: 'X', incurred_at: 'not-a-date' }]} onChange={() => {}} field={temporalField} readonly />,
-        );
-        expect(screen.getByTestId('line-items-readonly').textContent).toContain('not-a-date');
-      });
-
-      // The `zh` half of this — the sub-grid following the SESSION locale
-      // (objectui#4468) — lives in `__tests__/date-locale-channel.test.tsx`,
-      // not here. Mounting an `I18nProvider` anywhere in THIS file changes what
-      // the provider-less renders further down resolve, and the file-columns
-      // chip test reads a translated `aria-label`; keeping the provider in its
-      // own file keeps that coupling out of the way.
-    });
-  });
-
-  describe('trailing ghost row (start-with-one + auto-append)', () => {
-    it('renders a trailing empty row so an empty grid still has one input line', () => {
-      render(<GridField value={[]} onChange={() => {}} field={field} />);
-      // No "No items" empty-state in grid mode — the ghost row IS the first line.
-      expect(screen.getByText('Description')).toBeTruthy();
-      expect(screen.getAllByLabelText('Description')).toHaveLength(1); // just the ghost
-    });
-
-    it('typing in the ghost row materialises a new row (no Add click needed)', () => {
-      const onChange = vi.fn();
-      render(<GridField value={[{ description: 'A', amount: 1 }]} onChange={onChange} field={field} />);
-      const inputs = screen.getAllByLabelText('Description');
-      expect(inputs).toHaveLength(2); // data row + ghost
-      fireEvent.change(inputs[1], { target: { value: 'B' } }); // type in the ghost
-      expect(onChange).toHaveBeenCalledWith([
-        { description: 'A', amount: 1 },
-        { description: 'B', amount: null },
-      ]);
-    });
-  });
-
-  describe('computed columns (amount = qty × unit_price)', () => {
-    const computedField = {
-      columns: [
-        { field: 'product', label: 'Product', type: 'text' as const },
-        { field: 'quantity', label: 'Qty', type: 'number' as const },
-        { field: 'unit_price', label: 'Unit Price', type: 'currency' as const },
-        { field: 'amount', label: 'Amount', type: 'currency' as const, computed: true, expr: 'record.quantity * record.unit_price', scale: 2 },
-      ],
-      total_field: 'amount',
-    } as any;
-
-    it('renders a computed column read-only (no input) and recomputes on edit', () => {
-      const onChange = vi.fn();
-      render(<GridField value={[{ product: 'Widget', quantity: 3, unit_price: 10, amount: 30 }]} onChange={onChange} field={computedField} />);
-      // Amount is display-only — there is no editable Amount cell.
-      expect(screen.queryByLabelText('Amount')).toBeNull();
-      // Editing quantity recomputes amount in the emitted row.
-      fireEvent.change(screen.getAllByLabelText('Qty')[0], { target: { value: '4' } });
-      expect(onChange).toHaveBeenCalledWith([{ product: 'Widget', quantity: 4, unit_price: 10, amount: 40 }]);
-    });
-
-    it('shows a dash for a computed cell whose inputs are blank', () => {
-      render(<GridField value={[{ product: 'Widget', quantity: null, unit_price: null, amount: null }]} onChange={() => {}} field={computedField} />);
-      // The computed amount cell reads "—" until its inputs exist.
-      expect(screen.getAllByText('—').length).toBeGreaterThan(0);
-    });
-  });
-
-  describe('keyboard navigation', () => {
-    it('Enter moves focus to the same column in the next row', () => {
-      render(<GridField value={[{ description: 'A', amount: 1 }, { description: 'B', amount: 2 }]} onChange={() => {}} field={field} />);
-      const row0 = screen.getAllByLabelText('Description')[0];
-      row0.focus();
-      fireEvent.keyDown(row0, { key: 'Enter' });
-      expect(document.activeElement).toBe(screen.getAllByLabelText('Description')[1]);
-    });
-  });
-
-  it('removing a row emits the array without it', () => {
-    const onChange = vi.fn();
-    render(
-      <GridField
-        value={[{ description: 'A', amount: 1 }, { description: 'B', amount: 2 }]}
-        onChange={onChange}
-        field={field}
-      />,
-    );
-    const removeButtons = screen.getAllByLabelText('Remove row');
-    fireEvent.click(removeButtons[0]);
-    expect(onChange).toHaveBeenCalledWith([{ description: 'B', amount: 2 }]);
-  });
-
-  it('row action buttons are always visible in grid mode (not hover-revealed)', () => {
-    render(
-      <GridField
-        value={[{ description: 'A', amount: 1 }]}
-        onChange={() => {}}
-        field={field}
-      />,
-    );
-    // Grid-mode rows previously gated these behind opacity-0/group-hover, so they
-    // were invisible until hover and unreachable on touch — they must always show.
-    expect(screen.getByLabelText('Remove row').className).not.toContain('opacity-0');
-    expect(screen.getByTestId('line-items-duplicate-0').className).not.toContain('opacity-0');
-  });
-
-  it('readonly mode shows values and a summed total footer', () => {
-    render(
-      <GridField
-        value={[{ description: 'A', amount: 10 }, { description: 'B', amount: 20 }]}
-        onChange={() => {}}
-        field={field}
-        readonly
-      />,
-    );
-    expect(screen.getByTestId('line-items-readonly')).toBeTruthy();
-    expect(screen.getByText('Total')).toBeTruthy();
-    expect(screen.getByText('30')).toBeTruthy();
-  });
-
-  describe('lookupAutofillPatch (item typeahead auto-fill)', () => {
-    const cols = [
-      { field: 'product', type: 'lookup' as const, reference: 'product' },
-      { field: 'description', type: 'text' as const },
-      { field: 'quantity', type: 'number' as const },
-      { field: 'unit_price', type: 'currency' as const },
-      { field: 'amount', type: 'currency' as const, computed: true, expr: 'record.quantity * record.unit_price' },
-    ];
-    const product = { value: 'p1', label: 'Widget A', name: 'Widget A', description: 'Standard widget', unit_price: 29.99, sku: 'WIDGET-A' };
-
-    it('sets the FK id and copies same-named sibling fields from the record', () => {
-      const patch = lookupAutofillPatch(cols, cols[0], product);
-      expect(patch).toEqual({ product: 'p1', description: 'Standard widget', unit_price: 29.99 });
-      // quantity (not on the record) and computed amount are left to the row/compute.
-      expect(patch).not.toHaveProperty('quantity');
-      expect(patch).not.toHaveProperty('amount');
-    });
-
-    it('copies only the FK id when autofill is disabled', () => {
-      const patch = lookupAutofillPatch(cols, { ...cols[0], autofill: false }, product);
-      expect(patch).toEqual({ product: 'p1' });
-    });
-  });
-
-  describe('P1 affordances (duplicate / validation)', () => {
-    it('duplicates a row (id stripped) directly below the original', () => {
-      const onChange = vi.fn();
-      render(<GridField value={[{ id: 'r1', description: 'A', amount: 5 }]} onChange={onChange} field={field} />);
-      fireEvent.click(screen.getByTestId('line-items-duplicate-0'));
-      expect(onChange).toHaveBeenCalledWith([
-        { id: 'r1', description: 'A', amount: 5 },
-        { description: 'A', amount: 5 }, // copy without the id → persists as a new record
-      ]);
-    });
-
-    it('flags a required, empty cell on a real row (not the ghost row)', () => {
-      const reqField = { columns: [{ field: 'description', label: 'Description', type: 'text' as const, required: true }] } as any;
-      render(<GridField value={[{ description: '' }]} onChange={() => {}} field={reqField} />);
-      // The data row's required-empty cell is flagged...
-      expect(screen.getByTestId('line-items-invalid-0-description')).toBeTruthy();
-      // ...but the trailing ghost row (index 1) is not.
-      expect(screen.queryByTestId('line-items-invalid-1-description')).toBeNull();
-    });
-  });
-
-  describe('file columns (upload in a grid cell — #2360)', () => {
-    const fileField = {
-      columns: [
-        { field: 'description', label: 'Description', type: 'text' as const },
-        { field: 'receipt', label: 'Receipt', type: 'file' as const },
-      ],
-    } as any;
-
-    it('renders a real upload control in the cell, not a text input', () => {
-      render(<GridField value={[{ description: 'Taxi', receipt: null }]} onChange={() => {}} field={fileField} />);
-      // One upload button per row (data row + ghost), backed by a native file input.
-      expect(screen.getAllByRole('button', { name: 'Receipt' }).length).toBeGreaterThan(0);
-      expect(document.querySelector('input[type="file"]')).toBeTruthy();
-      expect(screen.queryByRole('textbox', { name: 'Receipt' })).toBeNull();
-    });
-
-    it('uploads a picked file and writes the file object into the row', async () => {
-      const onChange = vi.fn();
-      const adapter = {
-        name: 'test',
-        upload: async (f: File) => ({ url: 'https://cdn/receipt.png', name: f.name, size: f.size, mimeType: f.type }),
-      };
-      render(
-        <UploadProvider adapter={adapter as any}>
-          <GridField value={[{ description: 'Taxi', receipt: null }]} onChange={onChange} field={fileField} />
-        </UploadProvider>,
-      );
-      const file = new File(['x'], 'receipt.png', { type: 'image/png' });
-      const input = document.querySelectorAll('input[type="file"]')[0] as HTMLInputElement;
-      fireEvent.change(input, { target: { files: [file] } });
-      await waitFor(() => expect(onChange).toHaveBeenCalled());
-      expect(onChange).toHaveBeenCalledWith([
-        {
-          description: 'Taxi',
-          receipt: {
-            name: 'receipt.png',
-            original_name: 'receipt.png',
-            size: file.size,
-            mime_type: 'image/png',
-            url: 'https://cdn/receipt.png',
-          },
-        },
-      ]);
-    });
-
-    it('shows an uploaded file as a removable chip', () => {
-      const onChange = vi.fn();
-      render(
-        <GridField
-          value={[{ description: 'Taxi', receipt: { name: 'receipt.png', mime_type: 'image/png' } }]}
-          onChange={onChange}
-          field={fileField}
-        />,
-      );
-      expect(screen.getByText('receipt.png')).toBeTruthy();
-      fireEvent.click(screen.getByLabelText('Remove receipt.png'));
-      expect(onChange).toHaveBeenCalledWith([{ description: 'Taxi', receipt: null }]);
-    });
-
-    it('passes the column accept list to the native picker', () => {
-      const withAccept = {
-        columns: [{ field: 'receipt', label: 'Receipt', type: 'file' as const, accept: ['image/*', '.pdf'] }],
-      } as any;
-      render(<GridField value={[]} onChange={() => {}} field={withAccept} />);
-      const input = document.querySelector('input[type="file"]') as HTMLInputElement;
-      expect(input.getAttribute('accept')).toBe('image/*,.pdf');
-    });
-
-    it('renders the file name read-only in readonly mode', () => {
-      render(
-        <GridField
-          value={[{ description: 'Taxi', receipt: { name: 'receipt.png' } }]}
-          onChange={() => {}}
-          field={fileField}
-          readonly
-        />,
-      );
-      expect(screen.getByText('receipt.png')).toBeTruthy();
-      expect(document.querySelector('input[type="file"]')).toBeNull();
-    });
-  });
-
-  it('sumColumn ignores blanks and NaN', () => {
-    expect(sumColumn([{ amount: 1 }, { amount: 2 }, { amount: null }], 'amount')).toBe(3);
-  });
-
-  describe('parent-scoped conditional rules (B2 follow-up — "paid invoice → lock lines")', () => {
-    const lockField = {
-      columns: [
-        { field: 'product', label: 'Product', type: 'text' as const },
-        { field: 'qty', label: 'Qty', type: 'number' as const, readonlyWhen: "parent.status == 'paid'" },
-        { field: 'unit_price', label: 'Unit Price', type: 'currency' as const, readonlyWhen: "parent.status == 'paid'" },
-      ],
-    } as any;
-
-    it('leaves cells editable when the parent rule is FALSE', () => {
-      render(
-        <GridField
-          value={[{ product: 'Widget', qty: 2, unit_price: 10 }]}
-          onChange={() => {}}
-          field={lockField}
-          contextRecord={{ status: 'draft' }}
-        />,
-      );
-      expect((screen.getAllByLabelText('Qty')[0] as HTMLInputElement).disabled).toBe(false);
-      expect((screen.getAllByLabelText('Unit Price')[0] as HTMLInputElement).disabled).toBe(false);
-    });
-
-    it('locks cells whose readonlyWhen references the parent header', () => {
-      render(
-        <GridField
-          value={[{ product: 'Widget', qty: 2, unit_price: 10 }]}
-          onChange={() => {}}
-          field={lockField}
-          contextRecord={{ status: 'paid' }}
-        />,
-      );
-      // The header is paid → quantity / unit price lock; product (no rule) stays editable.
-      expect((screen.getAllByLabelText('Qty')[0] as HTMLInputElement).disabled).toBe(true);
-      expect((screen.getAllByLabelText('Unit Price')[0] as HTMLInputElement).disabled).toBe(true);
-      expect((screen.getAllByLabelText('Product')[0] as HTMLInputElement).disabled).toBe(false);
-    });
-
-    it('re-evaluates per row, mixing the parent header with row data', () => {
-      const rowRule = {
-        columns: [
-          { field: 'qty', label: 'Qty', type: 'number' as const },
-          // Locks only when the header is paid AND this row is already invoiced.
-          { field: 'note', label: 'Note', type: 'text' as const, readonlyWhen: "parent.status == 'paid' && record.invoiced == true" },
-        ],
-      } as any;
-      render(
-        <GridField
-          value={[{ qty: 1, note: 'a', invoiced: true }, { qty: 2, note: 'b', invoiced: false }]}
-          onChange={() => {}}
-          field={rowRule}
-          contextRecord={{ status: 'paid' }}
-        />,
-      );
-      const notes = screen.getAllByLabelText('Note') as HTMLInputElement[];
-      expect(notes[0].disabled).toBe(true);  // invoiced row → locked
-      expect(notes[1].disabled).toBe(false); // not-yet-invoiced row → editable
-    });
-  });
-});