Feature cms104 investigation #308
Conversation
| public void TestCMS125FHIR() { | ||
| // Extract the bundle | ||
| // NOTE: This is a 2021-AUFHIR measure, this is the test to use as the template to add the rest of the content for testing | ||
| private List<DataRequirement> StartMatOutputTest(String matBundleName, String measureLibraryName){ |
There was a problem hiding this comment.
had to more some common functionality into a method to make it so I could save the refreshed measure to a directory
| List<String> edrs = List.of("ServiceRequest","Procedure","Encounter"); | ||
| checkExpectedResourcesPresent(drs, edrs); | ||
|
|
||
| } |
There was a problem hiding this comment.
This is a simple test I added to CMS 104 to check to make sure that the expected FHIR resources are present. I am not sure that this list is complete, but this is enough for the test to fail, because the expected resource "Encouter" is not present
TychonautVII
requested review from
brynrhodes,
pmechineniesac and
abdullah-git1
There was a problem hiding this comment.
Fixed several issues with data requirements in the elm-fhir package, now part of 1.5.5-SNAPSHOT. Updated the test with the latest package from measure developers and verified it is functional. Would be good to get more detailed on the verification based on the manual review from measure developers, but it is minimally functional with just checking for data requirements with the given resource types.
There was a problem hiding this comment.
CMS147 can’t find service request
CMS125 shows the found expected data requirements
After the CQL translator change, all of the measures except for CMS147 are failing in the refreshmeasure stage. The measure is unable to be returned from line 61 of the refreshMeasure function. In fact, even CMS71 is now failing which was not failing prior to the translator change.
pmechineniesac
requested review from
brynrhodes
and removed request for
brynrhodes and
abdullah-git1
# Conflicts: # src/test/java/org/opencds/cqf/tooling/utilities/ECQMCreatorIT.java
…ag to reduce build time.
Updating the data requirements test for CMS 104 to show that an expected data requirement is missing. This PR is tied to the bug reporrt here: #309
Description
Slightly refactored tests to move common functionality into a method so I could save the refreshed measures out more easily
added additional logic to the CMS104 test to cause it to fail and made the bug I'm reporting in the issue more obvious
I've read the contribution guidelines
Code compiles without errors
Tests are created / updated
[N/A] Documentation is created / updated