Skip to content

Commit

Permalink
fix for unit tests: do not call NewAccessWitness in NewEVMTxContext (#49
Browse files Browse the repository at this point in the history
)

* potential fix: do not call NewAccessWitness in NewEVMTxContext

* more fixes: check for the existence of Accesses

* fix absence of witness in copy

* fix another witness issue

* workaround: ensure the prefetcher is off in verkle mode

* fix the remaining issues in tests

* review feedback

* fix witness allocation in stateless test
  • Loading branch information
gballet committed Jan 10, 2022
1 parent 79fe00f commit ed11f41
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions miner/worker_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -267,10 +267,7 @@ func testGenerateBlockAndImport(t *testing.T, isClique bool) {
if _, err := chain.InsertChain([]*types.Block{block}); err != nil {
t.Fatalf("failed to insert new mined block %d: %v", block.NumberU64(), err)
}
// TODO(gballet) the timeout had to be increased from 3s to 7s with verkle
// trees, presumably because calculating an address is orders of magnitude
// slower with perdersen_hash not using the multi-exponentiation.
case <-time.After(5 * time.Second): // Worker needs 1s to include new changes.
case <-time.After(3 * time.Second): // Worker needs 1s to include new changes.
t.Fatalf("timeout")
}
}
Expand Down

0 comments on commit ed11f41

Please sign in to comment.