chore: saving videos and screenshots for e2e tests only on failure

the limit of storage is not far. Most of the place is took by the videos and screenshots of the e2e tests. We are usually looking at them only on failure. This should save a lot of space. Signed-off-by: Aurélien Pupier <[email protected]>
KaotoIO · Sep 17, 2024 · 06ecb09 · 06ecb09
1 parent 972d250
commit 06ecb09
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/.github/workflows/e2e-tests.yml b/.github/workflows/e2e-tests.yml
@@ -111,14 +111,14 @@ jobs:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
       - name: 💾 Save videos
-        if: always()
+        if: failure()
         uses: actions/upload-artifact@v4
         with:
           name: videos-firefox
           path: packages/ui-tests/cypress/videos
 
       - name: 💾 Save screenshots
-        if: always()
+        if: failure()
         uses: actions/upload-artifact@v4
         with:
           name: screenshots-firefox
@@ -170,14 +170,14 @@ jobs:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
       - name: 💾 Save videos
-        if: always()
+        if: failure()
         uses: actions/upload-artifact@v4
         with:
           name: videos-chrome
           path: packages/ui-tests/cypress/videos
 
       - name: 💾 Save screenshots
-        if: always()
+        if: failure()
         uses: actions/upload-artifact@v4
         with:
           name: screenshots-chrome
@@ -229,14 +229,14 @@ jobs:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
       - name: 💾 Save videos
-        if: always()
+        if: failure()
         uses: actions/upload-artifact@v4
         with:
           name: videos-edge
           path: packages/ui-tests/cypress/videos
 
       - name: 💾 Save screenshots
-        if: always()
+        if: failure()
         uses: actions/upload-artifact@v4
         with:
           name: screenshots-edge