Skip to content

Transition types #1038

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

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

Transition types #1038

wants to merge 39 commits into from

Conversation

SamWilsn
Copy link
Contributor

What was wrong?

Header/block types near fork transitions weren't strict enough.

How was it fixed?

Made them stricter.

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

@@ -24,6 +27,7 @@ class NoTestsFound(Exception):


def run_blockchain_st_test(test_case: Dict, load: Load) -> None:
load.fork.fork_criteria = ByBlockNumber(0)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line feels like a hack, but I'm not sure what the correct solution would look like. Open to suggestions!

gurukamath and others added 27 commits January 21, 2025 16:33
* enable testing for all forks

* catch invalid signatures

* refactor validate_transaction

* define accessed_addresses at the start

* update variable name to BLOB_BASE_FEE_UPDATE_FRACTION

* refund_counter definition

* create encode_receipt function

* create process_system_transaction

* update gas calc

* explicitly define code_address in call*

* fix decode_receipt
@SamWilsn SamWilsn marked this pull request as ready for review February 12, 2025 16:00
@SamWilsn SamWilsn mentioned this pull request Feb 12, 2025
@SamWilsn SamWilsn added this to the Post-Prague Re-factoring milestone Feb 12, 2025
SamWilsn and others added 5 commits February 12, 2025 19:50
* refactor blob gas logic

There are parallels between how the regular block gas is handled and how the blob gas is handled. This commit
refactors blob gas canculation to bring them in line with block gas

* create BlockEnvironment and TransactionEnvironment

Create the BlockEnvironment and TransactionEnvironment dataclasses

BlockEnvironment holds data that is chain or block scoped
TransactionEnvironment holds data that is transaction scoped
Message holds data that is specific to a message call

* refactor validate_header

* update t8n

* update evm tracing

* backport changes to cancun

* port to shanghai

* port changes to paris

* port to london

* port berlin

* port istanbul

* port constantinople

* port byzantium

* port spurious_dragon

* port older forks

* fix t8n receipts

* update testing

* process_withdrawals update

* fixes for doc

* remove execution_spec_tests entry

The tests from execution_spec_tests should now be a part of EEST

* use default_factory in BlockOutput

* minor fixes 1

* define current_block_number in block_hash opcode

* rename tx_index to index_in_block

* fix vm test runs
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.

4 participants