Add Playwright (#1110)

* remix-run/indie-stack#230

* Use Doppler to start dev server

* Enable v2_headers

* Explicitly set serverModuleFormat to cjs

* Remove extra auth logging

* Clean up workflow files

* Remove custom CodeQL setup

* Add Playwright

* Add Playwright workflow

* Remove example tests

* Add component testing

* Update Playwright workflow to add component tests

.dockerignore

@@ -1,2 +1,3 @@
 node_modules
 npm-debug.log
+.git

.github/workflows/lint.yml

@@ -1,15 +1,16 @@
+---
 name: Lint
 
 on:
   push:
-    branches: [main]
+    branches:
+      - main
   pull_request:
-    branches: [main]
+    branches:
+      - main
 
 jobs:
   lint:
-    name: Lint
-
     runs-on: ubuntu-latest
 
     steps:

.github/workflows/playwright.yml

@@ -0,0 +1,57 @@
+---
+name: Playwright
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+    branches:
+      - main
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+
+      - uses: actions/setup-node@v3
+        with:
+          node-version: 18
+          cache: npm
+
+      - run: npm ci
+
+      - name: Install Playwright browsers
+        run: npx playwright install --with-deps
+
+      - run: npm run test
+
+      - uses: actions/upload-artifact@v3
+        if: always()
+        with:
+          name: playwright-report
+          path: playwright-report/
+
+  ct:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+
+      - uses: actions/setup-node@v3
+        with:
+          node-version: 18
+          cache: npm
+
+      - run: npm ci
+
+      - name: Install Playwright browsers
+        run: npx playwright install --with-deps
+
+      - run: npm run test-ct
+
+      - uses: actions/upload-artifact@v3
+        if: always()
+        with:
+          name: playwright-report
+          path: playwright-report/

.github/workflows/type-check.yml

@@ -1,15 +1,16 @@
+---
 name: Type check
 
 on:
   push:
-    branches: [main]
+    branches:
+      - main
   pull_request:
-    branches: [main]
+    branches:
+      - main
 
 jobs:
-  lint:
-    name: Type check
-
+  tsc:
     runs-on: ubuntu-latest
 
     steps:
@@ -20,6 +21,6 @@ jobs:
           node-version: 18
           cache: npm
 
-      - run: npm install
+      - run: npm ci
 
-      - run: npm run type-check
+      - run: npx tsc

.gitignore

@@ -8,3 +8,6 @@ build/
 
 # runtime
 upload
+/test-results/
+/playwright-report/
+/playwright/.cache/

app/auth/redirect-to-login.server.ts

@@ -12,7 +12,6 @@ export async function redirectToLogin(
   args: LoaderArgs,
   { onHoldPage }: RedirectToLoginArgs = {},
 ) {
-  const auth = await getAuth(args);
   const { userId } = await getAuth(args);
 
   if (!userId) {
@@ -22,7 +21,6 @@ export async function redirectToLogin(
   const user = await createClerkClient({
     apiKey: process.env.CLERK_SECRET_KEY,
   }).users.getUser(userId);
-  console.dir({ auth, user }, { depth: null });
 
   const allowed = await userIsAllowed(user);
   if (!allowed && !onHoldPage) {

doppler.yaml

@@ -1,2 +1,3 @@
+---
 setup:
   project: festival