From f63ef37d5810e8a80a809028e8bc920af37cc11e Mon Sep 17 00:00:00 2001
From: Merlin Beutlberger <m.beutlberger@sap.com>
Date: Wed, 1 Jul 2026 13:43:31 +0200
Subject: [PATCH] ci: Add aggregator job for test matrix

Add a 'Tests' job that depends on both matrix jobs (test and
test-ui5cli-matrix) so branch protection can require a single check
instead of every matrix combination. The matrix rows remain visible in
the PR checks list; only the required-check surface collapses.

'if: always()' ensures the aggregator still runs when matrix cells fail,
so a failure surfaces as 'failed' rather than 'skipped'.
---
 .github/workflows/test.yml | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index 4fce2d48..b2392a21 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -64,3 +64,20 @@ jobs:
 
     - name: Run integration tests
       run: npm run test:integration
+
+  # Aggregator so branch protection can require a single "Tests" check
+  # instead of every matrix combination.
+  tests:
+    name: Tests
+    needs: [test, test-ui5cli-matrix]
+    if: always()
+    runs-on: ubuntu-24.04
+    steps:
+    - name: Check matrix result
+      run: |
+        if [ "${{ needs.test.result }}" != "success" ] || \
+           [ "${{ needs.test-ui5cli-matrix.result }}" != "success" ]; then
+          echo "test: ${{ needs.test.result }}"
+          echo "test-ui5cli-matrix: ${{ needs.test-ui5cli-matrix.result }}"
+          exit 1
+        fi