make list plugins tests use mocked plugins
Metadata
Current evaluation
PR snapcraft#2315 was merged as commit 5eb674f4d, updating list-plugins tests to use mocked results so plugin additions or removals no longer require formatting-test edits.
Resolution: closed resolved
Reason: The issue was resolved by merged PR snapcraft#2315, commit 5eb674f4d, which replaced real plugin results in list-plugins tests with mocked results.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Adding or removing a plugin currently involves complicated formatting changes to the list plugins tests. The concern of this test is more of formatting, so used mocked results.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | PR snapcraft#2315 was merged as commit 5eb674f4d, updating list-plugins tests to use mocked results so plugin additions or removals no longer require formatting-test edits. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2315 updated list-plugins tests to use mocked plugin results, preventing plugin additions or removals from requiring formatting-test changes. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2315 (5eb674f4d) updated list-plugins tests to use mocked plugins, resolving LP #1795859 by isolating formatting tests from plugin-set changes. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed. The author implemented mocked plugin results to isolate test formatting logic, resolving the issue independently without discussion. | |
| qwen/qwen3.6-35b-a3b | — | — | Updated list plugins tests to use mocked results, isolating formatting checks from plugin management. The change was implemented and the issue closed. | |
| qwen/qwen3.6-35b-a3b | — | — | Using mocked plugins in list plugin tests to prevent formatting changes was closed without implementation. Zero comments indicate the proposal was abandoned. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed and fixed by replacing live plugin data with mocked results in list plugins tests to prevent formatting changes during plugin updates. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Likely Fixed By:
snapcraft#2315
(confidence 100%)
Merged PR implementing the requested test change and closing LP bug #1795859.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2315 tests: use mocked plugins for list-plugins | snapcraft | merged | Merged in commit 5eb674f4d (#2315), updating list-plugins tests to use mocked plugins; the test change was extracted from #2308 and addressed LP #1795859. | |
| #607 Simplify the list plugins integration test | snapcraft | merged | Merged PR #607 as commit 3fa8595f0, simplifying the list-plugins integration test by removing brittle output-format assertions that required updates whenever plugins were added. | |
| #1596112 list-plugins integration test is broken when run in adt | snapcraft (launchpad) | closed | Fixed by merged PR #607 (commit 3fa8595f0), replacing brittle list-plugins output assertions; snapcraft 2.12 was verified and released to xenial -updates. | |
| #140 cli: implement list-plugins | snapcraft | closed | Implemented list-plugins in commits 32669b7a3 and c9de8ff03; later PR canonical/snapcraft#959 made plugins an alias for the command. | |
| #1510160 list-plugins command | snapcraft (launchpad) | closed | Added the list-plugins command in commit 32669b7a3 and released it in Snapcraft 0.4; the issue was closed after test confirmation. | |
| #138 cli: implement list-plugins | snapcraft | closed | Merged the CLI list-plugins implementation; commit 32669b7a3 added the command, confirmed by snapcraft#31 and released in Snapcraft 0.4. |