← Back to issue list

Include branch coverage measurement in unit tests

View original Github issue

Metadata

Project
charmcraft
Number
#336
Type
pull request
State
merged
Author
mthaddon
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #336 via commit c6221ed8, adding branch coverage measurement to unit tests so untested implicit else paths are reported.

Resolution: closed resolved

Reason: PR #336 was merged into the main branch, resolving the coverage limitation with commit c6221ed8 (merge commit 0df6d3b3). The change enables branch coverage so both outcomes of conditionals are measured.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Add branch coverage measurement to unit tests. See https://coverage.readthedocs.io/en/coverage-5.5/branch.html?highlight=cov-branch#branch-coverage-measurement for details, and https://github.com/mthaddon/plan-services-test for an example of a side effect of this.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #336 via commit c6221ed8, adding branch coverage measurement to unit tests so untested implicit else paths are reported.
stealth/space-bunny-alpha — closed resolved Merged PR #336, commit c6221ed8, adding branch coverage measurement to unit tests so untested implicit else paths are reported.
qwen/qwen3.6-35b-a3b — — Merged a one-line change enabling branch coverage measurement in unit tests. Approved by two reviewers, the update tracks implicit else branches, resolving gaps in coverage reporting.
qwen3.6-35b-a3b-mtp-q6 — — Merged. Added branch coverage measurement to unit tests to track both true and false conditional evaluation paths. Resolves incomplete coverage reporting for implicit else branches. Change accepted and merged.

Update history

No update history recorded yet.

Related issues

No related issues found above the similarity threshold.