@object-ui/fields 17.5.0 → 17.6.0

package/src/__tests__/date-locale-channel.test.tsx

@@ -1,305 +0,0 @@
-/**
- * ObjectUI
- * Copyright (c) 2024-present ObjectStack Inc.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- */
-
-/**
- * objectui#4468 — every date branch threads the ACTIVE locale.
- *
- * ── The measured split ───────────────────────────────────────────────────
- * On a `zh` console the same task row rendered `逾期 6 天` in one column and
- * `In 3 days` in the next, and the datetime column read `8/11/2026 12:00 am`.
- * Not a missing translation: the field labels around it were fully Chinese.
- *
- * The cause is that the date renderers had TWO locale channels and only one
- * of them followed the user's language:
- *
- *   - the OVERDUE phrase resolves through `useFieldTranslate()` → the ACTIVE
- *     UI language → `逾期 6 天`;
- *   - every `Intl` branch (relative past, relative future, the >±7d absolute
- *     fallback) took its tag from `useLocalization().locale` — the TENANT's
- *     regional default (ADR-0053), which is `undefined` on any workspace that
- *     has not configured one. `Intl` with `undefined` reads the MACHINE's
- *     locale, so those branches rendered en-US next to the Chinese one;
- *   - `DateTimeCellRenderer` passed no tag at all, hence `8/11/2026 12:00 am`.
- *
- * The one resolver is `useDisplayLocale()` (tenant locale → active UI
- * language → `'en'`), which `@object-ui/i18n` already owns and whose own
- * docstring claimed `DateCellRenderer` "already formats from this channel" —
- * it did not. `'zh'` is a valid BCP-47 tag, so there is no `'zh'`→`zh-CN`
- * mapping anywhere: `useDisplayLocale()` IS the single mapping point.
- *
- * ── Directions ───────────────────────────────────────────────────────────
- * Reverting the renderers to `origin/main` and keeping this file turns every
- * `zh session` case RED and leaves every `en session` case GREEN. The `en`
- * cases are green on BOTH sides on purpose — that is the must-not-change
- * half: the English output has to be byte-identical after the change, since
- * `'en'` and the CI machine's `en-US` agree on all of these forms.
- *
- * `已逾期`/`Overdue Nd` is likewise green on both sides: it never used the
- * broken channel, and this file pins that the fix did not disturb it.
- */
-
-import { describe, it, expect, afterEach } from 'vitest';
-import React from 'react';
-import { render, screen, cleanup } from '@testing-library/react';
-import '@testing-library/jest-dom';
-import { I18nProvider, LocalizationProvider } from '@object-ui/i18n';
-import { DateCellRenderer, DateTimeCellRenderer } from '../index';
-import { DateField } from '../widgets/DateField';
-import { DateTimeField } from '../widgets/DateTimeField';
-import { FormulaField } from '../widgets/FormulaField';
-import { GridField } from '../widgets/GridField';
-
-/**
- * `n` days from today, pinned to local noon so a run that starts near
- * midnight can never land the value on the neighbouring day.
- */
-function daysFromNow(n: number): string {
-  const d = new Date();
-  d.setDate(d.getDate() + n);
-  d.setHours(12, 0, 0, 0);
-  return d.toISOString();
-}
-
-/**
- * A fixed instant, built from LOCAL parts so the rendered day/time is the
- * same in every timezone the suite might run in. This is the exact value
- * shape the card captured: `8/11/2026 12:00 am` on an en machine.
- */
-const FIXED_INSTANT = new Date(2026, 7, 11, 0, 0, 0).toISOString();
-
-/**
- * A session: the UI language the user picked, plus the tenant's regional
- * default (usually absent — the state the card was measured in).
- *
- * `persistLanguage={false}` keeps each case on its own language instead of
- * inheriting whatever the previous one wrote to `localStorage`.
- */
-function renderSession(
-  language: string,
-  node: React.ReactNode,
-  tenantLocale?: string,
-) {
-  return render(
-    <I18nProvider
-      config={{ defaultLanguage: language, detectBrowserLanguage: false }}
-      persistLanguage={false}
-    >
-      <LocalizationProvider value={{ locale: tenantLocale }}>{node}</LocalizationProvider>
-    </I18nProvider>,
-  );
-}
-
-const dateField = (name: string) => ({ type: 'date', name }) as any;
-
-afterEach(() => cleanup());
-
-describe('zh session — every date branch renders Chinese (objectui#4468)', () => {
-  it('future-relative: the branch the card caught as "In 3 days"', () => {
-    renderSession('zh', <DateCellRenderer value={daysFromNow(3)} field={dateField('end_date')} />);
-    expect(screen.getByText('3天后')).toBeInTheDocument();
-    expect(screen.queryByText('In 3 days')).not.toBeInTheDocument();
-  });
-
-  it('past-relative, non-due: the branch the card caught as "6 days ago"', () => {
-    renderSession('zh', <DateCellRenderer value={daysFromNow(-6)} field={dateField('start_date')} />);
-    expect(screen.getByText('6天前')).toBeInTheDocument();
-    expect(screen.queryByText('6 days ago')).not.toBeInTheDocument();
-  });
-
-  it('near-today: today / tomorrow / yesterday', () => {
-    renderSession('zh', <DateCellRenderer value={daysFromNow(0)} field={dateField('start_date')} />);
-    expect(screen.getByText('今天')).toBeInTheDocument();
-    cleanup();
-
-    renderSession('zh', <DateCellRenderer value={daysFromNow(1)} field={dateField('start_date')} />);
-    expect(screen.getByText('明天')).toBeInTheDocument();
-    cleanup();
-
-    renderSession('zh', <DateCellRenderer value={daysFromNow(-1)} field={dateField('start_date')} />);
-    expect(screen.getByText('昨天')).toBeInTheDocument();
-  });
-
-  it('absolute fallback beyond the ±7-day window', () => {
-    const value = daysFromNow(30);
-    // The formatter drops the year when it is the current one, so the
-    // expectation is built the same way rather than hardcoding a month that
-    // depends on the day this suite runs.
-    const d = new Date(value);
-    const sameYear = d.getFullYear() === new Date().getFullYear();
-    const expected = d.toLocaleDateString('zh', {
-      year: sameYear ? undefined : 'numeric',
-      month: 'short',
-      day: 'numeric',
-    });
-    const enForm = d.toLocaleDateString('en', {
-      year: sameYear ? undefined : 'numeric',
-      month: 'short',
-      day: 'numeric',
-    });
-
-    renderSession('zh', <DateCellRenderer value={value} field={dateField('start_date')} />);
-    expect(screen.getByText(expected)).toBeInTheDocument();
-    expect(screen.queryByText(enForm)).not.toBeInTheDocument();
-  });
-
-  it('absolute datetime: the branch the card caught as "8/11/2026 12:00 am"', () => {
-    const { container } = renderSession(
-      'zh',
-      <DateTimeCellRenderer value={FIXED_INSTANT} field={{ type: 'datetime', name: 'created_at' } as any} />,
-    );
-    expect(container.textContent).toContain('2026/8/11');
-    expect(container.textContent).toContain('上午12:00');
-    expect(container.textContent).not.toContain('8/11/2026');
-    expect(container.textContent).not.toContain('12:00 am');
-  });
-
-  it('read-only DateField / DateTimeField widgets', () => {
-    const { container } = renderSession(
-      'zh',
-      <DateField value={FIXED_INSTANT} onChange={() => {}} field={dateField('start_date')} readonly />,
-    );
-    expect(container.textContent).toContain('2026/8/11');
-    cleanup();
-
-    const dt = renderSession(
-      'zh',
-      <DateTimeField
-        value={FIXED_INSTANT}
-        onChange={() => {}}
-        field={{ type: 'datetime', name: 'created_at' } as any}
-        readonly
-      />,
-    );
-    expect(dt.container.textContent).toContain('2026/8/11');
-  });
-
-  /**
-   * The sub-grid (`GridField`) read-only face. Its cases live here rather than
-   * beside the rest of `widgets/GridField.test.tsx`: mounting an `I18nProvider`
-   * in that file changes what its later PROVIDER-LESS renders resolve, and one
-   * of them asserts a translated `aria-label`.
-   */
-  it('the sub-grid read-only table', () => {
-    const temporalField = {
-      columns: [
-        { field: 'merchant', label: 'Merchant', type: 'text' as const },
-        { field: 'incurred_on', label: 'Incurred On', type: 'date' as const },
-      ],
-    } as any;
-    renderSession(
-      'zh',
-      <GridField
-        value={[{ merchant: 'Chipotle', incurred_on: '2026-06-17T00:00:00.000Z' }]}
-        onChange={() => {}}
-        field={temporalField}
-        readonly
-      />,
-    );
-    const table = screen.getByTestId('line-items-readonly');
-    expect(table.textContent).toContain('2026/6/17');
-    expect(table.textContent).not.toContain('6/17/2026');
-  });
-
-  it('a formula field returning a date', () => {
-    const { container } = renderSession(
-      'zh',
-      <FormulaField
-        value={FIXED_INSTANT}
-        onChange={() => {}}
-        field={{ type: 'formula', name: 'computed_on', return_type: 'date' } as any}
-      />,
-    );
-    expect(container.textContent).toContain('2026/8/11');
-    expect(container.textContent).not.toContain('8/11/2026');
-  });
-});
-
-describe('en session — output is byte-identical (must-not-change)', () => {
-  it('future-relative', () => {
-    renderSession('en', <DateCellRenderer value={daysFromNow(3)} field={dateField('end_date')} />);
-    expect(screen.getByText('In 3 days')).toBeInTheDocument();
-  });
-
-  it('past-relative, non-due', () => {
-    renderSession('en', <DateCellRenderer value={daysFromNow(-6)} field={dateField('start_date')} />);
-    expect(screen.getByText('6 days ago')).toBeInTheDocument();
-  });
-
-  it('near-today', () => {
-    renderSession('en', <DateCellRenderer value={daysFromNow(0)} field={dateField('start_date')} />);
-    expect(screen.getByText('Today')).toBeInTheDocument();
-    cleanup();
-
-    renderSession('en', <DateCellRenderer value={daysFromNow(1)} field={dateField('start_date')} />);
-    expect(screen.getByText('Tomorrow')).toBeInTheDocument();
-    cleanup();
-
-    renderSession('en', <DateCellRenderer value={daysFromNow(-1)} field={dateField('start_date')} />);
-    expect(screen.getByText('Yesterday')).toBeInTheDocument();
-  });
-
-  it('absolute datetime', () => {
-    const { container } = renderSession(
-      'en',
-      <DateTimeCellRenderer value={FIXED_INSTANT} field={{ type: 'datetime', name: 'created_at' } as any} />,
-    );
-    expect(container.textContent).toContain('8/11/2026');
-    expect(container.textContent).toContain('12:00 am');
-  });
-
-  it('read-only DateField', () => {
-    const { container } = renderSession(
-      'en',
-      <DateField value={FIXED_INSTANT} onChange={() => {}} field={dateField('start_date')} readonly />,
-    );
-    expect(container.textContent).toContain('8/11/2026');
-  });
-});
-
-describe('the already-localized overdue branch is undisturbed (green both sides)', () => {
-  it('resolves through the translate fn, not through Intl — zh', () => {
-    renderSession('zh', <DateCellRenderer value={daysFromNow(-6)} field={dateField('due_date')} />);
-    const el = screen.getByText('逾期 6 天');
-    expect(el).toBeInTheDocument();
-    expect(el.className).toMatch(/text-red-600/);
-  });
-
-  it('resolves through the translate fn, not through Intl — en', () => {
-    renderSession('en', <DateCellRenderer value={daysFromNow(-6)} field={dateField('due_date')} />);
-    expect(screen.getByText('Overdue 6d')).toBeInTheDocument();
-  });
-});
-
-describe('channel precedence is unchanged (green both sides)', () => {
-  /**
-   * `useDisplayLocale()` puts the TENANT locale above the UI language: an org
-   * that configured `en` means it, even for a user reading Chinese chrome.
-   * This is the one case the old code got right, and it must survive.
-   */
-  it('an explicit tenant locale outranks the active UI language', () => {
-    renderSession('zh', <DateCellRenderer value={daysFromNow(3)} field={dateField('end_date')} />, 'en');
-    expect(screen.getByText('In 3 days')).toBeInTheDocument();
-    expect(screen.queryByText('3天后')).not.toBeInTheDocument();
-  });
-
-  /**
-   * The third step of the precedence — the provider-less `'en'` last resort —
-   * is pinned in `DateCellRenderer.test.tsx`, deliberately NOT here.
-   *
-   * It cannot be measured in this file: `useObjectTranslation()` outside a
-   * provider reports `i18n.language` from react-i18next's GLOBAL instance, and
-   * every `I18nProvider` mounted above leaves that global on the language it
-   * was given. So a provider-less render placed after these cases resolves
-   * `'zh'` — which is the state react-i18next is in, not the fallback under
-   * test. Written here it would assert a fact about test ordering.
-   *
-   * `DateCellRenderer.test.tsx` mounts no provider at all, so its
-   * `6 days ago` / `Overdue 6d` / `Tomorrow` cases ARE that pin, and they hold
-   * for the real reason.
-   */
-});

package/src/__tests__/field-carrier-sdui.test.tsx

@@ -1,169 +0,0 @@
-/**
- * ObjectUI
- * Copyright (c) 2024-present ObjectStack Inc.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- */
-
-/**
- * The SDUI path delivers field metadata on ONE key — objectui#3233.
- *
- * `SchemaRenderer` hands every registered component the authored node as
- * `schema`. That key is the universal SDUI contract (`element:*`, `page:*`,
- * `object-grid`, `ReportRenderer` …) and is NOT retired. What was retired is
- * `schema` on the *field-widget* contract: a widget used to receive the node
- * under `schema` here and the metadata under `field` from the form renderer,
- * which is why ~30 widgets resolved their config as `field || schema`.
- *
- * The translation now happens once, at the registration seam
- * (`withFieldCarrier`), so downstream `field` is the only carrier. That is only
- * correct if the node reaches the widget UNCHANGED — "swap the key, don't drop
- * the cargo". The first test proves it by OBJECT IDENTITY: a spy sits where the
- * registry entry sits, records the object `SchemaRenderer` passed as `schema`,
- * and the widget's `field` must be that exact object. (Identity against the
- * authored literal would be wrong to assert — `SchemaRenderer` shallow-copies
- * the node while evaluating expressions, so the object it delivers is its own.)
- *
- * The last test pins that the REAL registration path goes through the adapter
- * at all: an adapter nobody applied would leave every SDUI-hosted widget
- * reading `undefined`, which is exactly the silent failure this convergence is
- * accused of risking.
- *
- * The sibling half of the invariant — the form path through
- * `renderFieldComponent` — is pinned in `packages/components/src/renderers/
- * form/__tests__/form-field-carrier.test.tsx`.
- */
-
-import React, { Suspense } from 'react';
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import { render, screen, cleanup } from '@testing-library/react';
-import { ComponentRegistry } from '@object-ui/core';
-import { SchemaRenderer } from '@object-ui/react';
-
-import { withFieldCarrier } from '../withFieldCarrier';
-import { registerField } from '../index';
-// Module scope, not a hook: `registerField('boolean')` registers
-// `React.lazy(() => import('./widgets/BooleanField'))`, and under a saturated
-// transform pipeline a first `import()` can outlive RTL's 1000ms budget. The
-// specifier matches `fieldWidgetMap`'s exactly, so ESM's cache makes that
-// factory resolve immediately (AGENTS.md 测试纪律 / objectui#3010).
-import '../widgets/BooleanField';
-
-/**
- * Capture holder, not bare module variables: `react-hooks/globals` forbids
- * REASSIGNING an outer binding from a component body, which is what a probe
- * has to do. Mutating a property of a stable object is the repo's usual shape
- * for this (see plugin-detail's `RelatedList` probes).
- */
-const captured: { props: Record<string, any> | null; node: unknown } = {
-  props: null,
-  node: null,
-};
-
-function CarrierProbe(props: any) {
-  captured.props = props;
-  return <div data-testid="carrier-probe">{props.field?.label ?? 'NO FIELD'}</div>;
-}
-
-const AdaptedProbe = withFieldCarrier(CarrierProbe);
-
-/**
- * Sits exactly where the registry entry sits, so it sees the host's props
- * verbatim, then forwards them to the adapter under test. This is what makes
- * the identity assertion possible: `captured.node` and the widget's `field`
- * are observed within a single render pass.
- */
-function CarrierSpy(props: any) {
-  captured.node = props.schema;
-  return <AdaptedProbe {...props} />;
-}
-
-beforeEach(() => {
-  captured.props = null;
-  captured.node = null;
-  ComponentRegistry.register('carrierprobe', CarrierSpy, {
-    namespace: 'field',
-    skipFallback: true,
-  });
-});
-
-afterEach(() => {
-  cleanup();
-  vi.restoreAllMocks();
-});
-
-describe('SDUI path — the authored node arrives as `field` (objectui#3233)', () => {
-  it('hands the widget the very object it used to read through `schema`', () => {
-    const node = {
-      type: 'field:carrierprobe',
-      name: 'city',
-      label: 'City',
-      placeholder: 'Where?',
-      options: [{ label: 'Beijing', value: 'bj' }],
-    };
-
-    render(<SchemaRenderer schema={node as any} />);
-
-    expect(screen.getByTestId('carrier-probe')).toHaveTextContent('City');
-    expect(captured.props).not.toBeNull();
-    expect(captured.node).toBeTruthy();
-    // The payload-equivalence proof: same object, by reference. A
-    // reconstruction would still render "City" while quietly dropping whatever
-    // the copy did not know to carry.
-    expect(captured.props!.field).toBe(captured.node);
-    // …and it really is the whole authored node, not a narrowed projection.
-    expect(captured.props!.field.name).toBe('city');
-    expect(captured.props!.field.placeholder).toBe('Where?');
-    expect(captured.props!.field.options).toEqual([{ label: 'Beijing', value: 'bj' }]);
-  });
-
-  it('consumes `schema` rather than forwarding it to the widget', () => {
-    // Key ABSENCE, not `undefined`. Two reasons it must not be forwarded: a
-    // widget could resolve `field || schema` again (the second contract
-    // survives in practice while the type claims it is gone), and widgets
-    // spread their leftover props onto DOM nodes, where an object-valued
-    // `schema` attribute is a React warning.
-    render(
-      <SchemaRenderer schema={{ type: 'field:carrierprobe', name: 'city', label: 'City' } as any} />,
-    );
-
-    expect(captured.props).not.toBeNull();
-    expect('schema' in captured.props!).toBe(false);
-  });
-
-  it('lets an explicit `field` win when a host supplies both', () => {
-    // The form renderer passes `field` (resolved metadata) and never `schema`.
-    // If some host ever passed both, the resolved metadata is the better
-    // answer — the raw node is the fallback, not the override.
-    const node = { type: 'field:carrierprobe', name: 'from-node', label: 'From node' };
-    const metadata = { name: 'from-field', label: 'From field' };
-
-    render(<AdaptedProbe schema={node} field={metadata} />);
-
-    expect(captured.props!.field).toBe(metadata);
-  });
-
-  it('applies the adapter on the REAL registration path (`registerField`)', async () => {
-    // The seam is only worth anything if `registerField` goes through it.
-    // `BooleanField` reads `widget` / `name` off its metadata carrier and
-    // nowhere else: `widget: 'checkbox'` selects the Checkbox over the Switch,
-    // and `name` becomes the control's id. Both are node keys, so seeing them
-    // honoured here proves the node reached `field` end to end — through the
-    // real registry entry, the real SchemaRenderer, and the real widget.
-    registerField('boolean');
-
-    render(
-      <Suspense fallback={<div>loading</div>}>
-        <SchemaRenderer
-          schema={
-            { type: 'field:boolean', name: 'is_active', label: 'Active', widget: 'checkbox' } as any
-          }
-        />
-      </Suspense>,
-    );
-
-    const control = await screen.findByRole('checkbox');
-    expect(control).toHaveAttribute('id', 'is_active');
-  });
-});

package/src/__tests__/formatCurrency.centsDigit.test.tsx

@@ -1,158 +0,0 @@
-/**
- * ObjectUI
- * Copyright (c) 2024-present ObjectStack Inc.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- */
-
-/**
- * objectui#4332 — `formatCurrency` dropped a real cents digit.
- *
- * The symbol branch passed `minimumFractionDigits: 0` against a
- * wholeness-switched `maximumFractionDigits`, handing Intl the RANGE `[0, 2]`.
- * Intl then emits the shortest representation in range, so a genuine cents
- * value of `.50` printed as `.5`: `$1,234.5`, `$19.9`, `$0.5` — money on a
- * record page reading as a data error rather than a formatting one.
- *
- * The fix is one width for both bounds (`isWhole ? 0 : 2`), which also makes
- * the function's three branches agree: the no-currency branch already routed
- * through `formatNumber` (both bounds = the width), and the bad-currency
- * fallback already used `toFixed`. Only the symbol branch disagreed.
- *
- * What is pinned here, and why each half matters:
- *
- *  1. **Fractional amounts keep both cents digits** — the defect.
- *  2. **Whole amounts still drop `.00`** — the OTHER half of the wholeness
- *     switch, which the doc comment promises just as explicitly ("Salesforce
- *     convention: `$1,234.50` keeps cents; `$1,234` does not") and which
- *     objectui#4033 independently pinned through the renderer. Widening the
- *     minimum to a constant 2 would have "fixed" this card by breaking that
- *     one, so it is a control, not a decoration.
- *  3. **The branches that were already correct are unchanged** — no-currency,
- *     bad-currency fallback, and `formatCompactCurrency` (a different function
- *     with its own `maximumFractionDigits: 1` compact policy).
- */
-
-import { describe, it, expect } from 'vitest';
-import React from 'react';
-import { render, screen } from '@testing-library/react';
-import '@testing-library/jest-dom';
-import { LocalizationProvider } from '@object-ui/i18n';
-import {
-  formatCurrency,
-  formatCompactCurrency,
-  formatNumber,
-  CurrencyCellRenderer,
-} from '../index';
-
-describe('formatCurrency — a fractional amount keeps both cents digits (objectui#4332)', () => {
-  it("renders the card's fixture as the doc comment promises", () => {
-    expect(formatCurrency(1234.5, 'USD', 'en-US')).toBe('$1,234.50');
-  });
-
-  it.each([
-    [19.9, '$19.90'],
-    [0.5, '$0.50'],
-    [1234.5, '$1,234.50'],
-  ])('an amount ending in a zero cent digit keeps it: %s', (value, expected) => {
-    // The issue's impact list — every price whose cents end in 0 was truncated
-    // by one digit (`$19.9`, `$0.5`).
-    expect(formatCurrency(value as number, 'USD', 'en-US')).toBe(expected as string);
-  });
-
-  it('keeps the sign in front of a truncated-then-restored amount', () => {
-    expect(formatCurrency(-1234.5, 'USD', 'en-US')).toBe('-$1,234.50');
-  });
-
-  it('a fractional amount that ROUNDS to zero cents still shows them', () => {
-    // Sub-cent precision: `1234.001` is not whole, so it keeps cents and
-    // rounds into them. Before the fix the `[0, 2]` range rounded to `.00` and
-    // then trimmed it away, printing a bare `$1,234` for a value that is NOT
-    // the whole number that spelling claims. Deliberate: wholeness — not the
-    // rounded output — decides whether cents are shown.
-    expect(formatCurrency(1234.001, 'USD', 'en-US')).toBe('$1,234.00');
-  });
-
-  it('still caps at two digits (the maximum is unchanged)', () => {
-    expect(formatCurrency(1234.567, 'USD', 'en-US')).toBe('$1,234.57');
-    expect(formatCurrency(1234.56, 'USD', 'en-US')).toBe('$1,234.56');
-  });
-
-  it('follows the active locale while doing it (objectui#4033 is not disturbed)', () => {
-    // de-DE: dot grouping, comma decimal mark, TRAILING symbol separated by
-    // U+00A0. Matched loosely on the separator (written as an escape, never a
-    // raw byte) so an ICU spacing change cannot fail the digits this pins.
-    expect(formatCurrency(1234.5, 'EUR', 'de-DE')).toMatch(/^1\.234,50\s*€$/);
-  });
-});
-
-describe('formatCurrency — CONTROL: the whole-number half of the switch is untouched', () => {
-  it.each([
-    [1234, '$1,234'],
-    [5000000, '$5,000,000'],
-    [0, '$0'],
-  ])('a whole amount still drops `.00`: %s', (value, expected) => {
-    // The doc comment promises this as explicitly as it promises the cents,
-    // and objectui#4033 pinned it through `CurrencyCellRenderer`. A constant
-    // `minimumFractionDigits: 2` would turn all three of these into `.00`.
-    expect(formatCurrency(value as number, 'USD', 'en-US')).toBe(expected as string);
-  });
-
-  it('a non-finite amount is unaffected (it is not "whole")', () => {
-    expect(formatCurrency(Number.NaN, 'USD', 'en-US')).toBe('$NaN');
-  });
-});
-
-describe('formatCurrency — CONTROL: the branches that were already correct', () => {
-  it('the no-currency branch is unchanged — it never had the defect', () => {
-    // It routes through `formatNumber`, which sets BOTH bounds to the width it
-    // is given, so it already rendered `1,234.50` while the symbol branch
-    // rendered `$1,234.5`. That disagreement is what the fix removes.
-    expect(formatCurrency(1234.5, undefined, 'en-US')).toBe('1,234.50');
-    expect(formatCurrency(1234.5, undefined, 'en-US')).toBe(formatNumber(1234.5, 2, 'en-US'));
-    // A whole amount with no currency is still money and keeps its separators
-    // (objectui#4033: `formatNumber` passes no `scale`, so the ordinal
-    // no-grouping policy cannot fire here).
-    expect(formatCurrency(5000000, undefined, 'en-US')).toBe('5,000,000');
-  });
-
-  it('the bad-currency fallback is unchanged — it always used toFixed', () => {
-    // `Intl` throws `RangeError: Invalid currency code` for a non-ISO-4217
-    // code, and `formatDisplayNumber`'s locale retry re-throws it rather than
-    // swallowing it, so this lands in `formatCurrency`'s own catch.
-    expect(formatCurrency(1234.5, 'NOT_A_CODE', 'en-US')).toBe('NOT_A_CODE 1234.50');
-  });
-
-  it('formatCompactCurrency is a different function and keeps its own policy', () => {
-    // Compact notation caps at ONE fraction digit and strips a trailing `.0`;
-    // it never had the `[0, max]` range, and this card does not touch it.
-    expect(formatCompactCurrency(1234.5, 'USD', 'en-US')).toBe('$1.2K');
-    expect(formatCompactCurrency(150000, 'USD', 'en-US')).toBe('$150K');
-  });
-});
-
-describe('CurrencyCellRenderer — the user-visible surface (objectui#4332)', () => {
-  const renderCurrency = (value: unknown, field: Record<string, unknown>, locale?: string) =>
-    render(
-      <LocalizationProvider value={{ locale }}>
-        <CurrencyCellRenderer value={value as any} field={{ type: 'currency', ...field } as any} />
-      </LocalizationProvider>,
-    );
-
-  it('a grid cell shows the cents digit the amount actually has', () => {
-    renderCurrency(1234.5, { currency: 'USD' }, 'en-US');
-    expect(screen.getByText('$1,234.50')).toBeInTheDocument();
-  });
-
-  it('CONTROL: a declared `scale` is inert on the currency cell path', () => {
-    // Measured, not assumed: `CurrencyCellRenderer` reads `currency` and never
-    // `scale`, and `formatCurrency` takes no scale argument — on this path
-    // `scale` is neither a display width nor (money always groups) a grouping
-    // input. So a currency field declaring `scale: 0` renders exactly like one
-    // declaring nothing, before and after this card. Pinned so that a future
-    // change making `scale` load-bearing here has to say so out loud.
-    renderCurrency(1234.5, { currency: 'USD', scale: 0 }, 'en-US');
-    expect(screen.getByText('$1,234.50')).toBeInTheDocument();
-  });
-});