Bump to Chrome 141

Author: compulim

docker-compose-wsl2.yml

@@ -4,7 +4,7 @@ services:
   # On Windows, run with COMPOSE_CONVERT_WINDOWS_PATHS=1
 
   chrome:
-    image: selenium/node-chrome:110.0
+    image: selenium/node-chrome:141.0
     depends_on:
       - selenium-hub
       - webchat
@@ -20,7 +20,7 @@ services:
       - ./__tests__/setup/local/:/home/seluser/Downloads
 
   selenium-hub:
-    image: selenium/hub:4.8.1
+    image: selenium/hub:4.36.0
     container_name: selenium-hub
     environment:
       GRID_TIMEOUT: '300'

packages/test/harness/src/host/common/host/windowSize.js

@@ -1,11 +1,13 @@
-module.exports = webDriver =>
+module.exports = (/** @type import('selenium-webdriver').WebDriver */ webDriver) =>
   async function windowSize(width, height, element) {
     const rect = await webDriver.manage().window().getRect();
 
     height = +height || rect.height;
     width = +width || rect.width;
 
-    await webDriver.manage().window().setRect({ height, width });
+    // TODO: [P2] We cannot run --headless mode reliably, setting window size will mess things up.
+    //            Look at /packages/test/harness/src/host/jest/allocateWebDriver.js.
+    // await webDriver.manage().window().setRect({ height, width });
 
     /* istanbul ignore next */
     element &&

packages/test/harness/src/host/common/takeStabilizedScreenshot.js

@@ -1,13 +1,22 @@
+const { By } = require('selenium-webdriver');
+
 const until = require('./until');
 
 const TIME_FOR_STABILIZED_SCREENSHOT = 5000; // Time (in ms) to wait for a stabilized screenshot.
 
 // Keep taking screenshot until 2 consecutive screenshots are the same.
-module.exports = async function takeStabilizedScreenshot(webDriver) {
+module.exports = async function takeStabilizedScreenshot(
+  /** @type import('selenium-webdriver').WebDriver */ webDriver
+) {
   let lastScreenshot;
 
   const screenshot = await until(async () => {
-    const screenshot = await webDriver.takeScreenshot();
+    // TODO: [P1] --headless is not quite working.
+    //            Headless is supposed to be chrome-less (no window border).
+    //            After we set webDriver.manage().window().size(1024, 768), webDriver.takeScreenshot() is not returning a 1024x768 image.
+    //            See /packages/test/harness/src/host/jest/allocateWebDriver.js.
+    const element = await webDriver.findElement(By.id('webchat'));
+    const screenshot = await element.takeScreenshot();
 
     if (lastScreenshot === screenshot) {
       return screenshot;

packages/test/harness/src/host/jest-server/index.js

@@ -126,7 +126,9 @@ async function prepareSession(sessionId) {
   await sendWebDriverCommand(sessionId, 'url', { url: 'about:blank' });
 
   // It may have set a different window size.
-  await sendWebDriverCommand(sessionId, 'window/rect', { height: 640, width: 360 });
+  // TODO: [P2] We cannot run --headless mode reliably, setting window size will mess things up.
+  //            Look at /packages/test/harness/src/host/jest/allocateWebDriver.js.
+  // await sendWebDriverCommand(sessionId, 'window/rect', { height: 640, width: 360 });
 
   // Last test may have moved the mouse cursor.
   await sendWebDriverCommand(sessionId, 'actions', {

packages/test/harness/src/host/jest/allocateWebDriver.js

@@ -10,8 +10,12 @@ module.exports = async function allocateWebDriver({ webDriverURL }) {
 
   const builder = new Builder().forBrowser('chrome').setChromeOptions(
     new ChromeOptions()
-      .addArguments('--headless') // More info at https://github.com/SeleniumHQ/selenium/commit/5a97adf9864a346fdd8914cdb1b601c05dd837ac
-      .addArguments('--single-process')
+      // TODO: [P1] --headless is not quite working.
+      //            Headless is supposed to be chrome-less (no window border).
+      //            After we set webDriver.manage().window().size(1024, 768), webDriver.takeScreenshot() is not returning a 1024x768 image.
+      .addArguments('--headless') // WebDriver deprecated .headless(), more info at https://github.com/SeleniumHQ/selenium/commit/5a97adf9864a346fdd8914cdb1b601c05dd837ac
+      .addArguments('--window-size=1920,1080')
+      // .addArguments('--single-process') // --single-process works on 124 but fail silently on >= 133.
       .setAcceptInsecureCerts(true) // We are accessing https://webchat2/ which has a self-signed certificate.
       .setLoggingPrefs(preferences)
   );