From b150e1b97aa2259228223aa9a2bacaaa492d2b53 Mon Sep 17 00:00:00 2001
From: Douglas Cerna <douglascerna@yahoo.com>
Date: Mon, 30 Sep 2024 14:51:37 -0600
Subject: [PATCH] Use old headless mode in Chrome browser

This also sets the environment variables necessary to tell Selenium to
reuse the browsers set up in the Dockerfile instead of downloading
them on its own.
---
 Dockerfile                 | 4 ++++
 amuser/selenium_ability.py | 2 +-
 simplebrowsertest.py       | 2 +-
 3 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/Dockerfile b/Dockerfile
index 25e0835..4ac629c 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -93,6 +93,10 @@ FROM base AS archivematica-acceptance-tests
 ARG USER_ID=1000
 ARG GROUP_ID=1000
 
+ENV SE_MANAGER_PATH=${SELENIUM_DIR}/bin/selenium-manager
+ENV SE_CHROME_PATH=${SELENIUM_DIR}/bin/google-chrome
+ENV SE_FIREFOX_PATH=${SELENIUM_DIR}/bin/firefox
+
 RUN set -ex \
 	&& apt-get -qqy update \
 	&& apt-get -qqy --no-install-recommends install \
diff --git a/amuser/selenium_ability.py b/amuser/selenium_ability.py
index d4c22ab..45e6d5f 100644
--- a/amuser/selenium_ability.py
+++ b/amuser/selenium_ability.py
@@ -39,7 +39,7 @@ def get_driver(self):
         if self.driver_name == "Chrome":
             options = webdriver.ChromeOptions()
             if headless:
-                options.add_argument("--headless=new")
+                options.add_argument("--headless=old")
             driver = webdriver.Chrome(options=options)
             driver.set_window_size(1700, 900)
         elif self.driver_name == "Firefox":
diff --git a/simplebrowsertest.py b/simplebrowsertest.py
index 7459ba1..ded54be 100755
--- a/simplebrowsertest.py
+++ b/simplebrowsertest.py
@@ -8,7 +8,7 @@
 
 def get_chrome_driver():
     options = webdriver.ChromeOptions()
-    options.add_argument("--headless")
+    options.add_argument("--headless=old")
     driver = webdriver.Chrome(options=options)
     return driver