← Back to issue list

tests: re-enable reactive charms test

View original Github issue

Metadata

Project
charmcraft
Number
#1235
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR canonical/charmcraft#1235 as commit bff2a6a3, re-enabling the reactive charms test with a one-line change after two approvals.

Resolution: closed resolved

Reason: PR #1235 was approved and merged as commit bff2a6a3. The merged change restored the previously disabled reactive charms test.

Evaluated by: mr-cal-bot

No scores available.

Issue body

No description provided.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR canonical/charmcraft#1235 as commit bff2a6a3, re-enabling the reactive charms test with a one-line change after two approvals.
stealth/space-bunny-alpha — closed resolved Merged PR canonical/charmcraft#1235 (commit bff2a6a3), a one-line change re-enabling the reactive charms test after two approvals.
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.

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 in canonical/charmcraft#2426 via commit 7c33dd1b, applying a one-line fix to the reactive-charm spread test for CHARMCRAFT-641.
85%
#1326 fix(test): fix reactive plugin test if 'charm' tool is installed charmcraft merged Merged test-only fix making reactive plugin tests deterministic when the charm executable is installed, via PR #1326 and commit 4a573702.
77%
#1214 tests: add more charms to large spread tests charmcraft closed Closed unmerged after 1,135 days; its large-spread test expansion landed through merged PR canonical/charmcraft#1217 in commit 0f4591a9.
77%
#1198 tests: add charm library multiple versions update test charmcraft merged Merged PR #1198 (commit 154bf5d6) added a 120-line regression test covering updates across multiple charm-library versions, linked to #1020/CRAFT-1916.
76%
#132 test: update charm integration test craft-actions merged Merged in canonical/craft-actions#132 (54f7849), updating the charm pack integration test to use another simple charm after the previous Juju fixture was removed.
75%
#1427 tests(spread): make dependencies restore work even if no charm is built charmcraft merged Merged in canonical/charmcraft#1427 (commit 2d432044), making spread-test dependency restoration work when no charm artifact is built.
74%
#1540 fix: template integration tests charmcraft merged Merged PR #1540 as commit f3f82031, updating integration-test templates to use charmcraft.yaml instead of metadata.yaml and completing the stalled change from #1468.
72%
#49 ci: test the charmcraft actions in more places craft-actions merged Merged PR #49 in commit a8e5310, expanding Charmcraft action CI coverage across more Ubuntu architectures, channels, and pinned-revision runs.
72%
#1541 test: add regression test for manila charm charmcraft merged Merged as d531c104, adding a one-line regression test that packages the Manila charm and guards the metadata regression tracked in canonical/charmcraft#1488.
72%
#1092 Some tests will require the charm plugin to be enabled craft-application open Open, unlabelled issue: charmcraft PR #2722 may make craft-application tests depend on the charm plugin; the maintainer asks to remove that dependency, but no affected test or fix is identified.
70%