GitHub Actions to use Verifier for OS/JVM test matrix

benalexau · benalexau · commit 9f7435266877 · 2023-02-04T11:15:44.000+11:00
The Verifier is designed to test the accuracy of LMDB storage and is
therefore a useful test to perform on each target platform. It is
noted that other GitHub Actions execute a full Maven verification.
diff --git a/.github/workflows/maven.yml b/.github/workflows/maven.yml
@@ -8,7 +8,7 @@ on:
 
 jobs:
   build:
-    name: Java 11 Build and Verify
+    name: Latest Java LTS Build and Verify
     runs-on: ubuntu-latest
 
     steps:
@@ -48,8 +48,8 @@ jobs:
           java-version: ${{ matrix.java }}
           cache: maven
 
-      - name: Test with Maven
-        run: mvn -B test
+      - name: Execute verifier
+        run: mvn -B test -Dtest=VerifierTest -DverificationSeconds=10
 
       - name: Upload Surefire reports on test failure
         uses: actions/upload-artifact@v3
diff --git a/pom.xml b/pom.xml
@@ -229,8 +229,8 @@
             <artifactId>maven-surefire-plugin</artifactId>
             <configuration>
               <argLine>--add-opens java.base/java.nio=ALL-UNNAMED --add-opens java.base/sun.nio.ch=ALL-UNNAMED</argLine>
-             <forkCount>1</forkCount>
-             <reuseForks>false</reuseForks>
+              <forkCount>1</forkCount>
+              <reuseForks>false</reuseForks>
             </configuration>
           </plugin>
         </plugins>
diff --git a/src/test/java/org/lmdbjava/VerifierTest.java b/src/test/java/org/lmdbjava/VerifierTest.java
@@ -52,7 +52,8 @@ public void verification() throws IOException {
         .setMapSize(MEBIBYTES.toBytes(10))
         .open(path, MDB_NOSUBDIR)) {
       final Verifier v = new Verifier(env);
-      assertThat(v.runFor(2, TimeUnit.SECONDS), greaterThan(1L));
+      final int seconds = Integer.getInteger("verificationSeconds", 2);
+      assertThat(v.runFor(seconds, TimeUnit.SECONDS), greaterThan(1L));
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,8 @@ public void verification() throws IOException {`
`52`	`52`	`.setMapSize(MEBIBYTES.toBytes(10))`
`53`	`53`	`.open(path, MDB_NOSUBDIR)) {`
`54`	`54`	`final Verifier v = new Verifier(env);`
`55`		`- assertThat(v.runFor(2, TimeUnit.SECONDS), greaterThan(1L));`
	`55`	`+ final int seconds = Integer.getInteger("verificationSeconds", 2);`
	`56`	`+ assertThat(v.runFor(seconds, TimeUnit.SECONDS), greaterThan(1L));`
`56`	`57`	`}`
`57`	`58`	`}`
`58`	`59`