tests: re-enable reactive charms test
Metadata
Current evaluation
Merged a one-line update to re-enable the reactive charms test. Approved by two reviewers with no comments or CI issues. Successfully integrated into the repository.
Suggested action: —
No scores available.
Issue body
No description provided.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a one-line update to re-enable the reactive charms test. Approved by two reviewers with no comments or CI issues. Successfully integrated into the repository. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged by maintainer lengau to re-enable the disabled reactive charms test, restoring test coverage for reactive charm functionality. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to re-enable the reactive charms test suite. The change was integrated directly without comments or revisions. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2426 fix(test): fix the reactive charm spread test | charmcraft | merged | Merged a one-line fix for the reactive charm spread test. Approved by two reviewers and passed CI checks. Resolves CHARMCRAFT-641. | |
| #1326 fix(test): fix reactive plugin test if 'charm' tool is installed | charmcraft | merged | Merged a fix for the reactive plugin test that failed when the charm tool was installed. Approved by two reviewers with a +5/-1 line change. Resolves the test compatibility issue. | |
| #1427 tests(spread): make dependencies restore work even if no charm is built | charmcraft | merged | Merged a one-line change ensuring spread test dependencies restore correctly when no charm is built. Approved by two reviewers and successfully integrated. | |
| #1292 test(spread): make library tests register/unregister charms | charmcraft | merged | Fixed library test failures caused by store timeouts by updating tests to register and unregister charms. Merged following approval from two reviewers, with changes spanning two files (+22/-8 lines). | |
| #1198 tests: add charm library multiple versions update test | charmcraft | merged | Merged after approval, adding a test for updating charm library multiple versions. The change introduces 120 lines to address CRAFT-1916 and issue #1020, with zero unresolved review comments. | |
| #2664 tests: add reproducer for reactive plugin missing .build.manifest (#872) | charmcraft | open | Adds a failing unit test reproducing #872 (reactive plugin omits .build.manifest from .charm artifacts) plus a fix copying the manifest into install_dir. Pending review, 0 reviewers, CI green; branch appears stale vs main (-310 lines). | |
| #1666 fix: only keep metadata.yaml for reactive charms | charmcraft | merged | Merged to fix issue #1654 by restricting metadata.yaml retention to reactive charms only. Approved by two reviewers, passed CI, and modified six files (+101/-5). | |
| #2678 feat: reactive support for 26.04 | charmcraft | merged | Merged after two reviewers approved. The change enables reactive charm support for Ubuntu 26.04. CI failures were unrelated or caused by transient network issues, so they were dismissed and the contribution was accepted. | |
| #1678 ci: bump priority for reactive tests | charmcraft | merged | Merged a CI configuration update that increases the priority of reactive tests to run them earlier due to their long execution times. Approved by three reviewers with zero comments and no CI failures. |