Merge pull request #313 from scratchfoundation/make-test-failures-visible

Report test results more visibly

Author: cwillisf

.github/actions/test-package/action.yml

@@ -19,9 +19,19 @@ runs:
     - name: Test
       working-directory: ./packages/${{ inputs.package_name }}
       shell: bash
+      env:
+        TAP_REPORTER: "junit"
+        TAP_REPORTER_FILE: "test-results/junit.xml"
       run: |
         mkdir -p test-results
-        npm run test | tee -a ./test-results/${{ inputs.package_name }}-test-results.txt
+        npm run test
+    - name: Publish test reports
+      if: ${{ !cancelled() }}
+      uses: EnricoMi/publish-unit-test-result-action@v2
+      with:
+        files: "./packages/${{ inputs.package_name }}/test-results/**/*"
+        check_name: "Test report for ${{ inputs.package_name }}"
+        test_file_prefix: "+packages/${{ inputs.package_name }}/"
     - name: Store Test Results
       if: ${{ !cancelled() }}
       uses: actions/upload-artifact@65c4c4a1ddee5b72f698fdd19549f0f0fb45cf08 # v4

.github/workflows/ci.yml

@@ -91,7 +91,7 @@ jobs:
     name: Results
     runs-on: ubuntu-latest
     needs: test
-    if: ${{ always() }}
+    if: ${{ !cancelled() }}
     steps:
       - run: |
           case "${{ needs.test.result }}" in

packages/scratch-gui/.gitignore

@@ -8,6 +8,7 @@ npm-*
 # Testing
 /.nyc_output
 /coverage
+/test-results
 
 # Build
 /build
@@ -26,6 +27,3 @@ npm-*
 
 # for act
 .secrets
-
-# Generated test results
-/test-results

packages/scratch-render/.gitignore

@@ -9,6 +9,7 @@ npm-*
 /.nyc_output
 /.tap
 /coverage
+/test-results
 
 # IDEA
 /.idea

packages/scratch-svg-renderer/.gitignore

@@ -11,6 +11,7 @@ npm-*
 
 # Test
 /.tap
+/test-results
 
 # Editors
 /#*

packages/scratch-vm/.gitignore

@@ -9,6 +9,7 @@ npm-*
 /.nyc_output
 /.tap
 /coverage
+/test-results
 
 # Editor
 /.idea