tests: reformat snap inspector test parameters
Metadata
Current evaluation
Merged PR #347 as commit 51ce4ce, reformatting snap inspector test parameters across three files (+176/-74).
Resolution: closed resolved
Reason: The test-only cleanup was approved with no unresolved review comments and merged as commit 51ce4ce.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] 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 PR #347 as commit 51ce4ce, reformatting snap inspector test parameters across three files (+176/-74). | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/fetch-service#347 as commit 51ce4ce, reformatting snap inspector test parameters across three files (+176/-74). | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #347 via commit 51ce4ce, reformating snap inspector test parameters across three files (+176/-74). | |
| qwen/qwen3.6-35b-a3b | — | — | 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. |
Update history
| Date | Change |
|---|---|
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #521 test(spread): add an improved snap inspector test | fetch-service | merged | Merged in PR #521 (commit 06cd210), adding a more realistic snapd setup to the spread snap inspector test and exposing additional snapd network requests. | |
| #178 fix(inspectors/snap): minor inspector fixes and more test coverage | fetch-service | merged | Merged in PR #178 as commit 521b263, adding Snap inspector bounds checks, propagating SquashFS reader-creation errors, and broader tests. | |
| #2421 tests: remove obsolete snap and external tests | snapcraft | merged | Merged PR #2421 in commit 707395546, deleting 77 obsolete snap/external-test files and 2,959 lines despite a negligible coverage decrease. | |
| #2352 tests: use valid snap names in unit tests | snapcraft | merged | Merged PR #2352 (commit b8c0a4fbf) changed snap names used in unit tests so they pass snap pack --check-skeleton; author merged directly as a minor test-only change. | |
| #458 test: improve test coverage | fetch-service | merged | Merged as commit 47b7773, adding 370 lines of tests across five files; unit tests, linters, Snap builds/tests, and security scans all passed. |