Closed elsaperelli closed 9 months ago
St.:grey_question: |
Category | Percentage | Covered / Total |
---|---|---|---|
π’ | Statements | 86.35% (+0.02% πΌ) |
2366/2740 |
π‘ | Branches | 73.64% | 2190/2974 |
π’ | Functions | 88.89% | 424/477 |
π’ | Lines | 86.69% (+0.02% πΌ) |
2286/2637 |
448 tests passing in 31 suites.
Report generated by π§ͺjest coverage report action from c4607aae351cf9b0eef3dcc9d99ea9083fcf0274
Summary
This PR adds the
meta.profile
found on thetemplateId
of an ELMRetrieve (if it exists) to theprofile
attribute of a data requirement.New behavior
When the user runs dataRequirements on a measure that was authored using QI-Core, the data requirements output will now contain the
profile
attribute which is populated with themeta.profile
, found in thetemplateId
of the ELMRetrieve.Code changes
src/gaps/RetrievesFinder.ts
- gets thetemplateId
from the ELMRetrieve and adds it to theresults
so it can eventually get added to the data requirement.src/helpers/DataRequirementHelpers.ts
- functionaddProfileToDataRequirement
adds the profile to the data requirement if atemplateId
exists on the ELMRetrieve.src/types/Calculator.ts
- adds optionaltemplateId
to theDataTypeQuery
interface.test/unit/DataRequirements.test.ts
- adds a unit test foraddProfileToDataRequirement
and fixes some typos.test/unit/RetrievesFinder.test.ts
- adds templateId to results.Testing guidance
npm run check
npm run test:integration
profile
exists on the data requirements output and is correct. Question: do we want this to be populated for non qi-core measures as well? That is the current functionality.