diff --git a/frontend/tests/app/page.test.tsx b/frontend/tests/app/page.test.tsx index 787fc30..e46d7d1 100644 --- a/frontend/tests/app/page.test.tsx +++ b/frontend/tests/app/page.test.tsx @@ -55,6 +55,15 @@ jest.mock('@/lib/api/hooks/useAuth', () => ({ })), })); +// Mock AuthContext - Home page uses useAuth to check if user is authenticated +jest.mock('@/lib/auth/AuthContext', () => ({ + useAuth: jest.fn(() => ({ + isAuthenticated: false, + isLoading: false, + user: null, + })), +})); + // Mock Theme components jest.mock('@/components/theme', () => ({ ThemeToggle: () =>
Theme Toggle
, diff --git a/frontend/tests/components/agents/AgentTypeForm.test.tsx b/frontend/tests/components/agents/AgentTypeForm.test.tsx index 3f17d8d..276d5ec 100644 --- a/frontend/tests/components/agents/AgentTypeForm.test.tsx +++ b/frontend/tests/components/agents/AgentTypeForm.test.tsx @@ -549,14 +549,14 @@ describe('AgentTypeForm', () => { }); }); - describe('Null Model Params Handling', () => { - it('handles null model_params gracefully', () => { - const agentTypeWithNullParams: AgentTypeResponse = { + describe('Empty Model Params Handling', () => { + it('handles empty model_params gracefully', () => { + const agentTypeWithEmptyParams: AgentTypeResponse = { ...mockAgentType, - model_params: null, + model_params: {}, }; - render(); + render(); // Should render without errors expect(screen.getByText('Edit Agent Type')).toBeInTheDocument(); diff --git a/frontend/tests/components/common/ErrorBoundary.test.tsx b/frontend/tests/components/common/ErrorBoundary.test.tsx index 11a6203..7be0630 100644 --- a/frontend/tests/components/common/ErrorBoundary.test.tsx +++ b/frontend/tests/components/common/ErrorBoundary.test.tsx @@ -248,7 +248,7 @@ describe('ErrorBoundary', () => { describe('error without message', () => { it('handles error with null message gracefully', () => { - function ThrowNullError() { + function ThrowNullError(): never { const error = new Error(); error.message = ''; throw error; @@ -338,7 +338,7 @@ describe('ErrorBoundary', () => { describe('edge cases', () => { it('handles deeply nested errors', () => { - function DeepChild() { + function DeepChild(): never { throw new Error('Deep error'); } @@ -377,7 +377,7 @@ describe('ErrorBoundary', () => { }); it('allows nested error boundaries', () => { - function InnerThrowing() { + function InnerThrowing(): never { throw new Error('Inner error'); } diff --git a/frontend/tests/components/layout/Header.test.tsx b/frontend/tests/components/layout/Header.test.tsx index 4f1c495..ac7bce8 100644 --- a/frontend/tests/components/layout/Header.test.tsx +++ b/frontend/tests/components/layout/Header.test.tsx @@ -124,7 +124,7 @@ describe('Header', () => { render(
); const homeLink = screen.getByRole('link', { name: /home/i }); - expect(homeLink).toHaveAttribute('href', '/'); + expect(homeLink).toHaveAttribute('href', '/dashboard'); }); it('renders admin link for superusers', () => {