Skip to content

Commit

Permalink
Fix waitForAppUrl race condition
Browse files Browse the repository at this point in the history
  • Loading branch information
nikosdouvlis committed Jul 12, 2024
1 parent ad543f3 commit 4c7d111
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 5 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ jobs:

integration-tests:
name: Integration Tests
needs: formatting-linting
# needs: formatting-linting
runs-on: ${{ vars.RUNNER_LARGE || 'ubuntu-latest-l' }}
timeout-minutes: ${{ vars.TIMEOUT_MINUTES_LONG && fromJSON(vars.TIMEOUT_MINUTES_LONG) || 15 }}

Expand Down Expand Up @@ -191,11 +191,11 @@ jobs:
if: ${{ steps.task-status.outputs.affected == '1' }}
working-directory: ${{runner.temp}}
run: mkdir clerk-js && cd clerk-js && npm init -y && npm install @clerk/clerk-js

- name: Copy components @clerk/astro
if: ${{ matrix.test-name == 'astro' }}
run: cd packages/astro && npm run copy:components

- name: Write all ENV certificates to files in integration/certs
if: ${{ steps.task-status.outputs.affected == '1' }}
uses: actions/github-script@v7
Expand Down
5 changes: 3 additions & 2 deletions integration/playwright.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,8 @@ import * as path from 'path';

config({ path: path.resolve(__dirname, '.env.local') });

const numAvailableWorkers = os.cpus().length - 1;
const numAvailableWorkers = Math.floor(os.cpus().length / 2) + 1;
console.log(`Running tests using ${numAvailableWorkers} workers`);

export const common: PlaywrightTestConfig = {
testDir: './tests',
Expand All @@ -18,7 +19,7 @@ export const common: PlaywrightTestConfig = {
retries: process.env.CI ? 2 : 0,
timeout: 90000,
maxFailures: process.env.CI ? 1 : undefined,
workers: process.env.CI ? numAvailableWorkers : '50%',
workers: process.env.CI ? numAvailableWorkers : '70%',
reporter: process.env.CI ? 'line' : 'list',
use: {
ignoreHTTPSErrors: true,
Expand Down

0 comments on commit 4c7d111

Please sign in to comment.