From 3572568a3a51561be07a6e5eabe254978d2fd580 Mon Sep 17 00:00:00 2001 From: Erik Marks <25517051+rekmarks@users.noreply.github.com> Date: Tue, 7 Apr 2026 18:35:01 -0700 Subject: [PATCH] fix(kernel-ui): reset mock state in afterEach to prevent cascading failures Use resetMocks() in afterEach instead of manual restoration at the end of the test body, so a mid-test assertion failure cannot leave isKRefMock stuck at false for subsequent tests. Co-Authored-By: Claude Opus 4.6 --- packages/kernel-ui/src/components/SendMessageForm.test.tsx | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/packages/kernel-ui/src/components/SendMessageForm.test.tsx b/packages/kernel-ui/src/components/SendMessageForm.test.tsx index 6d24a8df5c..f2099a7d0e 100644 --- a/packages/kernel-ui/src/components/SendMessageForm.test.tsx +++ b/packages/kernel-ui/src/components/SendMessageForm.test.tsx @@ -17,7 +17,7 @@ import { useRegistry } from '../hooks/useRegistry.ts'; import type { ObjectRegistry } from '../types.ts'; import { SendMessageForm } from './SendMessageForm.tsx'; -const { setMockBehavior } = setupOcapKernelMock(); +const { resetMocks, setMockBehavior } = setupOcapKernelMock(); vi.mock('../context/PanelContext.tsx', () => ({ usePanelContext: vi.fn(), @@ -109,6 +109,7 @@ describe('SendMessageForm Component', () => { afterEach(() => { cleanup(); + resetMocks(); vi.resetModules(); }); @@ -203,8 +204,6 @@ describe('SendMessageForm Component', () => { // Target should remain unset, button stays disabled expect(targetSelect).toHaveValue(''); expect(getByTestId('message-send-button')).toBeDisabled(); - - setMockBehavior({ isKRef: true }); }); it('calls callKernelMethod with correct parameters when Send button is clicked', async () => {