tests: add a sudo marker for integration tests that require sudo
Metadata
Current evaluation
Merged #328 as commit 0a90009, adding guarded sudo markers for integration tests and allowing opt-in via CI or CRAFT_PROVIDERS_TESTS_ENABLE_SUDO; fixes #222.
Resolution: closed resolved
Reason: PR #328 was merged as commit 0a90009. It skips sudo-requiring tests unless CI or the explicit sudo environment variable is available through tox, resolving #222.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Fixes #222
- [ ] Have you followed the guidelines for contributing?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `tox`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #328 as commit 0a90009, adding guarded sudo markers for integration tests and allowing opt-in via CI or CRAFT_PROVIDERS_TESTS_ENABLE_SUDO; fixes #222. | |
| qwen/qwen3.6-35b-a3b | — | closed resolved | Merged PR #328 adds a sudo marker to integration tests requiring sudo, fixing #222 by passing the CI environment variable through tox. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Related To:
canonical/craft-providers#222
(confidence 100%)
PR #328 explicitly states that it fixes #222, the LXD integration-test sudo issue.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1463 test: add test-with-sudo | craft-parts | merged | Merged as commit b83516a4, adding sudo-based test-requires-root CI to run root-dependent tests and fixing #1464. |