From 9c3a94ba1acdd8a39b9186d46ecaa916ec810f91 Mon Sep 17 00:00:00 2001 From: Johnpaul Date: Wed, 21 Jan 2026 21:20:05 +0100 Subject: [PATCH] refactor: remove any types from keybindingService.forwarding.test.ts - Replace dialogStack mock 'as any' with ReturnType - Replace undefined/null assignments with double-cast using typeof saved variables - All 3 instances now properly typed --- src/services/keybindingService.forwarding.test.ts | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/services/keybindingService.forwarding.test.ts b/src/services/keybindingService.forwarding.test.ts index 1c62222dd..17aa11956 100644 --- a/src/services/keybindingService.forwarding.test.ts +++ b/src/services/keybindingService.forwarding.test.ts @@ -77,7 +77,7 @@ describe('keybindingService - Event Forwarding', () => { // Reset dialog store mock to empty vi.mocked(useDialogStore).mockReturnValue({ dialogStack: [] - } as any) + } as unknown as ReturnType) keybindingService = useKeybindingService() keybindingService.registerCoreKeybindings() @@ -127,7 +127,8 @@ describe('keybindingService - Event Forwarding', () => { it('should not forward Delete key when canvas processKey is not available', async () => { // Temporarily replace processKey with undefined const originalProcessKey = vi.mocked(app.canvas).processKey - vi.mocked(app.canvas).processKey = undefined as any + vi.mocked(app.canvas).processKey = + undefined as unknown as typeof originalProcessKey const event = createTestKeyboardEvent('Delete') @@ -142,7 +143,7 @@ describe('keybindingService - Event Forwarding', () => { it('should not forward Delete key when canvas is not available', async () => { // Temporarily set canvas to null const originalCanvas = vi.mocked(app).canvas - vi.mocked(app).canvas = null as any + vi.mocked(app).canvas = null as unknown as typeof originalCanvas const event = createTestKeyboardEvent('Delete')