tests: disable broken integration test
Metadata
Current evaluation
Merged in #307 (be9bf8e5), temporarily disabling the broken test_get_basic_items Launchpad integration test to restore CI; permanent repair remains tracked in #306.
Resolution: closed resolved
Reason: PR #307 was merged as commit be9bf8e5 and restored CI by commenting out the failing integration test. The permanent test repair remains open in craft-application#306.
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 `tox`?
-----
This is a temporary workaround to fix CI before we get around to #306
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged in #307 (be9bf8e5), temporarily disabling the broken test_get_basic_items Launchpad integration test to restore CI; permanent repair remains tracked in #306. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as commit be9bf8e5: temporarily commented out the broken test_get_basic_items Launchpad integration test to restore CI, with a TODO pointing to follow-up issue #306 for the real fix. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in #307 (be9bf8e5): temporarily disabled a broken integration test to restore CI, with a permanent fix tracked in #306. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a temporary workaround disabling a broken integration test to restore CI functionality, pending a permanent fix in issue #306. Approved by two reviewers with passing checks. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/craft-application#306
(confidence 100%)
Tracks the permanent fix and re-enabling of the integration test disabled by PR #307.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #306 Fix & re-enable test checking for snaps | craft-application | open | Re-enable and fix the commented-out test_get_basic_items integration test (tests/integration/launchpad/test_anonymous_access.py), disabled in PR #307 after lengau's private snap broke it; still commented out with a TODO linking this issue. | |
| #412 integration tests: use a temporary config path | snapcraft | merged | Merged #412, updating integration tests to use an isolated temporary config path for LP #1562000; implementation commit 98395774e was merged by 1d6317259. |