Skip to content

Conversation

@Tbusk
Copy link
Contributor

@Tbusk Tbusk commented Nov 20, 2025

Closes #507

This PR addresses the following:

  • Adds in missing commas in the existing test
  • Adds the test to a test suite for easier runnability
  • Adds a workflow which runs all tests that are apart of the test suite using a wildcard
  • Fixes the left-over gtk3 init by upgrading it to gtk4 init.

Additional Info

GTK Inits
https://docs.gtk.org/gtk3/func.init.html
https://docs.gtk.org/gtk4/initialization.html

Meson Tests
https://mesonbuild.com/Unit-tests.html

@stsdc
Copy link
Member

stsdc commented Nov 21, 2025

Thanks @tintou nad @Tbusk! This looks really good!

Do you think it is possible to have any coverage stats?

@Tbusk
Copy link
Contributor Author

Tbusk commented Nov 21, 2025

Do you think it is possible to have any coverage stats?

Yes @stsdc - in fact a repo you've worked on looks to have it set up.

lcallarec/live-chart@b0d6481
https://app.codecov.io/gh/lcallarec/live-chart?search=&trend=all%20time&displayType=tree

That approach probably would be the easiest approach.

Meson also seems to have integrations with code coverage, but I haven't had any luck with getting it going due to running into weird errors with paths and permissions.

https://mesonbuild.com/Unit-tests.html

I have been able to run gcovr and lcovr locally as well which meson uses, but haven't found the right configuration to get a usable coverage report yet. Still getting path and permission errors.

@stsdc stsdc merged commit c54ae3f into elementary:main Nov 21, 2025
7 checks passed
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.

Create CI/CD test runner

3 participants