test(spread): add an improved snap inspector test
Metadata
Current evaluation
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.
Suggested action: —
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 |
|---|---|---|---|---|
| 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 |
|---|---|---|---|---|
| #5845 test(spread): test building base and snapd snaps | snapcraft | merged | Merged spread tests for building base and snapd snaps. Despite initial CI failures linked to snapcraft issue #5843, the changes were approved and integrated to maintain build compatibility with mainline snapcraft. | |
| #4380 ci: download snap to tests dir | snapcraft | merged | Merged a CI fix that ensures scheduled spread tests download the snap to the tests directory. Approved by two reviewers, passed CI checks, and modified one file. | |
| #347 tests: reformat snap inspector test parameters | fetch-service | merged | Merged after two reviewer approvals and passing CI. Reformatted snap inspector test parameters across three files, modifying 176 additions and 74 deletions to standardize test structure. | |
| #150 inspectors/snap: add additional inspector tests | fetch-service | merged | Merged after approval by two reviewers. The pull request adds additional inspector tests for snap. Successfully integrated into the main branch. | |
| #1084 tests: check for submodules before running spread | charmcraft | merged | Merged changes verifying the snapd-testing-tools submodule before running Spread tests. Adds a clear error if missing. Approved by a reviewer, passed CI, and merged with minimal modifications. | |
| #2937 spread tests: do not attempt to remove snapd snap | snapcraft | merged | Merged a one-line fix to prevent spread tests from removing the snapd snap in LXD environments, resolving test failures caused by snapd's protected status. Approved by one reviewer with passing CI checks. | |
| #240 tests: check for submodules before running spread | rockcraft | merged | Merged. Adds a pre-check for the snapd-testing-tools submodule before running spread tests, providing a clear error if missing. Approved by two reviewers, passed CI, and merged successfully. | |
| #2745 tests: update spread tests to account for content snaps | snapcraft | merged | Merged updates to spread tests for gnome and kde extensions to handle content snaps and use gtk. Removed unnecessary snap cleanup logic per reviewer feedback, relying on restore-each for environment purging. Added a NOMATCH helper. Approved and merged. | |
| #1542 tests: add integration tests for build snaps | snapcraft | merged | Merged integration tests for build snaps. Approved by two reviewers and passed CI. Two test scenarios omitted due to known bugs. | |
| #4105 tests: check for submodules before running spread | snapcraft | merged | Merged. Adds a pre-run check for the snapd-testing-tools submodule in spread tests to provide a clear error if missing. Approved by two reviewers, passed CI, and introduced no coverage changes. |