Skip to content

chore: isolate E2E tests on port 3015 to avoid dev server conflicts#131

Merged
mahata merged 1 commit intomainfrom
chore/isolate-e2e-port
Mar 26, 2026
Merged

chore: isolate E2E tests on port 3015 to avoid dev server conflicts#131
mahata merged 1 commit intomainfrom
chore/isolate-e2e-port

Conversation

@mahata
Copy link
Copy Markdown
Owner

@mahata mahata commented Mar 26, 2026

Summary

  • Moves the E2E test server from port 3000 to port 3015 so it never collides with a running dev server.
  • Previously, reuseExistingServer: true (the local default) would silently reuse a dev server on port 3000, which connects to the wrong database (postgres instead of test) and may lack NODE_ENV=development — causing test data leaks or auth failures.

Changes

  • playwright.config.ts: Updated baseURL, webServer.url to localhost:3015, and added PORT: "3015" to webServer.env.
  • e2e/auth-helpers.ts: Updated TEST_ORIGIN to http://localhost:3015.

Copilot AI review requested due to automatic review settings March 26, 2026 07:06
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Playwright E2E configuration to run the app server on port 3015 instead of 3000, preventing Playwright’s reuseExistingServer: true behavior from accidentally reusing a developer’s local dev server (and therefore the wrong database/env).

Changes:

  • Switch Playwright baseURL and webServer.url to http://localhost:3015.
  • Set PORT=3015 in Playwright webServer.env so the app actually binds to that port.
  • Update E2E auth helper TEST_ORIGIN to match the new port.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
playwright.config.ts Moves E2E server URL/baseURL to 3015 and sets PORT in the spawned server env.
e2e/auth-helpers.ts Aligns the Origin header used by helper requests with the new E2E port.

@mahata mahata merged commit 483632b into main Mar 26, 2026
7 checks passed
@mahata mahata deleted the chore/isolate-e2e-port branch March 26, 2026 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants