status-im / nim-eth-verkle

Verkle tree implementation
Apache License 2.0
8 stars 3 forks source link

Testing the Verkle Trie Implementation #16

Closed advaita-saha closed 10 months ago

advaita-saha commented 10 months ago

Testing

The current Verkle Trie implementation is capable of all the trie operations but does not support the Stateless feature, it is yet to be added.

Tests

The correctness of the tree root commiment have already been tested w.r.t go-verkle For testing the on the broader perspective we are supposed to test the follows

For test vectors, it can be generated on our own also few can be taken from the go-verkle implementation from tree_test.go.

advaita-saha commented 10 months ago

Testing Results

Screenshot 2024-01-20 at 10 46 46 PM

Errors

All the operations are working perfectly fine from the correctness point of view. Except for the deletion of values with similar key path, for example let the tree have the following keys inserted with some values

daniellm commented 10 months ago

All the operations are working perfectly fine from the correctness point of view. Except for the deletion of values with similar key path, for example let the tree have the following keys inserted with some values

Fixed in 6f5784ad6498c8433948e94dae3eec16efa98f03

advaita-saha commented 10 months ago

All tests passing

Screenshot 2024-01-21 at 1 14 29 PM

The above tests are executed successfully in comparison the the go-verkle codebase