Increase timeout for waitFor in tests to improve reliability

Author: hotlong

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

@@ -143,7 +143,7 @@ describe('Console Application Simulation', () => {
         // Verify Object Header
         await waitFor(() => {
             expect(screen.getByRole('heading', { name: /Kitchen Sink/i })).toBeInTheDocument();
-        });
+        }, { timeout: 5000 });
 
         // Verify "New" Button exists
         const newButton = screen.getByRole('button', { name: /New/i });

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

@@ -80,7 +80,7 @@ describe('ObjectForm Field Type Mapping', () => {
         // Wait for schema to load and render
         await waitFor(() => {
             expect(screen.getByLabelText(/Text Field/i)).toBeInTheDocument();
-        });
+        }, { timeout: 3000 });
 
         // 1. Text Field -> Input[type=text]
         expect(container.querySelector('input[name="text_field"][type="text"]')).toBeInTheDocument();

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

@@ -1,5 +1,4 @@
 import { describe, it, expect, vi, beforeEach } from 'vitest';
-import React from 'react';
 import { render, screen, fireEvent } from '@testing-library/react';
 import '@testing-library/jest-dom';
 import { ObjectView } from '../components/ObjectView';
@@ -20,7 +19,7 @@ vi.mock('@object-ui/plugin-calendar', () => ({
 
 vi.mock('@object-ui/components', async (importOriginal) => {
     const React = await import('react');
-    const MockTabsContext = React.createContext({ onValueChange: (v: any) => {} });
+    const MockTabsContext = React.createContext({ onValueChange: ( _v: any) => {} });
     const actual = await importOriginal<any>();
     return {
         ...actual,

packages/plugin-form/src/ObjectForm.tsx

@@ -58,6 +58,7 @@ export const ObjectForm: React.FC<ObjectFormProps> = ({
   schema,
   dataSource,
 }) => {
+
   const [objectSchema, setObjectSchema] = useState<any>(null);
   const [formFields, setFormFields] = useState<FormField[]>([]);
   const [initialData, setInitialData] = useState<any>(null);

packages/plugin-grid/src/index.test.tsx

@@ -1,26 +1,29 @@
-import { describe, it, expect, vi } from 'vitest';
+import { vi, describe, it, expect, beforeEach } from 'vitest';
 import { render, screen } from '@testing-library/react';
 import React from 'react';
-import { ObjectGridRenderer } from './index';
 import { SchemaRendererProvider } from '@object-ui/react';
-
-// Mock dependencies
-vi.mock('./ObjectGrid', () => ({
-  ObjectGrid: ({ dataSource }: any) => (
-    <div data-testid="grid-mock">
-        {dataSource ? `DataSource: ${dataSource.type}` : 'No DataSource'}
-    </div>
-  )
-}));
+import * as ObjectGridModule from './ObjectGrid';
+import { ObjectGridRenderer } from './index';
 
 describe('Plugin Grid Registration', () => {
-  it('renderer passes dataSource from context', () => {
-    
+  it('renderer passes dataSource from context', async () => {
+    // Spy and mock implementation
+    vi.spyOn(ObjectGridModule, 'ObjectGrid').mockImplementation(
+      (({ dataSource }: any) => (
+        <div data-testid="grid-mock">
+            {dataSource ? `DataSource: ${dataSource.type}` : 'No DataSource'}
+        </div>
+      )) as any
+    );
+
     render(
       <SchemaRendererProvider dataSource={{ type: 'mock-datasource' } as any}>
         <ObjectGridRenderer schema={{ type: 'object-grid' }} />
       </SchemaRendererProvider>
     );
-    expect(screen.getByTestId('grid-mock')).toHaveTextContent('DataSource: mock-datasource');
+    
+    // Use findByTestId for async safety
+    const element = await screen.findByTestId('grid-mock');
+    expect(element).toHaveTextContent('DataSource: mock-datasource');
   });
 });