@testim/testim-cli 3.189.0 → 3.193.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testim/testim-cli",
-  "version": "3.189.0",
+  "version": "3.193.0",
   "description": "Command line interface for running Testing on your CI",
   "author": "Oren Rubin",
   "contributors": [{
@@ -22,7 +22,6 @@
     "gulp": "4.0.2",
     "gulp-cli": "2.3.0",
     "gulp-json-editor": "2.5.4",
-    "gulp-mocha": "7.0.2",
     "gulp-preprocess": "3.0.3",
     "husky": "4.3.8",
     "merge-stream": "2.0.0",
@@ -79,7 +78,7 @@
     "istanbul-lib-report": "3.0.0",
     "istanbul-reports": "3.0.2",
     "jimp": "0.2.28",
-    "jsdom": "14.0.0",
+    "jsdom": "16.7.0",
     "jsonwebtoken": "8.5.1",
     "lodash": "4.17.20",
     "memory-fs": "0.5.0",
@@ -96,7 +95,7 @@
     "promise-queue": "2.2.5",
     "prompts": "2.3.2",
     "proxy-agent": "5.0.0",
-    "rox-node": "4.9.6",
+    "rox-node": "4.9.18",
     "semver": "7.3.2",
     "serialize-error": "7.0.1",
     "socket.io-client": "2.4.0",
@@ -119,6 +118,7 @@
     "test": "IS_UNIT_TEST=1 ./node_modules/mocha/bin/_mocha --timeout 2000  --reporter spec --exit --recursive \"./src/**/*.test.js\" --exclude ./src/codim/template.js/tests/examples/**/*.test.js",
     "test:watch": "IS_UNIT_TEST=1 ./node_modules/mocha/bin/_mocha --timeout 2000 --exit --recursive \"./src/**/*.test.js\"  --exclude ./src/codim/template.js/tests/examples/**/*.test.js --watch",
     "test:cov": "nyc --reporter=lcov --reporter=text yarn test",
+    "upload-bundle-s3": "scripts/upload.js",
     "prepare-version": "rm -rf ./deploy && mkdir -p deploy && gulp prepare-version-on-prem",
     "make-onprem-deps": "cp ../../yarn.lock deploy && cd deploy && yarn install --production && bundle-deps",
     "pack-onprem": "yarn prepare-version && yarn make-onprem-deps && cd deploy && npm pack && zip -r testim-cli.zip testim-cli-*.tgz",

package/player/seleniumTestPlayer.js

@@ -1,4 +1,4 @@
-"use strict";
+'use strict';
 
 const TabService = require('./services/tabService');
 const PortSelector = require('./services/portSelector');
@@ -24,9 +24,6 @@ const WebDriver = require('./webdriver');
 const CryptoJS = require('crypto-js');
 const StepActionUtils = require('./utils/stepActionUtils');
 
-// Set JSDOM SVGElement on Global to support text validation on SVG
-global.SVGElement = require("jsdom/lib/jsdom/living/generated/SVGSVGElement").interface;
-
 class SeleniumTestPlayer {
     constructor(id, userParamsData, shouldMonitorPerformance, automationMode = 'code', driver = new WebDriver(), testRetryCount, previousTestResultId) {
         this.driver = driver;
@@ -34,7 +31,7 @@ class SeleniumTestPlayer {
 
         const stepActionUtils = new StepActionUtils(this.driver);
         this.stepActionFactory = new StepActionFactory(stepActionUtils);
-        require('./stepActions/stepActionRegistrar')(this.driver, this.stepActionFactory, "selenium");
+        require('./stepActions/stepActionRegistrar')(this.driver, this.stepActionFactory, 'selenium');
 
         if (assetService.setMd5) {
             // delete after https://github.com/testimio/clickim/pull/3430 release to the store
@@ -74,9 +71,9 @@ class SeleniumTestPlayer {
     }
 
     onStepCompleted(result, testId, resultId, step) {
-        if(step && step.isTabOpener) {
+        if (step && step.isTabOpener) {
             this.tabService.addNewPopup(this.id, step.id)
-                .catch(() => {});
+                .catch(() => { });
         }
     }
 
@@ -84,10 +81,8 @@ class SeleniumTestPlayer {
         const END_DRIVER_TIMEOUT = 1000 * 60 * 2;
         return this.driver.end()
             .timeout(END_DRIVER_TIMEOUT)
-            .catch(Promise.TimeoutError, () => {
-                return this.driver.forceEnd();
-            })
-            .catch(() => {})
+            .catch(Promise.TimeoutError, () => this.driver.forceEnd())
+            .catch(() => { })
             .then(() => {
                 this.sessionPlayer.playbackManager.off(commonConstants.playback.RESULT, this.onStepCompleted);
                 this.sessionPlayer = null;
@@ -101,10 +96,10 @@ class SeleniumTestPlayer {
         this.tabService.clearAllTabs(this.id);
     }
 
-    addTab(openerStepId, options = { loadInfo: true}) {
+    addTab(openerStepId, options = { loadInfo: true }) {
         return this.driver.getTabIds()
             .tap(ids => this.tabService.addNewTab(this.id, ids[ids.length - 1], openerStepId, options))
-            .then(ids => this.sessionPlayer.addPlaybackFrameHandler(ids[ids.length - 1], undefined, {emptyPage: true}))
+            .then(ids => this.sessionPlayer.addPlaybackFrameHandler(ids[ids.length - 1], undefined, { emptyPage: true }));
     }
 
     async addAllTabs(openerStepId, options = { loadInfo: true, checkForMainTab: true, takeScreenshots: true }, blackList = []) {
@@ -112,14 +107,14 @@ class SeleniumTestPlayer {
         // the ids are reversed so we search first in the last tab opened - otherwise it starts looking from the testim editor and not the AUT
 
         const PROHIBITED_URLS = ['app.testim.io'].concat(blackList);
-        for(const id of ids.reverse()) {
+        for (const id of ids.reverse()) {
             await this.tabService.addNewTab(this.id, id, openerStepId, { ...options, forceSwitch: true });
             const tabInfo = this.tabService.getTabInfo(this.id, id);
             if (PROHIBITED_URLS.some(bad => tabInfo.url.includes(bad))) {
                 await this.tabService.removeTabInfo(this.id, id);
                 continue;
             }
-            await this.sessionPlayer.addPlaybackFrameHandler(id, undefined, {emptyPage: true });
+            await this.sessionPlayer.addPlaybackFrameHandler(id, undefined, { emptyPage: true });
         }
         if (this.tabService.tabCount(this.id) === 1) {
             // if we only have one tab because we removed the editor tab - we have to switchTab to one of the other tabs, otherwise
@@ -130,7 +125,6 @@ class SeleniumTestPlayer {
         }
         // deal with checkForMainTab failing due to the page refreshing or JavaScript not responding or a similar issue
         this.tabService.fixMissingMainTab(this.id);
-
     }
 
     getSessionId() {

package/player/stepActions/mouseStepAction.js

@@ -111,8 +111,9 @@ class MouseStepAction extends StepAction {
             this.step.events.splice(mousedownEventIndex, 0, this.generateEventOfType(downEvent, 'mouseover'));
         }
 
-        const mouseUpEvent = this.step.events.find(event => event.event === 'mouseup') || this.step.events.find(event => event.event === 'pointerup');
-        const lastMouseMoveEventIndex = _.findLastIndex(this.step.events, event => event.event === 'mousemove') || _.findLastIndex(this.step.events, event => event.event === 'pointermove');
+        const { recordPointerMoveEvents = false } = this.context.project.defaults || {};
+        const mouseUpEvent = this.step.events.find(event => event.event === 'mouseup') || (recordPointerMoveEvents && this.step.events.find(event => event.event === 'pointerup'));
+        const lastMouseMoveEventIndex = _.findLastIndex(this.step.events, event => event.event === 'mousemove') || (recordPointerMoveEvents && _.findLastIndex(this.step.events, event => event.event === 'pointermove'));
         if (mouseUpEvent && lastMouseMoveEventIndex > 0 && !this.step.allEventsOnSameElement) {
             this.step.events.splice(lastMouseMoveEventIndex + 1, 0, this.generateEventOfType(mouseUpEvent, 'mouseover'));
         }
@@ -249,7 +250,7 @@ class MouseStepAction extends StepAction {
                         var done = arguments[1];
                         return dnd.call(null, eventData, done);
                     `;
-                        return this.driver.executeCodeAsync(html5DNDCode, timeout, eventParam)
+                        return this.driver.executeCodeAsync(html5DNDCode, timeout, eventParam);
                     }
 
                     const html5DNDCode = `
@@ -313,7 +314,7 @@ class MouseStepAction extends StepAction {
         if (events.find(event => event.event === 'dragend')) {
             return events;
         }
-        var dragendDefaultEvent = {
+        const dragendDefaultEvent = {
             event: 'dragend',
             eventInfo: {
                 detail: 0,
@@ -365,8 +366,8 @@ class MouseStepAction extends StepAction {
     }
 
     generateHTML5DragEventSequence() {
-        var fromElement = this.context.data.targetId;
-        var toElement = this.context.data.toElement;
+        const fromElement = this.context.data.targetId;
+        const toElement = this.context.data.toElement;
         let events = this.step.events.filter(event => event.event !== 'mousemove' && event.event !== 'pointermove');
         events = this.fixEventSequence(events);
         events = this.addElementLocatedElementToDragEvents(events, fromElement.locatedElement, toElement.locatedElement);

package/player/stepActions/stepAction.js

@@ -1,10 +1,9 @@
-"use strict";
+'use strict';
 
 const { commonConstants } = require('../../commons/getSessionPlayerRequire');
 const Promise = require('bluebird');
 
 class StepAction {
-
     constructor(step, context, frameHandler, exportsGlobal = {}, stepActionUtils, locateElementPlayer, exportsTest = {}) {
         this.step = step;
         this.context = context;
@@ -21,21 +20,19 @@ class StepAction {
     }
 
     performAction() {
-        throw new Error("not implemented");
+        throw new Error('not implemented');
     }
 
     getTarget() {
-        const targetId = this.step.targetId || "targetId";
+        const targetId = this.step.targetId || 'targetId';
         return this.context.data[targetId];
     }
 
     execute(stepActionFactory, step) {
         return Promise.resolve(this.performAction(stepActionFactory, step))
-            .then(res => {
-                return Promise.resolve(Object.assign({}, {success: true}, res));
-            })
+            .then(res => Promise.resolve(Object.assign({}, { success: true }, res)))
             .catch(err => {
-                const errorMsg = (err || {}).message || err && err.seleniumStack && err.seleniumStack.message;
+                const errorMsg = (err || {}).message || (err && err.seleniumStack && err.seleniumStack.message);
                 const displayMsg = (err || {}).displayMessage;
                 return Promise.resolve({
                     success: false,
@@ -43,11 +40,11 @@ class StepAction {
                     exception: err,
                     errorType: commonConstants.stepResult.ACTION_EXCEPTION,
                     resultInfo: {
-                        exception: "selenium exception: " + errorMsg,
+                        exception: `selenium exception: ${errorMsg}`,
                         // clickim -> playbackStepResultHandler.js -> FAILURE_REASON_MAPPING -> ACTION_EXCEPTION
                         // expects resultInfo.error or resultInfo.reason
                         error: displayMsg || errorMsg,
-                    }
+                    },
                 });
             });
     }

package/player/stepActions/stepActionRegistrar.js

@@ -1,4 +1,4 @@
-"use strict";
+'use strict';
 
 const LocateStepAction = require('./locateStepAction');
 const ScrollStepAction = require('./scrollStepAction');
@@ -48,69 +48,69 @@ function register(stepActionByType, stepActionFactory) {
 
 module.exports = function (driver, stepActionFactory, runMode) {
     const STEP_ACTION_MAPPING = {
-        "locate": LocateStepAction,
-        "scroll": ScrollStepAction,
-        "mouse": MouseStepAction,
-        "submit": SubmitStepAction,
-        "text": TextStepAction,
-        "special-key": SpecialKeyStepAction,
-        "user-code": JsCodeStepAction,
-        "validation-code-step": JsCodeStepAction,
-        "wait-for-code-step": JsCodeStepAction,
-        "action-code-step": JsCodeStepAction,
-        "condition-step": JsConditionStepAction,
-        "skip-code-step": JsConditionStepAction,
-        "element-code-step": JsConditionStepAction,
-        "evaluate-expression": EvaluateExpressionStepAction,
-        "text-validation": TextValidationStepAction,
-        "wait-for-text-validation": TextValidationStepAction,
-        "select-option": SelectOptionStepAction,
-        "drop-file": DropFileStepAction,
-        "input-file": InputFileStepAction,
-        "hover": HoverStepAction,
-        "navigation": NavigationStepAction,
-        "wheel": WheelStepAction,
-        "sleep": SleepStepAction,
-        "refresh": RefreshStepAction,
-        "api-validation": ApiStepAction,
-        "api-action": ApiStepAction,
-        "api-code-step": JsCodeStepAction,
-        "extract-text": ExtractTextStepAction,
-
-        "cli-validation-download-file": ExtensionOnlyStepAction,
-        "cli-wait-for-download-file": ExtensionOnlyStepAction,
-        "network-validation-step": ExtensionOnlyStepAction,
-
-        "cli-validation-code-step": CliJsStepAction,
-        "cli-wait-for-code-step": CliJsStepAction,
-        "cli-action-code-step": CliJsStepAction,
-        "cli-api-code-step": CliJsStepAction,
-
-        "cli-condition-step": CliConditionStepAction,
-        "node-package": NodePackageStepAction,
-
-        "email-code-step": JsCodeStepAction,
-        "cli-email-code-step": CliJsStepAction,
-        "tdk-hybrid": TdkHybridStepAction,
+        locate: LocateStepAction,
+        scroll: ScrollStepAction,
+        mouse: MouseStepAction,
+        submit: SubmitStepAction,
+        text: TextStepAction,
+        'special-key': SpecialKeyStepAction,
+        'user-code': JsCodeStepAction,
+        'validation-code-step': JsCodeStepAction,
+        'wait-for-code-step': JsCodeStepAction,
+        'action-code-step': JsCodeStepAction,
+        'condition-step': JsConditionStepAction,
+        'skip-code-step': JsConditionStepAction,
+        'element-code-step': JsConditionStepAction,
+        'evaluate-expression': EvaluateExpressionStepAction,
+        'text-validation': TextValidationStepAction,
+        'wait-for-text-validation': TextValidationStepAction,
+        'select-option': SelectOptionStepAction,
+        'drop-file': DropFileStepAction,
+        'input-file': InputFileStepAction,
+        hover: HoverStepAction,
+        navigation: NavigationStepAction,
+        wheel: WheelStepAction,
+        sleep: SleepStepAction,
+        refresh: RefreshStepAction,
+        'api-validation': ApiStepAction,
+        'api-action': ApiStepAction,
+        'api-code-step': JsCodeStepAction,
+        'extract-text': ExtractTextStepAction,
+
+        'cli-validation-download-file': ExtensionOnlyStepAction,
+        'cli-wait-for-download-file': ExtensionOnlyStepAction,
+        'network-validation-step': ExtensionOnlyStepAction,
+
+        'cli-validation-code-step': CliJsStepAction,
+        'cli-wait-for-code-step': CliJsStepAction,
+        'cli-action-code-step': CliJsStepAction,
+        'cli-api-code-step': CliJsStepAction,
+
+        'cli-condition-step': CliConditionStepAction,
+        'node-package': NodePackageStepAction,
+
+        'email-code-step': JsCodeStepAction,
+        'cli-email-code-step': CliJsStepAction,
+        'tdk-hybrid': TdkHybridStepAction,
     };
 
     const ANDROID_STEP_ACTION_MAPPING = {
-        "locate": AndroidLocateStepAction,
-        "android-tap": MobileTapStepAction,
-        "android-long-click": AndroidLongClickStepAction,
+        locate: AndroidLocateStepAction,
+        'android-tap': MobileTapStepAction,
+        'android-long-click': AndroidLongClickStepAction,
         'android-device-key': MobileGlobalActionStepAction,
         'android-key-board-special-keys': AndroidSpecialKeyStepAction,
-        "text": MobileTextChangeStepAction,
-        "android-swipe": AndroidSwipeStepAction,
-        "android-scroll": AndroidScrollStepAction,
+        text: MobileTextChangeStepAction,
+        'android-swipe': AndroidSwipeStepAction,
+        'android-scroll': AndroidScrollStepAction,
     };
 
     const IOS_STEP_ACTION_MAPPING = {
-        "locate": IosLocateStepAction,
-        "ios-tap": MobileTapStepAction,
-        "ios-device-key": MobileGlobalActionStepAction,
-        "text": MobileTextChangeStepAction,
-        "ios-scroll": IosScrollStepAction,
+        locate: IosLocateStepAction,
+        'ios-tap': MobileTapStepAction,
+        'ios-device-key': MobileGlobalActionStepAction,
+        text: MobileTextChangeStepAction,
+        'ios-scroll': IosScrollStepAction,
     };
 
     register(STEP_ACTION_MAPPING, stepActionFactory);
@@ -119,7 +119,7 @@ module.exports = function (driver, stepActionFactory, runMode) {
     }
 
     // override android actions
-    if (runMode === "android") {
+    if (runMode === 'android') {
         register(ANDROID_STEP_ACTION_MAPPING, stepActionFactory);
         if (stepActionFactory.registerLocateStepActionUtils) {
             stepActionFactory.registerLocateStepActionUtils(AndroidLocateStepAction.getUtils(driver));
@@ -127,7 +127,7 @@ module.exports = function (driver, stepActionFactory, runMode) {
     }
 
     // override ios actions
-    if (runMode === "ios") {
+    if (runMode === 'ios') {
         register(IOS_STEP_ACTION_MAPPING, stepActionFactory);
         if (stepActionFactory.registerLocateStepActionUtils) {
             stepActionFactory.registerLocateStepActionUtils(IosLocateStepAction.getUtils(driver));

package/runOptions.js

@@ -348,6 +348,7 @@ program
     .option('--aws-access-key-id [accessKeyId]', 'accessKeyId', '')
     .option('--aws-secret-access-key [secretAccessKey]', 'secretAccessKey', '')
 
+    .option('--high-speed', 'run in high speed mode')
     .option('--lightweight-mode [settings]', 'run lightweight mode')
     .option('--create-prefeched-data [location]', 'prefech data into local cache file')
     .option('--use-prefeched-data [location]', 'use prefeched data from local cache file, and force using only cached data')
@@ -1020,6 +1021,7 @@ module.exports = {
                 const DEFAULTS = {
                     general: true,
                     disableLabs: true,
+                    disableFeatureFlags: true,
                     disableAssets: true,
                     disablePixelValidation: true,
                     disableResults: true,
@@ -1029,10 +1031,13 @@ module.exports = {
                     disableVisibilityCheck: false,
                     disableLocators: false,
                     bypassSetup: false,
+                    disableAutoImprove: true,
                     disableQuotaBlocking: true,
                     onlyTestIdsNoSuite: true,
                     uploadAssetsAndResultsOnFailure: true,
                     preloadTests: true,
+                    disableProjectDefaults: true,
+                    type: 'lightweight',
                 };
 
                 const lightweightModeOptions = typeof program.lightweightMode === 'string' ? JSON.parse(program.lightweightMode) : {};
@@ -1040,8 +1045,33 @@ module.exports = {
             } catch (err) {
                 return Promise.reject(new ArgError(`failed to parse lightweightMode settings error: ${err.message}`));
             }
+        } else if (program.highSpeed && program.mode === CLI_MODE.EXTENSION) {
+            program.lightweightMode = {
+                general: true,
+                disableLabs: false,
+                disableFeatureFlags: false,
+                disableAssets: true,
+                disablePixelValidation: false,
+                disableResults: true,
+                disableStepDelay: true,
+                disableRemoteStep: false,
+                assumePreloadedSharedSteps: false,
+                disableVisibilityCheck: false,
+                disableLocators: false,
+                bypassSetup: false,
+                disableQuotaBlocking: false,
+                disableAutoImprove: false,
+                onlyTestIdsNoSuite: false,
+                uploadAssetsAndResultsOnFailure: true,
+                preloadTests: false,
+                disableProjectDefaults: false,
+                type: 'highSpeed',
+            };
         }
 
+        if (typeof program.baseUrl === 'boolean') {
+            return Promise.reject(new ArgError('base url cannot be used as a flag, and must contain a string value'));
+        }
 
         return ({
             testId: [].concat(program.testId),

package/runner.js

@@ -308,6 +308,11 @@ async function init(options) {
     if (!(options.lightweightMode && options.lightweightMode.disableLabs)) {
         await labFeaturesService.loadLabFeatures(projectById.id, companyByProjectId.activePlan);
     }
+
+    if (options.lightweightMode && options.lightweightMode.type === 'highSpeed' && !labFeaturesService.isFeatureAvailableForProject('highSpeedMode')) {
+        delete options.lightweightMode;
+    }
+
     gridService.keepAlive.start(project);
     analyticsIdentify(project);
     await setMockNetworkRules(options);

package/runners/strategies/BaseStrategy.js

@@ -23,11 +23,18 @@ const calcSource = (source, user) => {
     return source;
 };
 
+function setLightweightAnalytics(properties, lightweightMode) {
+    if (lightweightMode && lightweightMode.type) {
+        properties[`${lightweightMode.type}Mode`] = true;
+    }
+    return properties;
+}
+
 class BaseStrategy {
     analyticsTestStart({
         executionId, projectId, testId, resultId, companyId, companyName, projectName, companyPlanType, sessionType, source, user, lightweightMode,
     }) {
-        analytics.trackWithCIUser('test-run-ci', {
+        const properties = setLightweightAnalytics({
             executionId,
             projectId,
             testId,
@@ -37,15 +44,15 @@ class BaseStrategy {
             projectName,
             companyPlanType,
             sessionType,
-            lightweightMode,
             source: calcSource(source, user),
-        });
+        }, lightweightMode);
+        analytics.trackWithCIUser('test-run-ci', properties);
     }
 
     analyticsTestEnd({
         executionId, projectId, testId, resultId, result, companyId, companyName, projectName, companyPlanType, sessionType, source, user, lightweightMode,
     }) {
-        const properties = {
+        const properties = setLightweightAnalytics({
             executionId,
             projectId,
             testId,
@@ -56,9 +63,8 @@ class BaseStrategy {
             companyPlanType,
             sessionType,
             mockNetworkEnabled: result.wasMockNetworkActivated,
-            lightweightMode,
             source: calcSource(source, user),
-        };
+        }, lightweightMode);
 
         if (result.success) {
             analytics.trackWithCIUser('test-run-ci-success', properties);

package/runners/strategies/LocalStrategy.js

@@ -74,7 +74,7 @@ class LocalStrategy extends BaseStrategy {
             const user = options.user;
             const companyPlanType = options.company && options.company.planType;
             const projectName = options.projectData && options.projectData.name;
-            const lightweightMode = Boolean(options.lightweightMode);
+            const lightweightMode = options.lightweightMode;
             const sessionType = utils.getSessionType(options);
 
             const onTestStarted = (wid, testId, resultId, isRerun, testRetryKey) => {
@@ -96,11 +96,12 @@ class LocalStrategy extends BaseStrategy {
                 return testStatus.testStartAndReport(wid, executionId, resultId, isRerun, testRetryKey);
             };
             const onTestCompleted = (wid, testId, testResult, sessionId, isRerun) => testStatus.testEndAndReport(wid, testResult, executionId, sessionId, isRerun).then(() => {
-                if (options.lightweightMode && options.lightweightMode.disableResults) {
+                const isLocalRun = Boolean(options.useLocalChromeDriver || options.useChromeLauncher);
+                if (lightweightMode && lightweightMode.disableResults && isLocalRun) {
                     return undefined;
                 }
                 const update = {};
-                if (options.lightweightMode && options.lightweightMode.onlyTestIdsNoSuite) {
+                if (lightweightMode && lightweightMode.onlyTestIdsNoSuite) {
                     update.show = true;
                 }
                 if (testResult.seleniumStats) {

package/testRunHandler.js

@@ -518,7 +518,7 @@ TestRun.prototype.onRetry = async function () {
     this._originalTestResultId = this._originalTestResultId || this._previousTestResultId;
     this._testResultId = utils.guid();
 
-    if (this._options.lightweightMode && this._options.lightweightMode.disableResults) {
+    if (this._options.lightweightMode && this._options.lightweightMode.onlyTestIdsNoSuite) {
         return;
     }
 

package/testRunStatus.js

@@ -47,6 +47,7 @@ const RunStatus = function (testInfoList, options, testPlanId, branchToUse) {
     this.executionStartedPromise = Promise.resolve();
 
     const browserNames = utils.getUniqBrowsers(options, testInfoList);
+    const runnerMode = options.lightweightMode ? options.lightweightMode.type : options.mode; 
     this.execConfig = {
         parallel: TESTIM_CONCURRENT_WORKER_COUNT || options.parallel || 1,
         browser: browserNames,
@@ -74,7 +75,7 @@ const RunStatus = function (testInfoList, options, testPlanId, branchToUse) {
         tunnel: options.tunnel,
         tunnelPort: options.tunnelPort,
         tunnelHostHeader: options.tunnelHostHeader,
-        runnerMode: options.mode,
+        runnerMode,
         gridId: options.gridId || options.gridData.gridId,
         gridName: options.grid || options.gridData.name,
         gridType: options.gridData.type,
@@ -152,7 +153,7 @@ RunStatus.prototype.testStart = function (wid, executionId, resultId, isRerun) {
 
 RunStatus.prototype.updateTestStatusRunning = function (test, executionId, testRetryKey) {
     const { project: projectId, remoteRunId, projectData } = this.options;
-    if (this.options.lightweightMode && this.options.lightweightMode.disableResults) {
+    if (this.options.lightweightMode && this.options.lightweightMode.onlyTestIdsNoSuite) {
         return this.executionStartedPromise;
     }
 
@@ -184,9 +185,6 @@ RunStatus.prototype.testStartReport = function (test, executionId, testRetryKey)
         .then(params => {
             this.options.runParams[test.resultId] = test.config.testData = Object.assign({}, test.config.testData, this.exportsGlobal, this.fileUserParamsData, this.beforeSuiteParams, params);
             test.startTime = Date.now();
-            if (this.options.lightweightMode && this.options.lightweightMode.disableResults) {
-                return undefined;
-            }
             return this.updateTestStatusRunning(test, executionId, testRetryKey);
         }).catch(err => {
             logger.error('Failed to start test', { err });
@@ -283,7 +281,7 @@ RunStatus.prototype.testEndReport = function (test, executionId, result) {
     const globalParameters = result.exportsGlobal;
     return runHook(this.options.afterTest, Object.assign({}, test, { globalParameters }))
         .then(() => {
-            if (this.options.lightweightMode && this.options.lightweightMode.disableResults) {
+            if (this.options.lightweightMode && this.options.lightweightMode.onlyTestIdsNoSuite) {
                 return undefined;
             }
             return servicesApi.updateTestStatus('FINISHED', executionId, test.testId, test.resultId, test.startTime, result.endTime, test.success, test.failureReason, null, this.options.project, this.options.remoteRunId);
@@ -446,7 +444,7 @@ RunStatus.prototype.executionEnd = function (executionId) {
         .then((coverageSummary) => {
             resultExtraData.coverageSummary = coverageSummary;
 
-            if (this.options.lightweightMode && this.options.lightweightMode.disableResults && !this.options.lightweightMode.onlyTestIdsNoSuite) {
+            if (this.options.lightweightMode && this.options.lightweightMode.onlyTestIdsNoSuite) {
                 return undefined;
             }
             return servicesApi.reportExecutionFinished(

package/workers/WorkerExtension.js

@@ -38,6 +38,7 @@ class WorkerExtension extends BaseWorker {
         } catch (err) {
             logger.error('failed to get browser', {
                 err,
+                gridInfo,
                 testId: testRunHandler.getTestId(),
                 resultId: testRunHandler.getTestResultId(),
             });