Closed mgramigna closed 1 year ago
St.:grey_question: |
Category | Percentage | Covered / Total |
---|---|---|---|
π’ | Statements | 85.01% | 1990/2341 |
π‘ | Branches | 73.68% | 1755/2382 |
π’ | Functions | 87.1% | 351/403 |
π’ | Lines | 85.3% | 1920/2251 |
364 tests passing in 28 suites.
Report generated by π§ͺjest coverage report action from 7a9ae7e55a85339f9fc5b36976882c74274ced71
Summary
The existing
make
target for the proportion boolean integration test didn't work. This was due to the addition of MATGlobalCommonFunctions which included valuesets that ecqm-bundler couldn't resolve.My solution for this was to just remove this library and uses of it for simplicity. It required deleting one test.
I think one of the reasons that this happened was the
git status --porcelain ...
command used in the Makefile to only re-build bundles that have a git diff in their main CQL. I added an option to force rebuilding of bundles which will be useful for testing and testing new updates of ecqm bundler.New behavior
FORCE=true npm run build:test-data
to force a rebuild of the test dataCode changes
FORCE=
environment var for Make targets. Presence of it will re-build the bundles for everything regardless of if the CQL has been modifiedTesting guidance
npm run build:test-data
should do nothingFORCE=true npm run build:test-data
should re-build everything