Closed mgramigna closed 1 year ago
St.:grey_question: |
Category | Percentage | Covered / Total |
---|---|---|---|
🟢 | Statements | 84.52% (+1.37% 🔼) |
1884/2229 |
🟡 | Branches | 72.9% (+2.04% 🔼) |
1649/2262 |
🟢 | Functions | 87.43% (+2.4% 🔼) |
334/382 |
🟢 | Lines | 84.83% (+1.42% 🔼) |
1817/2142 |
330 tests passing in 28 suites.
Report generated by 🧪jest coverage report action from 88011b0d249b52677a5876cd27dfb0ef209bd800
Summary
Marking this as draft for now since it is branched off of #160. I made that the base branch so the code diff is clean to look at here
Fixes #146
This PR adds more information to all population and stratifier results when defined on the Measure group
New behavior
id
, thatid
will be included on the root populationResults for that population, and on any populationResults for individual episodes in the episodeResultscqfm-criteriaReference
, theid
of that referenced population (aka thevalueString
of that extension) will be included on all relevant results similar to aboveid
, that will show up asstrataId
on all stratifier results (including for individual episodes)Code changes
Testing guidance
For the above cases, you should see IDs and criteriaReferenceIds present on the results where appropriate