test: add typed route regression coverage#3994
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (6)
WalkthroughThis PR adds a regression test fixture for issue Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
Summary
Add regression coverage for typed i18n routes with
useRoute().paramsandRouteLocationRawprops.Why
Issue #3954 reports typed route regressions after the vue-router v5 update.
Changes
RouteLocationRawTesting
Known Issues
The minimal reproduction passes on current main, so this PR adds coverage only and does not change production types.
Summary by CodeRabbit
#3954validating TypeScript type diagnostics with internationalization and typed route declarations. The tests ensure route parameters maintain correct type narrowing when using typed pages with localization, and prevent unexpected expansion of route location unions when used in component props.