Skip to content

Commit

Permalink
fix(playwright): make rate-limiting test run last (#4738)
Browse files Browse the repository at this point in the history
* fix(playwright): make rate-limiting test run last

* enable parallelization on ci

* remove comment

* add comments
  • Loading branch information
Parker-Stafford authored Sep 25, 2024
1 parent fe5e043 commit eabf268
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions app/playwright.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,6 @@ export default defineConfig({
forbidOnly: !!process.env.CI,
/* Retry on CI only */
retries: process.env.CI ? 2 : 0,
/* Opt out of parallel tests on CI. */
workers: process.env.CI ? 1 : undefined,
/* Reporter to use. See https://playwright.dev/docs/test-reporters */
reporter: "html",
/* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */
Expand All @@ -37,14 +35,20 @@ export default defineConfig({
{
name: "chromium",
use: { ...devices["Desktop Chrome"] },
// The test below runs last in the 'rate limit' project so that we don't lock ourselves out
testIgnore: "**/*.rate-limit.spec.ts",
},
{
name: "firefox",
use: { ...devices["Desktop Firefox"] },
// The test below runs last in the 'rate limit' project so that we don't lock ourselves out
testIgnore: "**/*.rate-limit.spec.ts",
},
{
name: "webkit",
use: { ...devices["Desktop Safari"] },
// The test below runs last in the 'rate limit' project so that we don't lock ourselves out
testIgnore: "**/*.rate-limit.spec.ts",
},
{
name: "rate limit",
Expand Down

0 comments on commit eabf268

Please sign in to comment.