From 0d395d3cb052c674b9253d50d0273badac2fa7d4 Mon Sep 17 00:00:00 2001 From: Tom Butcher Date: Sat, 4 Jul 2026 16:54:55 +0100 Subject: [PATCH] Enhance SocketUser tests to verify removal of all listeners during disconnection. - Added verification for `removeAllListeners` method in both SocketUser and UpdateManager to ensure proper cleanup of event listeners during disconnection handling. --- src/socket/__tests__/socketuser.test.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/socket/__tests__/socketuser.test.js b/src/socket/__tests__/socketuser.test.js index 69e4d0f..9498626 100644 --- a/src/socket/__tests__/socketuser.test.js +++ b/src/socket/__tests__/socketuser.test.js @@ -33,7 +33,8 @@ jest.unstable_mockModule('../../updates/updatemanager.js', () => ({ subscribeToObjectUpdate: jest.fn(), removeObjectNewListener: jest.fn(), removeObjectDeleteListener: jest.fn(), - removeObjectUpdateListener: jest.fn() + removeObjectUpdateListener: jest.fn(), + removeAllListeners: jest.fn() })) })); @@ -256,6 +257,7 @@ describe('SocketUser', () => { await socketUser.handleDisconnect(); expect(socketUser.actionManager.removeAllListeners).toHaveBeenCalled(); + expect(socketUser.updateManager.removeAllListeners).toHaveBeenCalled(); expect(socketUser.eventManager.removeAllListeners).toHaveBeenCalled(); expect(socketUser.statsManager.removeAllListeners).toHaveBeenCalled(); expect(socketUser.notificationManager.removeAllListeners).toHaveBeenCalled();