Skip to content

Commit

Permalink
Revert "cmd/evm: allow state dump regardless if test passes in statet…
Browse files Browse the repository at this point in the history
…est (ethereum#28484)"

This reverts commit d362d8e.
  • Loading branch information
devopsbo3 authored Nov 10, 2023
1 parent 5a02597 commit 4093506
Showing 1 changed file with 5 additions and 6 deletions.
11 changes: 5 additions & 6 deletions cmd/evm/staterunner.go
Original file line number Diff line number Diff line change
Expand Up @@ -108,14 +108,13 @@ func runStateTest(fname string, cfg vm.Config, jsonOut, dump bool) error {
fmt.Fprintf(os.Stderr, "{\"stateRoot\": \"%#x\"}\n", root)
}
}
// Dump any state to aid debugging
if dump {
dump := state.RawDump(nil)
result.State = &dump
}
if err != nil {
// Test failed, mark as so
// Test failed, mark as so and dump any state to aid debugging
result.Pass, result.Error = false, err.Error()
if dump {
dump := state.RawDump(nil)
result.State = &dump
}
}
})
results = append(results, *result)
Expand Down

0 comments on commit 4093506

Please sign in to comment.