Add fuzz testing - #2
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (5)
📝 WalkthroughWalkthroughThe change adds ChangesProperty-based testing
Estimated code review effort: 3 (Moderate) | ~25 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
This pull request introduces property-based fuzz testing to both the
modalsandslottablepackages using thefast-checklibrary. The main changes include addingfast-checkas a dependency, updating lock and workspace files, and adding comprehensive property tests to ensure correctness and robustness of core APIs.Testing improvements:
packages/modals/src/store.property.test.ts: Adds property-based fuzz tests for the modal store, simulating arbitrary sequences of stack operations and validating consistency, error handling, and stack capacity enforcement.packages/slottable/src/use-slot.property.test.tsx: Adds property-based fuzz tests for theuseSlothook andSlotcomponent, verifying prop precedence, slot name resolution, and default/override behavior under a wide range of input combinations.Dependency updates:
packages/modals/package.json,packages/slottable/package.json: Addsfast-checkas a test dependency. [1] [2]pnpm-workspace.yaml,pnpm-lock.yaml: Updates workspace and lock files to includefast-checkand its dependencypure-rand. [1] [2] [3] [4] [5] [6] [7] [8]Summary by CodeRabbit