From 381f870bb5a636b8297013cacdd7c7e5e37a1c67 Mon Sep 17 00:00:00 2001 From: Pouzor Date: Mon, 30 Mar 2026 22:24:30 +0200 Subject: [PATCH] fix: update ScanConfigModal tests after removing interval field - Replace interval display test with interval-preservation test - Reset saveConfig mock call history in beforeEach to prevent test bleed --- .../modals/__tests__/ScanConfigModal.test.tsx | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/frontend/src/components/modals/__tests__/ScanConfigModal.test.tsx b/frontend/src/components/modals/__tests__/ScanConfigModal.test.tsx index 5eaca0c..53d2fa7 100644 --- a/frontend/src/components/modals/__tests__/ScanConfigModal.test.tsx +++ b/frontend/src/components/modals/__tests__/ScanConfigModal.test.tsx @@ -18,6 +18,7 @@ const defaultConfig = { data: { ranges: ['192.168.1.0/24'], interval_seconds: 60 describe('ScanConfigModal', () => { beforeEach(() => { vi.mocked(scanApi.getConfig).mockResolvedValue(defaultConfig as never) + vi.mocked(scanApi.saveConfig).mockReset() vi.mocked(scanApi.saveConfig).mockResolvedValue({} as never) vi.mocked(toast.success).mockReset() vi.mocked(toast.error).mockReset() @@ -37,11 +38,14 @@ describe('ScanConfigModal', () => { expect(input).toBeDefined() }) - it('loads interval from API on open', async () => { + it('preserves interval from API when saving ranges', async () => { vi.mocked(scanApi.getConfig).mockResolvedValue({ data: { ranges: ['10.0.0.0/8'], interval_seconds: 120 } } as never) render() - const input = await screen.findByDisplayValue('120') - expect(input).toBeDefined() + await screen.findByDisplayValue('10.0.0.0/8') + fireEvent.click(screen.getByRole('button', { name: 'Save' })) + await waitFor(() => { + expect(scanApi.saveConfig).toHaveBeenCalledWith({ ranges: ['10.0.0.0/8'], interval_seconds: 120 }) + }) }) it('adds a new empty range on "Add range" click', async () => {