test(dpp): improve coverage for identity state transitions and public keys in creation#3456
Conversation
Add comprehensive unit tests for all seven identity state transition modules that had the lowest coverage. Tests cover construction, accessors, StateTransitionLike/Owned/SingleSigned trait impls, IdentitySigned trait impls, value conversion roundtrips, serialization roundtrips, estimated fee validation, and error paths for unknown versions. Also adds thorough tests for public key structure validation including duplicate key detection, master key requirements, and security level enforcement. 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 7 minutes and 43 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 (15)
✨ 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 GateCommit:
|
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## v3.1-dev #3456 +/- ##
============================================
+ Coverage 83.13% 83.98% +0.84%
============================================
Files 2621 2621
Lines 270434 275224 +4790
============================================
+ Hits 224833 231137 +6304
+ Misses 45601 44087 -1514
🚀 New features to boost your workflow:
|
Summary
Adds 203 tests across 15 files covering identity state transition construction, serialization, validation, and conversion logic.
Test plan
cargo test -p dpp --lib— 2354 passedcargo fmt --allclean🤖 Generated with Claude Code