Skip to content
This repository has been archived by the owner on Dec 18, 2024. It is now read-only.

Commit

Permalink
Merge pull request #768 from MeasureAuthoringTool/MAT-7936
Browse files Browse the repository at this point in the history
MAT-7936 MADiE: Issues around Measure Buttons: Automatic navigation and an Error
  • Loading branch information
adongare authored Nov 22, 2024
2 parents 433b371 + e91ca0d commit a3811f7
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/components/testCaseLanding/qdm/TestCaseList.tsx
Original file line number Diff line number Diff line change
@@ -245,7 +245,7 @@ const TestCaseList = (props: TestCaseListProps) => {

useEffect(() => {
const validTestCases = testCases?.filter((tc) => tc.validResource);
if (validTestCases && calculationOutput) {
if (validTestCases && calculationOutput && selectedPopCriteria) {
const executionResults: CqmExecutionResultsByPatient = calculationOutput;
// calculation output only contains valid testcases already.
const highlightingForAllGroups = buildHighlightingForAllGroups(
2 changes: 1 addition & 1 deletion src/components/testCaseLanding/qiCore/TestCaseList.tsx
Original file line number Diff line number Diff line change
@@ -230,7 +230,7 @@ const TestCaseList = (props: TestCaseListProps) => {

useEffect(() => {
const validTestCases = testCases?.filter((tc) => tc.validResource);
if (validTestCases && calculationOutput?.results) {
if (validTestCases && calculationOutput?.results && selectedPopCriteria) {
// Pull Clause Coverage from coverage HTML
setCoveragePercentage(
getCoverageValueFromHtml(

0 comments on commit a3811f7

Please sign in to comment.