Skip to content

Tests: Use mutter dbus runner - #2965

Open
leolost2605 wants to merge 5 commits into
mainfrom
leolost/mutter-dbus-runner
Open

leolost2605 wants to merge 5 commits into
mainfrom
leolost/mutter-dbus-runner

Conversation

@leolost2605

@leolost2605 leolost2605 commented Oct 30, 2026 •

Copy link
Copy Markdown
Member

Mutter actually provides a wrapper that takes care of setting up a suitable environment for tests. Mutter itself and gnome shell use this wrapper. It allows us to reduce our manual environment setup and makes our tests much more robust.

This is one of a few coming PRs that serve as preparation for testing the full gala.

@leolost2605
leolost2605 force-pushed the leolost/mutter-dbus-runner branch 6 times, most recently from 1803327 to 0f5cdcd Compare October 31, 2026 12:56
@leolost2605
leolost2605 marked this pull request as draft October 31, 2026 13:10
@leolost2605
leolost2605 force-pushed the leolost/mutter-dbus-runner branch from 0f5cdcd to 7549b69 Compare November 1, 2026 12:56
By using the  gala_dbus_runner we don't
need dbus-run-session anymore.
By using mutter_dbusrunner we get a fully mocked
environment so that the fatal logs about not
being able to aquire some dbus objects aren't
thrown anymore.
Now that we use mutter_dbusrunner we get a fully
isolated environment allowing us to run the
tests in parallel.
@leolost2605
leolost2605 force-pushed the leolost/mutter-dbus-runner branch from c00265c to 0c04f66 Compare November 1, 2026 13:03
@leolost2605

Copy link
Copy Markdown
Member Author

This should be ready now. The problem was that mutter_dbusrunner from mutter 46 didn't yet isolate the environment enough so that we would have to do quite some stuff of our own to get the CI for mutter 46 to pass. Since we don't release to that anymore I decided it wasn't worth the effort and disabled tests for stable and unstable. By doing that and requiring the mutter dbus runner from 50 or higher we also get the benefit that we can run the tests in parallel which speeds up testing considerably. We now can also get rid of the fatal log handler because the environment is properly mocked.

@leolost2605
leolost2605 marked this pull request as ready for review November 1, 2026 13:08
@leolost2605
leolost2605 requested a review from a team November 1, 2026 13:26
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.

1 participant