possible fix for ITests. The idea to use quit() instead of disconnect()

Author: Artem

redisinsight/api/src/modules/pub-sub/model/redis-client.spec.ts

@@ -11,6 +11,7 @@ nodeClient.unsubscribe = jest.fn();
 nodeClient.punsubscribe = jest.fn();
 nodeClient.status = 'ready';
 nodeClient.disconnect = jest.fn();
+nodeClient.quit = jest.fn();
 
 describe('RedisClient', () => {
   let redisClient: RedisClient;
@@ -126,7 +127,7 @@ describe('RedisClient', () => {
       expect(redisClient['client']).toEqual(null);
       expect(redisClient['status']).toEqual(RedisClientStatus.End);
       expect(removeAllListenersSpy).toHaveBeenCalled();
-      expect(nodeClient.disconnect).toHaveBeenCalled();
+      expect(nodeClient.quit).toHaveBeenCalled();
     });
   });
 });

redisinsight/api/src/modules/pub-sub/model/redis-client.ts

@@ -95,7 +95,7 @@ export class RedisClient extends EventEmitter2 {
    */
   destroy() {
     this.client?.removeAllListeners();
-    this.client?.disconnect(false);
+    this.client?.quit();
     this.client = null;
     this.status = RedisClientStatus.End;
   }

redisinsight/api/src/modules/pub-sub/model/user-session.spec.ts

@@ -16,6 +16,7 @@ nodeClient.unsubscribe = jest.fn();
 nodeClient.punsubscribe = jest.fn();
 nodeClient.status = 'ready';
 nodeClient.disconnect = jest.fn();
+nodeClient.quit = jest.fn();
 
 const mockUserClient = new UserClient('socketId', mockSocket, 'databaseId');
 
@@ -119,7 +120,7 @@ describe('UserSession', () => {
     it('should handle message by particular subscription', async () => {
       userSession.handleDisconnect();
       expect(userSession['subscriptions'].size).toEqual(0);
-      expect(nodeClient.disconnect).toHaveBeenCalled();
+      expect(nodeClient.quit).toHaveBeenCalled();
     });
   });
 });