Skip to content

Ensure schemas are available during testing - #1605

Merged
lwindolf merged 1 commit into
lwindolf:mainfrom
paulgevers:build-schemas-for-tests
Oct 12, 2026
Merged

lwindolf merged 1 commit into
lwindolf:mainfrom
paulgevers:build-schemas-for-tests

Conversation

@paulgevers

Copy link
Copy Markdown
Contributor

At least the 'subscription' test needs an up-to-date schema with respect to pre 2.x versions of liferea. Having the tests point at the systems version of schemas let the test fail if there is no up-to-date schema there. 'gnome.compile_schema' is "only useful when running the application locally for example during tests" [1]. Hence, let's point call that during the build and point the tests at it.

[1] https://mesonbuild.com/Gnome-module.html#gnomecompile_schemas

Disclaimer: I used GPT-6 Luna to help me figure out why the test were failing for me, how to fix it and find canonical documentation which is how I confirmed that the solution seems reasonable.

At least the 'subscription' test needs an up-to-date schema with respect to pre
2.x versions of liferea. Having the tests point at the systems version of
schemas let the test fail if there is no up-to-date schema
there. 'gnome.compile_schema' is "only useful when running the application
locally for example during tests" [1]. Hence, let's point call that during the
build and point the tests at it.

[1] https://mesonbuild.com/Gnome-module.html#gnomecompile_schemas

Disclaimer: I used GPT-6 Luna to help me figure out why the test were failing
for me, how to fix it and find canonical documentation which is how I confirmed
that the solution seems reasonable.
@lwindolf

Copy link
Copy Markdown
Owner

Thank you for the fix. I did not think about this.

Also thanks for the AI transparency!

@lwindolf
lwindolf merged commit 72f0543 into lwindolf:main Oct 12, 2026
1 check 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.

2 participants