refactor(pool): wait for createConnection on pool reset

Author: mbroadst

lib/core/connection/pool.js

@@ -241,7 +241,10 @@ function resetPoolState(pool) {
 
 function connectionFailureHandler(pool, event, err, conn) {
   if (conn) {
-    if (conn._connectionFailHandled) return;
+    if (conn._connectionFailHandled) {
+      return;
+    }
+
     conn._connectionFailHandled = true;
     conn.destroy();
 
@@ -430,7 +433,7 @@ function messageHandler(self) {
           updateSessionFromResponse(session, document);
         }
 
-        if (document.$clusterTime) {
+        if (self.topology && document.$clusterTime) {
           self.topology.clusterTime = document.$clusterTime;
         }
       }
@@ -752,11 +755,12 @@ Pool.prototype.reset = function(callback) {
 
       resetPoolState(this);
 
-      // attempt reexecution
-      process.nextTick(() => _execute(this)());
-      if (typeof callback === 'function') {
-        callback(null, null);
-      }
+      // create a new connection, this will ultimately trigger execution
+      createConnection(this, () => {
+        if (typeof callback === 'function') {
+          callback(null, null);
+        }
+      });
     }
   );
 };
@@ -877,10 +881,6 @@ Pool.prototype.write = function(command, options, cb) {
   // Optional per operation socketTimeout
   operation.socketTimeout = options.socketTimeout;
   operation.monitoring = options.monitoring;
-  // Custom socket Timeout
-  if (options.socketTimeout) {
-    operation.socketTimeout = options.socketTimeout;
-  }
 
   // Get the requestId
   operation.requestId = command.requestId;