More assertions

lukebemishprojects · Jul 15, 2024 · f3c0ccd · f3c0ccd
1 parent 4564aa7
commit f3c0ccd
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 6 deletions.
diff --git a/.github/workflows/build_pr.yml b/.github/workflows/build_pr.yml
@@ -89,7 +89,7 @@
                 },
                 {
                     "name": "Run with Docker",
-                    "run": "docker run --workdir \"${GITHUB_WORKSPACE}\" -v \"${GITHUB_WORKSPACE}:${GITHUB_WORKSPACE}:z\" --rm --platform linux/${{ matrix.docker_arch }} -t ${{ matrix.docker }} /bin/bash -c \"mkdir ../working; cp -r ./ ../working/; cd ../working; for i in {1..1} ; do (java @args-${{ matrix.os }}.txt) ; done ; return=$?; cp -r ./ ${GITHUB_WORKSPACE}/; cd ${GITHUB_WORKSPACE}; exit $return\"",
+                    "run": "docker run --workdir \"${GITHUB_WORKSPACE}\" -v \"${GITHUB_WORKSPACE}:${GITHUB_WORKSPACE}:z\" --rm --platform linux/${{ matrix.docker_arch }} -t ${{ matrix.docker }} /bin/bash -c 'mkdir ../working; cp -r ./ ../working/; cd ../working; java @args-${{ matrix.os }}.txt; return=$?; cp -r ./ ${GITHUB_WORKSPACE}/; cd ${GITHUB_WORKSPACE}; exit $return'",
                     "if": "${{ matrix.docker }}"
                 },
                 {

diff --git a/.github/workflows/snapshot.yml b/.github/workflows/snapshot.yml
@@ -89,7 +89,7 @@
                 },
                 {
                     "name": "Run with Docker",
-                    "run": "docker run --workdir \"${GITHUB_WORKSPACE}\" -v \"${GITHUB_WORKSPACE}:${GITHUB_WORKSPACE}:z\" --rm --platform linux/${{ matrix.docker_arch }} -t ${{ matrix.docker }} /bin/bash -c \"mkdir ../working; cp -r ./ ../working/; cd ../working; for i in {1..1} ; do (java @args-${{ matrix.os }}.txt) ; done ; return=$?; cp -r ./ ${GITHUB_WORKSPACE}/; cd ${GITHUB_WORKSPACE}; exit $return\"",
+                    "run": "docker run --workdir \"${GITHUB_WORKSPACE}\" -v \"${GITHUB_WORKSPACE}:${GITHUB_WORKSPACE}:z\" --rm --platform linux/${{ matrix.docker_arch }} -t ${{ matrix.docker }} /bin/bash -c 'mkdir ../working; cp -r ./ ../working/; cd ../working; java @args-${{ matrix.os }}.txt; return=$?; cp -r ./ ${GITHUB_WORKSPACE}/; cd ${GITHUB_WORKSPACE}; exit $return'",
                     "if": "${{ matrix.docker }}"
                 },
                 {

diff --git a/build.gradle b/build.gradle
@@ -118,10 +118,10 @@ managedVersioning {
                 step {
                     name.set 'Run with Docker'
                     parameters.put('if', '${{ matrix.docker }}')
-                    run.set 'docker run --workdir "${GITHUB_WORKSPACE}" -v "${GITHUB_WORKSPACE}:${GITHUB_WORKSPACE}:z" --rm --platform linux/${{ matrix.docker_arch }} -t ${{ matrix.docker }} /bin/bash -c "' +
+                    run.set 'docker run --workdir "${GITHUB_WORKSPACE}" -v "${GITHUB_WORKSPACE}:${GITHUB_WORKSPACE}:z" --rm --platform linux/${{ matrix.docker_arch }} -t ${{ matrix.docker }} /bin/bash -c \'' +
                             'mkdir ../working; cp -r ./ ../working/; cd ../working; ' +
-                            'for i in {1..1} ; do (java @args-${{ matrix.os }}.txt) ; done ; return=$?; ' +
-                            'cp -r ./ ${GITHUB_WORKSPACE}/; cd ${GITHUB_WORKSPACE}; exit $return"'
+                            'java @args-${{ matrix.os }}.txt; return=$?; ' +
+                            'cp -r ./ ${GITHUB_WORKSPACE}/; cd ${GITHUB_WORKSPACE}; exit $return\''
                 }
                 step {
                     name.set 'Run'

diff --git a/natives/src/main/java/dev/lukebemish/opensesame/natives/NativeImplementations.java b/natives/src/main/java/dev/lukebemish/opensesame/natives/NativeImplementations.java
@@ -83,8 +83,9 @@ static void setup() throws IOException {
             throw new IOException("Could not find native library in opensesame jar, at path "+nativePath);
         }
         try (var lib = resource.openStream()) {
-            Files.copy(lib, path);
+            Files.copy(lib, path.toAbsolutePath());
         }
+        assert Files.exists(path);
         System.load(path.toAbsolutePath().toString());
         SETUP = true;
     }