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

Stratified report fix #315

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Stratified report fix #315

wants to merge 3 commits into from

Conversation

lmd59
Copy link
Contributor

@lmd59 lmd59 commented Oct 10, 2024

Summary

Addresses UnexpectedProperty issue in #311.

New behavior

For episode-based measures with stratifiers, (as was already done for patient-based), we now default the strataCode to be the id if the strata.code?.text does not exist

Code changes

  • DetailedResultsBuilder.ts - defaults strataCode to strata.id as secondary preferred value for episode results
  • MeasureReportBuilder.ts - slight efficiency improvement and added clarifying comment
  • Adds tests addressing episode+stratifier cases in DetailedResultsBuilder.test.ts an MeasureReportBuilder.test.ts

Testing guidance

  • npm run check
  • Pull measure/patient from ticket and test using fqm-execution details -m "path\to\measures\EpisodeMeasure.json" -p "path\to\patients\Episode_Patient.json" -o "path\to\output\Episode_Details.json" --vs-api-key {apiKey}

Copy link

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements
85.42% (+0.07% 🔼)
2425/2839
🟡 Branches
73.18% (+0.29% 🔼)
2284/3121
🟢 Functions
87.68% (+0.2% 🔼)
434/495
🟢 Lines
85.72% (+0.07% 🔼)
2342/2732

Test suite run success

459 tests passing in 31 suites.

Report generated by 🧪jest coverage report action from 55b3ac4

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.

2 participants