Skip to content

Conversation

@antvaset
Copy link
Contributor

@antvaset antvaset commented Oct 20, 2025

  • Add the EnableCoverageCollection engine option. When this is turned on, the engine keeps track of how many times ELM nodes were visited during evaluation, for all libraries involved.
  • Add BranchCollectionVisitor used to create a tree of branches from an ELM library.
  • Add the exportLcovInfo() method that generates lcov.info with DA and BRDA records using node visit counts.
  • Docs published at https://github.com/cqframework/clinical_quality_language/wiki/CQL-Code-Coverage

@github-actions
Copy link

Formatting check succeeded!

@codecov
Copy link

codecov bot commented Oct 20, 2025

Codecov Report

❌ Patch coverage is 79.48718% with 24 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.78%. Comparing base (01788ab) to head (1574faa).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...n/org/opencds/cqf/cql/engine/execution/Coverage.kt 84.88% 4 Missing and 9 partials ⚠️
...cqf/cql/engine/execution/CoverageTest/Library1.cql 72.22% 0 Missing and 5 partials ⚠️
...cqf/cql/engine/execution/CoverageTest/Library2.cql 25.00% 1 Missing and 2 partials ⚠️
...tlin/org/opencds/cqf/cql/engine/execution/State.kt 71.42% 1 Missing and 1 partial ⚠️
...ncds/cqf/cql/engine/execution/EvaluationVisitor.kt 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1634      +/-   ##
============================================
+ Coverage     65.70%   65.78%   +0.07%     
  Complexity     1647     1647              
============================================
  Files           470      473       +3     
  Lines         27990    28107     +117     
  Branches       5429     5475      +46     
============================================
+ Hits          18391    18489      +98     
- Misses         7309     7313       +4     
- Partials       2290     2305      +15     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@antvaset antvaset marked this pull request as ready for review October 21, 2025 10:46
@sonarqubecloud
Copy link

@JPercival JPercival merged commit e8479b7 into master Oct 23, 2025
9 checks passed
@JPercival JPercival deleted the engine-coverage branch October 23, 2025 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants