@testim/testim-cli 3.252.0 → 3.254.0

package/{utils.js → utils/index.js}

@@ -1,16 +1,18 @@
 'use strict';
 
+const os = require('os');
+const _ = require('lodash');
+const path = require('path');
 const moment = require('moment');
 const pRetry = require('p-retry');
-const _ = require('lodash');
 const Promise = require('bluebird');
-const fs = Promise.promisifyAll(require('fs'));
-const { W3C_ELEMENT_ID } = require('./player/constants');
-const { sessionType, testStatus: testStatusConst } = require('./commons/constants');
-const path = require('path');
-const httpRequest = require('./commons/httpRequest');
 const decompress = require('decompress');
-const os = require('os');
+const fs = Promise.promisifyAll(require('fs'));
+const stringUtils = require('./stringUtils');
+const promiseUtils = require('./promiseUtils');
+const httpRequest = require('../commons/httpRequest');
+const { W3C_ELEMENT_ID } = require('../player/constants');
+const { sessionType, testStatus: testStatusConst } = require('../commons/constants');
 
 const HOMEDIR = os.homedir();
 const TESTIM_BROWSER_DIR = path.join(HOMEDIR, '.testim-browser-profile');
@@ -43,6 +45,7 @@ const BROWSERS = [
     { browserName: 'Firefox', bs: { browser: 'Firefox', browser_version: '89' }, sl: { browserName: 'firefox', version: '89.0' }, browserValue: 'firefox' },
     { browserName: 'Safari', bs: { browser: 'Safari' }, sl: { browserName: 'safari' }, browserValue: 'safari' },
     { browserName: 'Edge', bs: { browser: 'Edge', browser_version: '18' }, sl: { browserName: 'MicrosoftEdge', version: '18.17763' }, browserValue: 'edge' },
+    // eslint-disable-next-line max-len
     { browserName: 'Edge Chromium', bs: { browser: 'Edge', browser_version: '94' }, sl: { browserName: 'MicrosoftEdge', version: '94' }, synonyms: ['edge-chromium'], browserValue: 'edge-chromium', seleniumName: 'MicrosoftEdge' },
     { browserName: 'Internet Explorer 11', bs: { browser: 'IE', browser_version: '11' }, sl: { browserName: 'internet explorer', version: '11.0' }, synonyms: ['ie11'], browserValue: 'ie11' },
     { browserName: 'Browser', bs: {}, sl: { browserName: 'Browser' }, browserValue: 'browser' },
@@ -53,7 +56,7 @@ const BROWSERS = [
 
 function getRunConfigByBrowserName(browser, saucelabs, browserstack) {
     browser = browser.toLowerCase();
-    const selectedBrowser = BROWSERS.find(b => b.browserName.toLowerCase() === browser || browser.indexOf(b.synonyms) > -1) || BROWSERS[0];
+    const selectedBrowser = BROWSERS.find(b => b.browserName.toLowerCase() === browser || browser.includes(b.synonyms)) || BROWSERS[0];
 
     // BS and SL do not support Linux for newer browser, so use Windows instead.
     let selectedOS = OSS.find(x => x.osName === 'Windows 10');
@@ -75,22 +78,6 @@ function getRunConfigByBrowserName(browser, saucelabs, browserstack) {
     return _.merge(selectedBrowser, selectedOS);
 }
 
-function getTestUrl(editorUrl, projectId, testId, resultId, branch) {
-    let testUrl = '';
-    branch = branch ? encodeURIComponent(branch) : 'master';
-    if (projectId && testId) {
-        testUrl = `${editorUrl}/#/project/${projectId}/branch/${branch}/test/${testId}`;
-        if (resultId) {
-            testUrl += `?result-id=${resultId}`;
-        }
-    }
-    return testUrl;
-}
-
-function isPromise(obj) {
-    return !!obj && (typeof obj === 'object' || typeof obj === 'function') && typeof obj.then === 'function' && typeof obj.catch === 'function';
-}
-
 function getDuration(ms) {
     const duration = moment.duration(ms);
     return `${duration.hours()}:${duration.minutes()}:${duration.seconds()}.${duration.milliseconds()}`;
@@ -102,7 +89,8 @@ function getDurationSec(ms) {
 
 function getRunnerVersion() {
     try {
-        const pack = require(`${__dirname}/package.json`);
+        /** @type {import('../../package.json')} */
+        const pack = require(`${__dirname}/../package.json`); // eslint-disable-line import/no-dynamic-require
         return pack.version;
     } catch (err) {
         return '';
@@ -111,7 +99,8 @@ function getRunnerVersion() {
 
 function getEnginesVersion() {
     try {
-        const pack = require(`${__dirname}/package.json`);
+        /** @type {import('../../package.json')} */
+        const pack = require(`${__dirname}/../package.json`); // eslint-disable-line import/no-dynamic-require
         return pack.engines.node;
     } catch (err) {
         return '';
@@ -120,7 +109,8 @@ function getEnginesVersion() {
 
 async function getEnginesVersionAsync() {
     try {
-        const pack = JSON.parse(await fs.readFileAsync(`${__dirname}/package.json`));
+        /** @type {import('../../package.json')} */
+        const pack = JSON.parse(await fs.readFileAsync(`${__dirname}/../package.json`));
         return pack.engines.node;
     } catch (err) {
         return '';
@@ -138,19 +128,6 @@ function getUniqBrowsers(options, testList) {
     return [options.browser.toLowerCase()];
 }
 
-function randomString(length) {
-    const a = 'qwertyuiopasdfghjklzxcvbnmQWERTYUIOPASDFGHJKLZXCVBNM1234567890';
-    const index = (Math.random() * (a.length - 1)).toFixed(0);
-    return length > 0 ? a[index] + randomString(length - 1) : '';
-}
-
-function blueBirdify(fn) {
-    return new Promise((resolve, reject) =>
-        fn()
-            .then(x => resolve(x))
-            .catch(y => reject(y)));
-}
-
 function removePropertyFromObject(obj, propName, cmpFunction) {
     for (const prop in obj) {
         if (obj.hasOwnProperty(prop)) {
@@ -169,72 +146,18 @@ function getCliLocation() {
         return process.cwd(); // fall back on the current working directory
     }
     if (require.main.filename.includes('/src') || require.main.filename.includes('\\src') || process.env.IS_UNIT_TEST) {
-        cliLocation = path.resolve(__dirname, '../');
+        cliLocation = path.resolve(__dirname, '../../');
     } else {
-        cliLocation = __dirname;
+        cliLocation = path.resolve(__dirname, '../');
     }
 
     return cliLocation;
 }
 
-function buildBasicHeader(userName, password) {
-    const userAndPasswordBase64 = Buffer.from(`${userName}:${password}`).toString('base64');
-    return `Basic ${userAndPasswordBase64}`;
-}
-
 function extractElementId(element) {
     return element.ELEMENT || element[W3C_ELEMENT_ID];
 }
 
-function isURL(path) {
-    const legacyPattern = /^(https?:\/\/)/i;
-
-    // https://gist.github.com/dperini/729294 (validator.js based on).
-    const pattern = new RegExp(
-        '^' +
-        // protocol identifier (optional)
-        // short syntax // still required
-        '(?:(?:(?:https?|ftp):)?\\/\\/)' +
-        // user:pass BasicAuth (optional)
-        '(?:\\S+(?::\\S*)?@)?' +
-        '(?:' +
-        // IP address exclusion
-        // private & local networks
-        '(?!(?:10|127)(?:\\.\\d{1,3}){3})' +
-        '(?!(?:169\\.254|192\\.168)(?:\\.\\d{1,3}){2})' +
-        '(?!172\\.(?:1[6-9]|2\\d|3[0-1])(?:\\.\\d{1,3}){2})' +
-        // IP address dotted notation octets
-        // excludes loopback network 0.0.0.0
-        // excludes reserved space >= 224.0.0.0
-        // excludes network & broadcast addresses
-        // (first & last IP address of each class)
-        '(?:[1-9]\\d?|1\\d\\d|2[01]\\d|22[0-3])' +
-        '(?:\\.(?:1?\\d{1,2}|2[0-4]\\d|25[0-5])){2}' +
-        '(?:\\.(?:[1-9]\\d?|1\\d\\d|2[0-4]\\d|25[0-4]))' +
-        '|' +
-        // host & domain names, may end with dot
-        // can be replaced by a shortest alternative
-        // (?![-_])(?:[-\\w\\u00a1-\\uffff]{0,63}[^-_]\\.)+
-        '(?:' +
-        '(?:' +
-        '[a-z0-9\\u00a1-\\uffff]' +
-        '[a-z0-9\\u00a1-\\uffff_-]{0,62}' +
-        ')?' +
-        '[a-z0-9\\u00a1-\\uffff]\\.' +
-        ')+' +
-        // TLD identifier name, may end with dot
-        '(?:[a-z\\u00a1-\\uffff]{2,}\\.?)' +
-        ')' +
-        // port number (optional)
-        '(?::\\d{2,5})?' +
-        // resource path (optional)
-        '(?:[/?#]\\S*)?' +
-        '$', 'i'
-    );
-
-    return legacyPattern.test(path) || pattern.test(path);
-}
-
 const DOWNLOAD_RETRY = 3;
 const download = async (url) => pRetry(() => httpRequest.download(url), { retries: DOWNLOAD_RETRY });
 
@@ -255,7 +178,7 @@ const copy = async (readFile, destFile) => new Promise((resolve, reject) => {
     }
 });
 function getSourcePath(location, fileName) {
-    if (isURL(location)) {
+    if (stringUtils.isURL(location)) {
         return fileName || path.join(process.cwd(), location.replace(/^.*[\\\/]/, ''));
     }
 
@@ -264,7 +187,7 @@ function getSourcePath(location, fileName) {
 
 const getSource = async (location, fileName) => {
     const destFile = getSourcePath(location, fileName);
-    if (isURL(location)) {
+    if (stringUtils.isURL(location)) {
         return downloadAndSave(location, destFile);
     }
 
@@ -272,7 +195,7 @@ const getSource = async (location, fileName) => {
 };
 
 const getSourceAsBuffer = async (location) => {
-    if (isURL(location)) {
+    if (stringUtils.isURL(location)) {
         return download(location);
     }
     return fs.readFileAsync(location);
@@ -304,16 +227,6 @@ function getPlanType(plan) {
     return 'free';
 }
 
-/**
- * @param time {number} in ms
- * @returns {Promise}
- */
-function delay(time) {
-    return new Promise(((resolve) => {
-        setTimeout(resolve, time);
-    }));
-}
-
 const calcPercentile = (arr, percentile) => {
     if (arr.length === 0) return 0;
     if (typeof percentile !== 'number') throw new TypeError('p must be a number');
@@ -327,7 +240,7 @@ const calcPercentile = (arr, percentile) => {
     return arr[index];
 };
 
-const hasTestPlanFlag = (options) => (options.testPlan && options.testPlan.length) || (options.testPlanIds && options.testPlanIds.length);
+const hasTestPlanFlag = (options) => Boolean(options.testPlan?.length || options.testPlanIds?.length);
 
 const isRemoteRun = (options) => options.resultId && options.source === 'remote-run';
 
@@ -396,22 +309,16 @@ function getArgsOnRemoteRunFailure() {
 module.exports = {
     TESTIM_BROWSER_DIR,
     removePropertyFromObject,
-    getTestUrl,
     getDuration,
     getDurationSec,
     getRunnerVersion,
     getEnginesVersion,
     getEnginesVersionAsync,
-    isPromise,
     getEnvironmentGitBranch,
     getUniqBrowsers,
-    guid: (n = 16) => randomString(n),
     getRunConfigByBrowserName,
-    blueBirdify,
-    buildBasicHeader,
     extractElementId,
     getCliLocation,
-    isURL,
     download,
     downloadAndSave,
     copy,
@@ -427,7 +334,8 @@ module.exports = {
     isQuarantineAndNotRemoteRun,
     groupTestsByRetries,
     getPlanType,
-    delay,
     getCdpAddressForHost,
     getArgsOnRemoteRunFailure,
+    ...promiseUtils,
+    ...stringUtils,
 };

package/utils/promiseUtils.js

@@ -0,0 +1,78 @@
+// @ts-check
+
+const Bluebird = require('bluebird');
+const { TimeoutError } = require('../errors');
+
+function isPromise(obj) {
+    return !!obj && (typeof obj === 'object' || typeof obj === 'function') && typeof obj.then === 'function' && typeof obj.catch === 'function';
+}
+
+/**
+ * @deprecated This is horrifying. Please stop what you are doing and think about what you just considered!
+ * @template T
+ * @param {() => Promise<T>} fn
+ * @returns {Bluebird<T>}
+ */
+function blueBirdify(fn) {
+    return new Bluebird((resolve, reject) =>
+        fn()
+            .then(x => resolve(x))
+            .catch(y => reject(y)));
+}
+
+/**
+ * @param time {number} in ms
+ * @returns {Promise}
+ */
+function delay(time) {
+    return new Promise(((resolve) => {
+        setTimeout(resolve, time);
+    }));
+}
+
+/**
+ * @template T
+ * @param {Promise<T> | undefined} promise
+ * @param {number} timeout
+ * @param {string=} errMsg
+ */
+function promiseTimeout(promise, timeout, errMsg = 'Timeout Error') {
+    return Promise.race([
+        promise,
+        delay(timeout).then(() => { throw new TimeoutError(errMsg); }),
+    ]);
+}
+
+/**
+ * @template T, U
+ * @param {ArrayLike<T> | Iterable<T>} arr
+ * @param {(item: T, index: number) => PromiseLike<U>} handler
+ */
+function promiseMap(arr, handler) {
+    return Promise.all(Array.from(arr, handler));
+}
+
+/**
+ * @template ErrT, T
+ * @param {(callback: (err: ErrT, result: T) => void) => void} resolver
+ * @returns {Promise<T>}
+ */
+function promiseFromCallback(resolver) {
+    return new Promise((resolve, reject) => {
+        resolver((err, result) => {
+            if (err) {
+                return reject(err);
+            }
+            return resolve(result);
+        });
+    });
+}
+
+module.exports = {
+    isPromise,
+    blueBirdify,
+    delay,
+    promiseTimeout,
+    promiseMap,
+    promiseFromCallback,
+};

package/utils/stringUtils.js

@@ -0,0 +1,96 @@
+// @ts-check
+
+/**
+ * @param {number} length
+ * @returns {string}
+ */
+function randomString(length) {
+    const a = 'qwertyuiopasdfghjklzxcvbnmQWERTYUIOPASDFGHJKLZXCVBNM1234567890';
+    const index = (Math.random() * (a.length - 1)).toFixed(0);
+    return length > 0 ? a[index] + randomString(length - 1) : '';
+}
+
+/**
+ * @param {string} editorUrl
+ * @param {string} projectId
+ * @param {string} testId
+ * @param {string | undefined} resultId
+ * @param {string} branch
+ */
+function getTestUrl(editorUrl, projectId, testId, resultId, branch) {
+    let testUrl = '';
+    branch = branch ? encodeURIComponent(branch) : 'master';
+    if (projectId && testId) {
+        testUrl = `${editorUrl}/#/project/${projectId}/branch/${branch}/test/${testId}`;
+        if (resultId) {
+            testUrl += `?result-id=${resultId}`;
+        }
+    }
+    return testUrl;
+}
+
+/**
+ * @param {string} userName
+ * @param {string} password
+ */
+function buildBasicHeader(userName, password) {
+    const userAndPasswordBase64 = Buffer.from(`${userName}:${password}`).toString('base64');
+    return `Basic ${userAndPasswordBase64}`;
+}
+
+/** @param {string} path */
+function isURL(path) {
+    const legacyPattern = /^(https?:\/\/)/i;
+
+    // https://gist.github.com/dperini/729294 (validator.js based on).
+    const pattern = new RegExp(
+        '^' +
+        // protocol identifier (optional)
+        // short syntax // still required
+        '(?:(?:(?:https?|ftp):)?\\/\\/)' +
+        // user:pass BasicAuth (optional)
+        '(?:\\S+(?::\\S*)?@)?' +
+        '(?:' +
+        // IP address exclusion
+        // private & local networks
+        '(?!(?:10|127)(?:\\.\\d{1,3}){3})' +
+        '(?!(?:169\\.254|192\\.168)(?:\\.\\d{1,3}){2})' +
+        '(?!172\\.(?:1[6-9]|2\\d|3[0-1])(?:\\.\\d{1,3}){2})' +
+        // IP address dotted notation octets
+        // excludes loopback network 0.0.0.0
+        // excludes reserved space >= 224.0.0.0
+        // excludes network & broadcast addresses
+        // (first & last IP address of each class)
+        '(?:[1-9]\\d?|1\\d\\d|2[01]\\d|22[0-3])' +
+        '(?:\\.(?:1?\\d{1,2}|2[0-4]\\d|25[0-5])){2}' +
+        '(?:\\.(?:[1-9]\\d?|1\\d\\d|2[0-4]\\d|25[0-4]))' +
+        '|' +
+        // host & domain names, may end with dot
+        // can be replaced by a shortest alternative
+        // (?![-_])(?:[-\\w\\u00a1-\\uffff]{0,63}[^-_]\\.)+
+        '(?:' +
+        '(?:' +
+        '[a-z0-9\\u00a1-\\uffff]' +
+        '[a-z0-9\\u00a1-\\uffff_-]{0,62}' +
+        ')?' +
+        '[a-z0-9\\u00a1-\\uffff]\\.' +
+        ')+' +
+        // TLD identifier name, may end with dot
+        '(?:[a-z\\u00a1-\\uffff]{2,}\\.?)' +
+        ')' +
+        // port number (optional)
+        '(?::\\d{2,5})?' +
+        // resource path (optional)
+        '(?:[/?#]\\S*)?' +
+        '$', 'i'
+    );
+
+    return legacyPattern.test(path) || pattern.test(path);
+}
+
+module.exports = {
+    isURL,
+    getTestUrl,
+    buildBasicHeader,
+    guid: (n = 16) => randomString(n),
+};

package/{utils.test.js → utils/utils.test.js}

@@ -1,5 +1,5 @@
-const chai = require('chai');
-const utils = require('./utils');
+const chai = require('chai'); // eslint-disable-line import/no-extraneous-dependencies
+const utils = require('./index');
 
 const expect = chai.expect;
 

package/workers/BaseWorker.js

@@ -1,11 +1,10 @@
 'use strict';
 
-const Bluebird = require('bluebird');
 const moment = require('moment');
 const pRetry = require('p-retry');
 const ms = require('ms');
 
-const { timeoutMessages, testRunStatus, stepResult, runnerTestStatus } = require('../commons/constants');
+const { timeoutMessages, testRunStatus, stepResult, runnerTestStatus, CLI_MODE } = require('../commons/constants');
 const logger = require('../commons/logger').getLogger('base-worker');
 const testResultService = require('../commons/socket/testResultService');
 const remoteStepService = require('../commons/socket/remoteStepService');
@@ -110,8 +109,15 @@ class BaseWorker {
         return undefined;
     }
 
+    setSessionTimeout() {
+        if (this.options.mode === CLI_MODE.APPIUM) {
+            return Math.max(ms('180s'), this.options.getSessionTimeout);
+        }
+        return Math.max(this.lambdatestService.getSessionTimeout, this.options.getSessionTimeout);
+    }
+
     async getTestPlayer(testRunHandler, customExtensionLocalLocation) {
-        const projectId = this.userData && this.userData.projectId;
+        const projectId = this.userData?.projectId;
         let testPlayer;
 
         try {
@@ -121,8 +127,7 @@ class BaseWorker {
             let gridInfo = await pRetry(async () => {
                 const startTime = Date.now();
                 try {
-                    return await Bluebird.resolve(this.getSlotOnce(testRunHandler))
-                        .timeout(this.options.getBrowserTimeout, timeoutMessages.GET_BROWSER_TIMEOUT_MSG);
+                    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 });
                     failedGetSlotAttempts++;
@@ -148,12 +153,14 @@ class BaseWorker {
                     this.options.gridData.provider = gridInfo.provider;
                     this.options.gridData.host = gridInfo.host;
                     this.options.gridData.failedGetBrowserAttempts = failedGetBrowserAttempts;
-                    const getSessionTimeout = Math.max(this.lambdatestService.getSessionTimeout, this.options.getSessionTimeout);
-                    const getBrowserRes = await Bluebird.resolve()
-                        .log('before getBrowserOnce')
-                        .then(() => this.getBrowserOnce(testRunHandler, customExtensionLocalLocation, player, gridInfo))
-                        .log('after getBrowserOnce')
-                        .timeout(getSessionTimeout, timeoutMessages.GET_BROWSER_TIMEOUT_MSG);
+                    const getSessionTimeout = this.setSessionTimeout();
+                    perf.log('before getBrowserOnce');
+                    const getBrowserRes = await utils.promiseTimeout(
+                        this.getBrowserOnce(testRunHandler, customExtensionLocalLocation, player, gridInfo),
+                        getSessionTimeout,
+                        timeoutMessages.GET_BROWSER_TIMEOUT_MSG,
+                    );
+                    perf.log('after getBrowserOnce');
                     reporter.onGetBrowserSuccess(this.id, projectId);
                     return player || getBrowserRes;
                 } catch (error) {
@@ -186,7 +193,7 @@ class BaseWorker {
 
     async runTest(testRunHandler, customExtensionLocalLocation, shouldRerun) {
         perf.log('inside runTest');
-        const projectId = this.userData && this.userData.projectId;
+        const projectId = this.userData?.projectId;
         const quarantineResult = this.handleQuarantine(testRunHandler);
         if (quarantineResult) {
             return quarantineResult;
@@ -236,7 +243,7 @@ class BaseWorker {
                 const testRetryKey = testRunHandler.getRetryKey();
                 testResult.testRetryKey = testRetryKey;
                 await this.onTestCompleted(this.id, this.testId, testResult, sessionId, shouldRerun);
-                if (this.executionQueue.hasMoreTests() && !(this.options.lightweightMode && this.options.lightweightMode.general)) {
+                if (this.executionQueue.hasMoreTests() && !this.options.lightweightMode?.general) {
                     await utils.delay(DELAY_BETWEEN_TESTS);
                 }
                 await this.runTestCleanup();
@@ -257,11 +264,11 @@ class BaseWorker {
                     return await runTestAndCalcResult(testRunHandler, shouldRerun);
                 }
                 return await runNextTest();
-            } catch (err) {
-                if (err instanceof StopRunOnError) {
+            } catch (error) {
+                if (error instanceof StopRunOnError) {
                     return undefined;
                 }
-                logger.error('failed to process test result', { err });
+                logger.error('failed to process test result', { error });
                 runNextTest();
                 return undefined;
             }
@@ -335,7 +342,7 @@ class BaseWorker {
             }
             logger.warn('error on run', { err });
 
-            const projectId = this.userData && this.userData.projectId;
+            const projectId = this.userData?.projectId;
             const { errorType, reason } = buildError(err, wasNetworkHealthy);
             testimServicesApi.updateTestResult(projectId, this.testResultId, this.testId, {
                 status: testRunStatus.COMPLETED,
@@ -351,7 +358,7 @@ class BaseWorker {
         const recoverTestResults = async (runError, testRunHandler) => {
             const testId = this.testId;
             const resultId = this.testResultId;
-            const projectId = this.userData && this.userData.projectId;
+            const projectId = this.userData?.projectId;
             const branch = this.branch;
             if (!testId || !resultId || !projectId || !branch) {
                 // Not enough data to call the API
@@ -380,8 +387,8 @@ class BaseWorker {
             }
         };
 
-        const disableResults = this.options.disableSockets || (this.options.lightweightMode && this.options.lightweightMode.disableResults && (this.options.useChromeLauncher || this.options.mode !== 'extension'));
-        const disableRemoteStep = this.options.disableSockets || (this.options.lightweightMode && this.options.lightweightMode.disableRemoteStep);
+        const disableResults = this.options.disableSockets || (this.options.lightweightMode?.disableResults && (this.options.useChromeLauncher || this.options.mode !== 'extension'));
+        const disableRemoteStep = this.options.disableSockets || (this.options.lightweightMode?.disableRemoteStep);
 
         const runTestAndCalcResult = (testRunHandler, shouldRerun) => Promise.all([
             !disableRemoteStep && remoteStepService.joinToRemoteStep(this.testResultId),
@@ -402,6 +409,7 @@ class BaseWorker {
             });
 
         const testRunHandler = this.executionQueue.getNext();
+
         if (!testRunHandler) { // no more tests to run
             return this.onQueueCompleted();
         }
@@ -411,6 +419,7 @@ class BaseWorker {
         this.overrideTestConfigId = testRunHandler.getOverrideTestConfigId();
         this.testRunConfig = testRunHandler.getRunConfig();
         this.branch = testRunHandler.getBranch();
+
         return runTestAndCalcResult(testRunHandler);
     }
 }