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

Add a 'Tests' job that depends on the full unit test matrix so branch
protection can require a single check instead of every workspace/Node/OS
combination. The 36 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 | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

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