Merge pull request #33872 from storybookjs/yann/readd-traces-to-e2e

Build: Fix missing traces in E2E step

Author: yannbf

code/playwright.config.ts

@@ -47,7 +47,7 @@ export default defineConfig({
     // baseURL: 'http://localhost:3000',
 
     /* Collect trace when retrying the failed test. See https://playwright.dev/docs/trace-viewer */
-    trace: 'on-first-retry',
+    trace: 'retain-on-failure',
   },
 
   /* Configure projects for major browsers */

scripts/ci/sandboxes.ts

@@ -97,6 +97,10 @@ function defineSandboxJob_dev({
                 },
               },
               artifact.persist(join(LINUX_ROOT_DIR, WORKING_DIR, 'test-results'), 'test-results'),
+              artifact.persist(
+                join(LINUX_ROOT_DIR, WORKING_DIR, 'code', 'playwright-results'),
+                'playwright-results'
+              ),
               testResults.persist(join(LINUX_ROOT_DIR, WORKING_DIR, 'test-results')),
             ]
           : [
@@ -289,6 +293,10 @@ export function defineSandboxFlow<Key extends string>(key: Key) {
           },
         },
         artifact.persist(join(LINUX_ROOT_DIR, WORKING_DIR, 'test-results'), 'test-results'),
+        artifact.persist(
+          join(LINUX_ROOT_DIR, WORKING_DIR, 'code', 'playwright-results'),
+          'playwright-results'
+        ),
         testResults.persist(join(LINUX_ROOT_DIR, WORKING_DIR, 'test-results')),
       ],
     }),