tests: broaden scope of fixtures
Metadata
Current evaluation
Merged PR #335 as commit 5d9c9c6, broadening selected test fixture scopes to session scope so setup or skipping occurs before dependent fixtures.
Resolution: closed resolved
Reason: The approved PR was merged into main as commit 5d9c9c6. Its change updates fixtures to use broader session-level scoping.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Some fixtures can be session-scoped, which guarantees that they will be created (or skipped) before other fixtures.
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `tox`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #335 as commit 5d9c9c6, broadening selected test fixture scopes to session scope so setup or skipping occurs before dependent fixtures. | |
| qwen/qwen3.6-35b-a3b | — | closed resolved | Merged PR #335 to broaden test fixture scopes to session-level, ensuring earlier creation/skipping, despite unresolved concerns about CI speed. |
Update history
| Date | Change |
|---|---|
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #309 tests: use fixture for base_configuration | craft-providers | merged | Merged PR #309 as commit f4042c9, refactoring tests to use a reusable base_configuration fixture as a prerequisite for CRAFT-1767. | |
| #3910 unit tests: minor improvement to fixture usage | snapcraft | merged | Merged test-only PR #3910 (commit 07d949a71): small refactor of unit-test fixture usage, 8 lines changed across 2 files, no coverage change. |