Skip to content

Commit f2ff191

Browse files
committed
Move all tests into one workflow, give them clear names
The Playwright tests had no name before, making it confusing to add them to the branch protection requirements.
1 parent 7d6a60a commit f2ff191

File tree

2 files changed

+30
-35
lines changed

2 files changed

+30
-35
lines changed

.github/workflows/playwright.yml

Lines changed: 0 additions & 33 deletions
This file was deleted.

.github/workflows/test.yaml

Lines changed: 30 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
1-
name: Run unit tests
1+
name: Test
22
on:
33
pull_request: {}
44
push:
55
branches: [livekit, full-mesh]
66
jobs:
77
vitest:
8-
name: Run vitest tests
8+
name: Run unit tests
99
runs-on: ubuntu-latest
1010
steps:
1111
- name: Checkout code
@@ -26,3 +26,31 @@ jobs:
2626
with:
2727
flags: unittests
2828
fail_ci_if_error: true
29+
playwright:
30+
name: Run end-to-end tests
31+
timeout-minutes: 10
32+
runs-on: ubuntu-latest
33+
steps:
34+
- uses: actions/checkout@v4
35+
- uses: actions/setup-node@v4
36+
with:
37+
cache: "yarn"
38+
node-version-file: ".node-version"
39+
- name: Install dependencies
40+
run: yarn install --frozen-lockfile
41+
- name: Install Playwright Browsers
42+
run: yarn playwright install --with-deps
43+
- name: Run backend components
44+
run: |
45+
docker compose -f playwright-backend-docker-compose.yml up -d
46+
docker ps
47+
- name: Copy config file
48+
run: cp config/config.devenv.json public/config.json
49+
- name: Run Playwright tests
50+
run: yarn playwright test
51+
- uses: actions/upload-artifact@v4
52+
if: ${{ !cancelled() }}
53+
with:
54+
name: playwright-report
55+
path: playwright-report/
56+
retention-days: 3

0 commit comments

Comments
 (0)