Merge pull request #745 from lcreid/upgrade-selenium

Upgrade selenium

Author: lcreid

.yarnrc

@@ -2,4 +2,4 @@
 # yarn lockfile v1
 
 
-lastUpdateCheck 1729300363924
+lastUpdateCheck 1741731093687

Gemfile

@@ -3,10 +3,9 @@ eval File.read(gems), binding, gems # rubocop: disable Security/Eval
 
 require "#{__dir__}/lib/bootstrap_form/version"
 
-gem "rails", BootstrapForm::REQUIRED_RAILS_VERSION
-gem "sprockets-rails", require: "sprockets/railtie"
-
 gem "bigdecimal" if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new("3.4.0")
 gem "drb" if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new("3.4.0")
 gem "mutex_m" if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new("3.4.0")
+gem "rails", BootstrapForm::REQUIRED_RAILS_VERSION
+gem "sprockets-rails", require: "sprockets/railtie"
 gem "sqlite3", "~> 1.4"

README.md

@@ -21,8 +21,8 @@ services:
     command: /bin/bash
 
   selenium:
-    # Try removing headless=old option when upgrading the version.
-    image: selenium/standalone-chrome:129.0
+    image: selenium/standalone-chrome:133.0
+    shm_size: 2gb
     logging:
       driver: none
     stdin_open: true
@@ -32,3 +32,4 @@ services:
     ports:
       - '4444:4444'
       - '5900:5900'
+      - '7900:7900'

compose.yml

@@ -18,11 +18,11 @@ def remote_selenium? = @remote_selenium ||= ENV["SELENIUM_HOST"].present? || ENV
               {}
             end
 
-  driven_by :selenium, using: :headless_chrome, screen_size: [960, 720], options: options do |capabilities|
+  # Debugging tip: Change to `chrome` (not headless) and then browse to:
+  # http://localhost:7900/?autoconnect=1&resize=scale&password=secret. You can watch the fun there.
+  driven_by :selenium, using: :headless_chrome, screen_size: [960, 800], options: options do |capabilities|
     capabilities.add_argument("force-device-scale-factor=1")
     capabilities.add_argument("lang=#{ENV.fetch('LANG', 'en_CA')}")
-    # Needed for Selenium 129. Presumably remove at some point.
-    capabilities.add_argument("--headless=old")
   end
 
   if remote_selenium?

demo/doc/screenshots/bootstrap/index-122/00_horizontal_form.png

@@ -74,7 +74,7 @@ class BootstrapTest < ApplicationSystemTestCase
         #{html}
       MD
     end
-    augmented_readme.gsub!(/127.0.0.1:\d+/, "test.host")
+    augmented_readme.gsub!(/(127.0.0.1:\d+|shell:3001)/, "test.host")
     File.write(File.expand_path("../../../README.md", __dir__), augmented_readme)
   end