@testim/testim-cli 3.278.0 → 3.280.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testim/testim-cli",
-  "version": "3.278.0",
+  "version": "3.280.0",
   "description": "Command line interface for running Testing on your CI",
   "author": "Oren Rubin",
   "contributors": [{
@@ -22,7 +22,6 @@
     "gulp-preprocess": "3.0.3",
     "merge-stream": "2.0.0",
     "proxyquire": "2.1.3",
-    "request": "2.88.2",
     "sinon": "9.0.2",
     "sinon-chai": "3.7.0",
     "ts-node": "9.1.1",
@@ -44,13 +43,13 @@
   "dependencies": {
     "@applitools/eyes-sdk-core": "13.11.21",
     "@testim/coralogix-logger": "1.1.27-beta.1",
-    "@testim/webdriverio": "0.0.5",
+    "@testim/webdriverio": "0.0.8",
     "abort-controller": "3.0.0",
     "ajv": "6.12.6",
     "analytics-node": "5.0.0",
     "appium-dom-utils": "1.0.6",
     "archiver": "5.3.0",
-    "body-parser": "1.19.1",
+    "body-parser": "1.20.1",
     "chalk": "4.1.2",
     "chrome-launcher": "0.15.0",
     "combine-source-map": "0.8.0",

package/services/gridService.js

@@ -4,7 +4,7 @@
 
 const _ = require('lodash');
 
-const { GridError, ArgError } = require('../errors');
+const { GridError, ArgError, GridConcurrencyError } = require('../errors');
 const { hasTestPlanFlag, promiseMap } = require('../utils');
 const { gridMessages, gridTypes } = require('../commons/constants');
 const logger = require('../commons/logger').getLogger('grid-service');
@@ -97,7 +97,7 @@ async function handleGetGridResponse(projectId, companyId, workerId, browser, ge
     }
 
     if (isError() && res.code === 'no-available-slot') {
-        throw new GridError(`Failed to run test on ${browser} - concurrency limit reached`);
+        throw new GridConcurrencyError(`Failed to run test on ${browser} - concurrency limit reached`);
     }
 
     logger.error('invalid code error response - get grid', { res });

package/workers/BaseWorker.js

@@ -18,7 +18,7 @@ const { releasePlayer } = require('./workerUtils');
 const featureFlags = require('../commons/featureFlags');
 const perf = require('../commons/performance-logger');
 const {
-    SeleniumError, StopRunOnError, GridError, GetBrowserError, NotImplementedError, PageNotAvailableError,
+    SeleniumError, StopRunOnError, GridError, GetBrowserError, NotImplementedError, PageNotAvailableError, GridConcurrencyError,
 } = require('../errors');
 
 const { GET_BROWSER_TIMEOUT_MSG, TEST_START_TIMEOUT_MSG, TEST_COMPLETE_TIMEOUT_MSG } = timeoutMessages;
@@ -166,7 +166,12 @@ class BaseWorker {
                 try {
                     return await utils.promiseTimeout(this.getSlotOnce(testRunHandler), this.options.getBrowserTimeout, timeoutMessages.GET_BROWSER_TIMEOUT_MSG);
                 } catch (error) {
-                    logger.error('error getting grid slot', { error, testId: this.testId, testResultId: this.testResultId, executionId: this.executionId });
+                    const logDetails = { testId: this.testId, testResultId: this.testResultId, executionId: this.executionId };
+                    if (error instanceof GridConcurrencyError) {
+                        logger.info('could not get grid slot due to concurrency issue', logDetails);
+                    } else {
+                        logger.error('error getting grid slot', { error, ...logDetails });
+                    }
                     failedGetSlotAttempts++;
                     await utils.delay(this.options.getBrowserTimeout - (Date.now() - startTime));
                     throw error;