fix(fields): await async data load in renderGrid slot test

The test asserted renderGrid was called with populated records
immediately, but the component renders initially with empty state
before the async fetch resolves. Changed to toHaveBeenLastCalledWith
inside waitFor to properly await the data-loaded render.

Agent-Logs-Url: https://github.com/objectstack-ai/objectui/sessions/bd086065-c65e-4f73-9b25-4fd3976d67e2

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Author: Copilot

CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+
+- **`@object-ui/fields`**: Fixed flaky `RecordPickerDialog` test that asserted
+  `renderGrid` slot props before async data fetch had completed. The test now
+  awaits the final render with populated records before asserting.
+
 ### Changed
 
 - **`MetadataProvider` now lazy-loads metadata.** Previously the console

packages/fields/src/record-picker.test.tsx

@@ -1113,29 +1113,36 @@ describe('RecordPickerDialog — renderGrid slot', () => {
       expect(screen.getByTestId('custom-grid')).toBeInTheDocument();
     });
 
-    // renderGrid should have been called with grid slot props
-    expect(renderGrid).toHaveBeenCalledWith(
-      expect.objectContaining({
-        columns: expect.arrayContaining([
-          expect.objectContaining({ field: 'name' }),
-        ]),
-        records: expect.arrayContaining([
-          expect.objectContaining({ id: '1', name: 'Acme Corp' }),
-        ]),
-        loading: false,
-        totalCount: 1,
-        currentPage: 1,
-        pageSize: 10,
-        sortField: null,
-        sortDirection: 'asc',
-        onSort: expect.any(Function),
-        onPageChange: expect.any(Function),
-        onRowClick: expect.any(Function),
-        isSelected: expect.any(Function),
-        multiple: false,
-        idField: 'id',
-      }),
-    );
+    // Wait for data to load, then check the last renderGrid call has records
+    await waitFor(() => {
+      expect(renderGrid).toHaveBeenLastCalledWith(
+        expect.objectContaining({
+          columns: expect.arrayContaining([
+            expect.objectContaining({ field: 'name' }),
+          ]),
+          records: expect.arrayContaining([
+            expect.objectContaining({ id: '1', name: 'Acme Corp' }),
+          ]),
+          loading: false,
+          totalCount: 1,
+        }),
+      );
+    });
+
+    // Verify all expected props are present in the last call
+    const lastCallArgs = renderGrid.mock.calls[renderGrid.mock.calls.length - 1][0];
+    expect(lastCallArgs).toMatchObject({
+      currentPage: 1,
+      pageSize: 10,
+      sortField: null,
+      sortDirection: 'asc',
+      multiple: false,
+      idField: 'id',
+    });
+    expect(lastCallArgs.onSort).toBeTypeOf('function');
+    expect(lastCallArgs.onPageChange).toBeTypeOf('function');
+    expect(lastCallArgs.onRowClick).toBeTypeOf('function');
+    expect(lastCallArgs.isSelected).toBeTypeOf('function');
   });
 
   it('hides built-in table when renderGrid is provided', async () => {