Report summary on success, not just failure

There are cases where the summary may be useful even if errors are not detected, so this ties only the exit condition to success/failure, not the whole summary step.

Author: anonymous-akorn

.github/workflows/cpp-packaging.yml

@@ -691,8 +691,10 @@ jobs:
       run: |
         python scripts/gha/test_lab.py --android_model ${{ env.android_device }} --android_api ${{ env.android_api }} --ios_model ${{ env.ios_device }} --ios_version ${{ env.ios_version }} --testapp_dir testapps --code_platform cpp --key_file scripts/gha-encrypted/gcs_key_file.json
     - name: Summarize build and test results
-      if: failure()
+      if: ${{ !cancelled() }}
       shell: bash
       run: |
         cat testapps/summary.log
-        exit 1
+        if [[ "${{ job.status }}" != "success" ]]; then
+          exit 1
+        fi

.github/workflows/integration_tests.yml

@@ -160,8 +160,10 @@ jobs:
         run: |
           python scripts/gha/test_lab.py --android_model ${{ github.event.inputs.android_device }} --android_api ${{ github.event.inputs.android_api }} --ios_model ${{ github.event.inputs.ios_device }} --ios_version ${{ github.event.inputs.ios_version }} --testapp_dir testapps --code_platform cpp --key_file scripts/gha-encrypted/gcs_key_file.json
       - name: Summarize build and test results
-        if: failure()
+        if: ${{ !cancelled() }}
         shell: bash
         run: |
           cat testapps/summary.log
-          exit 1
+          if [[ "${{ job.status }}" != "success" ]]; then
+            exit 1
+          fi