Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add TrieWalker tests #12034

Open
fgimenez opened this issue Oct 24, 2024 · 0 comments · May be fixed by #12047
Open

Add TrieWalker tests #12034

fgimenez opened this issue Oct 24, 2024 · 0 comments · May be fixed by #12047
Assignees
@fgimenez
Labels
A-trie Related to Merkle Patricia Trie implementation C-test A change that impacts how or what we test

Comments

@fgimenez
Copy link
Member

Describe the feature

TrieWalker accepts a cursor to walk over, we should have tests for hashed storage and hashed account cursors. The test should include two variants:

  • where hashed cursor wraps a noop cursor
  • where hased cursor wraps a database cursor

/cc @rkrasiuk

Additional context

No response
@fgimenez fgimenez added C-test A change that impacts how or what we test A-trie Related to Merkle Patricia Trie implementation labels Oct 24, 2024
@fgimenez fgimenez self-assigned this Oct 24, 2024
@fgimenez fgimenez linked a pull request Oct 24, 2024 that will close this issue
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fgimenez fgimenez

Labels
A-trie Related to Merkle Patricia Trie implementation C-test A change that impacts how or what we test
Projects
Reth Tracker
Status: In Progress
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(test): add TrieWalker tests paradigmxyz/reth
2 participants
@fgimenez and others