Fix test case

cqframework · Nov 5, 2024 · 8dde6d4 · 8dde6d4
1 parent 905e7e9
commit 8dde6d4
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/Src/java/cql-to-elm/src/test/java/org/cqframework/cql/cql2elm/SemanticTests.java b/Src/java/cql-to-elm/src/test/java/org/cqframework/cql/cql2elm/SemanticTests.java
@@ -382,10 +382,11 @@ void issue435() throws IOException {
 
     @Test
     void issue587() throws IOException {
-        CqlTranslator translator = TestUtils.runSemanticTest("Issue587.cql", 1);
-        // This doesn't resolve correctly, collapse null should work, but it's related to this issue:
-        // [#435](https://github.com/cqframework/clinical_quality_language/issues/435)
-        // So keeping as a verification of current behavior here, will address as part of vNext
+        CqlTranslator translator = TestUtils.runSemanticTest("Issue587.cql", 0);
+        // Both `collapse null` and `collapse { null }` now translate with no errors.
+        // The old errors were related to [#435](https://github.com/cqframework/clinical_quality_language/issues/435)
+        // and fixed by [#1428](https://github.com/cqframework/clinical_quality_language/pull/1428) and
+        // [#1425](https://github.com/cqframework/clinical_quality_language/pull/1425).
         assertThat(translator.getErrors().size(), equalTo(1));
     }