Update to selenium/standalone-chrome:133

Created on 25 March 2025, 16 days ago

Problem/Motivation

We're currently pinned to selenium/standalone-chrome:127, yet the latest version of Chrome is 134. 📌 Update to selenium/standalone-chrome:128 Active seems to require many tests to be updated, so let's update to 133 first, then tackle 134+ afterwards.

Steps to reproduce

Proposed resolution

  • Bump the standalone-chrome version to 133
  • Fix the few tests that need fixing

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

📌 Task
Status

Active

Version

11.0 🔥

Component

phpunit

Created by

🇦🇺Australia mstrelan

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @mstrelan
  • Merge request !11623Update to selenium/standalone-chrome:133 → (Open) created by mstrelan
  • Pipeline finished with Failed
    16 days ago
    Total: 447s
    #457525
  • Pipeline finished with Failed
    16 days ago
    Total: 625s
    #457572
  • 🇦🇺Australia mstrelan

    The fails here are all due to resizing the browser. It seems in 128+ the window.innerHeight is always 139px less than the height you resize to. I tried to debug this in non-headless mode, which was even weirder because resizing to 400px wide would set it to 500px width.

    It seems that the 139px includes the browser chrome (toolbar, infobar, etc), but I'm not sure if the difference is in how the resizeWindow works, or in how the browser reports window.innerHeight. Either way, using window.outerHeight makes BrowserWithJavascriptTest::testJavascript pass.

    There was an option to use --headless=old which would mean we don't need to refactor any tests, but this has been removed in Chrome 132.

    I was able to apply the same logic to LayoutBuilderDisableInteractionsTest::testFormsLinksDisabled and increase the height by 139px, but it seems like a bit of a hack.

    Tried the same approach for the Nightwatch tests but it seems like there must be something else going on there.

  • Pipeline finished with Failed
    16 days ago
    Total: 866s
    #457580
  • Pipeline finished with Failed
    16 days ago
    Total: 698s
    #457602
  • Pipeline finished with Failed
    16 days ago
    Total: 182s
    #457604
  • Pipeline finished with Failed
    16 days ago
    Total: 893s
    #457618
  • Pipeline finished with Failed
    16 days ago
    Total: 654s
    #457646
  • 🇦🇺Australia mstrelan

    Looks like the random fails we saw in 134 are creeping back in, and we've got a green run on 134 with all the additional waits added, so maybe we should just stick with that.

  • 🇬🇧United Kingdom catch

    Nightwatch testing has not really been maintained since it was added (apart from in desperation), but there is 📌 Consider dropping Nightwatch in favor of Functional Javascript tests Active .

Production build 0.71.5 2024