Test with Chrome

Author: basil

Jenkinsfile

@@ -42,7 +42,7 @@ def axes = [
   jenkinsVersions: ['lts', 'latest'],
   platforms: ['linux'],
   jdks: [17, 21],
-  browsers: ['firefox'],
+  browsers: ['chrome'],
 ]
 
 stage('Record builds and sessions') {
@@ -55,7 +55,7 @@ stage('Record builds and sessions') {
       }
       axes['jenkinsVersions'].each { jenkinsVersion ->
         infra.withArtifactCachingProxy {
-            sh "rm -rf target && DISPLAY=:0 ./src/main/resources/ath-container/run.sh firefox ${jenkinsVersion} -Dmaven.repo.local=${WORKSPACE_TMP}/m2repo -B clean process-test-resources"
+            sh "rm -rf target && DISPLAY=:0 ./src/main/resources/ath-container/run.sh chrome ${jenkinsVersion} -Dmaven.repo.local=${WORKSPACE_TMP}/m2repo -B clean process-test-resources"
         }
         def coreCommit = sh(script: './core-commit.sh', returnStdout: true).trim()
         /*

docker-compose.yml

@@ -6,16 +6,16 @@ services:
       sh -c "mkdir -p /tmp/videos && chmod 0777 /tmp/videos"
     volumes:
       - shared_tmp:/tmp
-  firefox:
-    container_name: firefox
+  chrome:
+    container_name: chrome
     environment:
       - SE_JAVA_OPTS=-Xmx256m -XX:+HeapDumpOnOutOfMemoryError -XX:+TieredCompilation -XX:TieredStopAtLevel=1
-      - SE_NODE_GRID_URL=http://firefox:4444  # Selenium needs to know where to advertise itself (e.g. BiDi)
+      - SE_NODE_GRID_URL=http://chrome:4444  # Selenium needs to know where to advertise itself (e.g. BiDi)
       - SE_SCREEN_HEIGHT=1090
       - SE_SCREEN_WIDTH=1680
     extra_hosts:
       - host.docker.internal:host-gateway
-    image: selenium/standalone-firefox:4.32.0@sha256:47d0b132a1ec4b976756349885a31bd484b4a40f9dd7839da270823d31d6cdcb
+    image: selenium/standalone-chrome:4.32.0@sha256:c15582eeaf5158ca377cfad5dd286fb8e7a418a6309bdfc8a0bbda866b4dd465
     networks:
       - ath-network
     ports:
@@ -27,11 +27,11 @@ services:
   video:
     container_name: video
     depends_on:
-      - firefox
+      - chrome
       - init_video
     environment:
-      - DISPLAY_CONTAINER_NAME=firefox
-      - SE_NODE_GRID_URL=http://firefox:4444
+      - DISPLAY_CONTAINER_NAME=chrome
+      - SE_NODE_GRID_URL=http://chrome:4444
       - SE_NODE_PORT=4444
       - SE_VIDEO_FILE_NAME=auto
       - SE_VIDEO_FILE_NAME_SUFFIX=false
@@ -50,13 +50,13 @@ services:
         - jenkins/ath:latest
     container_name: mvn
     depends_on:
-      - firefox
+      - chrome
       - video
     environment:
-      - DISPLAY=firefox:99.0
+      - DISPLAY=chrome:99.0
       - DOCKER_FIXTURES_NETWORK=ath-network
       - JENKINS_LOCAL_HOSTNAME=mvn
-      - REMOTE_WEBDRIVER_URL=http://firefox:4444/wd/hub
+      - REMOTE_WEBDRIVER_URL=http://chrome:4444/wd/hub
       - SELENIUM_PROXY_HOSTNAME=mvn
       - SHARED_DOCKER_SERVICE=true
       - TESTCONTAINERS_HOST_OVERRIDE=host.docker.internal

src/main/java/org/jenkinsci/test/acceptance/FallbackConfig.java

@@ -12,15 +12,12 @@
 import java.io.File;
 import java.io.IOException;
 import java.net.InetAddress;
-import java.net.MalformedURLException;
-import java.net.URL;
 import java.net.UnknownHostException;
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.nio.file.Paths;
 import java.nio.file.StandardCopyOption;
 import java.time.Duration;
-import java.util.HashMap;
 import java.util.Locale;
 import java.util.Map;
 import java.util.logging.Logger;
@@ -66,7 +63,6 @@
 import org.openqa.selenium.firefox.FirefoxDriver;
 import org.openqa.selenium.firefox.FirefoxOptions;
 import org.openqa.selenium.firefox.GeckoDriverService;
-import org.openqa.selenium.remote.Augmenter;
 import org.openqa.selenium.remote.CapabilityType;
 import org.openqa.selenium.remote.LocalFileDetector;
 import org.openqa.selenium.remote.RemoteWebDriver;
@@ -116,19 +112,10 @@ private WebDriver createWebDriver(TestCleaner cleaner, TestName testName) throws
                 return createContainerWebDriver(
                         cleaner, "selenium/standalone-firefox:4.32.0", buildFirefoxOptions(testName));
             case "chrome-container":
-                return createContainerWebDriver(cleaner, "selenium/standalone-chrome:4.32.0", new ChromeOptions());
+                return createContainerWebDriver(
+                        cleaner, "selenium/standalone-chrome:4.32.0", buildChromeOptions(testName));
             case "chrome":
-                Map<String, String> prefs = new HashMap<>();
-                prefs.put(LANGUAGE_SELECTOR, "en");
-                ChromeOptions options = new ChromeOptions();
-                options.setExperimentalOption("prefs", prefs);
-                if (HarRecorder.isCaptureHarEnabled()) {
-                    options.setAcceptInsecureCerts(true);
-                    options.setProxy(createSeleniumProxy(testName.get()));
-                }
-
-                setDriverPropertyIfMissing("chromedriver", ChromeDriverService.CHROME_DRIVER_EXE_PROPERTY);
-                return new ChromeDriver(options);
+                return new ChromeDriver(buildChromeOptions(testName));
             case "safari":
                 return new SafariDriver();
             case "saucelabs":
@@ -157,16 +144,16 @@ private WebDriver createWebDriver(TestCleaner cleaner, TestName testName) throws
         }
     }
 
-    private WebDriver buildRemoteWebDriver(Capabilities options) throws MalformedURLException {
+    private WebDriver buildRemoteWebDriver(Capabilities options) {
         String u = System.getenv("REMOTE_WEBDRIVER_URL");
         if (StringUtils.isBlank(u)) {
             throw new Error("remote-webdriver type browsers require REMOTE_WEBDRIVER_URL to be set");
         }
-        RemoteWebDriver driver = new RemoteWebDriver(
-                new URL(u), // http://192.168.99.100:4444/wd/hub
-                options);
-        driver.setFileDetector(new LocalFileDetector());
-        return new Augmenter().augment(driver);
+        // http://192.168.99.100:4444/wd/hub
+        WebDriver driver =
+                RemoteWebDriver.builder().address(u).addAlternative(options).build();
+        ((RemoteWebDriver) driver).setFileDetector(new LocalFileDetector());
+        return driver;
     }
 
     private String getBrowser() {
@@ -202,9 +189,15 @@ private FirefoxOptions buildFirefoxOptions(TestName testName) throws IOException
 
     private ChromeOptions buildChromeOptions(TestName testName) throws IOException {
         ChromeOptions chromeOptions = new ChromeOptions();
+        chromeOptions.setExperimentalOption("prefs", Map.of(LANGUAGE_SELECTOR, "en"));
+        chromeOptions.enableBiDi();
         if (HarRecorder.isCaptureHarEnabled()) {
             chromeOptions.setProxy(createSeleniumProxy(testName.get()));
         }
+        setDriverPropertyIfMissing("chromedriver", ChromeDriverService.CHROME_DRIVER_EXE_PROPERTY);
+        chromeOptions.setCapability("se:name", testName.get());
+        chromeOptions.setCapability(
+                "se:recordVideo", TestRecorderRule.isRecorderEnabled() && System.getenv("VIDEO_FOLDER") != null);
         return chromeOptions;
     }
 
@@ -266,10 +259,12 @@ public String toString() {
             Thread.sleep(3000); // Give the container and selenium some time to spawn
 
             try {
-                RemoteWebDriver remoteWebDriver =
-                        new RemoteWebDriver(new URL("http://127.0.0.1:" + controlPort + "/wd/hub"), capabilities);
+                WebDriver driver = RemoteWebDriver.builder()
+                        .address("http://127.0.0.1:" + controlPort + "/wd/hub")
+                        .addAlternative(capabilities)
+                        .build();
                 cleaner.addTask(cleanContainer);
-                return new Augmenter().augment(remoteWebDriver);
+                return driver;
             } catch (RuntimeException e) {
                 cleanContainer.close();
                 throw e;