From e28066a7dc9a8b3053369ca62c455738860c0ecb Mon Sep 17 00:00:00 2001
From: Richard Steinmetz <richard@steinmetz.cloud>
Date: Fri, 22 Jul 2022 10:59:32 +0200
Subject: [PATCH] Add summary jobs to workflows

Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
---
 .github/workflows/lint.yml            | 22 ++++++++++++++++++++++
 .github/workflows/static-analysis.yml | 13 +++++++++++++
 .github/workflows/test.yml            | 19 +++++++++++++++++++
 3 files changed, 54 insertions(+)

diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
index 514b525343..c6563988ad 100644
--- a/.github/workflows/lint.yml
+++ b/.github/workflows/lint.yml
@@ -74,3 +74,25 @@ jobs:
       run: npm run lint
     env:
       CI: true
+
+  summary:
+    runs-on: ubuntu-latest
+    needs:
+      - xml-linters
+      - php-linters
+      - php-cs-fixer
+      - node-linters
+
+    if: always()
+
+    name: lint-summary
+
+    steps:
+      - name: XML linter status
+        run: if ${{ needs.xml-linters.result != 'success' && needs.xml-linters.result != 'skipped' }}; then exit 1; fi
+      - name: PHP linter status
+        run: if ${{ needs.php-linters.result != 'success' && needs.php-linters.result != 'skipped' }}; then exit 1; fi
+      - name: PHP cs fixer status
+        run: if ${{ needs.php-cs-fixer.result != 'success' && needs.php-cs-fixer.result != 'skipped' }}; then exit 1; fi
+      - name: Node linter status
+        run: if ${{ needs.node-linters.result != 'success' && needs.node-linters.result != 'skipped' }}; then exit 1; fi
diff --git a/.github/workflows/static-analysis.yml b/.github/workflows/static-analysis.yml
index 98f95e2749..7e69dff436 100644
--- a/.github/workflows/static-analysis.yml
+++ b/.github/workflows/static-analysis.yml
@@ -35,3 +35,16 @@ jobs:
             run: composer require --dev christophwurst/nextcloud:${{ matrix.ocp-version }} --ignore-platform-reqs
           - name: Run coding standards check
             run: composer run psalm
+
+  summary:
+    runs-on: ubuntu-latest
+    needs:
+      - static-psalm-analysis
+
+    if: always()
+
+    name: static-analysis-summary
+
+    steps:
+      - name: Static analysis status
+        run: if ${{ needs.static-psalm-analysis.result != 'success' && needs.static-psalm-analysis.result != 'skipped' }}; then exit 1; fi
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index ccefc8671c..aae6d71d14 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -214,3 +214,22 @@ jobs:
             run: npm run test:unit
       env:
           CI: true
+
+  summary:
+    runs-on: ubuntu-latest
+    needs:
+      - unit-tests
+      - integration-tests
+      - frontend-unit-test
+
+    if: always()
+
+    name: test-summary
+
+    steps:
+      - name: Unit test status
+        run: if ${{ needs.unit-tests.result != 'success' && needs.unit-tests.result != 'skipped' }}; then exit 1; fi
+      - name: Integration test status
+        run: if ${{ needs.integration-tests.result != 'success' && needs.integration-tests.result != 'skipped' }}; then exit 1; fi
+      - name: Frontend unit test status
+        run: if ${{ needs.frontend-unit-test.result != 'success' && needs.frontend-unit-test.result != 'skipped' }}; then exit 1; fi