Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Coverage 97% with alias not covered #318

Closed
sb-cecilialiu opened this issue Oct 31, 2024 · 4 comments
Closed

Coverage 97% with alias not covered #318

sb-cecilialiu opened this issue Oct 31, 2024 · 4 comments

Comments

@sb-cecilialiu
Copy link

sb-cecilialiu commented Oct 31, 2024

Summary

For some reasons, the Test Coverage percentage keeps is standing at 97%. I have looked at the logic coverage page and everything appears to be covered except the alias of one of the definitions (see attached screenshot).

External Tracking Ticket

(https://oncprojectracking.healthit.gov/support/browse/MADIE-2065)

Expected Behavior

Should be 100%

Version or Commit

Translator v3.14.0
fqm-execution 1.5.0

Inputs (e.g. Measure Bundle, Patient Bundle, CQL Library)

ColorectalCancerMeasureFHIR4-v0.0.022-FHIR4 (1).zip
ColorectalCancerMeasureFHIR4-v0.0.022-FHIR4-TestCases (1).zip
MADiE_Question2

Relevant Calculation Options (e.g. Measurement Period, meta.profile Validation)

{
includeClauseResults: false,
trustMetaProfile: true,
buildStatementLevelHTML: true,
measurementPeriodStart: 1/1/2024,
measurementPeriodEnd: 12/31/2024,
}

cql-to-elm Version Used for Measure Logic Translation (if known)

v3.14.0

Any Additional Info

It seems like there was a similar issue (#285), but we are still having the issue.

@lmd59
Copy link
Contributor

lmd59 commented Oct 31, 2024

@sb-cecilialiu , Thanks for providing your measure and test cases. I've tested the coverage on fqm-execution 1.5.0, and it looks like this is a drop in coverage introduced with the v3.14.0 translator updates. #314 addresses the changes in coverage for v3.14.0-translated measures, and I've confirmed it brings this measure/Test case set back up to 100% coverage. We plan to include this update in our next release.

@sb-cecilialiu
Copy link
Author

@lmd59 Thanks so much for the update!

@lmd59
Copy link
Contributor

lmd59 commented Nov 4, 2024

Resolved with v1.6.0

@lmd59 lmd59 closed this as completed Nov 4, 2024
@sb-cecilialiu
Copy link
Author

@lmd59 Thank you so much for the quick update!!

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

No branches or pull requests

2 participants