@testim/testim-cli 3.199.0 → 3.200.0

package/npm-shrinkwrap.json

@@ -344,14 +344,14 @@
       }
     },
     "@babel/helper-compilation-targets": {
-      "version": "7.16.0",
-      "resolved": "https://registry.npmjs.org/@babel/helper-compilation-targets/-/helper-compilation-targets-7.16.0.tgz",
-      "integrity": "sha512-S7iaOT1SYlqK0sQaCi21RX4+13hmdmnxIEAnQUB/eh7GeAnRjOUgTYpLkUOiRXzD+yog1JxP0qyAQZ7ZxVxLVg==",
+      "version": "7.16.3",
+      "resolved": "https://registry.npmjs.org/@babel/helper-compilation-targets/-/helper-compilation-targets-7.16.3.tgz",
+      "integrity": "sha512-vKsoSQAyBmxS35JUOOt+07cLc6Nk/2ljLIHwmq2/NM6hdioUaqEXq/S+nXvbvXbZkNDlWOymPanJGOc4CBjSJA==",
       "dev": true,
       "requires": {
         "@babel/compat-data": "^7.16.0",
         "@babel/helper-validator-option": "^7.14.5",
-        "browserslist": "^4.16.6",
+        "browserslist": "^4.17.5",
         "semver": "^6.3.0"
       },
       "dependencies": {
@@ -477,13 +477,13 @@
       "dev": true
     },
     "@babel/helpers": {
-      "version": "7.16.0",
-      "resolved": "https://registry.npmjs.org/@babel/helpers/-/helpers-7.16.0.tgz",
-      "integrity": "sha512-dVRM0StFMdKlkt7cVcGgwD8UMaBfWJHl3A83Yfs8GQ3MO0LHIIIMvK7Fa0RGOGUQ10qikLaX6D7o5htcQWgTMQ==",
+      "version": "7.16.3",
+      "resolved": "https://registry.npmjs.org/@babel/helpers/-/helpers-7.16.3.tgz",
+      "integrity": "sha512-Xn8IhDlBPhvYTvgewPKawhADichOsbkZuzN7qz2BusOM0brChsyXMDJvldWaYMMUNiCQdQzNEioXTp3sC8Nt8w==",
       "dev": true,
       "requires": {
         "@babel/template": "^7.16.0",
-        "@babel/traverse": "^7.16.0",
+        "@babel/traverse": "^7.16.3",
         "@babel/types": "^7.16.0"
       }
     },
@@ -544,15 +544,15 @@
       }
     },
     "@babel/parser": {
-      "version": "7.16.2",
-      "resolved": "https://registry.npmjs.org/@babel/parser/-/parser-7.16.2.tgz",
-      "integrity": "sha512-RUVpT0G2h6rOZwqLDTrKk7ksNv7YpAilTnYe1/Q+eDjxEceRMKVWbCsX7t8h6C1qCFi/1Y8WZjcEPBAFG27GPw==",
+      "version": "7.16.3",
+      "resolved": "https://registry.npmjs.org/@babel/parser/-/parser-7.16.3.tgz",
+      "integrity": "sha512-dcNwU1O4sx57ClvLBVFbEgx0UZWfd0JQX5X6fxFRCLHelFBGXFfSz6Y0FAq2PEwUqlqLkdVjVr4VASEOuUnLJw==",
       "dev": true
     },
     "@babel/runtime": {
-      "version": "7.16.0",
-      "resolved": "https://registry.npmjs.org/@babel/runtime/-/runtime-7.16.0.tgz",
-      "integrity": "sha512-Nht8L0O8YCktmsDV6FqFue7vQLRx3Hb0B37lS5y0jDRqRxlBG4wIJHnf9/bgSE2UyipKFA01YtS+npRdTWBUyw==",
+      "version": "7.16.3",
+      "resolved": "https://registry.npmjs.org/@babel/runtime/-/runtime-7.16.3.tgz",
+      "integrity": "sha512-WBwekcqacdY2e9AF/Q7WLFUWmdJGJTkbjqTjoMDgXkVZ3ZRUvOPsLb5KdwISoQVsbP+DQzVZW4Zhci0DvpbNTQ==",
       "requires": {
         "regenerator-runtime": "^0.13.4"
       },
@@ -576,9 +576,9 @@
       }
     },
     "@babel/traverse": {
-      "version": "7.16.0",
-      "resolved": "https://registry.npmjs.org/@babel/traverse/-/traverse-7.16.0.tgz",
-      "integrity": "sha512-qQ84jIs1aRQxaGaxSysII9TuDaguZ5yVrEuC0BN2vcPlalwfLovVmCjbFDPECPXcYM/wLvNFfp8uDOliLxIoUQ==",
+      "version": "7.16.3",
+      "resolved": "https://registry.npmjs.org/@babel/traverse/-/traverse-7.16.3.tgz",
+      "integrity": "sha512-eolumr1vVMjqevCpwVO99yN/LoGL0EyHiLO5I043aYQvwOJ9eR5UsZSClHVCzfhBduMAsSzgA/6AyqPjNayJag==",
       "dev": true,
       "requires": {
         "@babel/code-frame": "^7.16.0",
@@ -586,7 +586,7 @@
         "@babel/helper-function-name": "^7.16.0",
         "@babel/helper-hoist-variables": "^7.16.0",
         "@babel/helper-split-export-declaration": "^7.16.0",
-        "@babel/parser": "^7.16.0",
+        "@babel/parser": "^7.16.3",
         "@babel/types": "^7.16.0",
         "debug": "^4.1.0",
         "globals": "^11.1.0"
@@ -3172,9 +3172,9 @@
       "integrity": "sha512-GJCAeDBKfREgkBtgrYSf9hQy9kTb3helv0zGdzqhM7iAkW8FA/ZF97VQDbwFiwIT8MQLLOe5VlPZOEvZAqtUAQ=="
     },
     "electron-to-chromium": {
-      "version": "1.3.892",
-      "resolved": "https://registry.npmjs.org/electron-to-chromium/-/electron-to-chromium-1.3.892.tgz",
-      "integrity": "sha512-YDW4yIjdfMnbRoBjRZ/aNQYmT6JgQFLwmTSDRJMQdrY4MByEzppdXp3rnJ0g4LBWcsYTUvwKKClYN1ofZ0COOQ==",
+      "version": "1.3.894",
+      "resolved": "https://registry.npmjs.org/electron-to-chromium/-/electron-to-chromium-1.3.894.tgz",
+      "integrity": "sha512-WY8pA4irAZ4cm/Pr7YFPtPLVqj3nU6d0SbfoHF6M7HZNONfPdAnYAarumqQ75go2LuN72uO9wGuCEqnfya/ytg==",
       "dev": true
     },
     "emoji-regex": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testim/testim-cli",
-  "version": "3.199.0",
+  "version": "3.200.0",
   "description": "Command line interface for running Testing on your CI",
   "author": "Oren Rubin",
   "contributors": [{

package/player/stepActions/salesforceAutoLoginStepAction.js

@@ -7,7 +7,7 @@ class SalesforceAutoLoginStepAction extends NavigationStepAction {
         try {
             salesforceUrl = await this.updateBaseUrl(salesforceUrl);
             await this.driver.url(salesforceUrl);
-            await Promise.delay(1000); // wait a little for the page to load (fixes screenshots and clicking on elements in username verification screen)
+            await Promise.delay(5000); // wait a little for the page to load (fixes screenshots and clicking on elements in username verification screen)
             let newUrl = await this.driver.getUrl();
             // Verify username screen
             const isUsernameVerificationNeeded = newUrl.includes(this.step.USERNAME_VERIFICATION_PATH_ID);
@@ -22,7 +22,7 @@ class SalesforceAutoLoginStepAction extends NavigationStepAction {
                 await Promise.delay(5000);
                 newUrl = await this.driver.getUrl(); // If we managed to continue correctly we want to get the new redirected url
             }
-            await Promise.delay(700);
+            await Promise.delay(1500);
             return {
                 success: true,
                 newUrl,

package/player/stepActions/stepActionRegistrar.js

@@ -24,7 +24,6 @@ const TdkHybridStepAction = require('./tdkHybridStepAction');
 const PixelValidationStepAction = require('./pixelValidationStepAction');
 
 const SalesforceAutoLoginStepAction = require('./salesforceAutoLoginStepAction');
-const SalesforceApexActionStepAction = require('./salesforceApexActionStepAction');
 
 const CliJsStepAction = require('./cliJsStepAction');
 const CliConditionStepAction = require('./cliConditionStepAction');
@@ -87,7 +86,6 @@ module.exports = function (driver, stepActionFactory, runMode) {
         'tdk-hybrid': TdkHybridStepAction,
 
         'salesforce-autologin': SalesforceAutoLoginStepAction,
-        'salesforce-apex-action': SalesforceApexActionStepAction,
     };
 
     register(STEP_ACTION_MAPPING, stepActionFactory);

package/runners/TestPlanRunner.js

@@ -150,10 +150,6 @@ class TestPlanRunner {
 
         const isCodeMode = tpOptions.files.length > 0;
 
-        if (isCodeMode && tpOptions.mode === constants.CLI_MODE.SELENIUM) {
-            // in selenium mode we don't need to wait for the runner and clickim to sync, so we don't need to wait for reports.
-            testStatus.setAsyncReporting(true);
-        }
         const testListInfoPromise = tpOptions.lightweightMode && tpOptions.lightweightMode.onlyTestIdsNoSuite ?
             { beforeTests, tests, afterTests } :
             testStatus.executionStart(executionId, projectId, this.startTime, testPlanName);

package/services/gridService.js

@@ -245,10 +245,6 @@ async function getTestPlanGridData(options) {
     if (testPlanGrids.includes(undefined)) {
         throw new ArgError('failed to find one of the test plan defined grid');
     }
-    const gridTypes = _(testPlanGrids).map(grid => grid.type).uniq().value();
-    if (gridTypes.includes('testimMobile') && gridTypes.length > 1) {
-        throw new ArgError('Test plans cannot include two different grid types');
-    }
     return getSerializableObject(_.first(testPlanGrids));
 }
 

package/stepPlayers/hybridStepPlayback.js

@@ -6,6 +6,7 @@ const perfLogger = require('../commons/performance-logger');
 const MemoryFS = require('memory-fs');
 const mfs = new MemoryFS();
 const AbortController = require("abort-controller");
+const logger = require('../commons/logger').getLogger('hybrid-step-playback');
 
 /**
  * @type {Map<string, import("abort-controller")>}
@@ -119,10 +120,12 @@ module.exports.execute = async function execute(step, context, driver, loginData
         }
 
         return { success: false, shouldRetry: false, reason: 'unknown hybrid format ' + hybridFunction.type };
+    } catch (err) {
+        logger.log('error running hybrid step', { err });
     } finally {
         runningStepsAbortControllersRegistry.delete(context.stepResultId);
     }
-}
+};
 
 module.exports.abort = function abort(stepResultId) {
     const abortController = runningStepsAbortControllersRegistry.get(stepResultId);

package/stepPlayers/tdkHybridStepPlayback.js

@@ -1,5 +1,6 @@
 'use strict';
 
+const _ = require('lodash');
 const sessionPlayerInit = require('../commons/getSessionPlayerRequire');
 const perfLogger = require('../commons/performance-logger');
 const { guid } = require('../utils');

package/testRunHandler.js

@@ -252,9 +252,6 @@ TestRun.prototype.clearTestResult = function () {
                 testRetryKey: this.getRetryKey(),
             });
         });
-    if (this._testRunStatus.asyncReporting) {
-        return Promise.resolve();
-    }
     return this.clearTestResultFinished;
 };
 

package/testRunStatus.js

@@ -43,7 +43,6 @@ const RunStatus = function (testInfoList, options, testPlanId, branchToUse) {
     this.exportsGlobal = {};
     this.testInfoList = testInfoList;
 
-    this.asyncReporting = false; // whether or not we wait for result reporting
     this.executionStartedPromise = Promise.resolve();
 
     const browserNames = utils.getUniqBrowsers(options, testInfoList);
@@ -92,9 +91,6 @@ const RunStatus = function (testInfoList, options, testPlanId, branchToUse) {
 RunStatus.prototype.waitForExecutionStartedFinished = function () {
     return this.executionStartedPromise;
 };
-RunStatus.prototype.setAsyncReporting = function (isAsyncReportingEnabled = false) {
-    this.asyncReporting = isAsyncReportingEnabled;
-};
 RunStatus.prototype.getTestResult = function (resultId) {
     return this.testRunStatus[resultId];
 };
@@ -157,7 +153,7 @@ RunStatus.prototype.updateTestStatusRunning = function (test, executionId, testR
         return this.executionStartedPromise;
     }
 
-    const res = servicesApi.updateTestDataArtifact(projectId, test.testId, test.resultId, test.config.testData, projectData.defaults)
+    return servicesApi.updateTestDataArtifact(projectId, test.testId, test.resultId, test.config.testData, projectData.defaults)
         .catch(err => {
             logger.error('failed to upload test data artifact (runner)', { err });
             return '';
@@ -169,12 +165,6 @@ RunStatus.prototype.updateTestStatusRunning = function (test, executionId, testR
             await this.executionStartedPromise;
             return servicesApi.updateTestStatus(projectId, executionId, test.testId, test.resultId, 'RUNNING', { startTime: test.startTime, config: testConfig, remoteRunId, testRetryKey });
         });
-    if (this.asyncReporting) {
-        // silence is golden
-    } else {
-        return res;
-    }
-    return servicesApi.updateTestStatus(projectId, executionId, test.testId, test.resultId, 'RUNNING', { startTime: test.startTime, config: test.config, remoteRunId, testRetryKey });
 };
 
 RunStatus.prototype.testStartReport = function (test, executionId, testRetryKey) {
@@ -406,10 +396,7 @@ RunStatus.prototype.executionStart = function (executionId, projectId, startTime
                     const ret = servicesApi.reportExecutionStarted(data);
                     this.executionStartedPromise = ret;
                     ret.catch(e => logger.error(e));
-                    if (!this.asyncReporting) {
-                        return ret;
-                    }
-                    return undefined;
+                    return ret;
                 });
             };
 

package/player/stepActions/salesforceApexActionStepAction.js

@@ -1,9 +0,0 @@
-const CliJsStepAction = require('./cliJsStepAction');
-
-class SalesforceApexActionStepAction extends CliJsStepAction {
-    getSkipStatus() {
-        return false;
-    }
-}
-
-module.exports = SalesforceApexActionStepAction;