test: derive serde for test

crates/eips/Cargo.toml

@@ -64,6 +64,7 @@ alloy-primitives = { workspace = true, features = [
     "arbitrary",
 ] }
 arbitrary = { workspace = true, features = ["derive"] }
+serde.workspace = true
 serde_json.workspace = true
 rand.workspace = true
 

crates/eips/src/eip1559/basefee.rs

@@ -15,7 +15,7 @@ use crate::{
 /// BaseFeeParams contains the config parameters that control block base fee computation
 #[derive(Clone, Copy, Debug, PartialEq, Eq)]
 #[cfg_attr(any(test, feature = "arbitrary"), derive(arbitrary::Arbitrary))]
-#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
+#[cfg_attr(any(test, feature = "serde"), derive(serde::Serialize, serde::Deserialize))]
 pub struct BaseFeeParams {
     /// The base_fee_max_change_denominator from EIP-1559
     #[cfg_attr(feature = "serde", serde(with = "alloy_serde::quantity"))]

crates/eips/src/eip1898.rs

@@ -10,7 +10,7 @@ use core::{
 
 /// A helper struct to store the block number/hash and its parent hash.
 #[derive(Debug, Clone, Copy, PartialEq, Eq, Hash)]
-#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
+#[cfg_attr(any(test, feature = "serde"), derive(serde::Serialize, serde::Deserialize))]
 #[cfg_attr(feature = "serde", serde(rename_all = "camelCase"))]
 #[cfg_attr(any(test, feature = "arbitrary"), derive(arbitrary::Arbitrary))]
 pub struct BlockWithParent {
@@ -641,7 +641,7 @@ impl FromStr for BlockId {
 
 /// A number and a hash.
 #[derive(Clone, Copy, Debug, Default, PartialEq, Eq, Hash)]
-#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
+#[cfg_attr(any(test, feature = "serde"), derive(serde::Serialize, serde::Deserialize))]
 #[cfg_attr(any(test, feature = "arbitrary"), derive(arbitrary::Arbitrary))]
 pub struct NumHash {
     /// The number
@@ -690,7 +690,7 @@ impl From<(B256, u64)> for NumHash {
 
 /// Either a hash _or_ a block number
 #[derive(Clone, Copy, Debug, PartialEq, Eq, Hash)]
-#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
+#[cfg_attr(any(test, feature = "serde"), derive(serde::Serialize, serde::Deserialize))]
 #[cfg_attr(any(test, feature = "arbitrary"), derive(arbitrary::Arbitrary))]
 pub enum HashOrNumber {
     /// The hash

crates/eips/src/eip4844/engine.rs

@@ -5,7 +5,7 @@ use alloc::boxed::Box;
 
 /// Blob type returned in responses to `engine_getBlobsV1`: <https://github.com/ethereum/execution-apis/pull/559>
 #[derive(Debug, Clone, PartialEq, Eq)]
-#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
+#[cfg_attr(any(test, feature = "serde"), derive(serde::Serialize, serde::Deserialize))]
 pub struct BlobAndProofV1 {
     /// The blob data.
     pub blob: Box<Blob>,

crates/eips/src/eip4844/sidecar.rs

@@ -17,7 +17,7 @@ pub(crate) const VERSIONED_HASH_VERSION_KZG: u8 = 0x01;
 
 /// A Blob hash
 #[derive(Clone, Debug, Default, PartialEq, Eq, Hash)]
-#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
+#[cfg_attr(any(test, feature = "serde"), derive(serde::Serialize, serde::Deserialize))]
 pub struct IndexedBlobHash {
     /// The index of the blob
     pub index: u64,
@@ -30,7 +30,7 @@ pub struct IndexedBlobHash {
 /// This type encodes and decodes the fields without an rlp header.
 #[derive(Clone, Default, PartialEq, Eq, Hash)]
 #[repr(C)]
-#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
+#[cfg_attr(any(test, feature = "serde"), derive(serde::Serialize, serde::Deserialize))]
 #[doc(alias = "BlobTxSidecar")]
 pub struct BlobTransactionSidecar {
     /// The blob data.
@@ -104,7 +104,7 @@ impl IntoIterator for BlobTransactionSidecar {
 /// A single blob sidecar.
 #[derive(Clone, Debug, Default, PartialEq, Eq, Hash)]
 #[repr(C)]
-#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
+#[cfg_attr(any(test, feature = "serde"), derive(serde::Serialize, serde::Deserialize))]
 pub struct BlobTransactionSidecarItem {
     /// The index of this item within the [BlobTransactionSidecar].
     #[cfg_attr(feature = "serde", serde(with = "alloy_serde::quantity"))]

crates/eips/src/eip4895.rs

@@ -13,7 +13,7 @@ pub const GWEI_TO_WEI: u64 = 1_000_000_000;
 /// Withdrawal represents a validator withdrawal from the consensus layer.
 #[derive(Clone, Copy, Debug, Default, PartialEq, Eq, Hash, RlpEncodable, RlpDecodable)]
 #[cfg_attr(any(test, feature = "arbitrary"), derive(arbitrary::Arbitrary))]
-#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
+#[cfg_attr(any(test, feature = "serde"), derive(serde::Serialize, serde::Deserialize))]
 #[cfg_attr(feature = "ssz", derive(ssz_derive::Encode, ssz_derive::Decode))]
 pub struct Withdrawal {
     /// Monotonically increasing identifier issued by consensus layer.
@@ -56,7 +56,7 @@ impl Withdrawal {
     RlpDecodableWrapper,
 )]
 #[cfg_attr(any(test, feature = "arbitrary"), derive(arbitrary::Arbitrary))]
-#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
+#[cfg_attr(any(test, feature = "serde"), derive(serde::Serialize, serde::Deserialize))]
 pub struct Withdrawals(pub Vec<Withdrawal>);
 
 impl Withdrawals {

crates/eips/src/eip6110.rs

@@ -16,7 +16,7 @@ pub const MAX_DEPOSIT_RECEIPTS_PER_PAYLOAD: usize = 8192;
 
 /// This structure maps onto the deposit object from [EIP-6110](https://eips.ethereum.org/EIPS/eip-6110).
 #[derive(Clone, Copy, Debug, PartialEq, Eq, Hash, Default)]
-#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
+#[cfg_attr(any(test, feature = "serde"), derive(serde::Serialize, serde::Deserialize))]
 #[cfg_attr(feature = "ssz", derive(ssz_derive::Encode, ssz_derive::Decode))]
 #[cfg_attr(any(test, feature = "arbitrary"), derive(arbitrary::Arbitrary))]
 pub struct DepositRequest {

crates/eips/src/eip7002.rs

@@ -25,7 +25,7 @@ pub const MAX_WITHDRAWAL_REQUESTS_PER_BLOCK: usize = 16;
 ///
 /// See [EIP-7002](https://eips.ethereum.org/EIPS/eip-7002).
 #[derive(Clone, Copy, Debug, PartialEq, Eq, Hash, Default)]
-#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
+#[cfg_attr(any(test, feature = "serde"), derive(serde::Serialize, serde::Deserialize))]
 #[cfg_attr(feature = "ssz", derive(ssz_derive::Encode, ssz_derive::Decode))]
 #[cfg_attr(any(test, feature = "arbitrary"), derive(arbitrary::Arbitrary))]
 pub struct WithdrawalRequest {

crates/eips/src/eip7251.rs

@@ -19,7 +19,7 @@ pub const MAX_CONSOLIDATION_REQUESTS_PER_BLOCK: usize = 2;
 
 /// This structure maps onto the consolidation request object from [EIP-7251](https://eips.ethereum.org/EIPS/eip-7251).
 #[derive(Clone, Copy, Debug, PartialEq, Eq, Hash, Default)]
-#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
+#[cfg_attr(any(test, feature = "serde"), derive(serde::Serialize, serde::Deserialize))]
 #[cfg_attr(feature = "ssz", derive(ssz_derive::Encode, ssz_derive::Decode))]
 #[cfg_attr(any(test, feature = "arbitrary"), derive(arbitrary::Arbitrary))]
 pub struct ConsolidationRequest {

crates/eips/src/eip7685.rs

@@ -18,7 +18,7 @@ pub const EMPTY_REQUESTS_HASH: B256 =
 /// each element is the `request_type` and the remaining bytes are the `request_data`.
 #[derive(Debug, Clone, PartialEq, Eq, Default, Hash, Deref, DerefMut, From, IntoIterator)]
 #[cfg_attr(any(test, feature = "arbitrary"), derive(arbitrary::Arbitrary))]
-#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
+#[cfg_attr(any(test, feature = "serde"), derive(serde::Serialize, serde::Deserialize))]
 pub struct Requests(Vec<Bytes>);
 
 impl Requests {
@@ -118,7 +118,7 @@ impl Requests {
 /// when the exact contents of the requests are unnecessary, and only a consistent hash value is
 /// needed to simulate the presence of requests without holding actual data.
 #[derive(Debug, Clone, PartialEq, Eq, Hash, derive_more::From)]
-#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
+#[cfg_attr(any(test, feature = "serde"), derive(serde::Serialize, serde::Deserialize))]
 pub enum RequestsOrHash {
     /// Stores a list of requests, allowing for dynamic requests hash calculation.
     Requests(Requests),

crates/eips/src/eip7840.rs

@@ -7,7 +7,7 @@ const IS_DEFAULT_BLOB_FEE: fn(&u128) -> bool = |&x| x == eip4844::BLOB_TX_MIN_BL
 
 /// Configuration for the blob-related calculations.
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]
-#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
+#[cfg_attr(any(test, feature = "serde"), derive(serde::Serialize, serde::Deserialize))]
 pub struct BlobParams {
     /// Target blob count for the block.
     #[cfg_attr(feature = "serde", serde(rename = "target"))]