@aiready/components 0.14.4 → 0.14.5

package/src/components/__tests__/switch.test.tsx

@@ -1,85 +1,45 @@
-import { describe, it, expect, vi } from 'vitest';
-import { render, screen, fireEvent } from '@testing-library/react';
+import { describe, it, expect } from 'vitest';
+import { render, screen } from '@testing-library/react';
 import { Switch } from '../switch';
 
 describe('Switch', () => {
-  it('should render a switch', () => {
-    render(<Switch />);
-    expect(screen.getByRole('checkbox')).toBeInTheDocument();
+  it('renders switch', () => {
+    render(<Switch data-testid="switch" />);
+    expect(screen.getByTestId('switch')).toBeInTheDocument();
   });
 
-  it('should render with a label', () => {
-    render(<Switch label="Toggle me" />);
-    expect(screen.getByText('Toggle me')).toBeInTheDocument();
+  it('applies custom className', () => {
+    render(<Switch className="custom-switch" data-testid="switch" />);
+    expect(screen.getByTestId('switch')).toHaveClass('custom-switch');
   });
 
-  it('should not render label when not provided', () => {
-    const { container } = render(<Switch />);
-    expect(container.querySelector('span')).not.toBeInTheDocument();
+  it('renders as button role', () => {
+    render(<Switch data-testid="switch" />);
+    const switchEl = screen.getByTestId('switch');
+    expect(switchEl).toHaveAttribute('role', 'switch');
   });
 
-  it('should apply custom className', () => {
-    const { container } = render(<Switch className="custom-class" />);
-    expect(container.querySelector('.custom-class')).toBeInTheDocument();
+  it('has unchecked state by default', () => {
+    render(<Switch data-testid="switch" />);
+    const switchEl = screen.getByTestId('switch');
+    expect(switchEl).toHaveAttribute('aria-checked', 'false');
   });
 
-  it('should handle checked state', () => {
-    render(<Switch checked readOnly />);
-    expect(screen.getByRole('checkbox')).toBeChecked();
+  it('can be checked', () => {
+    render(<Switch checked data-testid="switch" />);
+    const switchEl = screen.getByTestId('switch');
+    expect(switchEl).toHaveAttribute('aria-checked', 'true');
   });
 
-  it('should handle unchecked state', () => {
-    render(<Switch checked={false} readOnly />);
-    expect(screen.getByRole('checkbox')).not.toBeChecked();
+  it('is disabled when disabled prop is true', () => {
+    render(<Switch disabled data-testid="switch" />);
+    const switchEl = screen.getByTestId('switch');
+    expect(switchEl).toBeDisabled();
   });
 
-  it('should call onChange when clicked', () => {
-    const handleChange = vi.fn();
-    render(<Switch onChange={handleChange} />);
-    fireEvent.click(screen.getByRole('checkbox'));
-    expect(handleChange).toHaveBeenCalledTimes(1);
-  });
-
-  it('should call onCheckedChange when clicked', () => {
-    const handleCheckedChange = vi.fn();
-    render(<Switch onCheckedChange={handleCheckedChange} />);
-    fireEvent.click(screen.getByRole('checkbox'));
-    expect(handleCheckedChange).toHaveBeenCalledTimes(1);
-    expect(handleCheckedChange).toHaveBeenCalledWith(true);
-  });
-
-  it('should call onCheckedChange with false when unchecking', () => {
-    const handleCheckedChange = vi.fn();
-    render(<Switch checked onCheckedChange={handleCheckedChange} />);
-    fireEvent.click(screen.getByRole('checkbox'));
-    expect(handleCheckedChange).toHaveBeenCalledWith(false);
-  });
-
-  it('should be disabled when disabled prop is set', () => {
-    render(<Switch disabled />);
-    expect(screen.getByRole('checkbox')).toBeDisabled();
-  });
-
-  it('should forward ref', () => {
-    const ref = { current: null };
-    render(<Switch ref={ref} />);
-    expect(ref.current).not.toBeNull();
-  });
-
-  it('should use provided id', () => {
-    render(<Switch id="custom-id" label="Test" />);
-    const checkbox = screen.getByRole('checkbox');
-    expect(checkbox).toHaveAttribute('id', 'custom-id');
-  });
-
-  it('should generate unique id when not provided', () => {
-    render(<Switch label="Test" />);
-    const checkbox = screen.getByRole('checkbox');
-    expect(checkbox).toHaveAttribute('id');
-  });
-
-  it('should have sr-only class on input', () => {
-    render(<Switch />);
-    expect(screen.getByRole('checkbox')).toHaveClass('sr-only');
+  it('has default styling', () => {
+    render(<Switch data-testid="switch" />);
+    const switchEl = screen.getByTestId('switch');
+    expect(switchEl).toHaveClass('peer', 'inline-flex');
   });
 });

package/src/components/__tests__/textarea.test.tsx

@@ -1,96 +1,48 @@
-import { describe, it, expect, vi } from 'vitest';
-import { render, screen, fireEvent } from '@testing-library/react';
+import { describe, it, expect } from 'vitest';
+import { render, screen } from '@testing-library/react';
 import { Textarea } from '../textarea';
 
 describe('Textarea', () => {
-  it('should render a textarea element', () => {
-    render(<Textarea />);
-    expect(screen.getByRole('textbox')).toBeInTheDocument();
+  it('renders textarea', () => {
+    render(<Textarea data-testid="textarea" />);
+    expect(screen.getByTestId('textarea')).toBeInTheDocument();
   });
 
-  it('should apply custom className', () => {
-    const { container } = render(<Textarea className="custom-class" />);
-    expect(container.firstChild).toHaveClass('custom-class');
+  it('applies custom className', () => {
+    render(<Textarea className="custom-textarea" data-testid="textarea" />);
+    expect(screen.getByTestId('textarea')).toHaveClass('custom-textarea');
   });
 
-  it('should handle placeholder text', () => {
-    render(<Textarea placeholder="Enter text" />);
-    expect(screen.getByPlaceholderText('Enter text')).toBeInTheDocument();
-  });
-
-  it('should forward ref', () => {
-    const ref = { current: null };
-    render(<Textarea ref={ref} />);
-    expect(ref.current).not.toBeNull();
-  });
-
-  it('should be disabled when disabled prop is set', () => {
-    render(<Textarea disabled />);
-    expect(screen.getByRole('textbox')).toBeDisabled();
-  });
-
-  it('should handle value prop', () => {
-    render(<Textarea value="test value" readOnly />);
-    expect(screen.getByRole('textbox')).toHaveValue('test value');
-  });
-
-  it('should spread additional props', () => {
-    const { container } = render(<Textarea data-testid="textarea" />);
-    expect(container.firstChild).toHaveAttribute('data-testid', 'textarea');
-  });
-
-  it('should call onChange when text is entered', () => {
-    const handleChange = vi.fn();
-    render(<Textarea onChange={handleChange} />);
-    fireEvent.change(screen.getByRole('textbox'), {
-      target: { value: 'new text' },
-    });
-    expect(handleChange).toHaveBeenCalledTimes(1);
+  it('renders as textarea element', () => {
+    render(<Textarea data-testid="textarea" />);
+    const textarea = screen.getByTestId('textarea');
+    expect(textarea.tagName).toBe('TEXTAREA');
   });
 
-  it('should have default styling classes', () => {
-    const { container } = render(<Textarea />);
-    expect(container.firstChild).toHaveClass(
-      'flex',
-      'min-h-[80px]',
-      'w-full',
-      'rounded-md'
-    );
-  });
-
-  it('should render as a textarea element', () => {
-    render(<Textarea />);
-    expect(screen.getByRole('textbox').tagName).toBe('TEXTAREA');
-  });
-
-  it('should support rows attribute', () => {
-    render(<Textarea rows={5} />);
-    expect(screen.getByRole('textbox')).toHaveAttribute('rows', '5');
+  it('accepts placeholder', () => {
+    render(<Textarea placeholder="Enter text" />);
+    expect(screen.getByPlaceholderText('Enter text')).toBeInTheDocument();
   });
 
-  it('should support maxLength attribute', () => {
-    render(<Textarea maxLength={100} />);
-    expect(screen.getByRole('textbox')).toHaveAttribute('maxlength', '100');
+  it('is disabled when disabled prop is true', () => {
+    render(<Textarea disabled data-testid="textarea" />);
+    expect(screen.getByTestId('textarea')).toBeDisabled();
   });
 
-  it('should have border-input class', () => {
-    const { container } = render(<Textarea />);
-    expect(container.firstChild).toHaveClass('border-input');
+  it('forwards ref correctly', () => {
+    const ref = { current: null };
+    render(<Textarea ref={ref} />);
+    expect(ref.current).toBeInstanceOf(HTMLTextAreaElement);
   });
 
-  it('should have focus ring classes', () => {
-    const { container } = render(<Textarea />);
-    expect(container.firstChild).toHaveClass(
-      'focus-visible:ring-2',
-      'focus-visible:ring-ring'
-    );
+  it('has default styling', () => {
+    render(<Textarea data-testid="textarea" />);
+    const textarea = screen.getByTestId('textarea');
+    expect(textarea).toHaveClass('flex', 'min-h-[80px]', 'w-full');
   });
 
-  it('should have disabled styling classes', () => {
-    const { container } = render(<Textarea disabled />);
-    expect(container.firstChild).toHaveClass(
-      'disabled:cursor-not-allowed',
-      'disabled:opacity-50'
-    );
+  it('accepts rows prop', () => {
+    render(<Textarea rows={5} data-testid="textarea" />);
+    expect(screen.getByTestId('textarea')).toHaveAttribute('rows', '5');
   });
 });

package/src/hooks/useForceSimulation.ts

@@ -89,7 +89,7 @@ export function useForceSimulation(
         typeof l.target === 'string'
           ? l.target
           : (l.target as SimulationNode)?.id;
-      const linkType = (l as any).type || '';
+      const linkType = (l as { type?: string }).type || '';
       return `${sourceId}->${targetId}:${linkType}`;
     })
     .join('|');
@@ -156,7 +156,7 @@ export function useForceSimulation(
       const linkForce = d3
         .forceLink<SimulationNode, SimulationLink>(linksCopy)
         .id((d) => d.id)
-        .distance((d) => (d as any).distance ?? linkDistance)
+        .distance((d) => (d as { distance?: number }).distance ?? linkDistance)
         .strength(linkStrength);
 
       simulation