Skip to content

Use CLI commands for E2E store lifecycle - #8290

Merged
isaacroldan merged 2 commits into
mainfrom
use-cli-for-e2e-store-lifecycle
Aug 11, 2026
Merged

Use CLI commands for E2E store lifecycle#8290
isaacroldan merged 2 commits into
mainfrom
use-cli-for-e2e-store-lifecycle

Conversation

@isaacroldan

Copy link
Copy Markdown
Contributor

Why

The E2E suite provisions and deletes development stores through Playwright even though the CLI now supports these operations.

What

  • Create per-test development stores with shopify store create dev --json.
  • Delete stores with shopify store delete --force --json after app uninstall verification.
  • Validate command results and allow enough time for provisioning and deletion polling.

Testing

  • pnpm --filter @shopify/e2e type-check
  • pnpm --filter @shopify/e2e lint
  • pnpm --filter @shopify/e2e exec playwright test tests/app-dev-server.spec.ts --list

A live local Playwright run is blocked before test setup by the existing local node-pty error posix_spawnp failed. CI rebuilds node-pty before the E2E job.

Assisted-By: devx/2b0225ae-b59f-4355-9343-ecee05f202f0
@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 10, 2026
Assisted-By: devx/2b0225ae-b59f-4355-9343-ecee05f202f0
@isaacroldan
isaacroldan marked this pull request as ready for review August 11, 2026 10:02
@isaacroldan
isaacroldan requested a review from a team as a code owner August 11, 2026 10:02

@gonzaloriestra gonzaloriestra left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Simpler and more robust 👌

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