← Back to issue list

callbacks: add integration tests (CRAFT-223)

View original Github issue

Metadata

Project
craft-parts
Number
#79
Type
pull request
State
merged
Author
cmatsuoka
Labels
—
Created
Updated
Closed

Current evaluation

Merged in #79 via adc1421c; added callback integration tests and renamed callbacks.clear() to callbacks.unregister_all() for consistency with plugin APIs.

Resolution: closed resolved

Reason: PR #79 was merged as adc1421c, with implementation commit e836fdf8. It added callback integration coverage and completed the callbacks.clear() to callbacks.unregister_all() rename.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Add integraton tests for callbacks, renaming `callbacks.clear()` to `callbacks.unregister_all()` to be consistent with the unregister method used for plugins. Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com> - [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 in #79 via adc1421c; added callback integration tests and renamed callbacks.clear() to callbacks.unregister_all() for consistency with plugin APIs.
stealth/space-bunny-alpha — closed resolved Merged in #79 via adc1421c: added callback integration tests and renamed callbacks.clear() to callbacks.unregister_all() for consistency with plugin unregister APIs.
qwen/qwen3.6-35b-a3b — — Merged after approval and passing CI. Added integration tests for callbacks and renamed callbacks.clear() to callbacks.unregister_all() to align with plugin unregister methods.
qwen3.6-35b-a3b-mtp-q6 — — Merged pull request adding integration tests for callbacks. Renamed callbacks.clear() to callbacks.unregister_all() to align with the plugin unregister method. Resolves CRAFT-223.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#77 tests: add integration tests for application plugins (CRAFT-69) craft-parts merged Merged #77 (commit 7aaeb020, merge 04ffbf6e), adding integration coverage for application-defined plugin registration and behavior.
74%