CM-921: Improving BS tests

test-config/wdio.browserstack.conf.js

@@ -1,4 +1,6 @@
 import commonConfig from './wdio.common.conf.js'
+import http from 'http';
+import https from 'https';
 const currentTime = Date.now()
 const commonBStackCapabilities = {
   projectName: 'LiveConnect',
@@ -94,6 +96,10 @@ export const config = {
   key: process.env.BS_KEY,
   hostname: 'hub.browserstack.com',
 
+  agent: {
+    http: new http.Agent({ keepAlive: true }),
+    https: new https.Agent({ keepAlive: true })
+  },
   //
   // The number of times to retry the entire specfile when it fails as a whole
   specFileRetries: 2

test/it/helpers/browser.ts

@@ -162,32 +162,7 @@ export async function probeLS() {
   return enabled
 }
 
-export async function deleteAllCookies() {
-  const error = await browser.execute(() => {
-    try {
-      const cookies = document.cookie.split('; ')
-      for (let c = 0; c < cookies.length; c++) {
-        const d = window.location.hostname.split('.')
-        while (d.length > 0) {
-          const cookieBase = encodeURIComponent(cookies[c].split(';')[0].split('=')[0]) + '=; expires=Thu, 01-Jan-1970 00:00:01 GMT; domain=' + d.join('.') + ' ;path='
-          const p = location.pathname.split('/')
-          document.cookie = cookieBase + '/'
-          while (p.length > 0) {
-            document.cookie = cookieBase + p.join('/')
-            p.pop()
-          }
-          d.shift()
-        }
-      }
-      return null
-    } catch (e) {
-      return e
-    }
-  })
-  if (error) {
-    console.error(`failed cleaning cookies: ${error}`)
-  }
-}
+export const deleteAllCookies = () => browser.deleteAllCookies()
 
 export function isMobileSafari() {
   return browser.capabilities.browserName === 'safari' &&

test/it/helpers/mock-server.ts

@@ -207,21 +207,11 @@ export function MockServerFactory(config) {
       const afterPage = await getText('#after')
       assert.strictEqual(afterPage, 'After')
     },
-    getHistory: () => {
-      return history
-    },
-    getIdexHistory: () => {
-      return idex
-    },
-    getBakerHistory: () => {
-      return bakerHistory
-    },
-    getTrackingRequests: () => {
-      return history.filter(req => req.query.ae === undefined)
-    },
-    getApplicationErrors: () => {
-      return history.filter(req => req.query.ae !== undefined)
-    },
+    getHistory: () => history,
+    getIdexHistory: () => idex,
+    getBakerHistory: () => bakerHistory,
+    getTrackingRequests: () => history.filter(req => req.query.ae === undefined),
+    getApplicationErrors: () => history.filter(req => req.query.ae !== undefined),
     clearHistory: () => {
       idex = []
       history = []

test/it/live-connect.spec.ts

@@ -58,9 +58,7 @@ describe('LiveConnect', function () {
     console.log('\x1b[35m\x1b[4m%s\x1b[0m', `##### Finishing the test: '${this.currentTest.fullTitle()}'`)
   })
 
-  after(function () {
-    server.stop()
-  })
+  after(() => server.stop())
 
   it('should send decisionIds', async function () {
     const decisionIdOne = '4ca76883-1e26-3fb8-b6d1-f881ac7d6699'