Skip to content

test(e2e): wait for the example to hydrate after page.goto - #3544

Merged
segunadebayo merged 1 commit into
mainfrom
test/e2e-wait-for-hydration
Oct 25, 2026
Merged

segunadebayo merged 1 commit into
mainfrom
test/e2e-wait-for-hydration

Conversation

@segunadebayo

Copy link
Copy Markdown
Member

page.goto resolves on load, before SSR examples (Vue, Svelte, Solid) hydrate, so early clicks and key presses were lost and tests failed locally.

  • Each example app marks html[data-ready] once the page has hydrated: useEffect in Next's _app, Nuxt's page:finish hook, SvelteKit's afterNavigate, a component inside Solid's route Suspense, and preact-iso's onLoadEnd/onRouteChange.
  • e2e/_test.ts extends Playwright's test so page.goto waits for the marker; specs import test/expect from it.

Local menu.e2e.ts (no retries): Vue 1 → 8 passed, Solid 4 → 7, Svelte 8. The rest fail on routes or example content that don't exist in those frameworks (menu/overflow, menu/links). React: menu, tabs-links, tour, combobox 87 passed.

@vercel

vercel Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
zag-nextjs Ready Ready Preview Oct 25, 2026 6:55pm UTC
zag-solid Ready Ready Preview Oct 25, 2026 6:55pm UTC
zag-svelte Ready Ready Preview Oct 25, 2026 6:55pm UTC
zag-vue Ready Ready Preview Oct 25, 2026 6:55pm UTC
1 Skipped Deployment
Project Deployment Actions Updated
zag-website Ignored Ignored Preview Oct 25, 2026 6:55pm UTC

Request Review

@changeset-bot

changeset-bot Bot commented Oct 25, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: a7f3293

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

This branch was successfully deployed

4 active deployments
Preview – zag-solid — a7f32931 Deployed Oct 25, 2026 by vercel[bot]
Preview – zag-vue — a7f32931 Deployed Oct 25, 2026 by vercel[bot]
Preview – zag-svelte — a7f32931 Deployed Oct 25, 2026 by vercel[bot]
Preview – zag-nextjs — a7f32931 Deployed Oct 25, 2026 by vercel[bot]
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.

1 participant