alloy-rs / core

High-performance, well-tested & documented core libraries for Ethereum, in Rust
https://alloy.rs
Apache License 2.0
763 stars 135 forks source link

feat(primitives): derive `Arbitrary` for Signature #652

Closed yash-atreya closed 3 months ago

yash-atreya commented 3 months ago

Motivation

Needed for https://github.com/alloy-rs/alloy/issues/109.

primitive Signature and Parity types do not derive Arbitrary. But the Transaction type in alloy requires it.

Solution

Derive Arbitrary for the test and arbitrary feature.

PR Checklist

yash-atreya commented 3 months ago

@DaniPopes @prestwich merge pls

DaniPopes commented 3 months ago

What about proptest?