From 618c7285a2213c4e609d85042658ac77e2debb3e Mon Sep 17 00:00:00 2001
From: OJ Kwon <1210596+kwonoj@users.noreply.github.com>
Date: Mon, 11 Dec 2023 11:07:08 -0800
Subject: [PATCH] ci(test): upload playwright artifacts seperately (#59496)

###

Updating snapshot upload location per suggestion at
https://github.com/vercel/next.js/pull/59469#discussion_r1422943741

Closes PACK-2132

---------

Co-authored-by: Leah <8845940+ForsakenHarmony@users.noreply.github.com>
---
 .github/workflows/build_reusable.yml | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/build_reusable.yml b/.github/workflows/build_reusable.yml
index 322a2572cf756..559e8ba62efeb 100644
--- a/.github/workflows/build_reusable.yml
+++ b/.github/workflows/build_reusable.yml
@@ -178,7 +178,16 @@ jobs:
         with:
           name: test-reports
           path: |
-            test/traces
             test/test-junit-report
             test/turbopack-test-junit-report
           if-no-files-found: ignore
+
+      # upload playwright snapshots from failed tests
+      - name: Upload test report artifacts
+        uses: actions/upload-artifact@v3
+        if: ${{ inputs.afterBuild && always() }}
+        with:
+          name: test-playwright-snapshots
+          path: |
+            test/traces
+          if-no-files-found: ignore