Skip to content

Enforce E2E process and port ownership - #8330

Draft
isaacroldan wants to merge 1 commit into
mainfrom
isaac/e2e-process-port-ownership
Draft

Enforce E2E process and port ownership#8330
isaacroldan wants to merge 1 commit into
mainfrom
isaac/e2e-process-port-ownership

Conversation

@isaacroldan

Copy link
Copy Markdown
Contributor

Why

A failed E2E test can leave a CLI child process or bound worker port that affects the next test.

What

  • terminate complete PTY process trees and wait for root and process-group exit
  • verify fixed worker ports before use and after fixture cleanup
  • preserve test failures when cleanup also fails, with process and port ownership details
  • add local coverage for process-tree termination, ownership diagnostics, and worker port allocation

Testing

  • pnpm --filter @shopify/e2e lint
  • pnpm --filter @shopify/e2e type-check
  • pnpm exec playwright test tests/ownership.spec.ts --project=local --repeat-each=10
  • pnpm exec playwright test --project=local

Assisted-By: devx/215513a9-13fa-4e74-bb8f-79faee6e4f09
@github-actions github-actions Bot added the no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users. label Aug 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant