Fixes for gaps in the ELM visitor (#1308)

* Added test to ensure all elements of a random ELM graph are visited once and only once * Bit of clean-up and renaming for clarity * More cleanup * Architecture tests * Working Random ELM tests and design tests * bug fixes * Add spotless, prep for master merge * Fixed bugs in datarequirements and base visitor * Remove max report age restriction * Fix codecov one more time! * Further clean-up
cqframework · Dec 22, 2023 · cc869ca · cc869ca
1 parent cf7c30a
commit cc869ca
Show file tree

Hide file tree

Showing 15 changed files with 1,150 additions and 1,359 deletions.
diff --git a/Src/java/.vscode/settings.json b/Src/java/.vscode/settings.json
@@ -10,7 +10,10 @@
         "fhirpath",
         "hamcrest",
         "Inferencing",
+        "Instancio",
+        "Objenesis",
         "qicore",
+        "Randomizer",
         "testng",
         "tngtech",
         "trackback"

diff --git a/Src/java/buildSrc/src/main/groovy/cql.java-conventions.gradle b/Src/java/buildSrc/src/main/groovy/cql.java-conventions.gradle
@@ -29,7 +29,7 @@ dependencies {
     testImplementation 'org.testng:testng:7.4.0'
     testImplementation 'org.hamcrest:hamcrest-all:1.3'
     testImplementation 'uk.co.datumedge:hamcrest-json:0.2'
-    testImplementation 'junit:junit:4.12'
+    testImplementation 'junit:junit:4.13.2'
     testImplementation 'org.slf4j:slf4j-simple:1.7.36'
 
     // These are JAXB dependencies excluded because the libraries need to work

diff --git a/...a/elm-fhir/src/main/java/org/cqframework/cql/elm/requirements/ElmRequirementsVisitor.java b/...a/elm-fhir/src/main/java/org/cqframework/cql/elm/requirements/ElmRequirementsVisitor.java
diff --git a/Src/java/elm/build.gradle b/Src/java/elm/build.gradle
@@ -5,6 +5,8 @@ plugins {
 
 dependencies {
     api project(':model')
+    testImplementation 'org.jeasy:easy-random-core:5.0.0'
+    testImplementation 'com.tngtech.archunit:archunit:1.2.1'
 }
 
 generateSources {