Use The Hyve fork of the caciocavello project (#42)

* Use The Hyve fork of the caciocavello project (Swing virtual graphics environment for testing) until the parent project has been fixed for JDK 18+

* Use updated cacio-tta version, should run fine when headless

* For developemnt, JDK versions 17-21 are supported

CONTRIBUTING.md

@@ -2,7 +2,7 @@
 
 White Rabbit and Rabbit in a Hat are structured as a Maven package. Contributions are welcome.
 
-While the software in the project can be executed with Java 8 (1.8), for development Java 17 is needed.
+While the software in the project can be executed with Java 8 (1.8), for development Java 17 (or higher, currently tested wth 21) is needed.
 This has to do with test and verification dependencies that are not available in a version compatible with Java 8 .
 
 Please note that when using an IDE for development, source and target release must still be Java 8 (1.8) . This is enforced
@@ -68,4 +68,4 @@ and do not relate in any way to any production environment.
 The schema should not contain any tables when the test is started.
 
 It is possible to skip the Snowflake tests without failing the build by passing
-`-Dohdsi.org.whiterabbit.skip_snowflake_tests=1` to maven.
+`-Dohdsi.org.whiterabbit.skip_snowflake_tests=1` to maven.

README.md

@@ -44,7 +44,7 @@ Requires Java 1.8 or higher for running, and read access to the database to be s
 
 Dependencies
 ============
-For the distributable packages, the only requirement is Java 8. For building the package, Java 17 and Maven are needed.
+For the distributable packages, the only requirement is Java 8. For building the package, Java 17+ and Maven are needed.
 
 Getting Started
 ===============

pom.xml

@@ -70,6 +70,10 @@
             <layout>default</layout>
             <url>file:///${project.basedir}/../lib</url>
         </repository>
+        <repository>
+            <id>jitpack.io</id>
+            <url>https://jitpack.io</url>
+        </repository>
     </repositories>
 
     <pluginRepositories>
@@ -227,10 +231,10 @@
             </plugin>
             <plugin>
                 <!--
-                  While the source and target Java version are still 1.8, Java 17 is required to be able to use
+                  While the source and target Java version are still 1.8, Java 17+ is required to be able to use
                   recent versions of test and verification tools. This plugin enforces the required Java version,
                   and gives a cleaner error message than failing on a dependency that is not compatible with an
-                  older version of Java
+                  older version of Java.
                   -->
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-enforcer-plugin</artifactId>
@@ -244,7 +248,7 @@
                         <configuration>
                             <rules>
                                 <requireJavaVersion>
-                                    <version>17</version>
+                                    <version>[17,22)</version>
                                 </requireJavaVersion>
                             </rules>
                         </configuration>
@@ -305,6 +309,9 @@
                         it is published on Maven repos -->
                         <exclude>com.microsoft.sqlserver:sqljdbc4</exclude>
 
+                        <!-- temporary exclusion for the patched caciocavallo dependency -->
+                        <exclude>com.github.thehyve:caciocavallo</exclude>
+
                         <!-- PLEASE NOTE: teradata and googlebigquery are provided within the project, they
                              cannot be found on a maven repo; no license info available; they should ideally
                              not be part of the project, and should be removed here -->

whiterabbit/pom.xml

@@ -48,6 +48,7 @@
                     </systemPropertyVariables>
                     <argLine>
                         -Doracle.jdbc.timezoneAsRegion=false
+                        -XX:+EnableDynamicAgentLoading
                         <!-- Line(s) below are needed to keep TestSourceDataScan.updateEnv() working in java 17+
                              This is considered an acceptable hack since it only affects test code -->
                         --add-exports=java.base/java.util=ALL-UNNAMED
@@ -80,6 +81,7 @@
                     <systemPropertyVariables>
                         <propertyName>propertyValue</propertyName>
                         <projectBuildDirectory>${project.build.directory}</projectBuildDirectory>
+                        <java.awt.headless>false</java.awt.headless>
                     </systemPropertyVariables>
                     <forkCount>1</forkCount>
                     <reuseForks>false</reuseForks>
@@ -88,6 +90,7 @@
                     </includes>
                     <skipTests>${skipIntegrationTests}</skipTests>
                     <argLine>
+                        -XX:+EnableDynamicAgentLoading
                     <!-- experimental workaround for Snowflake JDBC issue,
                          see https://community.snowflake.com/s/article/JDBC-Driver-Compatibility-Issue-With-JDK-16-and-Later -->
                     --add-opens java.base/java.nio=ALL-UNNAMED
@@ -267,11 +270,6 @@
             <type>test-jar</type>
             <scope>test</scope>
         </dependency>
-<!--        <dependency>-->
-<!--            <groupId>org.apache.httpcomponents</groupId>-->
-<!--            <artifactId>httpclient</artifactId>-->
-<!--            <version>4.5.14</version>-->
-<!--        </dependency>-->
         <dependency>
             <groupId>one.util</groupId>
             <artifactId>streamex</artifactId>
@@ -284,10 +282,19 @@
             <version>5.2.5</version>
             <scope>compile</scope>
         </dependency>
-        <dependency>
+        <!-- cacio-tta cannot be used with JDK 18+. Until that is solved in the parent project, use a fixed
+             fork hosted at The Hyve -->
+        <!-- dependency>
             <groupId>com.github.caciocavallosilano</groupId>
             <artifactId>cacio-tta</artifactId>
-            <version>1.17.3</version>
+            <version>1.17.4-SNAPSHOT</version>
+            <scope>test</scope>
+        </dependency -->
+        <!-- fixed fork, enabled through the jitpack repository -->
+        <dependency>
+            <groupId>com.github.thehyve</groupId>
+            <artifactId>caciocavallo</artifactId>
+            <version>1.17.7</version>
             <scope>test</scope>
         </dependency>
         <!-- https://mvnrepository.com/artifact/org.assertj/assertj-swing-junit -->