Update snaps_tests to use the testing tools from tests.integration
Metadata
Current evaluation
Fixed by merged PR #2096 in commit 0e27e147d, replacing Snapcraft OS-release and repository dependencies in integration tests with test-specific helpers.
Resolution: closed resolved
Reason: PR #2096 was merged as commit 0e27e147d and explicitly fixed #1982 by decoupling integration tests from Snapcraft's os_release and repo modules.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Remove the coupling with snapcraft for:
- deb arch
- os release
- repo
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Fixed by merged PR #2096 in commit 0e27e147d, replacing Snapcraft OS-release and repository dependencies in integration tests with test-specific helpers. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/snapcraft#2096 via commit 0e27e147d, replacing Snapcraft OS-release and repository dependencies in integration tests with test-specific helpers. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2096 (commit 0e27e147d), decoupling integration tests from Snapcraft’s deb architecture, OS release, and repository implementations. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without resolution. The request to decouple snaps_tests from snapcraft dependencies for deb arch, os release, and repo was abandoned. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed and abandoned. The proposal to decouple snaps_tests from snapcraft for deb arch, os release, and repo received no discussion and was superseded by later test infrastructure updates. | |
| qwen/qwen3.6-35b-a3b | — | — | Resolved to decouple snaps_tests from snapcraft for deb architecture, OS release, and repository configuration. Migrated to shared testing utilities from tests.integration. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed without implementation. The request to decouple snaps_tests from snapcraft for deb architecture, OS release, and repository configuration was abandoned or superseded. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
snapcraft#2096
(confidence 100%)
Merged PR #2096 removed the integration tests' dependency on Snapcraft's os_release and repo modules and states that it fixes #1982.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2096 tests: don't use os_release and repo from snapcraft | snapcraft | merged | Merged PR #2096 in commit 0e27e147d, replacing integration tests' use of Snapcraft's os_release and repo modules with test-specific helpers; it fixes #1982. | |
| #1625 tests: use the snapcraft snap for integration tests | snapcraft | merged | Merged PR #1625 updated integration tests to run the installed Snapcraft snap, better matching the user environment. Resolution: commit eebb070e6. | |
| #1541332 Missing build-packages in integration tests | snapcraft (launchpad) | closed | Fixed by merged PR ubuntu-core/snapcraft#289, adding gcc to integration-test build-packages in commit 6e6fe11d7. | |
| #2353 tests: remove dependency on snapcraft for integration tests | snapcraft | merged | PR #2353 by sergiusens merged as commit 6a7c860c4, dropping snapcraft imports from integration tests (27 files, +956/-733) so the suite no longer depends on snapcraft and tests stay independent, alongside the bases/multipass cleanup. | |
| #1542 tests: add integration tests for build snaps | snapcraft | merged | PR #1542 merged as 8c9900f38, adding integration coverage for build snaps; two scenarios remained excluded due to Launchpad bugs #1716054 and #1716055. | |
| #4755 tests: fix devel base snap test | snapcraft | merged | Merged #4755, fixing the devel-base integration test to verify Snapcraft can create the base snap and that the resulting devel base is usable. | |
| #1105 tests: rename the integration test snaps | snapcraft | merged | Renamed integration test snaps across 139 files; PR #1105 merged as commit 02de32a3, addressing Launchpad bug #1661893. | |
| #1849 tests: add snap not found tests | snapcraft | merged | Merged missing-snap test coverage; snapcraft PR #1849 landed as commit 54cf2bd9e. | |
| #588 Implement `snapcraft update` for parts | snapcraft | merged | Merged PR #588 (commit b23494269), adding `snapcraft update` for parts and correcting integration-test environment mocking. | |
| #706 Add integration tests using the snap | charmcraft | closed | Snap-based integration coverage was added through Charmcraft's Spread suite by merged PR canonical/charmcraft#862 (commit 7790c044). |