Skip to content

Remove fixed wait from dev hot reload E2E#7844

Draft
alfonso-noriega wants to merge 1 commit into
mainfrom
06-17-remove_fixed_wait_from_dev_hot_reload_e2e
Draft

Remove fixed wait from dev hot reload E2E#7844
alfonso-noriega wants to merge 1 commit into
mainfrom
06-17-remove_fixed_wait_from_dev_hot_reload_e2e

Conversation

@alfonso-noriega

@alfonso-noriega alfonso-noriega commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

WHY are these changes introduced?

Fixes N/A

The dev hot reload delete E2E test used a fixed 5 second sleep after detecting extension creation. That made the test slower and still relied on timing rather than the observable state the test already waits for.

WHAT is this pull request doing?

Removes the fixed settle wait from tests/dev-hot-reload.spec.ts before deleting the generated extension. The test now proceeds once the CLI has emitted Extension created, then deletes the extension and waits for the existing Extension deleted signal.

How to test your changes?

Validated locally from ~/worktrees/cli-remove-dev-hot-reload-fixed-wait:

pnpm install --frozen-lockfile
pnpm --filter @shopify/e2e lint
NX_SKIP_NX_CACHE=true pnpm --filter @shopify/e2e type-check
git diff --check

I did not run the full E2E spec locally because it requires E2E environment credentials.

Post-release steps

None.

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes
  • I've considered analytics changes to measure impact
  • This change is not user-facing, so no changeset is required

Assisted-By: devx/aa6f2838-a296-4af6-ae5b-25654e978e0a
@alfonso-noriega alfonso-noriega requested a review from a team as a code owner June 17, 2026 12:36

alfonso-noriega commented Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@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 Jun 17, 2026
@alfonso-noriega alfonso-noriega marked this pull request as draft June 17, 2026 13:03
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