Skip to content

Commit

Permalink
test: migrate 4844 rlp tests (#1928)
Browse files Browse the repository at this point in the history
  • Loading branch information
mattsse authored Jan 20, 2025
1 parent fe60081 commit 39e5117
Show file tree
Hide file tree
Showing 28 changed files with 48 additions and 0 deletions.
21 changes: 21 additions & 0 deletions crates/consensus/src/transaction/pooled.rs
Original file line number Diff line number Diff line change
Expand Up @@ -518,6 +518,7 @@ mod tests {
use super::*;
use alloy_primitives::{address, hex};
use bytes::Bytes;
use std::path::PathBuf;

#[test]
fn invalid_legacy_pooled_decoding_input_too_short() {
Expand Down Expand Up @@ -592,4 +593,24 @@ mod tests {
let res = PooledTransaction::decode_2718(&mut &data[..]);
assert!(res.is_ok());
}

#[test]
fn decode_encode_raw_4844_rlp() {
let path = PathBuf::from(env!("CARGO_MANIFEST_DIR")).join("testdata/4844rlp");
let dir = std::fs::read_dir(path).expect("Unable to read folder");
for entry in dir {
let entry = entry.unwrap();
let content = std::fs::read_to_string(entry.path()).unwrap();
let raw = hex::decode(content.trim()).unwrap();
let tx = PooledTransaction::decode_2718(&mut raw.as_ref())
.map_err(|err| {
panic!("Failed to decode transaction: {:?} {:?}", err, entry.path());
})
.unwrap();
// We want to test only EIP-4844 transactions
assert!(tx.is_eip4844());
let encoded = tx.encoded_2718();
assert_eq!(encoded.as_slice(), &raw[..], "{:?}", entry.path());
}
}
}
1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/0.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/1.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/10.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/11.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/12.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/13.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/14.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/15.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/16.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/17.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/18.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/19.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/2.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/20.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/21.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/22.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/23.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/24.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/25.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/26.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/3.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/4.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/5.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/6.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/7.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/8.rlp

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions crates/consensus/testdata/4844rlp/9.rlp

Large diffs are not rendered by default.

0 comments on commit 39e5117

Please sign in to comment.