search

canonical / dex-auth-operator

Operator for Dex Auth
Apache License 2.0
3 stars 14 forks source link

selenium: Replace Chrome Webdriver with Firefox #151

Open phoevos opened 1 year ago

phoevos commented 1 year ago

The integration tests are currently using the Chrome driver for selenium.

We have seen our CI fail due to issues with this driver:

selenium.common.exceptions.WebDriverException: Message: unknown error: no chrome binary at /opt/google/chrome/google-chrome

This is most probably what's causing the integration tests to fail:

WARNING  selenium.webdriver.common.selenium_manager:selenium_manager.py:133 The chromedriver version (115.0.5790.110) detected in PATH at /usr/bin/chromedriver might not be compatible with the detected chrome version (115.0.5790.170); currently, chromedriver 115.0.5790.170 is recommended for chrome 115.*, so it is advised to delete the driver in PATH and retry

Goal

Replace the Chrome driver with Firefox, which is OSS, as well as the default browser in Ubuntu. Note that this is what we're already using for the bundle-kubeflow tests.

orfeas-k commented 1 year ago

Workaround

A workaround to the chrome compatibility issue is to pin selenium<4.11 as we did in https://github.com/canonical/dex-auth-operator/pull/156.