From c798f341f1958f9e4beb6a03b919921c21938006 Mon Sep 17 00:00:00 2001
From: Liam DeBeasi <2721089+liamdebeasi@users.noreply.github.com>
Date: Mon, 13 Nov 2023 21:34:06 -0500
Subject: [PATCH] upload report and install deps correctly

---
 .github/workflows/build-and-test.yml | 13 +++++++++++--
 playwright.config.ts                 |  2 +-
 2 files changed, 12 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml
index 93c47d0..f8276ab 100644
--- a/.github/workflows/build-and-test.yml
+++ b/.github/workflows/build-and-test.yml
@@ -4,6 +4,9 @@ on:
   push:
     branches:
       - '*'
+  pull_request:
+    branches:
+      - '*'
 
 concurrency:
   # Cancel in progress runs for this branch
@@ -22,11 +25,17 @@ jobs:
         with:
           version: 8
       - name: Install Dependencies
-        run: pnpm install && pnpm playwright install
+        run: pnpm install && pnpm playwright install --with-deps
         shell: sh
       - name: Build Application
         run: pnpm run build
         shell: sh
       - name: Run Tests
         run: pnpm run test
-        shell: sh
\ No newline at end of file
+        shell: sh
+      - uses: actions/upload-artifact@v3
+        if: always()
+        with:
+          name: playwright-report
+          path: playwright-report/
+          retention-days: 30
\ No newline at end of file
diff --git a/playwright.config.ts b/playwright.config.ts
index 6594416..9ead6c3 100644
--- a/playwright.config.ts
+++ b/playwright.config.ts
@@ -70,7 +70,7 @@ export default defineConfig({
 
   /* Run your local dev server before starting the tests */
   webServer: {
-    command: 'pnpm run build && pnpm serve dist',
+    command: 'pnpm serve dist',
     url: 'http://127.0.0.1:3000',
     reuseExistingServer: !process.env.CI,
   },