fix: emit connect once all handshake commands have been resolved

Author: PavelPashov

lib/Redis.ts

@@ -447,11 +447,11 @@ class Redis extends Commander implements DataHandledable {
 
     let writable =
       this.status === "ready" ||
+      (!stream && Command.checkFlag("HANDSHAKE_COMMANDS", command.name)) ||
       (!stream &&
         this.status === "connect" &&
         exists(command.name) &&
-        (hasFlag(command.name, "loading") ||
-          Command.checkFlag("HANDSHAKE_COMMANDS", command.name)));
+        hasFlag(command.name, "loading"));
     if (!this.stream) {
       writable = false;
     } else if (!this.stream.writable) {

lib/redis/event_handler.ts

@@ -17,8 +17,6 @@ const debug = Debug("connection");
 
 export function connectHandler(self) {
   return function () {
-    self.setStatus("connect");
-
     self.resetCommandQueue();
 
     // AUTH command should be processed before any other commands
@@ -66,15 +64,6 @@ export function connectHandler(self) {
       });
     }
 
-    /*
-      No need to keep the reference of DataHandler here
-      because we don't need to do the cleanup.
-      `Stream#end()` will remove all listeners for us.
-    */
-    new DataHandler(self, {
-      stringNumbers: self.options.stringNumbers,
-    });
-
     const clientCommandPromises = [];
 
     if (self.options.connectionName) {
@@ -109,9 +98,21 @@ export function connectHandler(self) {
       );
     }
 
+    /*
+      No need to keep the reference of DataHandler here
+      because we don't need to do the cleanup.
+      `Stream#end()` will remove all listeners for us.
+    */
+    new DataHandler(self, {
+      stringNumbers: self.options.stringNumbers,
+    });
+
     Promise.all(clientCommandPromises)
       .catch(noop)
       .finally(() => {
+        // Emit connect once all handshake commands have passed
+        self.setStatus("connect");
+
         if (!self.options.enableReadyCheck) {
           exports.readyHandler(self)();
         }