separate artifacts for videos and screenshots

conda-incubator · Jan 9, 2024 · c117a58 · c117a58
1 parent fe795f2
commit c117a58
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 3 deletions.
diff --git a/.github/workflows/push_images.yml b/.github/workflows/push_images.yml
@@ -66,6 +66,8 @@ jobs:
             pr: ${{ env.PR_NUMBER }}
             # how to exit the action if no artifact is found
             if_no_artifact_found: fail
+            # artifact name
+            name: 'screenshots'
 
       - name: "ls"
         run: |

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -59,9 +59,16 @@ jobs:
         run: |
           yarn run start & pytest --video on --output test-results --screenshots true test/playwright/test_ux.py
 
-      - name: "Upload artifacts"
+      - name: "Upload screenshot artifacts"
         uses: actions/upload-artifact@v3
         if: ${{ always() }}
         with:
-          name: playwright-tests
+          name: screenshots
           path: test-results
+
+      - name: "Upload video artifact"
+        uses: actions/upload-artifact@v3
+        if: ${{ always() }}
+        with:
+          name: video
+          path: video
diff --git a/test/playwright/conftest.py b/test/playwright/conftest.py
@@ -15,5 +15,5 @@ def screenshot(pytestconfig):
 def browser_context_args(browser_context_args):
     return {
         **browser_context_args,
-        "record_video_dir": "test-results/videos/",
+        "record_video_dir": "video",
     }