fix: e2e test status reporting incorrectly (#2045)

Fixes #2044

Infra PR deephaven/infra#301

You can see the e2e-results job failed on my 2nd commit to this PR. It
should fail if any of the matrix jobs fail or are cancelled.

Author: mattrunyon

.github/workflows/e2e.yml

@@ -121,3 +121,12 @@ jobs:
           name: playwright-report
           path: playwright-report
           retention-days: 30
+
+  e2e-results:
+    if: ${{ always() }}
+    runs-on: ubuntu-22.04
+    needs: [e2e-tests]
+    steps:
+      - name: Fail if any tests failed or cancelled
+        run: exit 1
+        if: ${{ contains(needs.*.result, 'failure') || contains(needs.*.result, 'cancelled') }}