test(dpp): improve coverage for document property serialization and encoding#3439
Conversation
…ncoding Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 29 minutes and 13 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ 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 |
|
⏳ Review in progress (commit 9a6ca1b) |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## v3.1-dev #3439 +/- ##
============================================
+ Coverage 80.66% 80.72% +0.06%
============================================
Files 2852 2852
Lines 285371 286088 +717
============================================
+ Hits 230190 230956 +766
+ Misses 55181 55132 -49
🚀 New features to boost your workflow:
|
Summary
Adds 87 tests to the document property type module — the largest non-excluded file in rs-dpp (4647 lines).
Test plan
cargo test -p dpp --lib -- property::testspasses (275 total)cargo fmt --allclean🤖 Generated with Claude Code