Merge pull request #151 from MeasureAuthoringTool/hangouts_hotfix_MAT…

…-7431 MAT-7431 incorrect message when library saved with blank cql
MeasureAuthoringTool · Jul 8, 2024 · 9f4202a · 9f4202a
2 parents 97aa7f1 + 43541a9
commit 9f4202a
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 23 deletions.
diff --git a/src/components/editCqlLibrary/EditCqlLibrary.test.tsx b/src/components/editCqlLibrary/EditCqlLibrary.test.tsx
@@ -559,7 +559,9 @@ describe("Edit Cql Library Component", () => {
     (synchingEditorCqlContent as jest.Mock)
       .mockClear()
       .mockImplementation(() => {
-        return "library UpdateName version '1.0.000'";
+        return {
+          cql: "library UpdateName version '1.0.000'",
+        };
       });
 
     isUsingEmpty.mockClear().mockImplementation(() => true);

diff --git a/src/components/editCqlLibrary/EditCqlLibrary.tsx b/src/components/editCqlLibrary/EditCqlLibrary.tsx
@@ -242,28 +242,30 @@ const EditCqlLibrary = () => {
         resetForm();
         let primaryMessage = "CQL updated successfully";
         const secondaryMessages = [];
-        if (isUsingEmpty(updatedContent.cql)) {
-          secondaryMessages.push(
-            "Missing a using statement. Please add in a valid model and version."
-          );
-        }
-        if (updatedContent.isLibraryStatementChanged) {
-          secondaryMessages.push(
-            "Library statement was incorrect. MADiE has overwritten it."
-          );
-        }
-        if (updatedContent.isUsingStatementChanged) {
-          secondaryMessages.push(
-            "Using statement was incorrect. MADiE has overwritten it."
-          );
-        }
-        if (updatedContent.isValueSetChanged) {
-          secondaryMessages.push(
-            "MADiE does not currently support use of value set version directly in library at this time. Your value set versions have been removed. Please use the relevant manifest for value set expansion for testing."
-          );
-        }
-        if (secondaryMessages.length > 0) {
-          primaryMessage += " but the following issues were found";
+        if (updatedContent.cql) {
+          if (isUsingEmpty(updatedContent.cql)) {
+            secondaryMessages.push(
+              "Missing a using statement. Please add in a valid model and version."
+            );
+          }
+          if (updatedContent.isLibraryStatementChanged) {
+            secondaryMessages.push(
+              "Library statement was incorrect. MADiE has overwritten it."
+            );
+          }
+          if (updatedContent.isUsingStatementChanged) {
+            secondaryMessages.push(
+              "Using statement was incorrect. MADiE has overwritten it."
+            );
+          }
+          if (updatedContent.isValueSetChanged) {
+            secondaryMessages.push(
+              "MADiE does not currently support use of value set version directly in library at this time. Your value set versions have been removed. Please use the relevant manifest for value set expansion for testing."
+            );
+          }
+          if (secondaryMessages.length > 0) {
+            primaryMessage += " but the following issues were found";
+          }
         }
         setSuccess({
           status: "success",