cqframework / clinical_quality_language

Clinical Quality Language (CQL) is an HL7 specification for the expression of clinical knowledge that can be used within both the Clinical Decision Support (CDS) and Clinical Quality Measurement (CQM) domains. This repository contains complementary tooling in support of that specification.
https://confluence.hl7.org/display/CDS/Clinical+Quality+Language
Apache License 2.0
251 stars 121 forks source link

Reporting coverage across subprojects #1374

Closed antvaset closed 4 days ago

antvaset commented 1 week ago

Currently, the JaCoCo test reports which are run separately for the engine-fhir, engine, cql-to-elm subprojects only include the classes from the default source sets. This means that e.g. the coverage of cql-to-elm classes is not reported when running engine-fhir tests. Concrete example: resolveMethod in the cql-to-elm subproject has low coverage (link) even though there are many tests for FHIRPath methods inside engine-fhir (link).

Here I'm trying to get code coverage to be reported across subprojects and see if codecov can aggregate the reports which seems to be the case. See coverage for resolveMethod after changing jacocoTestReport config (link) and overall change in coverage for the affected classes (link).

codecov[bot] commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 63.65%. Comparing base (e00c396) to head (f76f939). Report is 3 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1374 +/- ## ============================================ + Coverage 62.26% 63.65% +1.38% + Complexity 6951 2666 -4285 ============================================ Files 492 492 Lines 27727 27731 +4 Branches 5506 5508 +2 ============================================ + Hits 17265 17652 +387 + Misses 8198 7830 -368 + Partials 2264 2249 -15 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.