Provide a pytest fixture to simplify tests that use `emit`
Metadata
Current evaluation
Added installable pytest fixtures for emitter testing via merged PR canonical/craft-cli#76 (merge commit 826a57f).
Resolution: closed resolved
Reason: PR #76 was merged in commit 826a57f, adding init_emitter and emitter fixtures through craft-cli’s pytest plugin and resolving issue #57.
Evaluated by: mr-cal-bot
No scores available.
Issue body
The functionality is already in charmcraft, need to move it here, but needs to be in a way that installing `craft-cli` makes the plugin available.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Added installable pytest fixtures for emitter testing via merged PR canonical/craft-cli#76 (merge commit 826a57f). | |
| stealth/space-bunny-alpha | — | closed resolved | Added an emit-testing pytest fixture via merged PR canonical/craft-cli#76 (merge commit 826a57f). | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without implementation. The requested pytest fixture for emit tests was abandoned after the maintainer noted the functionality required migration from charmcraft to craft-cli. No resolution details were provided. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without resolution. The maintainer proposed moving an existing pytest fixture from charmcraft to craft-cli to simplify tests using emit, but the request was abandoned with no implementation or follow-up. | |
| qwen/qwen3.6-35b-a3b | — | — | A pytest fixture for emit was requested but abandoned. No implementation or discussion occurred, and the feature remains unaddressed. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/craft-cli#76
(confidence 100%)
The merged PR implemented and tested the requested emitter pytest fixtures.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #76 Added init_emitter and emitter fixtures (CRAFT-994). | craft-cli | merged | Merged PR #76 (commit 826a57f), adding tested init_emitter and emitter pytest fixtures, removing a redundant conftest fixture, and fixing #57. |