← Back to issue list

tests: use mocked plugins for list-plugins

View original Github issue

Metadata

Project
snapcraft
Number
#2315
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged in commit 5eb674f4d (#2315), updating list-plugins tests to use mocked plugins; the test change was extracted from #2308 and addressed LP #1795859.

Resolution: closed resolved

Reason: PR #2315 was merged as commit 5eb674f4d. The maintainer explained that its test change was extracted from the previously reviewed work in snapcraft#2308.

Evaluated by: mr-cal-bot

No scores available.

Issue body

LP: #1795859 Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)? - [ ] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)? - [ ] Have you successfully run `./runtests.sh static`? - [ ] Have you successfully run `./runtests.sh tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged in commit 5eb674f4d (#2315), updating list-plugins tests to use mocked plugins; the test change was extracted from #2308 and addressed LP #1795859.
stealth/space-bunny-alpha — closed resolved Merged PR #2315 updated list-plugins tests to use mocked plugins, resolving LP #1795859. The test change was extracted from already-reviewed PR #2308.
qwen/qwen3.6-35b-a3b — — Merged without review after being extracted from PR #2308. Updates tests to use mocked plugins for the list-plugins command.
qwen3.6-35b-a3b-mtp-q6 — — Merged by maintainer sergiusens without review. Updates tests to use mocked plugins for the list-plugins command, extracted from PR #2308. Resolves LP #1795859.

Update history

No update history recorded yet.

Related work

  • Related To: snapcraft#2308 (confidence 99%)

    The mocked list-plugins test change was extracted from this merged PR after review.

Related issues

Issue Project State Summary Similarity
#1795859 make list plugins tests use mocked plugins snapcraft (launchpad) closed 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.
84%
#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.
76%