test: add test-with-sudo
Metadata
Current evaluation
Merged as commit b83516a4, adding sudo-based test-requires-root CI to run root-dependent tests and fixing #1464.
Resolution: closed resolved
Reason: PR #1463 was approved and merged as commit b83516a4. Its test-requires-root target enables CI coverage for tests requiring root access, resolving issue #1464.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [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 `make lint && make test`?
- [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)?
---
Fixes #1464
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged as commit b83516a4, adding sudo-based test-requires-root CI to run root-dependent tests and fixing #1464. | |
| stealth/space-bunny-alpha | — | closed resolved | PR #1463 (commit b83516a4) merged to main, adding a `requires_root` pytest marker, conftest skip for non-root, and a `test-requires-root` CI job running tests under sudo. Closes issue #1464. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged canonical/craft-parts#1463, adding sudo-based tests to cover the root-requiring test suite and fix #1464. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged pull request adding test-with-sudo functionality to address issue #1464. Approved by three reviewers. The change modifies five files with a net addition of 22 lines and removal of 3. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into the main branch to resolve issue #1464. Adds a test-with-sudo test to verify sudo execution paths. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/craft-parts#1464
(confidence 100%)
PR #1463 explicitly states that it fixes the missing root-test CI requested in #1464.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1464 Add CI that runs tests that require root | craft-parts | closed | Root-only tests now run in CI via the test-requires-root job added by merged PR #1463 (commit b83516a4). | |
| #328 tests: add a sudo marker for integration tests that require sudo | craft-providers | merged | 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. |