See if docker uploads googletest

sethrj · Sep 20, 2024 · 66a0b84 · 66a0b84
1 parent fde1767
commit 66a0b84
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/.github/workflows/build-docker.yml b/.github/workflows/build-docker.yml
@@ -44,7 +44,6 @@ jobs:
                 matrix.special && '-' || '',
                 matrix.special)}}
       ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION: true # TODO: DELETEME, see #1305
-      GTEST_OUTPUT: "xml:${{github.workspace}}/test-output/google/"
     runs-on: ubuntu-latest
     container:
       image: >-
@@ -89,10 +88,11 @@ jobs:
       - name: Test Celeritas
         id: test
         working-directory: build
+        env:
+          GTEST_OUTPUT: "xml:${{github.workspace}}/test-output/google/"
         run: |
           ctest --parallel $(nproc) --timeout 180 --output-on-failure \
-            --test-output-size-passed=65536 --test-output-size-failed=1048576 \
-            --output-junit "${{github.workspace}}/test-output/ctest/all.xml"
+            --test-output-size-passed=65536 --test-output-size-failed=1048576
       - name: Install Celeritas
         working-directory: build
         run: |
@@ -119,9 +119,12 @@ jobs:
           fi
           ./scripts/ci/test-examples.sh
       - name: Upload test results
+        if: >-
+          ${{
+            (steps.test.outcome == 'success' || steps.test.outcome == 'failure')
+            && (env.CMAKE_PRESET != 'ndebug-vecgeom')
+          }}
         uses: actions/upload-artifact@v3
-        # TODO junit export doesn't work on the older version of ctest on the image
-        if: false
         with:
           name: test-results-${{matrix.image}}-${{env.CMAKE_PRESET}}
           path: "test-output/**/*.xml"