test(spread): add an improved snap inspector test
Metadata
Current evaluation
Merged in PR #521 (commit 06cd210), adding a more realistic snapd setup to the spread snap inspector test and exposing additional snapd network requests.
Resolution: closed resolved
Reason: The pull request was merged after approval with all listed CI checks passing. Commit 06cd210 adds the improved snap inspector spread test, including stopping snapd, clearing device state, configuring the global proxy, and restarting snapd.
Evaluated by: mr-cal-bot
No scores available.
Issue body
This test attempts to better emulate the setting up of a system by stopping snapd, removing some device-related files and directories, setting the proxy globally on /etc/environment and then re-starting snapd. This uncovers many snapd network requests that are not typically caught by the usual "snap set system proxy.http".
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged in PR #521 (commit 06cd210), adding a more realistic snapd setup to the spread snap inspector test and exposing additional snapd network requests. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged improved snap inspector test coverage, including a more realistic system setup that stops/restarts snapd and exposes additional network requests. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged an improved snap inspector test that emulates system setup by stopping snapd, clearing device files, setting a global proxy, and restarting snapd to uncover network requests. Approved by two reviewers, passed all CI checks, and merged. |
Update history
| Date | Change |
|---|---|
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #534 test(spread): workaround for flaky test | fetch-service | merged | Merged PR #534 (commit 4f89bb7) worked around #533 by skipping the flaky snap names-query assertion while retaining the other spread-test checks. | |
| #1272 tests: initial setup for the snapcraft snap tests with spread | snapcraft | merged | Merged PR #1272 as commit 545344442, adding initial Spread-based test infrastructure for the snapcraft snap. | |
| #5845 test(spread): test building base and snapd snaps | snapcraft | merged | Merged as fb8c81e8f, adding manual Spread coverage that builds current base snaps and the snapd snap against mainline Snapcraft. | |
| #5841 Include "build snapd snap" test in test suite | snapcraft | closed | Added spread coverage for building snapd and base snaps in merged PR #5845 (commit fb8c81e8f). | |
| #150 inspectors/snap: add additional inspector tests | fetch-service | merged | Merged PR #150 as commit 7e0b5b4, expanding test coverage for the Snap inspector after approval by two reviewers. | |
| #2211 tests: add spread suite for ament plugin | snapcraft | merged | Merged PR #2211 as commit bdaa8ea5, replacing the Ament plugin snapd integration test with a Spread suite and resolving LP #1783827. | |
| #432 ci: speed up spread testing workflow | fetch-service | merged | Merged PR #432 (commit 9eec284), removing snap-test's dependency on snap-build so spread tests start sooner while snap-build still produces downloadable artifacts. | |
| #2183 tests: add spread suite for cmake plugin | snapcraft | merged | Merged PR #2183 as commit 51518dbe6, moving CMake plugin and snapd integration tests into a Spread suite and resolving LP #1783830. | |
| #2745 tests: update spread tests to account for content snaps | snapcraft | merged | Merged PR #2745 updated GNOME and KDE spread tests for content snaps, switched GNOME 3.28 coverage to gtk, and removed redundant snap cleanup. | |
| #3212 tests: expand spread coverage for stage-snaps | snapcraft | closed | Spread-test PR was closed unmerged in favor of snapcraft#3211, merged as 0870f1738 to fix stage-snaps handling for v2 plugins. |