test(trie): cover canonical-delete branches and fork gate#825
Conversation
…/patch Adds tests for the new code paths so the patch clears the 80% coverage gate: - delete-to-empty returns the canonical empty root - deleting a key whose sibling is a multi-key internal subtree leaves the remaining keys exactly as a direct insert (the non-pull-up branch) - fork-heights get_/is_canonical_delete_height default-off + pinned activation - Blockchain::is_canonical_delete_height wrapper default-off
|
Warning Review limit reached
More reviews will be available in 55 minutes and 5 seconds. Learn how PR review limits work. Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file). ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits. 🚦 How do rate limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan refill rate. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, the refill rate gradually slows as usage increases. The highest same-day bursts are limited more strictly. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
✨ 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 |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Follow-up to #823. These tests were written for the canonical-SMT-delete fix but landed after #823 had already auto-merged (the merge happened before branch protection / required
codecov/patchexisted), so they never made it into main.Covers the new code paths in #823:
get_/is_canonical_delete_heightdefault-off + pinned activation (>=boundary)Blockchain::is_canonical_delete_heightwrapper default-offTest-only; no behaviour change.