@testim/testim-cli 3.253.0 → 3.255.0

package/player/utils/imageCaptureUtils.js

@@ -1,7 +1,10 @@
+// @ts-check
+
 'use strict';
 
-const Promise = require('bluebird');
+const _ = require('lodash');
 const Jimp = require('jimp');
+const utils = require('../../utils');
 
 const logger = require('../../commons/logger').getLogger('image-capture-utils');
 
@@ -13,22 +16,27 @@ class RectIsOutsideOfImageError extends Error {
     }
 }
 
+/**
+ * @param {never} tabId
+ * @param {import('./windowUtils')} windowUtils
+ * @param {import('./screenshotUtils')} screenshotUtils
+ * */
 const ImageCaptureUtils = function (tabId, windowUtils, screenshotUtils) {
     this.windowUtils = windowUtils;
     this.screenshotUtils = screenshotUtils;
 };
 
-function cropImageFromImageData(imageData, imageInfo) {
-    const image = imageInfo.image || imageInfo;
+async function cropImageFromImageData(imageData, imageInfo) {
+    const _image = imageInfo.image || imageInfo;
     const pixelRatio = imageInfo.devicePixelRatio;
 
-    if (!image) {
-        return Promise.reject(new Error('Failed to get image'));
+    if (!_image) {
+        throw new Error('Failed to get image');
     }
 
-    const imageRegExMatch = image.match(/^data\:[^;]*\;base64,(.*)$/);
+    const imageRegExMatch = _image.match(/^data\:[^;]*\;base64,(.*)$/);
     if (!imageRegExMatch) {
-        return Promise.reject(new Error('Image is not in base64 format'));
+        throw new Error('Image is not in base64 format');
     }
 
     // chromeCropImage2
@@ -39,102 +47,67 @@ function cropImageFromImageData(imageData, imageInfo) {
 
     // elementImage
     if (!imageData.elementRect) {
-        const { image, ...rest } = imageData;
-        logger.warn('missing elementRect', { ...rest });
-        return Promise.resolve({});
+        logger.warn('missing elementRect', _.omit(imageData, 'image'));
+        return {};
     }
 
     const { elementRect } = imageData;
-    return new Promise((resolve, reject) => {
-        Jimp.read(Buffer.from(imageRegExMatch[1], 'base64'), (err, image) => {
-            if (err) {
-                return reject(err);
-            }
-            try {
-                let x = elementRect.left * pixelRatio + offset.left * pixelRatio;
-                let y = elementRect.top * pixelRatio + offset.top * pixelRatio;
-                let width = elementRect.width * pixelRatio;
-                let height = elementRect.height * pixelRatio;
-
-                if (x < 0) {
-                    width += x;
-                    width = width < 0 ? 0 : width;
-                    x = 0;
-                }
+    const image = await Jimp.read(Buffer.from(imageRegExMatch[1], 'base64'));
+    let x = elementRect.left * pixelRatio + offset.left * pixelRatio;
+    let y = elementRect.top * pixelRatio + offset.top * pixelRatio;
+    let width = elementRect.width * pixelRatio;
+    let height = elementRect.height * pixelRatio;
+
+    if (x < 0) {
+        width += x;
+        width = width < 0 ? 0 : width;
+        x = 0;
+    }
 
-                if (y < 0) {
-                    height += y;
-                    height = height < 0 ? 0 : height;
-                    y = 0;
-                }
+    if (y < 0) {
+        height += y;
+        height = height < 0 ? 0 : height;
+        y = 0;
+    }
 
-                const imageWidth = image.bitmap.width;
-                const imageHeight = image.bitmap.height;
-                if ((x + width) > imageWidth) {
-                    width = imageWidth - x;
-                }
+    const imageWidth = image.bitmap.width;
+    const imageHeight = image.bitmap.height;
+    if ((x + width) > imageWidth) {
+        width = imageWidth - x;
+    }
 
-                if ((y + height) > imageHeight) {
-                    height = imageHeight - y;
-                }
+    if ((y + height) > imageHeight) {
+        height = imageHeight - y;
+    }
 
-                if (height <= 0 || width <= 0) {
-                    reject(new RectIsOutsideOfImageError('height or width is equal or lower than zero'));
-                    return undefined;
-                }
+    if (height <= 0 || width <= 0) {
+        throw new RectIsOutsideOfImageError('height or width is equal or lower than zero');
+    }
 
-                const cImage = image.crop(x, y, width, height);
-                cImage.getBase64(Jimp.MIME_PNG, (err, base64) => {
-                    if (err) {
-                        return reject(err);
-                    }
-                    resolve({
-                        elementImage: base64,
-                    });
-                    return undefined;
-                });
-                return undefined;
-            } catch (err) {
-                return reject(err);
-            }
-        });
-    });
+    const cImage = image.crop(x, y, width, height);
+    const base64 = await cImage.getBase64Async(Jimp.MIME_PNG);
+    return { elementImage: base64 };
 }
 
-
-function chromeStitchImage(fullSize, parts) {
-    return new Promise((resolve, reject) => {
-        new Jimp(fullSize.width, fullSize.height, ((err, fullImage) => {
-            if (err) {
-                reject(err);
-                return undefined;
-            }
-            return Promise.each(parts, part => new Promise((resolve, reject) => {
-                const imageRegExMatch = part.image.match(/^data\:[^;]*\;base64,(.*)$/);
-                Jimp.read(Buffer.from(imageRegExMatch[1], 'base64'), (err, partImage) => {
-                    if (err) {
-                        return reject(err);
-                    }
-                    fullImage.composite(partImage, part.position.left, part.position.top, (err, image) => {
-                        if (err) {
-                            return reject(err);
-                        }
-                        resolve(image);
-                        return undefined;
-                    });
-                    return undefined;
-                });
-            })).then(() => {
-                fullImage.getBase64(Jimp.MIME_PNG, (err, base64) => {
-                    if (err) {
-                        return reject(err);
-                    }
-                    resolve(base64);
-                    return undefined;
-                });
+async function chromeStitchImage(fullSize, parts) {
+    const fullImage = await Jimp.read(fullSize.width, fullSize.height);
+
+    for (const part of parts) {
+        const imageRegExMatch = part.image.match(/^data\:[^;]*\;base64,(.*)$/);
+        const partImage = await Jimp.read(Buffer.from(imageRegExMatch[1], 'base64'));
+        // From the code or types doesn't seem like `fullImage.composite` has an async Promise API
+        await new Promise((resolve, reject) => {
+            fullImage.composite(partImage, part.position.left, part.position.top, (err) => {
+                if (err) {
+                    reject(err);
+                    return;
+                }
+                resolve();
             });
-        }));
-    });
+        });
+    }
+    const base64 = await fullImage.getBase64Async(Jimp.MIME_PNG);
+    return base64;
 }
 
 function stitchImage(fullSize, parts) {
@@ -194,10 +167,8 @@ ImageCaptureUtils.prototype = {
         return this.screenshotUtils.takeScreenshot(format)
             .then((imageInfo) => cropImageFromImageData(areas, imageInfo).then((result) => {
                 result.screenImage = imageInfo.image;
-                result.absoluteScreenHighlight = getElementAbsoluteRectangle(
-                    areas.elementRect,
-                    imageInfo.devicePixelRatio);
-                return Promise.resolve(result);
+                result.absoluteScreenHighlight = getElementAbsoluteRectangle(areas.elementRect, imageInfo.devicePixelRatio);
+                return result;
             }));
     },
 
@@ -211,10 +182,8 @@ ImageCaptureUtils.prototype = {
             .then((imageInfo) => {
                 const result = {};
                 result.screenImage = imageInfo.image;
-                result.absoluteScreenHighlight = getElementAbsoluteRectangle(
-                    areas.elementRect,
-                    imageInfo.devicePixelRatio);
-                return Promise.resolve(result);
+                result.absoluteScreenHighlight = getElementAbsoluteRectangle(areas.elementRect, imageInfo.devicePixelRatio);
+                return result;
             }).then(uploadAllDataUrls);
     },
 
@@ -226,14 +195,12 @@ ImageCaptureUtils.prototype = {
         return this.screenshotUtils.getCurrentDevicePixelRatio();
     },
 
-    takeStitchedDataUrl(useImprovedScreenshotStitching) {
+    async takeStitchedDataUrl(useImprovedScreenshotStitching) {
         const windowUtil = this.windowUtils;
         const getCurrentScrollPosition = windowUtil.getCurrentScrollPosition.bind(windowUtil);
 
         const that = this;
-        const stabilize = () => new Promise(resolve => {
-            setTimeout(resolve, 250);
-        });
+        const stabilize = () => utils.delay(250);
         const usingImprovedStitching = Boolean(useImprovedScreenshotStitching);
         const scroll = usingImprovedStitching ?
             (pos) => windowUtil.scrollToPositionWithoutAnimation.bind(windowUtil)(pos) :
@@ -243,12 +210,7 @@ ImageCaptureUtils.prototype = {
             return scroll(position)
                 .then(stabilize)
                 .then(() => that.screenshotUtils.takeScreenshot())
-                .then(imageInfo => {
-                    const imageData = {
-                        elementRect: crop,
-                    };
-                    return cropImageFromImageData(imageData, imageInfo);
-                })
+                .then(imageInfo => cropImageFromImageData({ elementRect: crop }, imageInfo))
                 .then(cropResult => ({
                     position: { left: position.x + crop.left, top: position.y + crop.top },
                     size: { width: crop.width, height: crop.height },
@@ -256,13 +218,13 @@ ImageCaptureUtils.prototype = {
                 }));
         }
 
-        function takeAllParts(positionsData) {
-            const takeAllPartsPromises = positionsData.reduce(function (allParts, nextPos) {
-                const lastPart = allParts[allParts.length - 1];
-                allParts.push(lastPart.then(createPart.bind(this, nextPos.scrollPos, nextPos.cropData)));
-                return allParts;
-            }, [Promise.resolve()]);
-            return Promise.all(takeAllPartsPromises);
+        async function takeAllParts(positionsData) {
+            const allParts = [];
+            for (const nextPos of positionsData) {
+                const part = await createPart(nextPos.scrollPos, nextPos.cropData);
+                allParts.push(part);
+            }
+            return allParts;
         }
 
         function getPartsPositions(fullPageSize, viewPortSize) {
@@ -294,12 +256,11 @@ ImageCaptureUtils.prototype = {
             const positions = getPartsPositions(fullPageSize, viewPortSize);
             const parts = await takeAllParts(positions);
             await windowUtil.scrollToPosition(originalPosition);
-            parts.shift();
             return stitchImage(fullPageSize, parts);
         }
 
-        return Promise.all([windowUtil.getFullPageSize(), windowUtil.getViewportSize()])
-            .then(([fullPageSize, viewPortSize]) => createStitchImage(fullPageSize, viewPortSize));
+        const [fullPageSize, viewPortSize] = await Promise.all([windowUtil.getFullPageSize(), windowUtil.getViewportSize()]);
+        return await createStitchImage(fullPageSize, viewPortSize);
     },
 };
 

package/player/utils/windowUtils.js

@@ -2,6 +2,7 @@
 
 const Promise = require('bluebird');
 const pRetry = require('p-retry');
+const { delay } = require('../../utils');
 const { PageNotAvailableError } = require('../../errors');
 const logger = require('../../commons/logger').getLogger('window-utils');
 
@@ -72,7 +73,7 @@ class WindowUtils {
                 const shouldRetryNavigation = err.seleniumStack && err.message.includes('method IWebBrowser2::Navigate2() failed');
                 if (shouldRetryNavigation && retries > 0) {
                     logger.warn('selenium navigation failed. retrying to navigate', { err });
-                    await Promise.delay(1500);
+                    await delay(1500);
                     return navigate(retries - 1);
                 }
                 throw err;
@@ -80,13 +81,13 @@ class WindowUtils {
             return undefined;
         }
 
-        return Promise.race([navigate(), Promise.delay(NAVIGATION_MAX_TIME)]);
+        return Promise.race([navigate(), delay(NAVIGATION_MAX_TIME)]);
     }
 
     reloadTab(timeoutMSec = 15000) {
         return Promise.race([
             this.driver.reloadTab(),
-            Promise.delay(timeoutMSec),
+            delay(timeoutMSec),
         ]);
     }
 

package/player/webdriver.js

@@ -1,9 +1,9 @@
-/* eslint-disable no-var */
+/* eslint-disable arrow-body-style, no-shadow, @typescript-eslint/prefer-optional-chain, comma-spacing, no-empty, padded-blocks, semi, indent, unicorn/no-useless-promise-resolve-reject, default-param-last, comma-dangle, semi-style, consistent-return, radix, prefer-template, prefer-const, object-shorthand, no-trailing-spaces, operator-linebreak, no-else-return, unicorn/prefer-includes, prefer-arrow-callback, max-len, no-var */
 
 'use strict';
 
+const _ = require('lodash');
 const logger = require('../commons/logger').getLogger('webdriver');
-const Promise = require('bluebird');
 const parser = require('ua-parser-js');
 const desiredCapabilitiesBuilder = require('../commons/testimDesiredCapabilitiesBuilder');
 const { SeleniumError, SeleniumCrashError, IeError } = require('../errors');
@@ -13,7 +13,7 @@ const doubleClick = require('./stepActions/scripts/doubleClick');
 const dispatchFocus = require('./stepActions/scripts/focusElement');
 const { isOldProtocol } = require('./webDriverUtils');
 const featureFlags = require('../commons/featureFlags');
-const _ = require('lodash');
+const { W3C_ELEMENT_ID } = require('./constants');
 
 const [LEFT, RIGHT] = [0, 2];
 const { extractElementId, getCdpAddressForHost } = utils;
@@ -93,7 +93,7 @@ class WebDriver extends WebDriverApi {
             this.cdpUrl = await getCdpAddress(initResult);
             perf.log('after getCdpAddress in webdriver.js init');
             logger.info(`init new session testName: ${testName}`, { sessionId: this.getSessionId(), testResultId });
-            await Promise.delay(driverDelay);
+            await utils.delay(driverDelay);
             await focus();
             perf.log('after focus and delay in webdriver.js init');
         } catch (err) {
@@ -202,7 +202,8 @@ class WebDriver extends WebDriverApi {
     switchToLocatedFrame(locatedElement) {
         return this.getElement(locatedElement)
             .then(async el => {
-                await this.switchToFrame(el.value);
+                const elementId = extractElementId(el.value);
+                await this.switchToFrame({ ELEMENT: elementId, [W3C_ELEMENT_ID]: elementId });
                 return el;
             });
     }
@@ -213,7 +214,7 @@ class WebDriver extends WebDriverApi {
 
     switchToTopFrame() {
         return this.frame().catch(err => {
-            if (err.message && err.message.includes('ECONNREFUSED')) {
+            if (err.message?.includes('ECONNREFUSED')) {
                 throw new SeleniumCrashError();
             }
             throw err;
@@ -221,11 +222,11 @@ class WebDriver extends WebDriverApi {
     }
 
     /**
-     * @returns {Promise<{ value: HTMLElement }>}
+     * @returns {Promise<{ value: HTMLElement | { [W3C_ELEMENT_ID]: string } | { ELEMENT: string } }>}
     */
     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 compatibility
             return this.getElementBySelector(`[testim_dom_element_id='${locatedElement}']`)
                 .finally(() => this.seleniumPerfStats.markEnd(perfId, SELENIUM_PERF_MARKS.GET_ELEMENT));
         }
@@ -452,10 +453,10 @@ class WebDriver extends WebDriverApi {
                     throw testimInternalError;
                 }
                 return this.executeJS('return navigator.userAgent;')
-                    .catch(() => Promise.reject(testimInternalError))
+                    .catch(() => { throw testimInternalError; })
                     .then(ua => {
                         const error = this.isUsingUnsupportedCompabilityMode(ua.value) ? this.getIeError('Can’t run test in IE compatibility mode') : testimInternalError;
-                        return Promise.reject(error);
+                        throw error;
                     });
             });
     }
@@ -586,7 +587,7 @@ class WebDriver extends WebDriverApi {
                 userAgent: rawUserAgent,
                 browserVersion: parse.browser.version
             };
-            return Promise.resolve(this.browserAndOS);
+            return this.browserAndOS;
         });
     }
 
@@ -690,7 +691,7 @@ class WebDriver extends WebDriverApi {
                     this.unsupportedActions.move = true;
                     return this.rightClickWithActionsAPI(element, offsets);
                 }
-                return Promise.reject(err);
+                throw err;
             });
     }
 
@@ -704,7 +705,7 @@ class WebDriver extends WebDriverApi {
                     this.unsupportedActions.move = true;
                     return this.leftClickWithActionsAPI(element, offsets);
                 }
-                return Promise.reject(err);
+                throw err;
             });
     }
 
@@ -826,7 +827,7 @@ class WebDriver extends WebDriverApi {
                             this.unsupportedActions.move = true;
                             return this.dragWithActionsAPI(seleniumElement, xDiff, yDiff, midXRelative, midYRelative);
                         }
-                        return Promise.reject(err);
+                        throw err;
                     });
             });
     }
@@ -857,15 +858,17 @@ class WebDriver extends WebDriverApi {
         }, [{ x: Math.round(startLeft), y: Math.round(startTop) }]);
     }
 
-    dragAndDropWithGeneratedMoves(sourceSeleniumElement, destinationSeleniumElement, rectsAndOffsets) {
+    async dragAndDropWithGeneratedMoves(sourceSeleniumElement, destinationSeleniumElement, rectsAndOffsets) {
         const { fromRect, fromX, fromY, toRect, toX, toY } = rectsAndOffsets;
         const moveSequence = this.getMoveRelativeSequence(fromRect.left + fromX, fromRect.top + fromY, toRect.left + toX, toRect.top + toY);
 
-        return this.moveTo(extractElementId(sourceSeleniumElement), Math.round(fromX), Math.round(fromY))
-            .then(() => this.buttonDown())
-            .then(() => Promise.each(moveSequence, movePosition => this.moveTo(null, movePosition.x, movePosition.y)))
-            .then(() => this.moveTo(extractElementId(destinationSeleniumElement), Math.round(toX), Math.round(toY)))
-            .then(() => this.buttonUp());
+        await this.moveTo(extractElementId(sourceSeleniumElement), Math.round(fromX), Math.round(fromY));
+        await this.buttonDown();
+        for (const movePosition of moveSequence) {
+            await this.moveTo(null, movePosition.x, movePosition.y);
+        }
+        await this.moveTo(extractElementId(destinationSeleniumElement), Math.round(toX), Math.round(toY));
+        return await this.buttonUp();
     }
 
     dragAndDropWithActionsAPIWithGeneratedMoves(rectsAndOffsets) {
@@ -900,7 +903,7 @@ class WebDriver extends WebDriverApi {
                         this.unsupportedActions.move = true;
                         return this.dragAndDropWithActionsAPIWithGeneratedMoves(rectsAndOffsets);
                     }
-                    return Promise.reject(err);
+                    throw err;
                 });
         }
         if (this.unsupportedActions.move) {
@@ -912,7 +915,7 @@ class WebDriver extends WebDriverApi {
                     this.unsupportedActions.move = true;
                     return this.dragAndDropWithActionsAPI(rectsAndOffsets);
                 }
-                return Promise.reject(err);
+                throw err;
             });
     }
 
@@ -934,7 +937,7 @@ class WebDriver extends WebDriverApi {
                     this.unsupportedActions.move = true;
                     return this.doubleClickFallback(element, eventData, offsets);
                 }
-                return Promise.reject(err);
+                throw err;
             });
     }
 

package/processHandler.js

@@ -2,7 +2,7 @@
 
 'use strict';
 
-const Promise = require('bluebird');
+const { promiseTimeout, promiseMap } = require('./utils');
 const logger = require('./commons/logger').getLogger('process-handler');
 
 const exitHooks = [];
@@ -10,7 +10,7 @@ const exitHooks = [];
 module.exports = function (onExit, _process = process) {
     async function cleanup(err) {
         // give cleanup and socket reports a chance to run
-        await Promise.all(exitHooks.map(x => x())).timeout(10000).catch(() => {});
+        await promiseTimeout(promiseMap(exitHooks, hook => hook()), 10000).catch(() => null);
         onExit(err);
     }
     _process.on('uncaughtException', async (err) => {
@@ -29,7 +29,7 @@ module.exports = function (onExit, _process = process) {
         // rollout manages promises incorrectly and generates unhandled rejections from within their code
         logger.fatal('Caught unhandled promise rejection', reason);
         //TODO(benji) - this is a pretty shitty way to detect this error since rollout can change their API endpoint
-        if (reason && reason.message && reason.message.includes('ENOTFOUND x-api.rollout.io')) {
+        if (reason?.message?.includes('ENOTFOUND x-api.rollout.io')) {
             // this is not a fatal error - we recover from this in feature-flags service
             return;
         }

package/processHandler.test.js

@@ -1,5 +1,5 @@
 
-const { delay } = require('bluebird');
+const { delay } = require('./utils');
 const EventEmitter = require('events');
 const { expect, sinon } = require('../test/utils/testUtils');
 const processHandler = require('./processHandler');

package/reports/consoleReporter.js

@@ -2,7 +2,7 @@
 const chalk = require('chalk');
 const _ = require('lodash');
 
-const utils = require('../utils.js');
+const utils = require('../utils');
 const constants = require('../commons/constants');
 const featureAvailabilityService = require('../commons/featureAvailabilityService');
 const { getAbortedTests, getFailedTests, getPassedTests, getFailureEvaluatingCount, getSkippedCount } = require('./reporterUtils');
@@ -120,7 +120,8 @@ class ConsoleReporter {
         if (isCodeMode) {
             console.log(`Run test plan, Project: ${this.options.project} (Execution ID: ${executionId}):`);
         } else {
-            console.log(`Run${this.buildTestPlanName(isAnonymous, testPlanName)} test plan with ${configString}, Project: ${this.options.project}, Branch: ${this.branchToUse} (${executionId})`);
+            const sfdcCredentialLog = this.options.sfdcCredential ? ` SfdcCredential: ${this.options.sfdcCredential}` : '';
+            console.log(`Run${this.buildTestPlanName(isAnonymous, testPlanName)} test plan with ${configString}, Project: ${this.options.project}, Branch: ${this.branchToUse}${sfdcCredentialLog} (${executionId})`);
         }
         this.printWorkerDivider();
 

package/reports/junitReporter.js

@@ -3,13 +3,11 @@
 'use strict';
 
 const xml2js = require('xml2js');
-const Promise = require('bluebird');
-const fs = Promise.promisifyAll(require('fs'));
-const utils = require('../utils.js');
+const fsPromises = require('fs/promises');
+const utils = require('../utils');
 const {
     isAbortedTest, isSkippedTest, getFailedTests, isFailedTest, getFailureEvaluatingCount, getSkippedCount, getAbortedTests,
 } = require('./reporterUtils');
-const featureFlags = require('../commons/featureFlags.js');
 const featureAvailabilityService = require('../commons/featureAvailabilityService');
 
 class JunitReporter {
@@ -35,7 +33,7 @@ class JunitReporter {
             return undefined;
         }
         try {
-            await fs.writeFileAsync(reportFile, reportText);
+            await fsPromises.writeFile(reportFile, reportText);
             console.log('JUnit XML file saved to', reportFile);
             return testResults;
         } catch (err) {
@@ -56,7 +54,7 @@ function getPrintName(testResult) {
 }
 
 async function report(editorUrl, testPlanResults, projectId, branch, classname, options) {
-    function createTestCaseObject(testResult, projectId) {
+    function createTestCaseObject(testResult) {
         const testResultUrl = utils.getTestUrl(editorUrl, projectId, testResult.testId, testResult.resultId, branch);
         const testResultObject = {
             $: {
@@ -103,7 +101,7 @@ async function report(editorUrl, testPlanResults, projectId, branch, classname,
         }
         return {
             $: testSuiteAttributes,
-            testcase: Object.keys(testResults).map(resultId => createTestCaseObject(testResults[resultId], projectId)),
+            testcase: Object.keys(testResults).map(resultId => createTestCaseObject(testResults[resultId])),
         };
     }
 
@@ -131,9 +129,9 @@ async function report(editorUrl, testPlanResults, projectId, branch, classname,
     try {
         const builder = new xml2js.Builder();
         const jUnitXmlReporter = builder.buildObject(testResultObject);
-        return Promise.resolve(jUnitXmlReporter);
+        return jUnitXmlReporter;
     } catch (err) {
-        return Promise.resolve(createErrorjUnitReporter(err));
+        return createErrorjUnitReporter(err);
     }
 }