ci(browserstack): increased random wait before browserstack availabil…

…ity polling
mountain-pass · Jan 11, 2021 · 12e5765 · 12e5765
1 parent 0015bb7
commit 12e5765
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 9 deletions.
diff --git a/.github/workflows/build-and-publish.yml b/.github/workflows/build-and-publish.yml
@@ -327,14 +327,14 @@ jobs:
 
       - run: npm config set script-shell $(which bash)
       - run: npm ci
-      - run: npm run wait-for-browser-stack
-        env:
-          BROWSERSTACK_USERNAME: ${{ secrets.BROWSERSTACK_USERNAME }}
-          BROWSERSTACK_ACCESS_KEY: ${{ secrets.BROWSERSTACK_ACCESS_KEY }}
       - uses: "browserstack/github-actions/setup-env@v1.0.1"
         with:
           username: ${{ secrets.BROWSERSTACK_USERNAME }}
           access-key: ${{ secrets.BROWSERSTACK_ACCESS_KEY }}
+      - run: npm run wait-for-browser-stack
+        env:
+          BROWSERSTACK_USERNAME: ${{ secrets.BROWSERSTACK_USERNAME }}
+          BROWSERSTACK_ACCESS_KEY: ${{ secrets.BROWSERSTACK_ACCESS_KEY }}
       - uses: "browserstack/github-actions/setup-local@v1.0.1"
         with:
           local-testing: start

diff --git a/scripts/wait-for-browser-stack.js b/scripts/wait-for-browser-stack.js
@@ -51,17 +51,17 @@ const MAX_WAIT = 10 * 60 * 60 // 10hr
 const RESERVED_SESSIONS = 1
 const MIN_WAIT_TIME = 40
 const MAX_WAIT_TIME = 80
-const MAX_INIT_WAIT_TIME = 20
+const MAX_INIT_WAIT_TIME = 120
 /**
  *
  */
 async function waitForSpareSession () {
   const start = Date.now()
+  // inital random sleep so all the different browserstack tests don't try to start at once
+  const initialWaitTime = getRandomInt(0, MAX_INIT_WAIT_TIME)
+  logger.info(`waiting ${initialWaitTime} before starting availability check`)
   while (secondsSince(start) < MAX_WAIT) {
-    // inital random sleep so all the different browserstack tests don't try to start at once
-    await new Promise(resolve =>
-      setTimeout(resolve, getRandomInt(0, MAX_INIT_WAIT_TIME) * 1000)
-    )
+    await new Promise(resolve => setTimeout(resolve, initialWaitTime * 1000))
     const planInfo = await getPlanInfo()
     const maxAllowed =
       planInfo.parallel_sessions_max_allowed - RESERVED_SESSIONS