Skip to content
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

feat!(pytest-plugin): make fake_host_architecture modify craft-parts #655

Merged
merged 10 commits into from
Feb 20, 2025

Conversation

lengau
Copy link
Contributor

@lengau lengau commented Feb 19, 2025

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • Have you successfully run make lint && make test?
  • Have you added an entry to the changelog (docs/reference/changelog.rst)?

This modifies the fake_host_architecture fixture in the pytest plugin so
it makes craft_parts also think it's running on that machine.

Requires #653

CRAFT-4158

This should make testing the Application more straightforward, as we're
always in debug mode when we run tests (so `run()` raises exceptions
that would otherwise become InternalErrors).

Already caught a test issue :-)
This adds a pytest plugin for craft-application, with two fixtures:

1. debug_mode: This fixture puts the app into debug mode by default.
2. production_mode: Overrides debug_mode for a function.

This is a breaking change because debug_mode is an auto-used fixture,
meaning any tests that rely on production mode will break.
This modifies the fake_host_architecture fixture in the pytest plugin so
it makes craft_parts also think it's running on that machine.
@lengau lengau changed the base branch from feature/pro-sources to feature/hybrid-commands February 19, 2025 22:04
@lengau lengau marked this pull request as ready for review February 19, 2025 22:04
Copy link
Contributor

@dariuszd21 dariuszd21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@lengau lengau merged commit 0b3b1ed into feature/hybrid-commands Feb 20, 2025
15 checks passed
@lengau lengau deleted the work/pytest-plugin-craft-parts-arch branch February 20, 2025 19:09
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.

3 participants