Merge pull request #2478 from eqlabs/t00ts/fix-trace-block-txs

chore(rpc): fix `test_multiple_transactions` for rpc `v08`
eqlabs · Jan 10, 2025 · 7cbefab · 7cbefab
2 parents f2c2f84 + 8f82ab8
commit 7cbefab
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 4 deletions.
diff --git a/crates/rpc/src/method/simulate_transactions.rs b/crates/rpc/src/method/simulate_transactions.rs
@@ -743,7 +743,7 @@ pub(crate) mod tests {
                                         l1_gas: 0,
                                         l1_data_gas: 192,
                                     },
-                                l1_gas: 0,
+                                l1_gas: 878,
                                 l1_data_gas: 192,
                                 l2_gas: 0,
                             },
@@ -1006,7 +1006,7 @@ pub(crate) mod tests {
                                         l1_gas: 0,
                                         l1_data_gas: 224,
                                     },
-                                l1_gas: 0,
+                                l1_gas: 16,
                                 l1_data_gas: 224,
                                 l2_gas: 0,
                             },
@@ -1417,7 +1417,7 @@ pub(crate) mod tests {
                                         l1_gas: 0,
                                         l1_data_gas: 128,
                                     },
-                                l1_gas: 0,
+                                l1_gas: 12,
                                 l1_data_gas: 128,
                                 l2_gas: 0,
                             },
@@ -1647,7 +1647,10 @@ pub(crate) mod tests {
                             ..Default::default()
                         },
                         execution_resources:
-                            pathfinder_executor::types::InnerCallExecutionResources::default(),
+                            pathfinder_executor::types::InnerCallExecutionResources {
+                                l1_gas: 12840,
+                                ..Default::default()
+                            },
                     }],
                     class_hash: Some(DUMMY_ACCOUNT_CLASS_HASH.0),
                     entry_point_type: pathfinder_executor::types::EntryPointType::External,

diff --git a/crates/rpc/src/method/trace_block_transactions.rs b/crates/rpc/src/method/trace_block_transactions.rs
@@ -786,6 +786,7 @@ pub(crate) mod tests {
             include_state_diffs: true,
         };
 
+        // V07
         pretty_assertions_sorted::assert_eq!(
             output
                 .serialize(Serializer {
@@ -798,6 +799,20 @@ pub(crate) mod tests {
                 })
                 .unwrap(),
         );
+
+        // V08
+        pretty_assertions_sorted::assert_eq!(
+            output
+                .serialize(Serializer {
+                    version: RpcVersion::V08,
+                })
+                .unwrap(),
+            expected
+                .serialize(Serializer {
+                    version: RpcVersion::V08,
+                })
+                .unwrap(),
+        );
         Ok(())
     }