-
Notifications
You must be signed in to change notification settings - Fork 13
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
Make UI tests work on github actions #153
Comments
ensonic
pushed a commit
that referenced
this issue
Mar 2, 2024
With the skipped UI test, CI run finishes in ~6m (instead of 45m !): https://github.com/Buzztrax/buzztrax/actions/runs/8123130041 |
ensonic
added a commit
that referenced
this issue
Mar 9, 2024
Run only the ui test with increased log levels to debug test infra issues on CI. See #153
Open
ensonic
added a commit
that referenced
this issue
Mar 9, 2024
Run only the ui test with increased log levels to debug test infra issues on CI. See #153
ensonic
added a commit
that referenced
this issue
Mar 9, 2024
Run only the ui test with increased log levels to debug test infra issues on CI. See #153
One of the test that also fail locally on debian is:
When I run it with |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We still can't run the UI tests and this makes the whole CI run super slow (as each test runs into a timeout).
Locally the tests mostly work:
The test setup believes it has successfully setup the xvfb:
It is not even clear if the xvfb setup is to blame. A lot of the tests have these as the last log lines:
The test itself seem to actually work and finish quickly (we see cleanup markers in the log). It feels more like the something in the test hardness hangs (e.g. taking down the xvfb server)
Completely alternative idea:
We could try to use a matrix to run groups of tests in parallel.
https://stackoverflow.com/questions/59175332/using-output-from-a-previous-job-in-a-new-one-in-a-github-action
https://docs.github.com/en/actions/using-jobs/using-a-matrix-for-your-jobs
The text was updated successfully, but these errors were encountered: