Skip to content

chore: small nits in executor from #1674 #1823

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
onbjerg merged 1 commit into from
Mar 17, 2023
Merged

chore: small nits in executor from #1674 #1823

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion crates/executor/src/executor.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -373,6 +373,8 @@ where
/// The changes in [PostState] have a transition ID associated with them: there is one
/// transition ID for each transaction (with the first executed tx having transition ID 0, and
/// so on).
///
/// The second returned value represents the total gas used by this block of transactions.
pub fn execute_transactions(
&mut self,
block: &Block,
Expand Down Expand Up @@ -657,7 +659,11 @@ mod tests {
let mut executor = Executor::new(chain_spec, db);
let post_state = executor.execute_and_verify_receipt(&block, U256::ZERO, None).unwrap();

assert_eq!(post_state.transitions_count(), 2, "Should executed one transaction");
assert_eq!(
post_state.transitions_count(),
2,
"Should executed two transitions (1 tx and 1 block reward)"
);

let block_reward = U256::from(WEI_2ETH + (WEI_2ETH >> 5));

Expand Down