Closed hossenlopp closed 9 months ago
St.:grey_question: |
Category | Percentage | Covered / Total |
---|---|---|---|
π’ | Statements | 86.58% | 2355/2720 |
π‘ | Branches | 73.83% (-0.05% π») |
2189/2965 |
π’ | Functions | 89.05% | 423/475 |
π’ | Lines | 86.94% | 2276/2618 |
447 tests passing in 31 suites.
Report generated by π§ͺjest coverage report action from a465fa2ec08facbb6771aeaee10beea5f4acc2a7
Summary
Fixes an issue with coverage highlighting where clauses that may have situations with "UNHIT" final results may not be specifically highlighted without coverage and show as covered when they were not. This can happen for any logic that is only relevant when the IPP or DENOM is satisfied. There was no issue with coverage percentage in this situation.
In the case of CMS108, the DENEX related logic would have this bug exhibited when any patients that were testing the IPP fail scenario were included in the set of test patients.
New behavior
Now allows for the UNHIT results to be valid for marking a clause as uncovered instead of inheriting style from the parent clause.
Code changes
HTMLBuilder.ts
- Add UNHIT as an option for a clause to be marked uncovered which "whites" out the coverage styling if the parent clause was covered.Testing guidance
Run coverage calculation with CMS108 and ensure that
doNotPerform
and the uncovered equality comparisons~
show as uncovered instead of inheriting the coverage of their parent clauses.