mirror of
https://github.com/ethereum/go-ethereum.git
synced 2026-05-14 03:56:36 +00:00
cmd/evm: fix statetest with multi-fork statetest fixtures (#31374)
Fixes `evm statetest` for state test fixtures with multiple fork entries
in their `post` field (e.g.,
[chainId.json](81862e4848/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.
This commit is contained in:
parent
51177ed8c5
commit
475e87cbc1
1 changed files with 2 additions and 0 deletions
|
|
@ -53,7 +53,9 @@ var stateTestCommand = &cli.Command{
|
||||||
Flags: slices.Concat([]cli.Flag{
|
Flags: slices.Concat([]cli.Flag{
|
||||||
BenchFlag,
|
BenchFlag,
|
||||||
DumpFlag,
|
DumpFlag,
|
||||||
|
forkFlag,
|
||||||
HumanReadableFlag,
|
HumanReadableFlag,
|
||||||
|
idxFlag,
|
||||||
RunFlag,
|
RunFlag,
|
||||||
}, traceFlags),
|
}, traceFlags),
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue