browsertime 22.0.0 → 22.2.0

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Browsertime changelog (we do [semantic versioning](https://semver.org))
 
+## 22.2.0 - 2024-05-24
+### Added
+* New command: Mouse single click on a element with a specific id `commands.mouse.singleClick.byId(id)` and `commands.mouse.singleClick.byIdAndWait(id)` [#2135](https://github.com/sitespeedio/browsertime/pull/2135).
+* New command: `commands.wait.byIdAndVisible(id, maxTime)` - wait for an element with id to become visible [#2137](https://github.com/sitespeedio/browsertime/pull/2137).
+
+### Fixed
+* Make sure errors are correctly logged if you cannot add text to element [#2136](https://github.com/sitespeedio/browsertime/pull/2136).
+
+## 22.1.0 - 2024-05-20
+### Added
+* Updated to Edge and Edgedriver 125 [#2132](https://github.com/sitespeedio/browsertime/pull/2132).
+
+### Fixed
+* Updated to Selenium Webdriver 4.21.0 [#2131](https://github.com/sitespeedio/browsertime/pull/2131).
+* Trying to remove new Chrome popup [#2130](https://github.com/sitespeedio/browsertime/pull/2130).
+
+
 ## 22.0.0 - 2024-05-16
 
 ### Breaking

package/lib/chrome/webdriver/setupChromiumOptions.js

@@ -27,6 +27,7 @@ const CHROME_FEATURES_THAT_WE_DISABLES = [
   'MediaRouter',
   'OfflinePagesPrefetching',
   'OptimizationHints',
+  'SidePanelPinning',
   'Translate',
   'msAutofillEdgeCoupons',
   'msShoppingTrigger',

package/lib/core/engine/command/addText.js

@@ -33,7 +33,12 @@ export class AddText {
       await element.clear();
       return element.sendKeys(text);
     } catch (error) {
-      log.error('Could not add text %s to id %s', text, id);
+      log.error(
+        'Could not add text %s to id %s error:%s',
+        text,
+        id,
+        error.message
+      );
       log.verbose(error);
       throw new Error(`Could not add text ${text} to id ${id}`);
     }
@@ -57,7 +62,12 @@ export class AddText {
       await element.clear();
       return element.sendKeys(text);
     } catch (error) {
-      log.error('Could not add text %s to xpath %s', text, xpath);
+      log.error(
+        'Could not add text %s to xpath %s error: %s',
+        text,
+        xpath,
+        error.message
+      );
       log.verbose(error);
       throw new Error(`Could not add text ${text} to xpath ${xpath}`);
     }
@@ -81,7 +91,12 @@ export class AddText {
       await element.clear();
       return element.sendKeys(text);
     } catch (error) {
-      log.error('Could not add text %s to selector %s', text, selector);
+      log.error(
+        'Could not add text %s to selector %s error: %s',
+        text,
+        selector,
+        error.message
+      );
       log.verbose(error);
       throw new Error(`Could not add text ${text} to selector ${selector}`);
     }
@@ -105,7 +120,12 @@ export class AddText {
       await element.clear();
       return element.sendKeys(text);
     } catch (error) {
-      log.error('Could not add text %s to class name %s', text, className);
+      log.error(
+        'Could not add text %s to class name %s error: %s',
+        text,
+        className,
+        error.message
+      );
       log.verbose(error);
       throw new Error(`Could not add text ${text} to class name ${className}`);
     }
@@ -129,7 +149,12 @@ export class AddText {
       await element.clear();
       return element.sendKeys(text);
     } catch (error) {
-      log.error('Could not add text %s to name attribute  %s', text, name);
+      log.error(
+        'Could not add text %s to name attribute %s error: %s',
+        text,
+        name,
+        error.message
+      );
       log.verbose(error);
       throw new Error(`Could not add text ${text} to name attribute ${name}`);
     }

package/lib/core/engine/command/mouse/singleClick.js

@@ -230,7 +230,7 @@ export class SingleClick {
 
   /**
    * Clicks on a link whose visible text contains the given substring and waits on the
-   * page complete checl.
+   * page complete check.
    *
    * @async
    * @param {string} text - The substring of the visible text of the link to click.
@@ -247,4 +247,43 @@ export class SingleClick {
       throw new Error('Could not find link by partial text ' + text);
     }
   }
+
+  /**
+   * Clicks on a element with a specific id.
+   *
+   * @async
+   * @param {string} id - The id of the link to click.
+   * @returns {Promise<void>} A promise that resolves when the click action is performed.
+   * @throws {Error} Throws an error if the id is not found.
+   */
+  async byId(id) {
+    try {
+      const element = await this.browser.getDriver().findElement(By.id(id));
+      return this.actions.click(element).perform();
+    } catch (error) {
+      log.error('Could not find the element with id %s', id);
+      log.verbose(error);
+      throw new Error('Could not the element with id ' + id);
+    }
+  }
+
+  /**
+   * Clicks on a element with a specific id and wait on the page complete check
+   *
+   * @async
+   * @param {string} id - The id of the link to click.
+   * @returns {Promise<void>} A promise that resolves when the page has completed.
+   * @throws {Error} Throws an error if the id is not found.
+   */
+  async byIdAndWait(id) {
+    try {
+      const element = await this.browser.getDriver().findElement(By.id(id));
+      await this.actions.click(element).perform();
+      return this.browser.extraWait(this.pageCompleteCheck);
+    } catch (error) {
+      log.error('Could not find the element with id %s', id);
+      log.verbose(error);
+      throw new Error('Could not the element with id ' + id);
+    }
+  }
 }

package/lib/core/engine/command/wait.js

@@ -22,7 +22,7 @@ export class Wait {
   }
 
   /**
-   * Waits for an element with a specific ID to appear within a maximum time.
+   * Waits for an element with a specific ID to be located within a maximum time.
    *
    * @async
    * @param {string} id - The ID of the element to wait for.
@@ -46,6 +46,30 @@ export class Wait {
     }
   }
 
+  /**
+   * Waits for an element with a specific ID to be located and visible within a maximum time.
+   *
+   * @async
+   * @param {string} id - The ID of the element to wait for.
+   * @param {number} maxTime - Maximum time to wait in milliseconds.
+   * @returns {Promise<void>} A promise that resolves when the element is found or the time times out.
+   * @throws {Error} Throws an error if the element is not found within the specified time.
+   */
+  async byIdAndVisible(id, maxTime) {
+    const driver = this.browser.getDriver();
+    await this.byId(id, maxTime);
+    try {
+      await driver.wait(
+        webdriver.until.elementIsVisible(webdriver.By.id(id)),
+        maxTime
+      );
+    } catch (error) {
+      log.error('Element by id %s was not visible in %s ms', id, maxTime);
+      log.verbose(error);
+      throw new Error(`Element by id ${id} was not located in ${maxTime} ms`);
+    }
+  }
+
   /**
    * Waits for an element located by XPath to appear within a maximum time.
    *

package/package.json

@@ -1,15 +1,15 @@
 {
   "name": "browsertime",
   "description": "Get performance metrics from your web page using Browsertime.",
-  "version": "22.0.0",
+  "version": "22.2.0",
   "bin": "./bin/browsertime.js",
   "type": "module",
   "types": "./types/scripting.d.ts",
   "dependencies": {
     "@cypress/xvfb": "1.2.4",
     "@devicefarmer/adbkit": "3.2.6",
-    "@sitespeed.io/chromedriver": "123.0.6312-58",
-    "@sitespeed.io/edgedriver": "123.0.2420-53",
+    "@sitespeed.io/chromedriver": "125.0.6422-60",
+    "@sitespeed.io/edgedriver": "125.0.2535-47",
     "@sitespeed.io/geckodriver": "0.34.0",
     "@sitespeed.io/throttle": "5.0.0",
     "@sitespeed.io/tracium": "0.3.3",
@@ -30,7 +30,7 @@
     "lodash.merge": "4.6.2",
     "lodash.pick": "4.4.0",
     "lodash.set": "4.3.2",
-    "selenium-webdriver": "4.20.0",
+    "selenium-webdriver": "4.21.0",
     "yargs": "17.7.2"
   },
   "optionalDependencies": {

package/types/core/engine/command/addText.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"addText.d.ts","sourceRoot":"","sources":["../../../../lib/core/engine/command/addText.js"],"names":[],"mappings":"AAIA;;;;GAIG;AACH;IACE,0BAKC;IAJC;;OAEG;IACH,gBAAsB;IAGxB;;;;;;;;;OASG;IACH,WALW,MAAM,MACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAezB;IAED;;;;;;;;;OASG;IACH,cALW,MAAM,SACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAezB;IAED;;;;;;;;;OASG;IACH,iBALW,MAAM,YACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAezB;IAED;;;;;;;;;OASG;IACH,kBALW,MAAM,aACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAezB;IAED;;;;;;;;;OASG;IACH,aALW,MAAM,QACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAezB;CACF"}
+{"version":3,"file":"addText.d.ts","sourceRoot":"","sources":["../../../../lib/core/engine/command/addText.js"],"names":[],"mappings":"AAIA;;;;GAIG;AACH;IACE,0BAKC;IAJC;;OAEG;IACH,gBAAsB;IAGxB;;;;;;;;;OASG;IACH,WALW,MAAM,MACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAoBzB;IAED;;;;;;;;;OASG;IACH,cALW,MAAM,SACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAoBzB;IAED;;;;;;;;;OASG;IACH,iBALW,MAAM,YACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAoBzB;IAED;;;;;;;;;OASG;IACH,kBALW,MAAM,aACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAoBzB;IAED;;;;;;;;;OASG;IACH,aALW,MAAM,QACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAoBzB;CACF"}

package/types/core/engine/command/mouse/singleClick.d.ts

@@ -103,7 +103,7 @@ export class SingleClick {
     byPartialLinkText(text: string): Promise<void>;
     /**
      * Clicks on a link whose visible text contains the given substring and waits on the
-     * page complete checl.
+     * page complete check.
      *
      * @async
      * @param {string} text - The substring of the visible text of the link to click.
@@ -111,5 +111,23 @@ export class SingleClick {
      * @throws {Error} Throws an error if the link is not found.
      */
     byPartialLinkTextAndWait(text: string): Promise<void>;
+    /**
+     * Clicks on a element with a specific id.
+     *
+     * @async
+     * @param {string} id - The id of the link to click.
+     * @returns {Promise<void>} A promise that resolves when the click action is performed.
+     * @throws {Error} Throws an error if the id is not found.
+     */
+    byId(id: string): Promise<void>;
+    /**
+     * Clicks on a element with a specific id and wait on the page complete check
+     *
+     * @async
+     * @param {string} id - The id of the link to click.
+     * @returns {Promise<void>} A promise that resolves when the page has completed.
+     * @throws {Error} Throws an error if the id is not found.
+     */
+    byIdAndWait(id: string): Promise<void>;
 }
 //# sourceMappingURL=singleClick.d.ts.map

package/types/core/engine/command/mouse/singleClick.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"singleClick.d.ts","sourceRoot":"","sources":["../../../../../lib/core/engine/command/mouse/singleClick.js"],"names":[],"mappings":"AAIA;;;;;GAKG;AACH;IACE,kDAaC;IAZC;;OAEG;IACH,gBAAsB;IACtB;;OAEG;IACH,gBAAgE;IAChE;;OAEG;IACH,0BAA0C;IAG5C;;;;;;;;OAQG;IACH,eALW,MAAM,kBAEJ,QAAQ,IAAI,CAAC,CAoBzB;IAED;;;;;;;OAOG;IACH,sBAJW,MAAM,GACJ,QAAQ,IAAI,CAAC,CAezB;IAED;;;;;;;;OAQG;IACH,qBALW,MAAM,kBAEJ,QAAQ,IAAI,CAAC,CAsBzB;IAED;;;;;;;OAOG;IACH,4BAJW,MAAM,GACJ,QAAQ,IAAI,CAAC,CAiBzB;IAED;;;;;;;OAOG;IACH,yBAHa,QAAQ,IAAI,CAAC,CAiBzB;IAED;;;;;;;OAOG;IACH,mBAHa,QAAQ,IAAI,CAAC,CAYzB;IAED;;;;;;;OAOG;IACH,iBAJW,MAAM,GACJ,QAAQ,IAAI,CAAC,CAYzB;IAED;;;;;;;OAOG;IACH,wBAJW,MAAM,GACJ,QAAQ,IAAI,CAAC,CAYzB;IAED;;;;;;;OAOG;IACH,wBAJW,MAAM,GACJ,QAAQ,IAAI,CAAC,CAYzB;IAED;;;;;;;;OAQG;IACH,+BAJW,MAAM,GACJ,QAAQ,IAAI,CAAC,CAYzB;CACF"}
+{"version":3,"file":"singleClick.d.ts","sourceRoot":"","sources":["../../../../../lib/core/engine/command/mouse/singleClick.js"],"names":[],"mappings":"AAIA;;;;;GAKG;AACH;IACE,kDAaC;IAZC;;OAEG;IACH,gBAAsB;IACtB;;OAEG;IACH,gBAAgE;IAChE;;OAEG;IACH,0BAA0C;IAG5C;;;;;;;;OAQG;IACH,eALW,MAAM,kBAEJ,QAAQ,IAAI,CAAC,CAoBzB;IAED;;;;;;;OAOG;IACH,sBAJW,MAAM,GACJ,QAAQ,IAAI,CAAC,CAezB;IAED;;;;;;;;OAQG;IACH,qBALW,MAAM,kBAEJ,QAAQ,IAAI,CAAC,CAsBzB;IAED;;;;;;;OAOG;IACH,4BAJW,MAAM,GACJ,QAAQ,IAAI,CAAC,CAiBzB;IAED;;;;;;;OAOG;IACH,yBAHa,QAAQ,IAAI,CAAC,CAiBzB;IAED;;;;;;;OAOG;IACH,mBAHa,QAAQ,IAAI,CAAC,CAYzB;IAED;;;;;;;OAOG;IACH,iBAJW,MAAM,GACJ,QAAQ,IAAI,CAAC,CAYzB;IAED;;;;;;;OAOG;IACH,wBAJW,MAAM,GACJ,QAAQ,IAAI,CAAC,CAYzB;IAED;;;;;;;OAOG;IACH,wBAJW,MAAM,GACJ,QAAQ,IAAI,CAAC,CAYzB;IAED;;;;;;;;OAQG;IACH,+BAJW,MAAM,GACJ,QAAQ,IAAI,CAAC,CAYzB;IAED;;;;;;;OAOG;IACH,SAJW,MAAM,GACJ,QAAQ,IAAI,CAAC,CAYzB;IAED;;;;;;;OAOG;IACH,gBAJW,MAAM,GACJ,QAAQ,IAAI,CAAC,CAazB;CACF"}

package/types/core/engine/command/wait.d.ts

@@ -15,7 +15,7 @@ export class Wait {
      */
     private pageCompleteCheck;
     /**
-     * Waits for an element with a specific ID to appear within a maximum time.
+     * Waits for an element with a specific ID to be located within a maximum time.
      *
      * @async
      * @param {string} id - The ID of the element to wait for.
@@ -24,6 +24,16 @@ export class Wait {
      * @throws {Error} Throws an error if the element is not found within the specified time.
      */
     byId(id: string, maxTime: number): Promise<void>;
+    /**
+     * Waits for an element with a specific ID to be located and visible within a maximum time.
+     *
+     * @async
+     * @param {string} id - The ID of the element to wait for.
+     * @param {number} maxTime - Maximum time to wait in milliseconds.
+     * @returns {Promise<void>} A promise that resolves when the element is found or the time times out.
+     * @throws {Error} Throws an error if the element is not found within the specified time.
+     */
+    byIdAndVisible(id: string, maxTime: number): Promise<void>;
     /**
      * Waits for an element located by XPath to appear within a maximum time.
      *

package/types/core/engine/command/wait.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"wait.d.ts","sourceRoot":"","sources":["../../../../lib/core/engine/command/wait.js"],"names":[],"mappings":"AAKA;;;;;GAKG;AACH;IACE,kDASC;IARC;;OAEG;IACH,gBAAsB;IACtB;;OAEG;IACH,0BAA0C;IAG5C;;;;;;;;OAQG;IACH,SALW,MAAM,WACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAiBzB;IAED;;;;;;;;OAQG;IACH,eALW,MAAM,WACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAmBzB;IAED;;;;;;;;OAQG;IACH,qBALW,MAAM,WACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAuBzB;IAED;;;;;;;OAOG;IACH,WAHW,MAAM,GACJ,QAAQ,IAAI,CAAC,CAIzB;IAED;;;;;OAKG;IACH,oBAFa,QAAQ,IAAI,CAAC,CAIzB;IAED;;;;;;;;OAQG;IACH,0BALW,MAAM,WACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAiBzB;CACF"}
+{"version":3,"file":"wait.d.ts","sourceRoot":"","sources":["../../../../lib/core/engine/command/wait.js"],"names":[],"mappings":"AAKA;;;;;GAKG;AACH;IACE,kDASC;IARC;;OAEG;IACH,gBAAsB;IACtB;;OAEG;IACH,0BAA0C;IAG5C;;;;;;;;OAQG;IACH,SALW,MAAM,WACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAiBzB;IAED;;;;;;;;OAQG;IACH,mBALW,MAAM,WACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAgBzB;IAED;;;;;;;;OAQG;IACH,eALW,MAAM,WACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAmBzB;IAED;;;;;;;;OAQG;IACH,qBALW,MAAM,WACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAuBzB;IAED;;;;;;;OAOG;IACH,WAHW,MAAM,GACJ,QAAQ,IAAI,CAAC,CAIzB;IAED;;;;;OAKG;IACH,oBAFa,QAAQ,IAAI,CAAC,CAIzB;IAED;;;;;;;;OAQG;IACH,0BALW,MAAM,WACN,MAAM,GACJ,QAAQ,IAAI,CAAC,CAiBzB;CACF"}