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 two tests for address zero + EVM contract bug tests #81

Open
wants to merge 3 commits into
base: integration-more-tests
Choose a base branch
from

Conversation

aamirpashaa
Copy link

@aamirpashaa aamirpashaa commented Feb 3, 2025

This PR addresses these two tests for integration test #23:

  • Withdraw when ZERO has adequate balance (@aamirpashaa this was our second issue on testnet related to ZERO address balance, need to figure out how to replicate the issue in our integration test)
  • Deposit to ZERO address, ensure reth balance is correct after the transaction
  • Create test that exercises our bugs from revision 0 and assert they still fail (get account bug and delegate call bug)

@aamirpashaa aamirpashaa changed the title Add two tests for address zero Add two tests for address zero + EVM contract bug tests Feb 5, 2025
@guilledk guilledk mentioned this pull request Feb 10, 2025
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