[codex] Add Solid Start pending hydration regression test#7086
[codex] Add Solid Start pending hydration regression test#7086ljho01 wants to merge 1 commit intoTanStack:mainfrom
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
View your CI Pipeline Execution ↗ for commit 9ece5f9
☁️ Nx Cloud last updated this comment at |
|
Shouldn't the tests be failing if you add a regression test? |
|
Integrated what I could into this pr |
What changed
This adds a Solid Start regression test for
ssr: 'data-only'with a slow loader and apendingComponentthat returns a DOM element.It introduces:
/mre-data-onlyroute undere2e/solid-start/selective-ssrOK — loader finishedwithouttemplate is not a functionorHydration MismatchWhy
This is intended to lock in the Solid hydration fix around
pendingComponentondata-onlyroutes.Current
mainpasses this test.If
packages/solid-router/src/Match.tsxis changed from:to:
the same test fails in dev mode and the route never reaches the loaded UI.
Impact
This gives us a targeted regression test for the exact hydration behavior that previously broke in dev mode for Solid Start.
Validation
Ran:
Also manually verified:
main/ fixed||condition: passes&&: fails||: passes again