From fdba5d270f6c3200f85f42a17d96944baa0555c2 Mon Sep 17 00:00:00 2001 From: bymyself Date: Thu, 15 Jan 2026 23:20:28 -0800 Subject: [PATCH] test: remove redundant mock restoration calls vi.restoreAllMocks() in afterEach already handles cleanup --- .../services/workflowSessionStorageService.test.ts | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/src/platform/workflow/persistence/services/workflowSessionStorageService.test.ts b/src/platform/workflow/persistence/services/workflowSessionStorageService.test.ts index d6950560cb..da2317d884 100644 --- a/src/platform/workflow/persistence/services/workflowSessionStorageService.test.ts +++ b/src/platform/workflow/persistence/services/workflowSessionStorageService.test.ts @@ -59,25 +59,21 @@ describe('workflowSessionStorageService', () => { }) it('returns false after max eviction attempts', () => { - const spy = vi.spyOn(sessionStorage, 'setItem').mockImplementation(() => { + vi.spyOn(sessionStorage, 'setItem').mockImplementation(() => { throw new DOMException('Quota exceeded', 'QuotaExceededError') }) const result = setWithEviction('key', { test: 'data' }) expect(result).toBe(false) - - spy.mockRestore() }) it('returns false on unexpected errors', () => { - const spy = vi.spyOn(sessionStorage, 'setItem').mockImplementation(() => { + vi.spyOn(sessionStorage, 'setItem').mockImplementation(() => { throw new Error('Unexpected error') }) const result = setWithEviction('key', { test: 'data' }) expect(result).toBe(false) - - spy.mockRestore() }) })