Revert "test(ethereum): ignore invalid string sequence in ethereum st…

…ate tests (#3307)" (#3937)
paradigmxyz · Jul 26, 2023 · 9adab0b · 9adab0b
1 parent adce7ea
commit 9adab0b
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 26 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/testing/ef-tests/Cargo.toml b/testing/ef-tests/Cargo.toml
@@ -23,5 +23,4 @@ tokio = "1.28.1"
 walkdir = "2.3.3"
 serde = "1.0.163"
 serde_json.workspace = true
-thiserror.workspace = true
-serde_bytes = "0.11.9"
+thiserror.workspace = true
diff --git a/testing/ef-tests/src/models.rs b/testing/ef-tests/src/models.rs
@@ -36,20 +36,6 @@ pub struct BlockchainTest {
     #[serde(default)]
     /// Engine spec.
     pub self_engine: SealEngine,
-    #[serde(rename = "_info")]
-    #[allow(unused)]
-    info: BlockchainTestInfo,
-}
-
-#[derive(Debug, PartialEq, Eq, Deserialize)]
-struct BlockchainTestInfo {
-    #[serde(rename = "filling-rpc-server")]
-    #[allow(unused)]
-    // One test has an invalid string in this field, which breaks our CI:
-    // https://github.com/ethereum/tests/blob/6c252923bdd1bd5a70f680df1214f866f76839db/GeneralStateTests/stTransactionTest/ValueOverflow.json#L5
-    // By using `serde_bytes::ByteBuf`, we ignore the validation of this field as a string.
-    // TODO(alexey): remove when `ethereum/tests` is fixed
-    filling_rpc_server: serde_bytes::ByteBuf,
 }
 
 /// A block header in an Ethereum blockchain test.