Skip to content

Disable Selenium test for now #197

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JoeyBF
Copy link
Collaborator

@JoeyBF JoeyBF commented Jul 4, 2025

The switch from Ubuntu 20.04 which reached EOL to Ubuntu 24.04 broke our selenium tests. I don't have the bandwidth to deal with that right now, but at least the webserver jobs still work. Since deploy depends on selenium, that also means that we won't be deploying new versions until this gets fixed. I feel it's ok since the webserver hasn't changed in years.

@hoodmane
Copy link
Contributor

hoodmane commented Jul 4, 2025

It's failing to find the firefox driver, maybe it can find the chrome driver? Let's try just commenting out line 54:
https://github.com/SpectralSequences/sseq/blob/master/web_ext/sseq_gui/Makefile?plain=1#L54

@JoeyBF
Copy link
Collaborator Author

JoeyBF commented Jul 4, 2025

No it gives the same error, just for the chrome driver: https://github.com/JoeyBF/sseq/actions/runs/16076913009/job/45374256882

What's extra strange is that this uses selenium 4.12, and since 4.6 it should download the correct driver automatically

@hoodmane
Copy link
Contributor

hoodmane commented Jul 4, 2025

I tried starting up the github runner docker image locally but I got a different error than the one in CI, so I failed to reproduce it.

@JoeyBF
Copy link
Collaborator Author

JoeyBF commented Jul 4, 2025

The main difference between 20.04 and 22.04 for us is that firefox is now installed as a snap, so maybe that confuses selenium? I doubt it though because I would expect lots of people complaining about that if it were the case

@JoeyBF
Copy link
Collaborator Author

JoeyBF commented Jul 4, 2025

By the way I did try adding https://github.com/namespacelabs/breakpoint just before the failure, so I was able to reproduce it in that sense, but I still didn't manage to find a solution

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants