@testim/testim-cli 3.243.0 → 3.244.0

package/package.json

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

package/player/stepActions/scripts/doClick.js

@@ -1,4 +1,6 @@
-"use strict";
+/* globals getLocatedElement, dispatchFocus */
+
+'use strict';
 
 var doClick = function (eventData, done) {
     var eventConstructorSupported = typeof Event === 'function';

package/player/stepActions/scripts/html5dragAction.js

@@ -1,4 +1,6 @@
-"use strict";
+/* global getLocatedElement */
+
+'use strict';
 
 var html5dndAction = function(eventData) {
     var data = {};

package/player/stepActions/scripts/html5dragActionV2.js

@@ -1,4 +1,6 @@
-"use strict";
+/* global getLocatedElement */
+
+'use strict';
 
 var html5dndAction = function (eventData, done) {
     var mouseEventConstructorSupported = typeof MouseEvent === 'function';

package/player/stepActions/scripts/runCode.js

@@ -3,23 +3,40 @@
 'use strict';
 
 const runCode = function (eventData, preCompiledCode) {
+    typeof Object.tstassign !== 'function' && (Object.tstassign = function (n, t) {
+        'use strict';
 
-    "function"!=typeof Object.tstassign&&(Object.tstassign=function(n,t){"use strict";if(null==n)throw new TypeError("Cannot convert undefined or null to object");for(var r=Object(n),e=1;e<arguments.length;e++){var o=arguments[e];if(null!=o)for(var a in o)Object.prototype.hasOwnProperty.call(o,a)&&(r[a]=o[a])}return r});
-    Object.assign = typeof Object.assign !== "function" ? Object.tstassign : Object.assign;
+        if (n == null) throw new TypeError('Cannot convert undefined or null to object'); for (var r = Object(n), e = 1; e < arguments.length; e++) { var o = arguments[e]; if (o != null) for (var a in o)Object.prototype.hasOwnProperty.call(o, a) && (r[a] = o[a]); } return r;
+    });
+    Object.assign = typeof Object.assign !== 'function' ? Object.tstassign : Object.assign;
 
     function appendToStorage(name, data) {
         const sessionItem = 'data-testim-' + name;
+
+        const nativeFuncErrMsg = 'Native sessionStorage is not available';
+        function isNativeFunction(fn) {
+            if (!fn || !fn.toString) {
+                return false;
+            }
+            return fn.toString().indexOf('[native code]') > -1;
+        }
         try {
+            if (![window.sessionStorage.setItem, window.sessionStorage.getItem].every(isNativeFunction)) {
+                throw new Error(nativeFuncErrMsg);
+            }
             const oldData = JSON.parse(window.sessionStorage.getItem(sessionItem) || '{}');
             const newData = Object.tstassign({}, oldData, data);
             window.sessionStorage.setItem(sessionItem, JSON.stringify(newData));
         } catch (err) {
-            // any vernation QuotaExceededError from browsers
+            // any variation QuotaExceededError from browsers
             const isQuotaExceededError = err.message.toLowerCase().indexOf('quota') > -1;
+            const isNativeFunctionError = err.message === nativeFuncErrMsg;
+
             if (err.message.indexOf('sessionStorage') > -1 || // Chrome + Firefox
                 err.message.indexOf('The operation is insecure') > -1 || // Safari
                 err.message.indexOf('SecurityError') > -1 || // Edge
-                isQuotaExceededError
+                isQuotaExceededError ||
+                isNativeFunctionError
             ) {
                 var storage = document.head.querySelector('#testim-storage-backup');
                 if (!storage) {
@@ -30,13 +47,13 @@ const runCode = function (eventData, preCompiledCode) {
                 const oldData = JSON.parse(storage.getAttribute(sessionItem) || '{}');
                 const newData = Object.tstassign({}, oldData, data);
                 storage.setAttribute(sessionItem, JSON.stringify(newData));
-                if (isQuotaExceededError) {
+                if (isQuotaExceededError || isNativeFunctionError) {
                     try {
                         window.sessionStorage.removeItem(sessionItem);
                     } catch (e) {
                         // Prevents future retries from looking in sessionStorage with old data
                     }
-                    window.TSTA.useFallbackStorage = true;
+                    (window.TSTA = window.TSTA || {}).useFallbackStorage = true;
                 }
                 return;
             }

package/player/stepActions/scripts/scroll.js

@@ -1,9 +1,4 @@
-// Disabling the eslint to keep this in a format which works on IE 11.
-/* eslint-disable no-var */
-/* eslint-disable no-redeclare */
-/* eslint-disable object-shorthand */
-/* eslint-disable one-var */
-/* eslint-disable operator-linebreak */
+/* globals getLocatedElement, elementScrollTo */
 
 'use strict';
 

package/player/stepActions/scripts/setText.js

@@ -1,4 +1,7 @@
-"use strict";
+/* globals getLocatedElement, dispatchFocus */
+
+'use strict';
+
 /*global KeyboardEvent */
 var setText = function (eventData, done) {
     var eventConstructorSupported = typeof Event === 'function';

package/utils.js

@@ -71,20 +71,16 @@ function getRunConfigByBrowserName(browser, saucelabs, browserstack) {
     return _.merge(selectedBrowser, selectedOS);
 }
 
-function handleAngularURIComponent(part) {
-    return part.replace(/(~|\/)/g, m => ({ '~': '~~', '/': '~2F' }[m]));
-}
-
 function getTestUrl(editorUrl, projectId, testId, resultId, branch) {
     let testUrl = '';
-    branch = branch ? handleAngularURIComponent(branch) : 'master';
+    branch = branch ? encodeURIComponent(branch) : 'master';
     if (projectId && testId) {
         testUrl = `${editorUrl}/#/project/${projectId}/branch/${branch}/test/${testId}`;
         if (resultId) {
             testUrl += `?result-id=${resultId}`;
         }
     }
-    return encodeURI(testUrl);
+    return testUrl;
 }
 
 function isPromise(obj) {

package/utils.test.js

@@ -32,7 +32,7 @@ describe('utils', () => {
             expect(utils.getTestUrl('http://localhost:8080', 'project', 'test', 'result', 'normal-branch-name'))
                 .to.equal('http://localhost:8080/#/project/project/branch/normal-branch-name/test/test?result-id=result');
             expect(utils.getTestUrl('http://localhost:8080', 'project', 'test', 'result', 'branch/with/slashes'))
-                .to.equal('http://localhost:8080/#/project/project/branch/branch~2Fwith~2Fslashes/test/test?result-id=result');
+                .to.equal('http://localhost:8080/#/project/project/branch/branch%2Fwith%2Fslashes/test/test?result-id=result');
             expect(utils.getTestUrl('http://localhost:8080', 'project', 'test', 'result', 'branch with spaces'))
                 .to.equal('http://localhost:8080/#/project/project/branch/branch%20with%20spaces/test/test?result-id=result');
             expect(utils.getTestUrl('http://localhost:8080', 'project', 'test', 'result', 'encoded%20branch'))