chore: constant for retry delay during binary start

Author: rchougule

setup-local/config/constants.js

@@ -50,6 +50,7 @@ module.exports = {
   },
 
   BINARY_MAX_TRIES: 3,
+  RETRY_DELAY_BINARY: 5000,
 
   RESTRICTED_LOCAL_ARGS: ['k', 'key', 'local-identifier', 'daemon', 'only-automate', 'verbose', 'log-file', 'ci-plugin'],
 

setup-local/dist/index.js

@@ -1083,6 +1083,7 @@ const {
   LOCAL_BINARY_ZIP,
   LOCAL_LOG_FILE_PREFIX,
   LOCAL_BINARY_TRIGGER,
+  RETRY_DELAY_BINARY,
   BINARY_MAX_TRIES,
   ALLOWED_INPUT_VALUES: {
     LOCAL_TESTING,
@@ -1286,7 +1287,7 @@ class BinaryControl {
         if (triesAvailable) {
           core.info(`Error in starting local tunnel: ${e.message}. Trying again in 5 seconds...`);
           // eslint-disable-next-line no-await-in-loop
-          await Utils.sleepFor(5000);
+          await Utils.sleepFor(RETRY_DELAY_BINARY);
         } else {
           throw Error(`Local tunnel could not be started. Error message from binary: ${e.message}`);
         }
@@ -13489,6 +13490,7 @@ module.exports = {
   },
 
   BINARY_MAX_TRIES: 3,
+  RETRY_DELAY_BINARY: 5000,
 
   RESTRICTED_LOCAL_ARGS: ['k', 'key', 'local-identifier', 'daemon', 'only-automate', 'verbose', 'log-file', 'ci-plugin'],
 

setup-local/src/binaryControl.js

@@ -18,6 +18,7 @@ const {
   LOCAL_BINARY_ZIP,
   LOCAL_LOG_FILE_PREFIX,
   LOCAL_BINARY_TRIGGER,
+  RETRY_DELAY_BINARY,
   BINARY_MAX_TRIES,
   ALLOWED_INPUT_VALUES: {
     LOCAL_TESTING,
@@ -221,7 +222,7 @@ class BinaryControl {
         if (triesAvailable) {
           core.info(`Error in starting local tunnel: ${e.message}. Trying again in 5 seconds...`);
           // eslint-disable-next-line no-await-in-loop
-          await Utils.sleepFor(5000);
+          await Utils.sleepFor(RETRY_DELAY_BINARY);
         } else {
           throw Error(`Local tunnel could not be started. Error message from binary: ${e.message}`);
         }

setup-local/test/binaryControl.test.js

@@ -21,6 +21,7 @@ const {
   LOCAL_BINARY_ZIP,
   LOCAL_LOG_FILE_PREFIX,
   LOCAL_BINARY_TRIGGER,
+  RETRY_DELAY_BINARY,
   ALLOWED_INPUT_VALUES: {
     LOCAL_TESTING,
   },
@@ -509,7 +510,7 @@ describe('Binary Control Operations', () => {
           try {
             await binaryControl.startBinary();
           } catch (e) {
-            sinon.assert.calledWith(Utils.sleepFor, 5000);
+            sinon.assert.calledWith(Utils.sleepFor, RETRY_DELAY_BINARY);
             sinon.assert.calledWith(core.info, 'Error in starting local tunnel: "some message". Trying again in 5 seconds...');
             expect(e.message).to.eq('Local tunnel could not be started. Error message from binary: "some message"');
           }
@@ -529,7 +530,7 @@ describe('Binary Control Operations', () => {
           try {
             await binaryControl.startBinary();
           } catch (e) {
-            sinon.assert.calledWith(Utils.sleepFor, 5000);
+            sinon.assert.calledWith(Utils.sleepFor, RETRY_DELAY_BINARY);
             sinon.assert.calledWith(core.info, `Error in starting local tunnel: ${JSON.stringify(response.error)}. Trying again in 5 seconds...`);
             expect(e.message).to.eq(`Local tunnel could not be started. Error message from binary: ${JSON.stringify(response.error)}`);
           }