Skip to content

Commit

Permalink
🔧 try installing only one playwright browser and clean up config
Browse files Browse the repository at this point in the history
  • Loading branch information
bnussman committed Aug 10, 2023
1 parent 93c7bdd commit b8f5216
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/e2e.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ jobs:
- name: 📦 install packages
run: pnpm i --frozen-lockfile
- name: Install Playwright Browsers
run: pnpm exec playwright install --with-deps
run: pnpm exec playwright install --with-deps chromium
- name: Start DB and Redis
run: docker compose up -d
- name: Build the api
Expand Down
4 changes: 1 addition & 3 deletions test/playwright.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ export default defineConfig({
workers: process.env.CI ? 1 : undefined,
/* Reporter to use. See https://playwright.dev/docs/test-reporters */
reporter: 'html',
timeout: 120_000,
/* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */
use: {
/* Base URL to use in actions like `await page.goto('/')`. */
Expand Down Expand Up @@ -71,12 +70,11 @@ export default defineConfig({

/* Run your local dev server before starting the tests */
webServer: {
command: 'cd .. && docker compose up -d && pnpm dev',
command: 'cd .. && pnpm dev',
url: 'http://127.0.0.1:8081',
reuseExistingServer: true,
stdout: 'pipe',
stderr: 'pipe',
timeout: 120_000,
ignoreHTTPSErrors: true,
},
});

0 comments on commit b8f5216

Please sign in to comment.