MAT-7652 do not replace FHIR model if thats the only using model for QICore measure/library

src/AceEditor/madie-ace-editor.test.tsx

@@ -340,7 +340,7 @@ describe("synching the cql", () => {
     expect(updatedContent.isUsingStatementChanged).toEqual(true);
   });
 
-  test("Not to replace the using FHIR statement for QICore measure if it is the only using statement", async () => {
+  test("Not to replace the using FHIR statement for QICore measure if it is the only using statement with correct version", async () => {
     const expectValue = "using FHIR version '4.0.1'";
     const updatedContent = await updateEditorContent(
       "using FHIR version '4.0.1'",
@@ -355,6 +355,21 @@ describe("synching the cql", () => {
     expect(updatedContent.cql).toEqual(expectValue);
   });
 
+  test("Correct the using FHIR model version for QICore measure if it is the only using statement but incorrect version", async () => {
+    const expectValue = "using FHIR version '4.0.1'";
+    const updatedContent = await updateEditorContent(
+      "using FHIR version '4.1.1'", //incorrect FHIR version
+      "",
+      "Test",
+      "",
+      "0.0.000",
+      "QI-Core",
+      "4.1.1",
+      "measureEditor"
+    );
+    expect(updatedContent.cql).toEqual(expectValue);
+  });
+
   test("generated Cql has updated cql library name", async () => {
     const expectValue = "library Testing version '0.0.000'";
     const updatedContent = await updateEditorContent(

src/AceEditor/madie-ace-editor.tsx

@@ -86,21 +86,22 @@
   let isCqlUpdated = false;
   if (usingStatements?.length === 1) {
     const { name, version, start } = usingStatements[0];
-    if (
-      measureModel !== name ||
-      modelVersion !== version.replace(/["']/g, "")
-    ) {
-      // we want to keep FHIR if that's the only using model present for QICore.
+    const cleanedVersion = version.replace(/["']/g, "");
+    if (measureModel !== name || modelVersion !== cleanedVersion) {
+      // keep FHIR if that's the only using model present for QICore but update version if it was incorrect.
       if (measureModel === "QICore" && name === "FHIR") {
-        parsedEditorCqlCopy.cqlArrayToBeFiltered[
-          start.line - 1
-        ] = `using FHIR version '4.0.1'`;
+        if (cleanedVersion !== "4.0.1") {
+          parsedEditorCqlCopy.cqlArrayToBeFiltered[
+            start.line - 1
+          ] = `using FHIR version '4.0.1'`;
+          isCqlUpdated = true;
+        }
       } else {
         parsedEditorCqlCopy.cqlArrayToBeFiltered[
           start.line - 1
         ] = `using ${measureModel} version '${modelVersion}'`;
+        isCqlUpdated = true;
       }
-      isCqlUpdated = true;
     }
   } else if (usingStatements?.length > 1) {
     // to track if the usings statement was verified or not
@@ -477,7 +478,7 @@
         console.warn("Editor is not set! Cannot set annotations!", editor);
       }
     }
   }, [parserAnnotations, inboundAnnotations, editor]);
   const toggleSearchBox = () => {
     // given the searchBox has not been instantiated we execCommand which also triggers show
     //@ts-ignore
@@ -519,7 +520,7 @@
         toggleSearchBox as EventListener
       );
     };
   }, [editor, parseErrorMarkers, inboundErrorMarkers]);
 
   useEffect(() => {
     const cqlMode = new CqlMode();
@@ -531,14 +532,14 @@
         debouncedParse.cancel();
       }
     };
   }, [debouncedParse]);
 
   useEffect(() => {
     if (!_.isNil(value) && editor && validationsEnabled) {
       setParsing(true);
       debouncedParse(value, editor);
     }
   }, [value, editor, debouncedParse]);
 
   useEffect(() => {
     // This is to set aria-label on textarea for accessibility

src/api/useFhirElmTranslationServiceApi.test.ts

@@ -75,7 +75,7 @@ describe("Test FhirElmTranslationServiceApi", () => {
       new FhirElmTranslationServiceApi(null, mockGetAccessToken);
 
     try {
-      await fhirElmTranslationServiceApi.translateCqlToElm("test", true);
+      await fhirElmTranslationServiceApi.translateCqlToElm("mock cql", true);
     } catch (error) {
       expect(error).not.toBeNull();
       expect(error.message).toBe(
@@ -104,7 +104,7 @@ describe("Test FhirElmTranslationServiceApi", () => {
       new FhirElmTranslationServiceApi("test", mockGetAccessToken);
 
     try {
-      await fhirElmTranslationServiceApi.translateCqlToElm("test", false);
+      await fhirElmTranslationServiceApi.translateCqlToElm("mock cql", false);
     } catch (error) {
       expect(error).not.toBeNull();
       expect(error.message).toBe("Request failed with status code 404");