← Back to issue list

Integration test for poetry plugin requires root/apt state (not marked, fails locally)

View original Github issue

Metadata

Project
craft-parts
Number
#1470
Type
issue
State
open
Author
gcomneno
Labels
Created
Updated
Closed

Current evaluation

Poetry plugin integration tests fail for unprivileged users because they apt-install build deps (findutils, python3-poetry, etc.). Reporter re-verified on current main that all 16 cases fail and offered a one-line fix adding the existing requires_root marker. No maintainer response yet.

Suggested action: needs triage

Reason: No labels, no maintainer comments, and no assignee — the sole comment is the reporter's own re-verification on current main (dbc2a503) showing all 16 cases still raise BuildPackagesNotInstalled unprivileged, with an offer to submit a one-line fix. The fix is trivial and matches existing conventions (conftest.py already skips requires_root tests when euid != 0, and CI has a sudo-based test-requires-root job), so it just needs a maintainer to acknowledge and accept the contribution.

Impact: 20 Quick Win: 18.0 Staleness: 25 Complexity: 10 Confidence: 85 Support Request: 35

Issue body

### Summary `tests/integration/plugins/test_poetry.py::test_poetry_plugin[...]` fails when run as a regular user because it triggers apt operations. ### Observed failure Error includes `BuildPackagesNotInstalled` and log lines like "Packages list not refreshed, not running as superuser". ### Why it happens The test appears to run `apt-get update/install` (or otherwise requires refreshed package lists / build deps installation), which typically needs root privileges and/or a CI image prepared in advance. ### Question / intended behavior Is this test expected to run only in privileged CI environments? If so, is there a recommended way to run the integration suite locally (documentation / flags / markers) that I might have missed? ### Suggested direction If the intent is "privileged only", one option could be to mark it (e.g. `@pytest.mark.requires_root`) and/or skip gracefully when not root / when apt state is not available. ### Local workaround Skip the test file, or run only non-privileged tests.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.8-27b
Impact: 20
Quick Win: 18.0
Staleness: 25
Complexity: 10
Confidence: 85
Support Request: 35
needs triage Poetry plugin integration tests fail for unprivileged users because they apt-install build deps (findutils, python3-poetry, etc.). Reporter re-verified on current main that all 16 cases fail and offered a one-line fix adding the existing requires_root marker. No maintainer response yet.
qwen/qwen3.6-35b-a3b
Staleness: 15
Complexity: 10
Confidence: 90
Support Request: 10
needs triage Integration test for poetry plugin fails for non-root users due to unmarked apt operations. Author suggests adding a requires_root pytest marker to skip gracefully locally and offers a one-line fix.
qwen/qwen3.6-35b-a3b
Staleness: 85
Complexity: 20
Confidence: 85
Support Request: 10
needs triage Integration test for poetry plugin fails locally due to unmarked root/apt requirements. No maintainer response yet. Suggests adding pytest markers or skip logic for non-privileged environments.
qwen3.6-35b-a3b-mtp-q6
Staleness: 80
Complexity: 15
Confidence: 90
Support Request: 10
needs triage Integration test for poetry plugin fails locally due to unmarked root/apt requirements. No maintainer response or labels after 136 days.
qwen3.6-35b-a3b-mtp-q6
Staleness: 75
Complexity: 15
Confidence: 85
Support Request: 25
needs triage Poetry plugin integration test fails locally because it requires root and apt state but lacks proper markers. Suggests adding a pytest marker or skip condition. Open 116 days, no comments, awaiting triage.

Update history

Date Change
updated

Related issues

No related issues found above the similarity threshold.