search

ethereum / go-verkle

A go implementation of Verkle trees
The Unlicense
209 stars 63 forks source link

TreeFromProof test hardening #356

Closed jsign closed 8 months ago

jsign commented 1 year ago

This PR adds more test coverage for PreStateTreeFromProof(...).

The diff is very messy, so I recommend some "split view" maybe.

gballet commented 1 year ago

@jsign right, the reason keyvals is passed, is because I used to make the difference between "previous" values (held by keyvals) and "current values" (held by the tree). This isn't something that we are doing at the moment, but it will come back since we want to put the post state in the tree. We can figure out the proper way to do this when this happens.