Extract default connect strategy into helper function

Author: qsymmachus

packages/client/lib/client/socket.ts

@@ -82,15 +82,6 @@ export default class RedisSocket extends EventEmitter {
   }
 
   #createReconnectStrategy(options?: RedisSocketOptions): ReconnectStrategyFunction {
-    const defaultStrategy = (retries: number) => {
-      // Generate a random jitter between 0 – 200 ms:
-      const jitter = Math.floor(Math.random() * 200);
-      // Delay is an exponential back off, (times^2) * 50 ms, with a maximum value of 2000 ms:
-      const delay = Math.min(Math.pow(2, retries) * 50, 2000);
-
-      return delay + jitter;
-    }
-    
     const strategy = options?.reconnectStrategy;
     if (strategy === false || typeof strategy === 'number') {
       return () => strategy;
@@ -106,12 +97,12 @@ export default class RedisSocket extends EventEmitter {
           return retryIn;
         } catch (err) {
           this.emit('error', err);
-          return defaultStrategy(retries);
+          return this.defaultReconnectStrategy(retries);
         }
       };
     }
 
-    return defaultStrategy;
+    return this.defaultReconnectStrategy;
   }
 
   #createSocketFactory(options?: RedisSocketOptions) {
@@ -342,4 +333,13 @@ export default class RedisSocket extends EventEmitter {
     this.#isSocketUnrefed = true;
     this.#socket?.unref();
   }
+
+  defaultReconnectStrategy(retries: number) {
+    // Generate a random jitter between 0 – 200 ms:
+    const jitter = Math.floor(Math.random() * 200);
+    // Delay is an exponential back off, (times^2) * 50 ms, with a maximum value of 2000 ms:
+    const delay = Math.min(Math.pow(2, retries) * 50, 2000);
+  
+    return delay + jitter;
+  }
 }