Merge pull request #949 from objectstack-ai/copilot/fix-ci-test-issue

Author: hotlong

apps/console/src/__tests__/RecordDetailEdit.test.tsx

@@ -89,7 +89,7 @@ describe('RecordDetailView — recordId handling', () => {
     });
 
     // findOne should be called with the FULL URL recordId (no stripping)
-    expect(ds.findOne).toHaveBeenCalledWith('contact', 'contact-1772350253615-4');
+    expect(ds.findOne).toHaveBeenCalledWith('contact', 'contact-1772350253615-4', undefined);
 
     // Click the Edit button
     const editButton = await screen.findByRole('button', { name: /edit/i });
@@ -115,7 +115,7 @@ describe('RecordDetailView — recordId handling', () => {
     });
 
     // findOne should be called with the original ID unchanged
-    expect(ds.findOne).toHaveBeenCalledWith('contact', 'plain-id-12345');
+    expect(ds.findOne).toHaveBeenCalledWith('contact', 'plain-id-12345', undefined);
 
     const editButton = await screen.findByRole('button', { name: /edit/i });
     await userEvent.click(editButton);

packages/plugin-dashboard/src/__tests__/ObjectPivotTable.test.tsx

@@ -77,7 +77,7 @@ describe('ObjectPivotTable', () => {
 
     await waitFor(() => {
       const errorEl = container.querySelector('[data-testid="pivot-error"]');
-      expect(errorEl).toBeDefined();
+      expect(errorEl).not.toBeNull();
     });
 
     expect(screen.getByText('Network error')).toBeDefined();