test(platform): coverage round 3 — replace, index, bytes, distribution encode + exclusions#3431
Conversation
…n encode + exclusions Add 218 tests across 5 files covering real logic: - replace.rs: recursive path-based type replacement, clean_recursive - index.rs: Index/IndexMut for Value, auto-map creation, nested access - bytes_20.rs: conversions, hashing, ordering, string encoding - bytes_32.rs: conversions, hashing, ordering, identifier interop - distribution_function/encode.rs: all 9 variants encode/decode, edge cases, truncated input, tag validation, determinism Exclude 3 accessor-only files from coverage: - token_configuration/v0/accessors.rs - document/v0/accessors.rs - platform_state/accessors.rs 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 11 minutes and 38 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 ignored due to path filters (1)
📒 Files selected for processing (7)
✨ 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 #3431 +/- ##
============================================
+ Coverage 80.22% 80.55% +0.32%
============================================
Files 2855 2852 -3
Lines 281685 283778 +2093
============================================
+ Hits 225983 228596 +2613
+ Misses 55702 55182 -520
🚀 New features to boost your workflow:
|
Fixes cargo-machete CI failure. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
cbcced0 to
aab73a5
Compare
Summary
Adds 218 tests across 5 files covering real logic, plus 3 codecov exclusions for accessor-only files.
Tests added
replace.rs(platform-value)index.rs(platform-value)bytes_20.rs(platform-value)bytes_32.rs(platform-value)distribution_function/encode.rs(dpp)Codecov exclusions added
token_configuration/v0/accessors.rsdocument/v0/accessors.rsplatform_state/accessors.rsTest plan
cargo test -p platform-value --lib— 776 passedcargo test -p dpp --lib -- distribution_function::encode— 59 passedcargo fmt --allclean🤖 Generated with Claude Code