@testim/testim-cli 3.244.0 → 3.247.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testim/testim-cli",
-  "version": "3.244.0",
+  "version": "3.247.0",
   "description": "Command line interface for running Testing on your CI",
   "author": "Oren Rubin",
   "contributors": [{
@@ -57,7 +57,6 @@
     "appium-dom-utils": "1.0.6",
     "archiver": "5.3.0",
     "bluebird": "3.7.2",
-    "bluebird-retry": "0.11.0",
     "body-parser": "1.19.1",
     "chalk": "4.1.2",
     "chrome-launcher": "0.15.0",
@@ -87,6 +86,7 @@
     "npm": "8.3.0",
     "object-hash": "3.0.0",
     "ora": "5.4.1",
+    "p-retry": "4.6.2",
     "pako": "1.0.11",
     "portfinder": "1.0.28",
     "promise-queue": "2.2.5",

package/player/WebDriverHttpRequest.js

@@ -4,8 +4,8 @@ const _ = require('lodash');
 const httpRequest = require('../commons/httpRequest');
 const utils = require('../utils');
 const SeleniumProtocolError = require('./SeleniumProtocolError');
-const {SELENIUM_STATUS_CODES} = require('./constants');
-const logger = require('./../commons/logger').getLogger("WebDriverHttpRequest");
+const { SELENIUM_STATUS_CODES } = require('./constants');
+const logger = require('../commons/logger').getLogger('WebDriverHttpRequest');
 
 function isSuccessfulResponse(body, statusCode) {
     /**
@@ -56,37 +56,37 @@ class WebDriverHttpRequest {
     }
 
     handleFailedStatus(requestId, response, err = {}) {
-        const {body, statusCode, headers, text} = response;
+        const { body, statusCode, headers, text } = response;
         const resBody = _.isEmpty(body) && text ? text : body;
         /**
          * in Appium you find sometimes more exact error messages in origValue
          */
         if (resBody && resBody.value && typeof resBody.value.origValue === 'string' && typeof resBody.value.message === 'string') {
-            resBody.value.message += ' ' + resBody.value.origValue;
+            resBody.value.message += ` ${resBody.value.origValue}`;
         }
 
         if (resBody && typeof resBody === 'string') {
-            throw new SeleniumProtocolError(resBody, {requestId});
+            throw new SeleniumProtocolError(resBody, { requestId });
         }
 
         if (resBody && resBody.value) {
             const errorCode = SELENIUM_STATUS_CODES[resBody.status] || (resBody.value && SELENIUM_STATUS_CODES[resBody.value.error]) || SELENIUM_STATUS_CODES[-1];
-            let error = {
+            const error = {
                 type: errorCode ? errorCode.id : 'unknown',
                 message: errorCode ? errorCode.message : 'unknown',
-                orgStatusMessage: resBody.value ? resBody.value.message : ''
+                orgStatusMessage: resBody.value ? resBody.value.message : '',
             };
 
-            throw new SeleniumProtocolError(error, {requestId});
+            throw new SeleniumProtocolError(error, { requestId });
         }
 
         // IEServer webdriver bug where the error is put into the Allow header
         // https://github.com/SeleniumHQ/selenium/issues/6828
         if (statusCode === 405) {
             const allowHeader = headers && headers.allow;
-            const err = new SeleniumProtocolError(allowHeader, {requestId});
-            if (err.message.match(/Command not found/)) {
-                throw err;
+            const _err = new SeleniumProtocolError(allowHeader, { requestId });
+            if (_err.message.match(/Command not found/)) {
+                throw _err;
             }
         }
 
@@ -94,12 +94,12 @@ class WebDriverHttpRequest {
             status: -1,
             type: err.code || 'ECONNREFUSED',
             orgStatusMessage: err.rawResponse || err.message,
-            message: err.rawResponse || "Couldn't connect to selenium server"
-        }, {requestId});
+            message: err.rawResponse || "Couldn't connect to selenium server",
+        }, { requestId });
     }
 
     processHttpOnSuccess(response, requestId) {
-        const {body, statusCode} = response;
+        const { body, statusCode } = response;
         if (isSuccessfulResponse(body, statusCode)) {
             return body;
         }
@@ -108,44 +108,49 @@ class WebDriverHttpRequest {
     }
 
     processHttpOnError(err, requestId) {
-        const response = err && err.response || {};
+        const response = (err && err.response) || {};
         return this.handleFailedStatus(requestId, response, err);
     }
 
     httpPostRequest(path, body = {}) {
         const requestId = utils.guid();
-        if (path === "/session") {
-            logger.info("POST REQUEST", {requestId, path, testResultId: this.testResultId});
+        if (path === '/session') {
+            logger.info('POST REQUEST', { requestId, path, testResultId: this.testResultId });
         } else {
-            logger.info("POST REQUEST", {requestId, path, body, testResultId: this.testResultId});
+            logger.info('POST REQUEST', { requestId, path, body, testResultId: this.testResultId });
         }
 
         return httpRequest.postFullRes(`${this.gridUrl}${path}`, body, this.headers, this.connectionRetryTimeout)
-            .tap((response) => logger.info("POST RESPONSE", {
-                requestId,
-                path,
-                res: JSON.stringify(response.body),
-                testResultId: this.testResultId
-            }))
+            .then(response => {
+                logger.info('POST RESPONSE', {
+                    requestId,
+                    path,
+                    res: JSON.stringify(response.body),
+                    testResultId: this.testResultId,
+                });
+                return response;
+            })
             .catch(err => this.processHttpOnError(err, requestId))
             .then((response) => this.processHttpOnSuccess(response, requestId));
     }
 
     httpGetRequest(path) {
         const requestId = utils.guid();
-        logger.info("GET REQUEST", {requestId, path, testResultId: this.testResultId});
+        logger.info('GET REQUEST', { requestId, path, testResultId: this.testResultId });
         const query = {};
         return httpRequest.getFullRes(`${this.gridUrl}${path}`, query, this.headers, this.connectionRetryTimeout)
-            .tap((response) => {
-                if (_.endsWith(path, "/screenshot")) {
-                    return logger.info("GET RESPONSE", {requestId, path, testResultId: this.testResultId});
+            .then(response => {
+                if (_.endsWith(path, '/screenshot')) {
+                    logger.info('GET RESPONSE', { requestId, path, testResultId: this.testResultId });
+                    return response;
                 }
-                logger.info("GET RESPONSE", {
+                logger.info('GET RESPONSE', {
                     requestId,
                     path,
                     res: JSON.stringify(response.body),
-                    testResultId: this.testResultId
+                    testResultId: this.testResultId,
                 });
+                return response;
             })
             .catch(err => this.processHttpOnError(err, requestId))
             .then((response) => this.processHttpOnSuccess(response, requestId));
@@ -153,14 +158,17 @@ class WebDriverHttpRequest {
 
     httpDeleteRequest(path) {
         const requestId = utils.guid();
-        logger.info("DELETE REQUEST", {requestId, path, testResultId: this.testResultId});
+        logger.info('DELETE REQUEST', { requestId, path, testResultId: this.testResultId });
         return httpRequest.deleteFullRes(`${this.gridUrl}${path}`, undefined, this.headers, this.connectionRetryTimeout)
-            .tap((response) => logger.info("DELETE RESPONSE", {
-                requestId,
-                path,
-                res: JSON.stringify(response.body),
-                testResultId: this.testResultId
-            }))
+            .then(response => {
+                logger.info('DELETE RESPONSE', {
+                    requestId,
+                    path,
+                    res: JSON.stringify(response.body),
+                    testResultId: this.testResultId,
+                });
+                return response;
+            })
             .catch(err => this.processHttpOnError(err, requestId))
             .then((response) => this.processHttpOnSuccess(response, requestId));
     }

package/player/chromeLauncherTestPlayer.js

@@ -23,8 +23,8 @@ class LauncherDriver {
         this.chrome.process.once('exit', () => { this._isAlive = false; });
         this.chrome.process.once('close', () => { this._isAlive = false; });
         this._isAlive = true;
-        const browserEndpoint = await httpRequest.get(`http://localhost:${this.chrome.port}/json/version`);
-        await this.cdpTestRunner.initSession(browserEndpoint.webSocketDebuggerUrl);
+        const webSocketDebuggerUrl = await utils.getCdpAddressForHost(`localhost:${this.chrome.port}`);
+        await this.cdpTestRunner.initSession(webSocketDebuggerUrl);
 
         registerExitHook(() => this.chrome.kill());
     }

package/player/services/playbackTimeoutCalculator.js

@@ -4,6 +4,7 @@ const _ = require('lodash');
 
 const COMMUNICATION_BUFFER_TIME = 1000;
 const UI_VERIFICATION_STEPS = ['simple-ui-verification', 'wait-for-simple-ui-verification'];
+const FULL_TIMEOUT_STEP_TYPES = [...UI_VERIFICATION_STEPS, 'custom-validation', 'sfdc-step-login'];
 
 class PlaybackTimeoutCalculator {
     constructor(isDebuggerConnected) {
@@ -41,7 +42,7 @@ class PlaybackTimeoutCalculator {
         };
         stepPlayback.setStartTimestamp();
         const totalStepTime = this.getTotalStepRunTime(stepPlayback);
-        const currentRetryTimes = [...UI_VERIFICATION_STEPS, 'custom-validation'].includes(stepPlayback.stepType) ? [totalStepTime] : getRetryTimeoutSuggestions(totalStepTime);
+        const currentRetryTimes = FULL_TIMEOUT_STEP_TYPES.includes(stepPlayback.stepType) ? [totalStepTime] : getRetryTimeoutSuggestions(totalStepTime);
         this.resetStepVariables(totalStepTime, currentRetryTimes);
         stepPlayback.context.data.maxTotalStepTime = totalStepTime;
     }
@@ -61,6 +62,9 @@ class PlaybackTimeoutCalculator {
         if (UI_VERIFICATION_STEPS.includes(stepPlayback.stepType)) {
             fallbackTimeout = stepPlayback.context.config.applitoolsStepTimeout || HALF_HOUR_IN_MS;
         }
+        if (stepPlayback.step.type.startsWith('sfdc-')) {
+            fallbackTimeout = stepPlayback.step.defaultTimeout;
+        }
         return (stepPlayback.step.useStepTimeout && stepPlayback.step.stepTimeout) ? stepPlayback.step.stepTimeout : fallbackTimeout;
     }
     getTotalStepTimeLeftToPlay(stepPlayback, totalStepTime = this.totalStepTime) {

package/player/stepActions/RefreshStepAction.js

@@ -6,10 +6,8 @@ require('bluebird');
 class RefreshStepAction extends StepAction {
     execute() {
         return this.driver.reloadTab()
-            .return({ success: true })
-            .catch(error => {
-                return { success: false, reason: error.message };
-            });
+            .then(() => ({ success: true }))
+            .catch(error => ({ success: false, reason: error.message }));
     }
 }
 

package/player/stepActions/inputFileStepAction.js

@@ -86,18 +86,22 @@ class InputFileStepAction extends StepAction {
         const target = this.context.data[this.step.targetId || 'targetId'];
         const overrideAzureStorageUrl = featureFlagService.flags.overrideAzureStorageUrl.isEnabled();
         const useJsInputCodeInSafari = featureFlagService.flags.useJsInputCodeInSafari.isEnabled();
+        const useJsInputCodeInFirefox = featureFlagService.flags.useJsInputCodeInFirefox.isEnabled();
         const downloadToBase64 = featureFlagService.flags.downloadToBase64.isEnabled();
 
         let fileUrls = await utils.addTokenToFileUrl(this.context.project.id, this.step.fileUrls, this.stepActionUtils.testimServicesApi, overrideAzureStorageUrl, logger);
 
+
+        const isSafariJsInputCode = this.driver.isSafari() && (useJsInputCodeInSafari || fileUrls.length > 1);
+        const isFirefoxJsInputCode = this.driver.isFirefox() && (useJsInputCodeInFirefox || fileUrls.length > 1);
+
         if (downloadToBase64) {
             fileUrls = await Promise.all(fileUrls.map(async ({ name, url }) => {
                 const res = await download(url);
                 return { name, url: `data:${res.type};base64,${Buffer.from(res.body).toString('base64')}` };
             }));
         }
-
-        if (this.driver.isSafari() && (useJsInputCodeInSafari || fileUrls.length > 1)) {
+        if (isSafariJsInputCode || isFirefoxJsInputCode) {
             await this.driver.executeJSWithArray(`
                 const getLocatedElement = ${codeSnippets.getLocatedElementCode};
                 const downloadAndUploadFile = ${downloadAndUpload()};

package/player/stepActions/sfdcStepAction.js

@@ -0,0 +1,27 @@
+const StepAction = require('./stepAction');
+const { sfdc } = require('../../commons/getSessionPlayerRequire');
+
+class SfdcStepAction extends StepAction {
+    async performAction() {
+        const page = sfdc.sfdcNewSePage(this.driver);
+        try {
+            const actions = this.context.sfdcTestActions;
+            if (actions === undefined) {
+                throw new Error('No test actions were compiled');
+            }
+            await sfdc.sfdcExecute(page, actions);
+            return { success: true };
+        } catch (err) {
+            return {
+                success: false,
+                reason: err.reason || err.message,
+                exception: err,
+                shouldRetry: false, // TODO - check this.  Our (bFormat) steps are probably not retryable?
+            };
+        } finally {
+            page.releaseObjects();
+        }
+    }
+}
+
+module.exports = SfdcStepAction;

package/player/stepActions/stepActionRegistrar.js

@@ -27,6 +27,7 @@ const CliJsStepAction = require('./cliJsStepAction');
 const CliConditionStepAction = require('./cliConditionStepAction');
 const NodePackageStepAction = require('./nodePackageStepAction');
 const ExtensionOnlyStepAction = require('./extensionOnlyStepAction');
+const SfdcStepAction = require('./sfdcStepAction');
 
 function register(stepActionByType, stepActionFactory) {
     Object.keys(stepActionByType).forEach(type => {
@@ -82,6 +83,17 @@ module.exports = function (driver, stepActionFactory, runMode) {
         'email-code-step': JsCodeStepAction,
         'cli-email-code-step': CliJsStepAction,
         'tdk-hybrid': TdkHybridStepAction,
+
+        'sfdc-step-login': SfdcStepAction,
+        'sfdc-step-logout': SfdcStepAction,
+        'sfdc-step-sobjectcreate': SfdcStepAction,
+        'sfdc-step-sobjectdelete': SfdcStepAction,
+        'sfdc-step-findrecord': SfdcStepAction,
+        'sfdc-step-quickaction': SfdcStepAction,
+        'sfdc-step-edit': SfdcStepAction,
+        'sfdc-step-sobjectvalidate': SfdcStepAction,
+        'sfdc-step-launchapp': SfdcStepAction,
+        'sfdc-step-closeconsoletabs': SfdcStepAction,
     };
 
     register(STEP_ACTION_MAPPING, stepActionFactory);

package/player/utils/screenshotUtils.js

@@ -1,7 +1,7 @@
 'use strict';
 
 const Promise = require('bluebird');
-const utils = require('../../utils');
+const pRetry = require('p-retry');
 
 class ScreenshotUtils {
     constructor(tabId, driver, options = { takeScreenshots: true }) {
@@ -29,7 +29,7 @@ class ScreenshotUtils {
         const SCREENSHOT_RETRY_DELAY = 2000;
         const devicePixelRatioPromise = this.currentDevicePixelRatio ? Promise.resolve(this.currentDevicePixelRatio) : this.getDevicePixelRatio();
         const getScreenshot = () => Promise.all([devicePixelRatioPromise, this.driver.takeScreenshot()]);
-        return utils.runWithRetries(getScreenshot, MAX_RETRY_COUNT, SCREENSHOT_RETRY_DELAY)
+        return pRetry(getScreenshot, { retries: MAX_RETRY_COUNT, minTimeout: SCREENSHOT_RETRY_DELAY })
             .then(([devicePixelRatio, image]) => {
                 const base64 = image ? image.value : '';
                 const dataUrl = `data:image/png;base64,${this.base64AddPadding(base64.replace(/[\r\n]/g, ''))}`;

package/player/utils/windowUtils.js

@@ -1,7 +1,7 @@
 'use strict';
 
 const Promise = require('bluebird');
-const utils = require('../../utils');
+const pRetry = require('p-retry');
 const { PageNotAvailableError } = require('../../errors');
 const logger = require('../../commons/logger').getLogger('window-utils');
 
@@ -152,12 +152,12 @@ WindowUtils.prototype.quit = function () {
 };
 
 WindowUtils.prototype.getOsAndBrowser = function () {
-    return utils.runWithRetries(() => this.driver.getBrowserAndOS())
+    return pRetry(() => this.driver.getBrowserAndOS(), { retries: 3 })
         .then(osAndBrowser => ({ uaBrowserName: osAndBrowser.browser, uaOs: osAndBrowser.os, userAgent: osAndBrowser.userAgent, browserVersion: osAndBrowser.browserVersion }));
 };
 
 WindowUtils.prototype.getUserAgentInfo = function () {
-    return utils.runWithRetries(() => this.driver.getUserAgentInfo());
+    return pRetry(() => this.driver.getUserAgentInfo(), { retries: 3 });
 };
 
 module.exports = WindowUtils;

package/player/webdriver.js

@@ -1,6 +1,7 @@
-"use strict";
+/* eslint-disable no-var */
+'use strict';
 
-const logger = require('../commons/logger').getLogger("webdriver");
+const logger = require('../commons/logger').getLogger('webdriver');
 const Promise = require('bluebird');
 const parser = require('ua-parser-js');
 const desiredCapabilitiesBuilder = require('../commons/testimDesiredCapabilitiesBuilder');
@@ -14,8 +15,7 @@ const featureFlags = require('../commons/featureFlags');
 const _ = require('lodash');
 
 const [LEFT, RIGHT] = [0, 2];
-const { extractElementId } = utils;
-const httpRequest = require('../commons/httpRequest');
+const { extractElementId, getCdpAddressForHost } = utils;
 const perf = require('../commons/performance-logger');
 const { SeleniumPerfStats, SELENIUM_PERF_MARKS } = require('../commons/SeleniumPerfStats');
 
@@ -34,15 +34,11 @@ const playerUtils = () => {
 
 async function getCdpAddress(initResult) {
     try {
-        const debuggerAddress = initResult && initResult.value && initResult.value['goog:chromeOptions'] && initResult.value['goog:chromeOptions'].debuggerAddress;
-        if (!debuggerAddress) {
+        const debuggerHost = initResult && initResult.value && initResult.value['goog:chromeOptions'] && initResult.value['goog:chromeOptions'].debuggerAddress;
+        if (!debuggerHost) {
             return undefined;
         }
-        const browserEndpoint = await httpRequest.get('http://' + debuggerAddress + '/json/version');
-        if (!browserEndpoint) {
-            return undefined;
-        }
-        return browserEndpoint.webSocketDebuggerUrl;
+        return await getCdpAddressForHost(debuggerHost);
     } catch (e) {
         logger.info('Error getting cdpAddress', e);
         return undefined;
@@ -202,7 +198,10 @@ class WebDriver extends WebDriverApi {
 
     switchToLocatedFrame(locatedElement) {
         return this.getElement(locatedElement)
-            .tap(el => this.switchToFrame(el.value));
+            .then(async el => {
+                await this.switchToFrame(el.value);
+                return el;
+            });
     }
 
     switchToFrame(el) {
@@ -220,7 +219,7 @@ class WebDriver extends WebDriverApi {
 
     getElement(locatedElement) {
         const perfId = this.seleniumPerfStats.markStart(SELENIUM_PERF_MARKS.GET_ELEMENT);
-        if (typeof locatedElement === "string" || typeof locatedElement === "number") { // support testimId in the meanwhile for backwards compatability
+        if (typeof locatedElement === 'string' || typeof locatedElement === 'number') { // support testimId in the meanwhile for backwards compatability
             return this.getElementBySelector(`[testim_dom_element_id='${locatedElement}']`)
                 .finally(() => this.seleniumPerfStats.markEnd(perfId, SELENIUM_PERF_MARKS.GET_ELEMENT));
         }
@@ -353,13 +352,13 @@ class WebDriver extends WebDriverApi {
 
             function isTextElement(element) {
                 var tagName = element.tagName;
-                return (tagName === "INPUT" || tagName === "TEXTAREA");
+                return (tagName === 'INPUT' || tagName === 'TEXTAREA');
             }
 
             function getFixedTextContent(element) {
                 try { // fix for salesforce's custom-elements
                     if (element.shadowRoot && Object.getOwnPropertyDescriptor(element.ownerDocument.defaultView.Node.prototype,'textContent').get.toString().indexOf('[native code]') === -1) {
-                        return element.shadowRoot.textContent.replace(/(\r\n|\n|\r)/gm, "");
+                        return element.shadowRoot.textContent.replace(/(\r\n|\n|\r)/gm, '');
                     }
                 } catch (err) { }
                 if (ignoreHiddenTagsText && Array.prototype.some.call(element.children, function (elem) { return elem.hidden; })) {
@@ -386,7 +385,7 @@ class WebDriver extends WebDriverApi {
                         var svgContent = new XMLSerializer().serializeToString(element);
                         copyElement = new DOMParser().parseFromString(svgContent, 'text/html').body.firstChild;
                     }
-                    return clearTitleTags(copyElement).textContent.replace(/(\r\n|\n|\r)/gm, "");
+                    return clearTitleTags(copyElement).textContent.replace(/(\r\n|\n|\r)/gm, '');
                 } else {
                     return getFixedTextContent(element);
                 }
@@ -429,7 +428,7 @@ class WebDriver extends WebDriverApi {
             })
             .catch(err => !(err instanceof IeError), err => {
                 this.seleniumPerfStats.markEnd(perfId, SELENIUM_PERF_MARKS.GET_HTML);
-                const testimInternalError = Object.assign(new Error(), { success: false, reason: err.toString(), errorType: "internal-error" });
+                const testimInternalError = Object.assign(new Error(), { success: false, reason: err.toString(), errorType: 'internal-error' });
                 if (!this.client.requestHandler.sessionID) {
                     // we got here after the driver has been disposed of. It's impossible to run JavaScirpt on the page.
                     testimInternalError.extraInfo = 'Inside getHtml catch and trying to check if in quirks mode - but the session has already terminated';
@@ -438,10 +437,10 @@ class WebDriver extends WebDriverApi {
                 if (!this.isIE()) { // no need to check quirks mode if I'm not in IE
                     throw testimInternalError;
                 }
-                return this.executeJS(`return navigator.userAgent;`)
+                return this.executeJS('return navigator.userAgent;')
                     .catch(() => Promise.reject(testimInternalError))
                     .then(ua => {
-                        const error = this.isUsingUnsupportedCompabilityMode(ua.value) ? this.getIeError("Can’t run test in IE compatibility mode") : testimInternalError;
+                        const error = this.isUsingUnsupportedCompabilityMode(ua.value) ? this.getIeError('Can’t run test in IE compatibility mode') : testimInternalError;
                         return Promise.reject(error);
                     });
             });
@@ -527,7 +526,7 @@ class WebDriver extends WebDriverApi {
         try {
             return parseInt(parse.browser.major);
         } catch (err) {
-            logger.error("failed to get browser version", { err: err });
+            logger.error('failed to get browser version', { err: err });
             return 0;
         }
     }
@@ -536,13 +535,13 @@ class WebDriver extends WebDriverApi {
         function getBrowserName(ua, browserDetails) {
             var M = ua.match(/(opera|chrome|safari|firefox|msie|trident(?=\/))\/?\s*(\d+)/i) || [];
             if (/trident/i.test(M[1])) {
-                return "Internet Explorer " + browserDetails.major;
+                return 'Internet Explorer ' + browserDetails.major;
             }
             if (M[1] === 'Chrome' && ua.match(/\bOPR\/(\d+)/) !== null) {
-                return "opera";
+                return 'opera';
             }
             if (M[1] === 'Chrome' && ua.match(/\bEdge|Edg\/(\d+)/) !== null) {
-                return "edge";
+                return 'edge';
             }
             M = M[2] ? [M[1], M[2]] : [ua.appName, ua.appVersion, '-?'];
             var tem = ua.match(/version\/(\d+)/i);
@@ -558,7 +557,7 @@ class WebDriver extends WebDriverApi {
 
         return this.executeJS(function () {
             if (typeof window === 'undefined' || !window.navigator || !window.navigator.userAgent) {
-                return "unknown";
+                return 'unknown';
             }
             return window.navigator.userAgent;
         }).then(userAgent => {
@@ -566,7 +565,7 @@ class WebDriver extends WebDriverApi {
             const parse = parser(rawUserAgent);
             const osDetails = parse.os;
             this.browserAndOS = {
-                os: osDetails.name + " " + osDetails.version,
+                os: osDetails.name + ' ' + osDetails.version,
                 browserMajor: this.getBrowserMajorVersion(parse),
                 browser: getBrowserName(userAgent.value, parse.browser),
                 userAgent: rawUserAgent,
@@ -595,7 +594,7 @@ class WebDriver extends WebDriverApi {
             !playerUtils().isWithinBounds(-inViewCenter.y, inViewCenter.y, top)) {
             // [NOTE] the code is not supposed to get here! - using center (0,0) instead of step offsets.
             // this is a fallback so the action will take place even if for some reason calculation went out of element..
-            logger.warn("using center as fallback for offset");
+            logger.warn('using center as fallback for offset');
             return this.getMoveActions(0, 0, element);
         }
         return this.getMoveActions(left, top, element);
@@ -612,20 +611,20 @@ class WebDriver extends WebDriverApi {
         const moveActions = this.getRelativeMoveActions(offsets, element);
         const clickActions = this.getClickActions(actions, button);
         return this.actions([{
-            type: "pointer",
-            id: "mouse",
+            type: 'pointer',
+            id: 'mouse',
             parameters: { pointerType: 'mouse' },
-            actions: moveActions.concat(clickActions)
+            actions: moveActions.concat(clickActions),
         }]).catch(err => {
             logger.error('tried to use element origin but failed because of visibility, trying absolute', err);
             const { x, y } = this.computeAbsoluteMovement(offsets);
             const moveActions = this.getMoveActions(x, y);
             return this.actions([{
-                type: "pointer",
-                id: "mouse",
+                type: 'pointer',
+                id: 'mouse',
                 parameters: { pointerType: 'mouse' },
-                actions: moveActions.concat(clickActions)
-            }])
+                actions: moveActions.concat(clickActions),
+            }]);
         });
     }
 
@@ -639,7 +638,7 @@ class WebDriver extends WebDriverApi {
             var getLocatedElement = ${codeSnippets().getLocatedElementCode};
             var dispatchFocus = ${dispatchFocus.toString()};
             var doubleClick = ${doubleClick.toString()};
-            var eventData = ${this.isEdge() ? `JSON.parse(arguments[0])` : `arguments[0]`};
+            var eventData = ${this.isEdge() ? 'JSON.parse(arguments[0])' : 'arguments[0]'};
             var done = arguments[1];
             return doubleClick.call(null, eventData, done);
         `, eventData.timeout, eventParam);
@@ -651,8 +650,8 @@ class WebDriver extends WebDriverApi {
 
     getClickActionList(types = [], button) {
         return [{
-            type: "pointer",
-            id: "mouse",
+            type: 'pointer',
+            id: 'mouse',
             actions: this.getClickActions(types, button)
         }];
     }
@@ -740,26 +739,26 @@ class WebDriver extends WebDriverApi {
 
     getMoveActions(x = 1, y = 1, origin = 'viewport', duration = 0) {
         // force x != 0 ,y != 0 because of Safari issues
-        return [{ type: "pointerMove", duration, x: Math.floor(x) || 1, y: Math.floor(y) || 1, origin }];
+        return [{ type: 'pointerMove', duration, x: Math.floor(x) || 1, y: Math.floor(y) || 1, origin }];
     }
 
     moveWithActionsAPI(point) {
         const actions = this.getMoveActions(point.x, point.y);
         return this.actions([{
-            type: "pointer",
-            id: "mouse",
+            type: 'pointer',
+            id: 'mouse',
             actions: actions
         }]);
     }
 
     moveToElementWithActionsAPI(seleniumElement, offsets) {
         return this.actions([{
-            type: "pointer",
-            id: "mouse",
-            actions: this.getRelativeMoveActions(offsets, seleniumElement)
+            type: 'pointer',
+            id: 'mouse',
+            actions: this.getRelativeMoveActions(offsets, seleniumElement),
         }]).catch(err => {
             logger.error('tried to use element origin but failed because of visibility, trying location', err);
-            const point = this.computeAbsoluteMovement(offsets)
+            const point = this.computeAbsoluteMovement(offsets);
             return this.moveWithActionsAPI(point);
         });
     }
@@ -789,14 +788,14 @@ class WebDriver extends WebDriverApi {
         const doDrag = this.getMoveActions(xDiff, yDiff, 'pointer', 1);
         const endDrag = this.getClickActions(['pointerUp'], LEFT);
         return this.actions([{
-            type: "pointer",
-            id: "mouse",
+            type: 'pointer',
+            id: 'mouse',
             actions: goToDrag.concat(startDrag).concat(doDrag).concat(endDrag)
-        }])
+        }]);
     }
 
     drag(seleniumElement, targetRect, xElementOffset, yElementOffset, events) {
-        const { width, height } = targetRect
+        const { width, height } = targetRect;
         const midXRelative = this.isEdge() ? xElementOffset : (xElementOffset - width / 2 + 1);
         const midYRelative = this.isEdge() ? yElementOffset : (yElementOffset - height / 2);
         return this.getDragCoordinates(events)
@@ -870,8 +869,8 @@ class WebDriver extends WebDriverApi {
         const actions = startMovePositionActions.concat(pointerDownActions).concat(moveSequenceActions).concat(endMovePositionActions).concat(pointerUpActions);
 
         return this.actions([{
-            type: "pointer",
-            id: "mouse",
+            type: 'pointer',
+            id: 'mouse',
             actions: actions
         }]);
     }
@@ -957,7 +956,7 @@ class WebDriver extends WebDriverApi {
 
     getElementRect(target) {
         let defaultLocation = { width: 0, height: 0, top: 0, left: 0 };
-        return this.getElementLocation(target).catch((err) => logger.error("error getting element location", { err }))
+        return this.getElementLocation(target).catch((err) => logger.error('error getting element location', { err }))
             .then(loc => {
                 if (loc && loc.value) {
                     return {
@@ -972,9 +971,9 @@ class WebDriver extends WebDriverApi {
     }
 
     end() {
-        logger.info("delete session", { sessionId: this.getSessionId() });
+        logger.info('delete session', { sessionId: this.getSessionId() });
         if (!this.getSessionId()) {
-            logger.warn("failed to close session because session is undefined");
+            logger.warn('failed to close session because session is undefined');
         }
         clearInterval(this.keepAliveTimer);
         return super.end()