fix #2443 - fix multiple sockets on error in socket initiator (#2480)

* fix #2443 - fix multiple sockets on error in socket initiator

* handle `error` events in test

Author: leibale

packages/client/lib/client/index.spec.ts

@@ -647,6 +647,9 @@ describe('Client', () => {
     });
 
     testUtils.testWithClient('should propagated errors from "isolated" clients', client => {
+        client.on('error', () => {
+            // ignore errors
+        });
         return client.executeIsolated(isolated => killClient(isolated, client));
     }, GLOBAL.SERVERS.OPEN);
 

packages/client/lib/client/socket.ts

@@ -199,7 +199,7 @@ export default class RedisSocket extends EventEmitter {
                         .off('error', reject)
                         .once('error', (err: Error) => this.#onSocketError(err))
                         .once('close', hadError => {
-                            if (!hadError && this.#isOpen && this.#socket === socket) {
+                            if (!hadError && this.#isReady && this.#socket === socket) {
                                 this.#onSocketError(new SocketClosedUnexpectedlyError());
                             }
                         })