From fdddb641f93a3bdb9143285ffc690caf6f9292dd Mon Sep 17 00:00:00 2001
From: Federica Agostini <federica.agostini@cnaf.infn.it>
Date: Tue, 19 Nov 2024 18:14:00 +0100
Subject: [PATCH] Fail the workflow in case of tests failure

Always collect test reports
---
 .github/workflows/run-testsuite.yml | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/run-testsuite.yml b/.github/workflows/run-testsuite.yml
index fd97bcaa..bad5c3ad 100644
--- a/.github/workflows/run-testsuite.yml
+++ b/.github/workflows/run-testsuite.yml
@@ -27,17 +27,18 @@ jobs:
         run: docker compose up --build -d storage-setup webdav nginx
 
       - name: Run testsuite
+        if: ${{ always() }}
         working-directory: compose
         run: |
           docker compose up -d ts
           docker compose exec -T ts bash -c '/scripts/ci-run-testsuite.sh'
-        continue-on-error: true
 
       - name: Create artifacts directory
         if: ${{ always() }}
         run: mkdir -p ${ARTIFACTS}
 
       - name: Collect test reports
+        if: ${{ always() }}
         run: docker cp storm-webdav-ts-1:/home/test/robot/reports ${ARTIFACTS}
 
       - name: Collect service log