-
Notifications
You must be signed in to change notification settings - Fork 72
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
feat(fw): Changes required for Devnet-4 Tests #888
Conversation
8b76a55
to
6ef9589
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Loving the precompile and system contract enums and the simplification this brings to forks.
I would suggest renaming this package from ethereum_test_addresses
to ethereum_addresses
as it's not really test specific.
Few comments below mainly regarding naming. At the cost of duplication, I would write the current version of system contract addresses in test Spec
classes for ease of verification in the docs (see screenshot in comment below).
tests/prague/eip2935_historical_block_hashes_from_state/spec.py
Outdated
Show resolved
Hide resolved
c60dbfa
to
2c5594b
Compare
Scrapped |
fixup: BlockchainEngine changelog fixup(docs): BlockchainTestEngine docs Co-authored-by: danceratopz <danceratopz@gmail.com>
2c5594b
to
818c87f
Compare
🗒️ Description
This PR can be best reviewed (and also merged via any method that is not Squash) by commit.
Allow Bytes(), add sha256 method
Allow instantiating an empty
Bytes
instance by calling it with no parameters, and also add asha256
method to allBytes
instances, which returns aHash
instance containing the sha256 digest.Allow instantiating Address with a label
Adds keyword argument
label
toAddress
init method to allow it being instantiated with a label.Fix FixedSizeBytes comparison
Minor fix to
FixedSizeBytes
__eq__
method to fix casting of theother
element under comparison, and allow comparing againstNone
without raising an exception.Add BlockchainTestEngine
Adds a
BlockchainTestEngine
spec which has all the same properties asBlockchainTest
except that it only generates a blockchain test engine version fixture.Allow
nonce
infund_eoa
Allows specifying a starting nonce for the funded EOA, which could only be achieved during fill but not on execution.
Allow Bytes in Initcode
Minor fix to allow deploying arbitrary bytes in an Initcode.
🔗 Related Issues
None
✅ Checklist
mkdocs serve
locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.