Fix typecheck. (#14)

Reviewed-on: #14
Co-authored-by: Drew Galbraith <drew@tiramisu.one>
Co-committed-by: Drew Galbraith <drew@tiramisu.one>

.forgejo/workflows/ci.yml

@@ -74,7 +74,7 @@ jobs:
       working-directory: frontend
 
     - name: Typecheck
-      run: npm run typecheck || echo "Failed"
+      run: npm run typecheck 
       working-directory: frontend
 
     - name: Test

frontend/app/components/TaskForm.tsx

@@ -10,6 +10,7 @@ import {
   Paper,
   Typography,
   Alert,
+  type SelectChangeEvent,
 } from '@mui/material'
 import { useNavigate } from 'react-router'
 import type { CreateTaskRequest, Task } from '~/types/task'
@@ -134,12 +135,10 @@ export function TaskForm({
       }
     }
 
-  const handleStatusChange = (
+  const handleStatusChange = (event: SelectChangeEvent<TaskStatus>) => {
-    e: React.ChangeEvent<{ name?: string; value: unknown }>
-  ) => {
     setFormData(prev => ({
       ...prev,
-      status: e.target.value as TaskStatus,
+      status: event.target.value as TaskStatus,
     }))
   }
 

frontend/app/routes/home.test.tsx

@@ -2,13 +2,35 @@ import { render, screen } from '@testing-library/react'
 import { describe, it, expect } from 'vitest'
 import { MemoryRouter } from 'react-router'
 import Home from './home'
+import type { Route } from './+types/home'
 
 describe('Home component', () => {
   it('should render task management interface', () => {
-    const mockLoaderData = { tasks: [] }
+    const mockComponentProps: Route.ComponentProps = {
+      loaderData: { tasks: [] },
+      params: {},
+      matches: [
+        {
+          id: 'root',
+          params: {},
+          pathname: '/',
+          data: undefined,
+          loaderData: undefined,
+          handle: undefined,
+        },
+        {
+          id: 'routes/home',
+          params: {},
+          pathname: '/',
+          data: { tasks: [] },
+          loaderData: { tasks: [] },
+          handle: undefined,
+        },
+      ],
+    }
     render(
       <MemoryRouter>
-        <Home loaderData={mockLoaderData} />
+        <Home {...mockComponentProps} />
       </MemoryRouter>
     )
     expect(