refactor getRandomPoolLength out of markRandomNodeUnreachable

js/modules/loki_rpc.js

@@ -73,9 +73,8 @@ const sendToProxy = async (options = {}, targetNode, retryNumber = 0) => {
     // remove
     // but which the proxy or the target...
     // we got a ton of randomPool nodes, let's just not worry about this one
-    const randomPoolRemainingCount = lokiSnodeAPI.markRandomNodeUnreachable(
+    lokiSnodeAPI.markRandomNodeUnreachable(randSnode);
-      randSnode
+    const randomPoolRemainingCount = lokiSnodeAPI.getRandomPoolLength();
-    );
     log.warn(
       `lokiRpc sendToProxy`,
       `snode ${randSnode.ip}:${randSnode.port} to ${targetNode.ip}:${
@@ -99,9 +98,8 @@ const sendToProxy = async (options = {}, targetNode, retryNumber = 0) => {
     // but the target node
 
     // we got a ton of randomPool nodes, let's just not worry about this one
-    const randomPoolRemainingCount = lokiSnodeAPI.markRandomNodeUnreachable(
+    lokiSnodeAPI.markRandomNodeUnreachable(randSnode);
-      randSnode
+    const randomPoolRemainingCount = lokiSnodeAPI.getRandomPoolLength();
-    );
     log.warn(
       `lokiRpc sendToProxy`,
       `snode ${randSnode.ip}:${randSnode.port} to ${targetNode.ip}:${

js/modules/loki_snode_api.js

@@ -185,6 +185,9 @@ class LokiSnodeAPI {
       this.randomSnodePool,
       _.find(this.randomSnodePool, { ip: snode.ip, port: snode.port })
     );
+  }
+
+  getRandomPoolLength() {
     return this.randomSnodePool.length;
   }
 
@@ -281,7 +284,8 @@ class LokiSnodeAPI {
       const snodes = result.snodes.filter(tSnode => tSnode.ip !== '0.0.0.0');
       return snodes;
     } catch (e) {
-      const randomPoolRemainingCount = this.markRandomNodeUnreachable(snode);
+      this.markRandomNodeUnreachable(snode);
+      const randomPoolRemainingCount = this.getRandomPoolLength();
       log.error(
         'loki_snodes:::getSnodesForPubkey - error',
         e.code,