fix: skip skipping of empty auth tests

bluealloy · Sep 18, 2024 · 0c962fd · 0c962fd
1 parent f643648
commit 0c962fd
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 11 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -106,4 +106,3 @@ jobs:
         with:
           config: ./Typos.toml
           isolated: true
-
diff --git a/bins/revme/src/cmd/statetest/models/mod.rs b/bins/revme/src/cmd/statetest/models/mod.rs
@@ -108,8 +108,7 @@ pub struct TransactionParts {
 
     #[serde(default)]
     pub access_lists: Vec<Option<AccessList>>,
-    #[serde(default)]
-    pub authorization_list: Vec<TestAuthorization>,
+    pub authorization_list: Option<Vec<TestAuthorization>>,
     #[serde(default)]
     pub blob_versioned_hashes: Vec<B256>,
     pub max_fee_per_blob_gas: Option<U256>,

diff --git a/bins/revme/src/cmd/statetest/runner.rs b/bins/revme/src/cmd/statetest/runner.rs
@@ -373,16 +373,15 @@ pub fn execute_test_suite(
                     .cloned()
                     .unwrap_or_default();
 
-                if unit.transaction.authorization_list.is_empty() {
-                    continue;
-                }
-                env.tx.authorization_list = Some(AuthorizationList::Recovered(
+                env.tx.authorization_list =
                     unit.transaction
                         .authorization_list
-                        .iter()
-                        .map(|auth| auth.into_recovered())
-                        .collect(),
-                ));
+                        .as_ref()
+                        .map(|auth_list| {
+                            AuthorizationList::Recovered(
+                                auth_list.iter().map(|auth| auth.into_recovered()).collect(),
+                            )
+                        });
 
                 let to = match unit.transaction.to {
                     Some(add) => TxKind::Call(add),