chore: run tests on multiple java versions (#164)

* chore: run tests on multiple java versions

* fix typo

* Use a higher release version for basic-math

* Make assertions stronger

* Set source and target to 17

* Write logical assert statements

* Remove MacOS on CI until https://github.com/orgs/community/discussions/111013 is resolved

* Does order matter?

* Use native maven instead of maven invoker

* Add back macOS

* Let's look at PATH

* Revert

* Remove macOS and Windows for now

.github/workflows/tests.yml

@@ -10,14 +10,15 @@ jobs:
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
-        os: [ ubuntu-latest, macos-latest, windows-latest ]
+        java-version: [ 17, 21 ]
+        os: [ ubuntu-latest]
     steps:
       - uses: actions/checkout@v4
-      - name: Setup JDK 17
+      - name: Setup JDK${{ matrix.java-version }}
         uses: actions/setup-java@v4
         with:
-          distribution: 'oracle'
-          java-version: '17'
+          distribution: 'temurin'
+          java-version: ${{ matrix.java-version }}
       - name: Build project
         # We need to build the agent first, because the tests depend on it
         run: mvn clean install -DskipTests

classfile-fingerprint/src/test/java/io/github/algomaster99/terminator/index/RuntimeIndexerTest.java

@@ -5,6 +5,7 @@
 import java.io.IOException;
 import java.nio.file.Files;
 import java.nio.file.Path;
+import java.util.List;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.io.TempDir;
 
@@ -19,21 +20,25 @@ void runtimeIndexShouldBeDeterministic(@TempDir Path tempDir) throws IOException
         // act
         String[] jdk_argsFirst = {"jdk", "-o", indexFile.toString()};
         Index.main(jdk_argsFirst);
+        List<String> jdkIndex = Files.readAllLines(indexFile);
+
         String[] runtime_argsFirst = {
             "runtime", "-p", project.toString(), "-i", indexFile.toString(), "-mj", "basic-math", "--cleanup"
         };
         Index.main(runtime_argsFirst);
-        String contentFirst = Files.readString(indexFile);
+        List<String> first = Files.readAllLines(indexFile);
 
         String[] jdk_argsSecond = {"jdk", "-o", indexFile.toString()};
         Index.main(jdk_argsSecond);
         String[] runtime_argsSecond = {
             "runtime", "-p", project.toString(), "-i", indexFile.toString(), "-mj", "basic-math", "--cleanup"
         };
         Index.main(runtime_argsSecond);
-        String contentSecond = Files.readString(indexFile);
+        List<String> second = Files.readAllLines(indexFile);
 
         // assert
-        assertThat(contentFirst).isEqualTo(contentSecond);
+        // This ensures that the project compiles
+        assertThat(first).size().isGreaterThan(jdkIndex.size());
+        assertThat(first).isEqualTo(second);
     }
 }

classfile-fingerprint/src/test/resources/runtime-index/basic-math/pom.xml

@@ -8,6 +8,11 @@
     <artifactId>basic-math</artifactId>
     <version>1.0-SNAPSHOT</version>
 
+    <properties>
+        <maven.compiler.source>17</maven.compiler.source>
+        <maven.compiler.target>17</maven.compiler.target>
+    </properties>
+
     <build>
         <plugins>
             <plugin>