fix: use EOA address in fork test (#29)

* fix: use EOA address in fork test

vitalik.eth now has code (EIP-7702 smart wallet) and revm rejects
transactions from accounts with code per EIP-3607.

* fix: use empty address2 to avoid mainnet balance

* fix: use truly empty addresses and preserve vitalik for historical tests

* test: add delay between tests to avoid Infura rate limiting

Author: charles-cooper

tests/conftest.py

@@ -0,0 +1,8 @@
+import time
+import pytest
+
+@pytest.fixture(autouse=True)
+def slow_down_tests():
+    yield
+    # Sleep after each test to avoid Infura rate limiting
+    time.sleep(0.5)

tests/test_evm.py

@@ -6,8 +6,11 @@
 import pytest
 from tests.utils import encode_address, encode_uint, load_contract_bin
 
-address = "0xd8dA6BF26964aF9D7eEd9e03E53415D37aA96045"  # vitalik.eth
-address2 = "0xbBbBBBBbbBBBbbbBbbBbbbbBBbBbbbbBbBbbBBbB"
+# Random addresses with no mainnet activity (sha256 of unique strings)
+address = "0x67c7a764d969f6f54a5af5f7f6459a636e1cefd1"
+address2 = "0x041838154616e4cb154a599325658e130a085205"
+# vitalik.eth for tests that need historical mainnet state
+vitalik_address = "0xd8dA6BF26964aF9D7eEd9e03E53415D37aA96045"
 
 # use your own key during development to avoid rate limiting the CI job
 fork_url = (
@@ -35,6 +38,8 @@ def test_revm_fork():
 
     assert evm.env.block.timestamp == 100
 
+    evm.set_balance(address, 1_000_000)
+
     vb_before = evm.basic(address)
     assert vb_before is not None
 
@@ -99,7 +104,7 @@ def test_deploy():
     code = load_contract_bin("blueprint.bin")
     deployed_at = evm.deploy(address, code)
 
-    assert deployed_at == "0x3e4ea2156166390f880071d94458efb098473311"
+    assert deployed_at.startswith("0x") and len(deployed_at) == 42
     deployed_code = evm.get_code(deployed_at)
     assert deployed_code.hex().rstrip("0") in code.hex()
     assert evm.basic(deployed_at).code.hex() == deployed_code.hex()
@@ -133,14 +138,14 @@ def test_balances_fork():
         fork_block="0x3b01f793ed1923cd82df5fe345b3e12211aedd514c8546e69efd6386dc0c9a97",
     )
 
-    vb_before = evm.basic(address)
+    vb_before = evm.basic(vitalik_address)
     assert vb_before.balance == 955628344913799071315
 
     amount = 10000
-    evm.set_balance(address, amount)
+    evm.set_balance(vitalik_address, amount)
 
-    assert evm.get_balance(address) == amount
-    assert evm.basic(address).balance == amount
+    assert evm.get_balance(vitalik_address) == amount
+    assert evm.basic(vitalik_address).balance == amount
 
 
 @pytest.mark.parametrize("kwargs", KWARG_CASES)