Increase timeout for component registration tests and improve detail-view component assertions

Author: hotlong

packages/plugin-charts/src/index.test.ts

@@ -13,7 +13,7 @@ describe('Plugin Charts', () => {
   // Import all renderers to register them
   beforeAll(async () => {
     await import('./index');
-  });
+  }, 30000);
 
   describe('bar-chart component', () => {
     it('should be registered in ComponentRegistry', () => {

packages/plugin-charts/src/registration.test.tsx

@@ -6,15 +6,15 @@ import { ComponentRegistry } from '@object-ui/core';
 describe('Plugin Charts Registration', () => {
   beforeAll(async () => {
     await import('./index');
-  });
+  }, 30000);
 
   it('registers bar-chart component', () => {
     const config = ComponentRegistry.get('bar-chart');
     expect(config).toBeDefined();
   });
 
    it('registers chart component types', () => {
-    expect(ComponentRegistry.get('chart-renderer')).toBeDefined(); // Assuming base
+    expect(ComponentRegistry.get('chart')).toBeDefined(); // Assuming base
     // Verify aliases if they exist
     expect(ComponentRegistry.get('pie-chart')).toBeDefined();
     expect(ComponentRegistry.get('donut-chart')).toBeDefined();

packages/plugin-detail/src/registration.test.tsx

@@ -1,6 +1,4 @@
-import React from 'react';
-import { render, screen } from '@testing-library/react';
-import { describe, it, expect, vi, beforeAll } from 'vitest';
+import { describe, it, expect, beforeAll } from 'vitest';
 import { ComponentRegistry } from '@object-ui/core';
 
 describe('Plugin Detail Registration', () => {
@@ -9,27 +7,12 @@ describe('Plugin Detail Registration', () => {
   });
 
   it('registers detail-view component', () => {
-    const config = ComponentRegistry.get('detail-view');
-    console.log('DetailView Config Type:', typeof config);
-    console.log('DetailView Config Keys:', config ? Object.keys(config) : 'null');
-    console.log('DetailView Config Component:', (config as any)?.component);
+    // We must use getConfig to retrieve the metadata (label, category, etc.)
+    // .get() only returns the React Component.
+    const config = ComponentRegistry.getConfig('detail-view');
 
-    // If config IS the component (e.g. legacy mode?), we might need to handle it.
-    // But Registry.ts says it returns config object.
-    
-    // For now, let's see what we got.
-    if ((config as any)?.label) {
-        expect((config as any).label).toBe('Detail View');
-    } else {
-        // Fail with info
-        console.error('Config missing label:', config);
-        // expect(true).toBe(false); // verification step
-    }
-  });
-
-  it('registers related sub-components', () => {
-     // Assuming these might be registered in future or implicitly available
-     // If index.tsx exports them, they are available to import.
-     // If they are not registered in Registry, we don't test registry for them.
+    expect(config).toBeDefined();
+    expect(config?.label).toBe('Detail View');
+    expect(config?.category).toBe('Views');
   });
 });

packages/plugin-kanban/src/ObjectKanban.msw.test.tsx

@@ -75,7 +75,7 @@ describe('ObjectKanban with MSW', () => {
     // Initial state might show Skeleton, wait for data
     await waitFor(() => {
         expect(screen.getByText('Task 1')).toBeInTheDocument();
-    });
+    }, { timeout: 10000 });
 
     // Check classification
     // Task 1 (todo) and Task 3 (todo) should be in To Do column.