Browserstack optimisations (#7201)

* add local identifier to suppoort parallel runs BrowserStack

Signed-off-by: Nimit <nimitagg95@gmail.com>

* calling driver quit to stop test and then killing binary

Signed-off-by: Nimit <nimitagg95@gmail.com>

* run lint-fix

* standard fixes

Signed-off-by: Nimit <nimitagg95@gmail.com>

Author: nimit95

test/jest-environment.js

@@ -44,7 +44,8 @@ if (isBrowserStack) {
   }
   const sharedOpts = {
     'browserstack.local': true,
-    'browserstack.video': false
+    'browserstack.video': false,
+    'browserstack.localIdentifier': global.browserStackLocal.localIdentifierFlag
   }
 
   browserOptions = {

test/jest-global-setup.js

@@ -5,7 +5,8 @@ if (process.env.BROWSERSTACK) {
   const { Local } = require('browserstack-local')
   const browserStackLocal = new Local()
   const localBrowserStackOpts = {
-    key: process.env.BROWSERSTACK_ACCESS_KEY
+    key: process.env.BROWSERSTACK_ACCESS_KEY,
+    localIdentifier: new Date().getTime() // Adding a unique local identifier to run parallel tests on BrowserStack
   }
   global.browserStackLocal = browserStackLocal
 

test/jest-global-teardown.js

@@ -11,8 +11,6 @@ if (process.env.BROWSERSTACK) {
 }
 
 module.exports = async () => {
-  await globalTeardown()
-
   if (browser) {
     // Close all remaining browser windows
     try {
@@ -21,7 +19,11 @@ module.exports = async () => {
         if (!window) continue
         await browser.window(window)
         await browser.origClose()
+        await browser.quit()
+        await globalTeardown()
       }
     } catch (_) {}
+  } else {
+    await globalTeardown()
   }
 }