Conversation
…test to '@component' + 'render' pattern
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughA test in the Angular-experimental package was refactored to replace direct injection context execution with a component-based rendering approach. The test now renders a Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
|
View your CI Pipeline Execution ↗ for commit d0b5051
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version PreviewNo changeset entries found. Merging this PR will not cause a version bump for any packages. |
size-limit report 📦
|
…test to '@component' + 'render' pattern (TanStack#10563)
🎯 Changes
Align the
isRestoringtest ininjectQuerieswith the rest of the suite by using a@Component+@testing-library/angularrender()setup instead ofTestBed.runInInjectionContext. This verifies behavior through a real component/template rather than a synthetic injection context.✅ Checklist
pnpm run test:pr.🚀 Release Impact
Summary by CodeRabbit