Skip to content

Commit 475e87c

Browse files
authored
cmd/evm: fix statetest with multi-fork statetest fixtures (ethereum#31374)
Fixes `evm statetest` for state test fixtures with multiple fork entries in their `post` field (e.g., [chainId.json](https://github.com/ethereum/tests/blob/81862e4848585a438d64f911a19b3825f0f4cd95/GeneralStateTests/stChainId/chainId.json#L39)). When these re-activated flags aren't exposed, `statetest` only executes the fixture for a single fork entry instead of all of the forks as expected. This only affects ethereum/tests state test fixtures, not ethereum/execution-spec-tests (EEST) state tests. EEST writes a separate fixture/test case (i.e. a separate top-level dict entry in the .json) for each fork configuration as apposed to combining multiple forks in one fixture test case: New EEST state tests targeting Prague behavior are not affected.
1 parent 51177ed commit 475e87c

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

cmd/evm/staterunner.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,9 @@ var stateTestCommand = &cli.Command{
5353
Flags: slices.Concat([]cli.Flag{
5454
BenchFlag,
5555
DumpFlag,
56+
forkFlag,
5657
HumanReadableFlag,
58+
idxFlag,
5759
RunFlag,
5860
}, traceFlags),
5961
}

0 commit comments

Comments
 (0)