feat: améliorer le workflow de test de performance des endpoints

.gitea/workflows/pr-endpoint-check.yml

@@ -8,32 +8,20 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      # ─────────────────────────────────────────────
-      # Récupération du code source
-      # ─────────────────────────────────────────────
       - name: Checkout PR branch
         uses: actions/checkout@v4
 
-      # ─────────────────────────────────────────────
-      # Installation de .NET 10
-      # ─────────────────────────────────────────────
       - name: Setup .NET 10
         uses: actions/setup-dotnet@v4
         with:
           dotnet-version: "10.0.x"
 
-      # ─────────────────────────────────────────────
-      # Restauration des dépendances et compilation
-      # ─────────────────────────────────────────────
       - name: Restore dependencies
         run: dotnet restore Webzine.sln
 
       - name: Build solution
         run: dotnet build Webzine.sln --no-restore --configuration Release
 
-      # ─────────────────────────────────────────────
-      # Exécution des tests unitaires (entités)
-      # ─────────────────────────────────────────────
       - name: Run unit tests
         run: |
           dotnet test Webzine.Entity.Tests/Webzine.Entity.Tests.csproj \
@@ -41,9 +29,6 @@ jobs:
             --configuration Release \
             --logger "console;verbosity=normal"
 
-      # ─────────────────────────────────────────────
-      # Démarrage de l'application web en arrière-plan
-      # ─────────────────────────────────────────────
       - name: Start Webzine application
         run: |
           dotnet run \
@@ -60,20 +45,21 @@ jobs:
           '
           echo "Application prête !"
 
-      # ─────────────────────────────────────────────
-      # Exécution des tests de performance des endpoints
-      # ─────────────────────────────────────────────
       - name: Test endpoint response times
         id: perf_test
         run: |
           chmod +x scripts/test-endpoints.sh
-          bash scripts/test-endpoints.sh http://localhost:5038 1000 || true
+          bash scripts/test-endpoints.sh http://localhost:5038 1000 2>&1 | tee /tmp/webzine_endpoint_output.txt
-          FAIL_COUNT=$(grep -c "^\[FAIL\]\|^\[SLOW\]" /tmp/webzine_endpoint_report.txt || echo 0)
+          EXIT_CODE=${PIPESTATUS[0]}
+          
+          FAIL_COUNT=$(grep -c "^\[FAIL\]\|^\[SLOW\]" /tmp/webzine_endpoint_output.txt || echo 0)
           echo "failed=$FAIL_COUNT" >> "$GITHUB_OUTPUT"
+          echo "exit_code=$EXIT_CODE" >> "$GITHUB_OUTPUT"
+          
+          cat /tmp/webzine_endpoint_output.txt > /tmp/webzine_endpoint_report.txt
+          
+          exit $EXIT_CODE
 
-      # ─────────────────────────────────────────────
-      # Publication du rapport en commentaire de PR
-      # ─────────────────────────────────────────────
       - name: Post performance report as PR comment
         if: always()
         env:
@@ -84,43 +70,64 @@ jobs:
         run: |
           REPORT_CONTENT=$(cat /tmp/webzine_endpoint_report.txt 2>/dev/null || echo "Aucun rapport généré.")
           FAILED_COUNT="${{ steps.perf_test.outputs.failed }}"
-
+          
+          REPORT_CONTENT_ESCAPED=$(echo "$REPORT_CONTENT" | sed 's/`/\\`/g' | sed 's/\\/\\\\/g')
+          
           if [ "${FAILED_COUNT:-0}" -gt 0 ]; then
             HEADER="## ❌ Vérification des performances ÉCHOUÉE"
             INTRO="${FAILED_COUNT} endpoint(s) ont dépassé 1 seconde ou retourné une erreur serveur."
+            STATUS="ÉCHEC"
           else
             HEADER="## ✅ Vérification des performances RÉUSSIE"
             INTRO="Tous les endpoints ont répondu en moins d'une seconde."
+            STATUS="SUCCÈS"
           fi
-
+          
           BODY=$(cat <<EOF
           $HEADER
-
+          
           $INTRO
-
+          
+          ### Rapport complet des tests :
+          
           \`\`\`
-          $REPORT_CONTENT
+          $REPORT_CONTENT_ESCAPED
           \`\`\`
-
+          
-          > Seuil : **1000ms** | Vérifié par le workflow **PR Endpoint Performance**.
+          ---
+          **Statut**: $STATUS  
+          **Seuil**: 1000ms  
+          **Vérifié par**: Workflow PR Endpoint Performance
           EOF
           )
-
+          
           curl -s -X POST \
             -H "Authorization: token $GITEA_TOKEN" \
             -H "Content-Type: application/json" \
             -d "$(jq -n --arg body "$BODY" '{body: $body}')" \
             "$GITEA_SERVER_URL/api/v1/repos/$REPO/issues/$PR_NUMBER/comments"
+          
+          mkdir -p /tmp/artifacts
+          cp /tmp/webzine_endpoint_report.txt /tmp/artifacts/performance-report.txt
+
+      - name: Upload performance report
+        if: always()
+        uses: actions/upload-artifact@v4
+        with:
+          name: performance-report
+          path: /tmp/artifacts/
+          retention-days: 7
 
-      # ─────────────────────────────────────────────
-      # Blocage de la PR si un endpoint a échoué
-      # ─────────────────────────────────────────────
       - name: Enforce performance gate
+        if: always()
         run: |
           FAILED="${{ steps.perf_test.outputs.failed }}"
           if [ "${FAILED:-0}" -gt 0 ]; then
             echo "❌ PR REJETÉE : ${FAILED} endpoint(s) n'ont pas respecté le seuil d'une seconde."
             echo "   Corrigez les endpoints lents ou en erreur avant de fusionner."
+            echo ""
+            echo "Résumé des problèmes :"
+            grep -E "^\[(FAIL|SLOW)\]" /tmp/webzine_endpoint_report.txt || echo "  (aucun détail disponible)"
             exit 1
           else
             echo "✅ Tous les endpoints ont passé le contrôle de performance."