@testim/testim-cli 3.193.0 → 3.194.0

package/codim/codim-npm-package/index.ts

@@ -119,6 +119,7 @@ type ApiCallOptions = {
     headers?: Headers;
     body?: string;
     sendViaWebApp?: boolean;
+    omitCookies?: boolean;
 }
 
 export enum GeneratedValueTypes {

package/commons/testimDesiredCapabilitiesBuilder.js

@@ -193,7 +193,6 @@ function _buildChromiumOptions(opts, browserOptions, testRunConfig, customExtens
     }
 
     if (isDFGrid(gridInfo) && browserName === 'MicrosoftEdge') {
-        delete chromiumOptions.prefs['profile.content_settings.exceptions.clipboard']; // for some reason this breaks
         opts.desiredCapabilities['ms:edgeChromium'] = true;
     }
 

package/commons/testimServicesApi.js

@@ -100,7 +100,7 @@ function saveTestPlanResult(projectId, testPlanId, result) {
     return utils.runWithRetries(() => postAuth({ url: '/testPlan/result', body: { projectId, testPlanId, result } }));
 }
 
-function updateTestStatus(status, executionId, testId, resultId, startTime, endTime, success, failureReason, config, projectId, remoteRunId, testRetryKey) {
+function updateTestStatus(status, executionId, testId, resultId, startTime, endTime, success, failureReason, config, projectId, remoteRunId, testRetryKey, apiRetries = DEFAULT_REQUEST_RETRY) {
     return utils.runWithRetries(() => putAuth('/result/run/test', {
         runId: executionId,
         testId,
@@ -115,7 +115,7 @@ function updateTestStatus(status, executionId, testId, resultId, startTime, endT
         runnerVersion,
         remoteRunId,
         testRetryKey,
-    }), DEFAULT_REQUEST_RETRY);
+    }), apiRetries);
 }
 
 function updateExecutionTests(executionId, runnerStatuses, status, reason, success, startTime, endTime, projectId) {

package/npm-shrinkwrap.json

@@ -1912,9 +1912,9 @@
       },
       "dependencies": {
         "@types/node": {
-          "version": "16.11.4",
-          "resolved": "https://registry.npmjs.org/@types/node/-/node-16.11.4.tgz",
-          "integrity": "sha512-TMgXmy0v2xWyuCSCJM6NCna2snndD8yvQF67J29ipdzMcsPa9u+o0tjF5+EQNdhcuZplYuouYqpc4zcd5I6amQ=="
+          "version": "16.11.6",
+          "resolved": "https://registry.npmjs.org/@types/node/-/node-16.11.6.tgz",
+          "integrity": "sha512-ua7PgUoeQFjmWPcoo9khiPum3Pd60k4/2ZGXt18sm2Slk0W0xZTqt5Y0Ny1NyBiN1EVQ/+FaF9NcY4Qe6rwk5w=="
         },
         "mkdirp": {
           "version": "0.5.5",
@@ -2853,9 +2853,9 @@
       "integrity": "sha512-GJCAeDBKfREgkBtgrYSf9hQy9kTb3helv0zGdzqhM7iAkW8FA/ZF97VQDbwFiwIT8MQLLOe5VlPZOEvZAqtUAQ=="
     },
     "electron-to-chromium": {
-      "version": "1.3.878",
-      "resolved": "https://registry.npmjs.org/electron-to-chromium/-/electron-to-chromium-1.3.878.tgz",
-      "integrity": "sha512-O6yxWCN9ph2AdspAIszBnd9v8s11hQx8ub9w4UGApzmNRnoKhbulOWqbO8THEQec/aEHtvy+donHZMlh6l1rbA==",
+      "version": "1.3.879",
+      "resolved": "https://registry.npmjs.org/electron-to-chromium/-/electron-to-chromium-1.3.879.tgz",
+      "integrity": "sha512-zJo+D9GwbJvM31IdFmwcGvychhk4KKbKYo2GWlsn+C/dxz2NwmbhGJjWwTfFSF2+eFH7VvfA8MCZ8SOqTrlnpw==",
       "dev": true
     },
     "emoji-regex": {
@@ -5310,9 +5310,9 @@
           }
         },
         "estraverse": {
-          "version": "5.2.0",
-          "resolved": "https://registry.npmjs.org/estraverse/-/estraverse-5.2.0.tgz",
-          "integrity": "sha512-BxbNGGNm0RyRYvUdHpIwv9IWzeM9XClbOxwoATuFdOE7ZE6wHL+HQ5T8hoPM+zHvmKzzsEqhgy0GrQ5X13afiQ=="
+          "version": "5.3.0",
+          "resolved": "https://registry.npmjs.org/estraverse/-/estraverse-5.3.0.tgz",
+          "integrity": "sha512-MMdARuVEQziNTeJD8DgMqmhwR11BRQ/cBP+pLtYdSTnf3MIO8fFeiINEbX36ZdNlfU/7A9f3gUw49B3oQsvwBA=="
         },
         "html-encoding-sniffer": {
           "version": "2.0.1",

package/package.json

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

package/player/chromeLauncherTestPlayer.js

@@ -4,9 +4,10 @@ const launcher = require('chrome-launcher');
 const desiredCapabilitiesBuilder = require('../commons/testimDesiredCapabilitiesBuilder');
 const utils = require('../utils');
 const httpRequest = require('../commons/httpRequest');
+const { registerExitHook } = require('../processHandler');
 const CDPTestRunner = require('../cdpTestRunner');
 
-class LancherDriver {
+class LauncherDriver {
     constructor(sessionId) {
         this.sessionId = sessionId;
         this.cdpTestRunner = new CDPTestRunner();
@@ -24,6 +25,8 @@ class LancherDriver {
         this._isAlive = true;
         const browserEndpoint = await httpRequest.get(`http://localhost:${this.chrome.port}/json/version`);
         await this.cdpTestRunner.initSession(browserEndpoint.webSocketDebuggerUrl);
+
+        registerExitHook(() => this.chrome.kill());
     }
 
     isAlive() {
@@ -49,7 +52,7 @@ class LancherDriver {
 class ChromeLauncherTestPlayer {
     constructor(id) {
         this.sessionId = utils.guid();
-        this.driver = new LancherDriver(this.sessionId);
+        this.driver = new LauncherDriver(this.sessionId);
         this.id = id;
     }
 

package/player/stepActions/apiStepAction.js

@@ -36,6 +36,7 @@ class ApiStepAction extends StepAction {
             headers: context.apiHeaders,
             body: context.apiBody,
             timeout: context.data.maxTotalStepTime,
+            omitCookies: step.omitCookies,
         };
 
         return (step.sendViaWebApp ? this.runApiInAut(step, context) : this.runApiInBg(eventData))

package/player/stepActions/salesforceAutoLoginStepAction.js

@@ -0,0 +1,37 @@
+const NavigationStepAction = require('./navigationStepAction');
+const Promise = require('bluebird');
+
+class SalesforceAutoLoginStepAction extends NavigationStepAction {
+    async performAction() {
+        let salesforceUrl = this.context.data.testimNavigationStepDestination || this.context.data.url;
+        try {
+            salesforceUrl = await this.updateBaseUrl(salesforceUrl);
+            await this.driver.url(salesforceUrl);
+            await Promise.delay(300); // 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);
+            if (isUsernameVerificationNeeded) {
+                await this.driver.executeCodeAsync(`
+                    function ${this.step.handleUsernameVerificationAUTFunc.toString()}
+                    handleUsernameVerificationAUTFunc();
+                    var done = arguments[1];
+                    done();
+                `);
+                await Promise.delay(500);
+                newUrl = await this.driver.getUrl(); // If we managed to continue correctly we want to get the new redirected url
+            }
+            return {
+                success: true,
+                newUrl,
+            };
+        } catch (err) {
+            return {
+                success: false,
+                reason: err.message,
+            };
+        }
+    }
+}
+
+module.exports = SalesforceAutoLoginStepAction;

package/player/stepActions/stepActionRegistrar.js

@@ -21,6 +21,7 @@ const RefreshStepAction = require('./RefreshStepAction');
 const ApiStepAction = require('./apiStepAction');
 const ExtractTextStepAction = require('./extractTextStepAction');
 const TdkHybridStepAction = require('./tdkHybridStepAction');
+const SalesforceAutoLoginStepAction = require('./salesforceAutoLoginStepAction');
 
 const CliJsStepAction = require('./cliJsStepAction');
 const CliConditionStepAction = require('./cliConditionStepAction');
@@ -92,6 +93,8 @@ module.exports = function (driver, stepActionFactory, runMode) {
         'email-code-step': JsCodeStepAction,
         'cli-email-code-step': CliJsStepAction,
         'tdk-hybrid': TdkHybridStepAction,
+
+        'salesforce-autologin': SalesforceAutoLoginStepAction,
     };
 
     const ANDROID_STEP_ACTION_MAPPING = {

package/runner.js

@@ -1,7 +1,7 @@
 'use strict';
 
 /* eslint-disable no-console */
-const { CLI_MODE, gridTypes } = require('./commons/constants');
+const { CLI_MODE } = require('./commons/constants');
 const Promise = require('bluebird');
 const _ = require('lodash');
 const { EDITOR_URL } = require('./commons/config');
@@ -309,10 +309,14 @@ async function init(options) {
         await labFeaturesService.loadLabFeatures(projectById.id, companyByProjectId.activePlan);
     }
 
-    if (options.lightweightMode && options.lightweightMode.type === 'highSpeed' && !labFeaturesService.isFeatureAvailableForProject('highSpeedMode')) {
+    if (options.lightweightMode && options.lightweightMode.type === 'highSpeed' && (!labFeaturesService.isFeatureAvailableForProject('highSpeedMode') || options.company.planType === 'free')) {
         delete options.lightweightMode;
     }
 
+    if (options.lightweightMode && options.lightweightMode.type === 'highSpeed') {
+        console.log('High-speed mode will ignore step delays. Test artifacts like screenshots and logs will only be saved for failed runs. For more information see our docs: https://help.testim.io/docs/high-speed-mode');
+    }
+
     gridService.keepAlive.start(project);
     analyticsIdentify(project);
     await setMockNetworkRules(options);

package/runners/strategies/BaseStrategy.js

@@ -51,26 +51,31 @@ class BaseStrategy {
 
     analyticsTestEnd({
         executionId, projectId, testId, resultId, result, companyId, companyName, projectName, companyPlanType, sessionType, source, user, lightweightMode,
+        logger,
     }) {
-        const properties = setLightweightAnalytics({
-            executionId,
-            projectId,
-            testId,
-            resultId,
-            companyId,
-            companyName,
-            projectName,
-            companyPlanType,
-            sessionType,
-            mockNetworkEnabled: result.wasMockNetworkActivated,
-            source: calcSource(source, user),
-        }, lightweightMode);
+        try {
+            const properties = setLightweightAnalytics({
+                executionId,
+                projectId,
+                testId,
+                resultId,
+                companyId,
+                companyName,
+                projectName,
+                companyPlanType,
+                sessionType,
+                mockNetworkEnabled: result.wasMockNetworkActivated,
+                source: calcSource(source, user),
+            }, lightweightMode);
 
-        if (result.success) {
-            analytics.trackWithCIUser('test-run-ci-success', properties);
-            return;
+            if (result.success) {
+                analytics.trackWithCIUser('test-run-ci-success', properties);
+                return;
+            }
+            analytics.trackWithCIUser('test-run-ci-fail', Object.assign({}, properties, { failureReason: result.failureReason }));
+        } catch (err) {
+            logger.error('failed to update test end analytics', { err });
         }
-        analytics.trackWithCIUser('test-run-ci-fail', Object.assign({}, properties, { failureReason: result.failureReason }));
     }
 
     runTests(testList, testStatus, executionId, options, branchToUse, authData, workerCount, stopOnError) {

package/runners/strategies/LocalStrategy.js

@@ -95,7 +95,9 @@ 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(() => {
+            const onTestCompleted = (wid, testId, testResult, sessionId, isRerun) => testStatus.testEndAndReport(wid, testResult, executionId, sessionId, isRerun).catch(err => {
+                logger.error('testEndAndReport threw an error', { err });
+            }).then(() => {
                 const isLocalRun = Boolean(options.useLocalChromeDriver || options.useChromeLauncher);
                 if (lightweightMode && lightweightMode.disableResults && isLocalRun) {
                     return undefined;
@@ -161,6 +163,7 @@ class LocalStrategy extends BaseStrategy {
                     source,
                     user,
                     lightweightMode,
+                    logger,
                 });
                 if (stopOnError && !testResult.success) {
                     reject(new StopRunOnError());

package/testRunStatus.js

@@ -47,7 +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; 
+    const runnerMode = options.lightweightMode ? options.lightweightMode.type : options.mode;
     this.execConfig = {
         parallel: TESTIM_CONCURRENT_WORKER_COUNT || options.parallel || 1,
         browser: browserNames,
@@ -277,19 +277,25 @@ RunStatus.prototype.testEnd = function (wid, result, executionId, sessionId, isR
     return test;
 };
 
-RunStatus.prototype.testEndReport = function (test, executionId, result) {
+RunStatus.prototype.testEndReport = async 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.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);
-        })
-        .catch(err => {
-            logger.error('Failed to update test finished', { err });
-            throw err;
-        });
+    try {
+        try {
+            await runHook(this.options.afterTest, Object.assign({}, test, { globalParameters }));
+        } catch (err) {
+            logger.error('HOOK threw an error', { test: test.testId, err });
+            // eslint-disable-next-line no-console
+            console.error('HOOK threw an error', err); // show the customer that his hook failed.
+        }
+        if (this.options.lightweightMode && this.options.lightweightMode.onlyTestIdsNoSuite) {
+            return undefined;
+        }
+
+        return await servicesApi.updateTestStatus('FINISHED', executionId, test.testId, test.resultId, test.startTime, result.endTime, test.success, test.failureReason, null, this.options.project, this.options.remoteRunId, undefined, 5);
+    } catch (err) {
+        logger.error('Failed to update test finished', { err });
+        throw err;
+    }
 };
 
 RunStatus.prototype.testEndAndReport = function (wid, result, executionId, sessionId, isRerun) {