fixup

Signed-off-by: Frederic BIDON <fredbi@yahoo.com>

Author: fredbi

.github/workflows/go-test.yml

@@ -108,7 +108,7 @@ jobs:
     # just on that job rather than each matrix job independently.
     needs: [ unit_tests ]    # <- requires all unit_tests jobs to be successful.
     env:
-      test_result: '${{ join(needs.unit_tests.outputs.*) }}'  # <- this is a comma-separated list of strings
+      test_result: '${{ join(needs.unit_tests.outputs.*) }}'  # <- this is a comma-separated list of quoted strings
     runs-on: ubuntu-latest
     steps:
       - name: Tests complete
@@ -128,10 +128,10 @@ jobs:
     name: Some tests have failed
     # description: |
     # This job is only here to report a summary of failed tests in the github actions UI.
-    needs: [ unit_tests ]     # <- requires all unit_tests jobs to be completed, but not necessarily successfully.
-    if: ${{ needs.unit_tests.result == 'failure' }}
+    needs: [ unit_tests ]     # <- requires all unit_tests jobs to be completed, but not all where successfull.
+    if: ${{ failure() }}
     env:
-      test_result: '${{ needs.unit_tests.outputs.test_result }}'  # <- this is a JSON map
+      test_result: '${{ join(needs.unit_tests.outputs.*) }}'
     runs-on: ubuntu-latest
     steps:
       - name: Tests failed