Skip to content

Split local and remote E2E tests - #8325

Merged
isaacroldan merged 3 commits into
mainfrom
isaac/split-local-remote-e2e
Aug 14, 2026
Merged

Split local and remote E2E tests#8325
isaacroldan merged 3 commits into
mainfrom
isaac/split-local-remote-e2e

Conversation

@isaacroldan

@isaacroldan isaacroldan commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Why

Shopify authentication failures currently block local E2E checks that do not use remote services.

What

  • Add separate local, remote-auth, and remote Playwright projects.
  • Store shared authentication artifacts at stable paths for remote workers.
  • Run the four local smoke and fixture tests in a separate CI job without Shopify secrets.
  • Limit the existing sharded jobs to the remote project.

Testing

  • pnpm --filter @shopify/e2e lint
  • pnpm --filter @shopify/e2e type-check
  • Local Playwright project: 4 tests passed without Shopify credentials
  • Both remote shard test lists include the remote authentication setup
  • Workflow YAML parsing and git diff --check passed

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 13, 2026
Assisted-By: devx/215513a9-13fa-4e74-bb8f-79faee6e4f09
This reverts commit 4f068ec.

Assisted-By: devx/215513a9-13fa-4e74-bb8f-79faee6e4f09
@isaacroldan
isaacroldan marked this pull request as ready for review August 13, 2026 13:29
@isaacroldan
isaacroldan requested a review from a team as a code owner August 13, 2026 13:29
@isaacroldan
isaacroldan added this pull request to the merge queue Aug 14, 2026
Merged via the queue into main with commit c2a420d Aug 14, 2026
30 checks passed
@isaacroldan
isaacroldan deleted the isaac/split-local-remote-e2e branch August 14, 2026 08:59
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.

2 participants