From 535717e34f4e3b202b4161c182ed34493dcf12c2 Mon Sep 17 00:00:00 2001 From: Ajax Davis Date: Thu, 27 Nov 2025 02:17:51 +1000 Subject: [PATCH] fix(ui): add RadioGroup to build and fix TypeScript errors in tests MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Add RadioGroup.tsx to tsup entry points (was missing from build) - Fix HTMLTextareaElement casing in Textarea tests (should be HTMLTextAreaElement) - Remove unused variables in test files (user, container) Fixes CI type-check and build failures. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- packages/ui/src/Checkbox/Checkbox.test.tsx | 1 - packages/ui/src/Radio/Radio.test.tsx | 4 ++-- packages/ui/src/Slider/Slider.test.tsx | 3 +-- packages/ui/src/Textarea/Textarea.test.tsx | 6 +++--- packages/ui/tsup.config.ts | 3 +++ 5 files changed, 9 insertions(+), 8 deletions(-) diff --git a/packages/ui/src/Checkbox/Checkbox.test.tsx b/packages/ui/src/Checkbox/Checkbox.test.tsx index 108e869..efa3ed6 100644 --- a/packages/ui/src/Checkbox/Checkbox.test.tsx +++ b/packages/ui/src/Checkbox/Checkbox.test.tsx @@ -311,7 +311,6 @@ describe('Checkbox', () => { }); it('should call onBlur when blurred', async () => { - const user = userEvent.setup(); const handleBlur = vi.fn(); render(); diff --git a/packages/ui/src/Radio/Radio.test.tsx b/packages/ui/src/Radio/Radio.test.tsx index fb635c3..46c7fa8 100644 --- a/packages/ui/src/Radio/Radio.test.tsx +++ b/packages/ui/src/Radio/Radio.test.tsx @@ -178,7 +178,7 @@ describe('Radio', () => { describe('Label Position', () => { it('should position label on the right by default', () => { - const { container } = render( + render( @@ -189,7 +189,7 @@ describe('Radio', () => { }); it('should position label on the left when specified', () => { - const { container } = render( + render( diff --git a/packages/ui/src/Slider/Slider.test.tsx b/packages/ui/src/Slider/Slider.test.tsx index 8b27903..98ac596 100644 --- a/packages/ui/src/Slider/Slider.test.tsx +++ b/packages/ui/src/Slider/Slider.test.tsx @@ -139,7 +139,7 @@ describe('Slider', () => { { value: 50, label: 'Mid' }, { value: 100, label: 'Max' }, ]; - const { container } = render(); + render(); expect(screen.getByText('Min')).toBeInTheDocument(); expect(screen.getByText('Mid')).toBeInTheDocument(); @@ -331,7 +331,6 @@ describe('Slider', () => { describe('Keyboard Navigation', () => { it('should support arrow key navigation', async () => { - const user = userEvent.setup(); render(); const slider = screen.getByRole('slider') as HTMLInputElement; diff --git a/packages/ui/src/Textarea/Textarea.test.tsx b/packages/ui/src/Textarea/Textarea.test.tsx index d929a86..8c5e834 100644 --- a/packages/ui/src/Textarea/Textarea.test.tsx +++ b/packages/ui/src/Textarea/Textarea.test.tsx @@ -368,7 +368,7 @@ describe('Textarea', () => { describe('Ref Forwarding', () => { it('forwards ref to textarea element', () => { - let ref: HTMLTextareaElement | null = null; + let ref: HTMLTextAreaElement | null = null; render(