Ensure tests-completed job fails if required jobs either fail or are …

…cancelled (hashicorp#19948)

Otherwise, the tests-completed job should succeed, including when the test-ui
job is skipped.

.github/workflows/ci.yml

@@ -283,10 +283,8 @@ jobs:
     - setup
     - test-go
     - test-ui
-    if: |
-      always() &&
-      !contains(needs.*.result, 'failure') &&
-      !contains(needs.*.result, 'cancelled')
+    if: always()
     runs-on: ${{ fromJSON(needs.setup.outputs.compute-tiny) }}
     steps:
-    - run: echo "All Go tests successfully passed"
+    - run: |
+        tr -d '\n' <<< '${{ toJSON(needs.*.result) }}' | grep -q -v -E '(failure|cancelled)'