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

Fix 2/7 failing test cases by getting latest block #99

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

brettfazio
Copy link

A full-node running without archive mode will prune generated state to conserve disk space. This helps with the sync time of a node and greatly reduces storage and computation cost. Because of the way account and contract storage works in Ethereum, this means that only an archive node can serve API request for certain RPC methods older than 128 blocks. thus the reason for these cases failing before.

@brettfazio
Copy link
Author

I suppose the problem is accurately asserting the balance like here since the block number will be a different one every time.

But I do believe this new solution to be better as all the tests were failing before.

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

Successfully merging this pull request may close these issues.

1 participant