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.

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

Add a state comparison for blockchain tests if they fail in certain contexts #2154

Open
fselmo opened this issue Mar 14, 2024 · 0 comments
Open

Comments

@fselmo
Copy link
Collaborator

fselmo commented Mar 14, 2024

If the only block header field mismatch is the state root, use the state.meta_witness and the postState of the test fixture to try to reconcile where there may be discrepancies. If running verify_state yields a success, it means we likely have an extra account in our state that isn't present in the postState of the test. Inspecting the meta_witness.accounts_queried can help elucidate if there are any extra accounts in the state that may need to be marked for deletion, thus giving us a different state root even though everything in the postState is accurate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant