test(preact-query-devtools/PreactQueryDevtools{,Panel}): add tests for the full prop wiring matrix#10852
Conversation
…r the full prop wiring matrix
|
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 (2)
📝 WalkthroughWalkthroughThe PR expands test coverage for two Preact devtools wrapper components. Both test suites now import their underlying TanStack devtools dependencies as runtime values and add comprehensive test cases validating prop forwarding, constructor options, and lifecycle behavior. ChangesPreact Devtools Component Tests
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 c0926c9
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version Preview2 package(s) bumped directly, 23 bumped as dependents. 🟩 Patch bumps
|
size-limit report 📦
|
🎯 Changes
Add test cases to both
PreactQueryDevtoolsandPreactQueryDevtoolsPanelthat cover the full prop wiring matrix, mirroring the matrix already in place forSolidQueryDevtools(#10823, #10824),SolidQueryDevtoolsPanel(#10827, #10828), andReactQueryDevtools{,Panel}(#10851).PreactQueryDevtoolsinitialIsOpen→setInitialIsOpen+ default (falsewhen omitted)errorTypes→setErrorTypes+ default ([]when omitted)theme→setThemeQueryClient→setClientstyleNonce,shadowDOMTarget,hideDisabledQueries)unmountcleanupPreactQueryDevtoolsPanelonClose→setOnClose+ default (no-op function when omitted)errorTypes→setErrorTypes+ defaulttheme→setThemeQueryClient→setClientstyleNonce,shadowDOMTarget,hideDisabledQueries)stylemerge contract (preserve default and override on conflict)unmountcleanupAll cases follow the existing mock pattern (`vi.mock('@tanstack/query-devtools', ...)` with per-method spies), and
stylecases use `toHaveStyle` from@testing-library/jest-dom.✅ Checklist
🚀 Release Impact
Summary by CodeRabbit