feat(api): add missing spec() and label() accessors#266
Conversation
Add spec() and label() accessor methods to fixture types that were missing them, ensuring all fixture types expose a consistent API surface.
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
|
Caution Review failedPull request was closed or merged during review No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (13)
Summary by CodeRabbit
WalkthroughThis PR adds public accessor methods ( Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
📝 Coding Plan
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 |
Summary
specandlabelfields — some had accessors, some didn'tspec()andlabel()toRsaKeyPair,Ed25519KeyPair,HmacSecret,TokenFixturelabel()toEcdsaKeyPairandPgpKeyPair(which already hadspec())Test plan
cargo check --workspace --all-featurespassescargo testpasses for all 6 affected crates including new doctests