ethereum / ethereum/execution-specs

blockchain_test_engine bugs in tests-focil-devnet@v0.2.0: IL transaction and blobGasUsed

Open
#3,435 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
1.2k
Forks
505
Avg merge
2d 17h
Merged PRs (30d)
117

Description

"tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_blob_hash_versioning_multiple_txs[fork_Bogota-blockchain_test_engine--single_blob" has 2 transactions and the `blobGasUsed` is 0x40000.

But it's IL companion test only have one transaction in the block, and the other one is in IL, but the `blobGasUsed` is still 0x40000.
"tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_blob_hash_versioning_multiple_txs[fork_Bogota-blockchain_test_engine_inclusion_list--single_blob]"

fixtures with same issue:
```
tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_blob_hash_versioning_multiple_txs[fork_Bogota-blockchain_test_engine_inclusion_list--multiple_blobs] from blockchain_tests_engine\for_bogota\cancun\eip4844_blobs\blob_txs\invalid_blob_hash_versioning_multiple_txs.json
tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_blob_hash_versioning_multiple_txs[fork_Bogota-blockchain_test_engine_inclusion_list--multiple_blobs_single_bad_hash_1] from blockchain_tests_engine\for_bogota\cancun\eip4844_blobs\blob_txs\invalid_blob_hash_versioning_multiple_txs.json
tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_blob_hash_versioning_multiple_txs[fork_Bogota-blockchain_test_engine_inclusion_list--multiple_blobs_single_bad_hash_2] from blockchain_tests_engine\for_bogota\cancun\eip4844_blobs\blob_txs\invalid_blob_hash_versioning_multiple_txs.json
tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_blob_hash_versioning_multiple_txs[fork_Bogota-blockchain_test_engine_inclusion_list--single_blob] from blockchain_tests_engine\for_bogota\cancun\eip4844_blobs\blob_txs\invalid_blob_hash_versioning_multiple_txs.json
tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_blob_hash_versioning_single_tx[fork_Bogota-blockchain_test_engine_inclusion_list_from_state_test--multiple_blobs] from blockchain_tests_engine\for_bogota\cancun\eip4844_blobs\blob_txs\invalid_blob_hash_versioning_single_tx.json
tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_blob_hash_versioning_single_tx[fork_Bogota-blockchain_test_engine_inclusion_list_from_state_test--multiple_blobs_single_bad_hash_1] from blockchain_tests_engine\for_bogota\cancun\eip4844_blobs\blob_txs\invalid_blob_hash_versioning_single_tx.json
tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_blob_hash_versioning_single_tx[fork_Bogota-blockchain_test_engine_inclusion_list_from_state_test--multiple_blobs_single_bad_hash_2] from blockchain_tests_engine\for_bogota\cancun\eip4844_blobs\blob_txs\invalid_blob_hash_versioning_single_tx.json
tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_blob_hash_versioning_single_tx[fork_Bogota-blockchain_test_engine_inclusion_list_from_state_test--single_blob] from blockchain_tests_engine\for_bogota\cancun\eip4844_blobs\blob_txs\invalid_blob_hash_versioning_single_tx.json
tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_block_blob_count[fork_Bogota-blobs_per_tx_(1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1)-blockchain_test_engine_inclusion_list-] from blockchain_tests_engine\for_bogota\cancun\eip4844_blobs\blob_txs\invalid_block_blob_count.json
tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_block_blob_count[fork_Bogota-blobs_per_tx_(1,21)-blockchain_test_engine_inclusion_list-] from blockchain_tests_engine\for_bogota\cancun\eip4844_blobs\blob_txs\invalid_block_blob_count.json
tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_block_blob_count[fork_Bogota-blobs_per_tx_(22,)-blockchain_test_engine_inclusion_list-] from blockchain_tests_engine\for_bogota\cancun\eip4844_blobs\blob_txs\invalid_block_blob_count.json
tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_block_blob_count[fork_Bogota-blobs_per_tx_(6,16)-blockchain_test_engine_inclusion_list-] from blockchain_tests_engine\for_bogota\cancun\eip4844_blobs\blob_txs\invalid_block_blob_count.json
tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_block_blob_count[fork_Bogota-blobs_per_tx_(6,6,5,5)-blockchain_test_engine_inclusion_list-] from blockchain_tests_engine\for_bogota\cancun\eip4844_blobs\blob_txs\invalid_block_blob_count.json
tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_block_blob_count[fork_Bogota-blobs_per_tx_(6,6,6,4)-blockchain_test_engine_inclusion_list-] from blockchain_tests_engine\for_bogota\cancun\eip4844_blobs\blob_txs\invalid_block_blob_count.json
tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_normal_gas[fork_Bogota-blockchain_test_engine_inclusion_list_from_state_test-insufficient_max_fee_per_gas] from blockchain_tests_engine\for_bogota\cancun\eip4844_blobs\blob_txs\invalid_normal_gas.json
tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_tx_blob_count[fork_Bogota-too_many_blobs-blockchain_test_engine_inclusion_list_from_state_test] from blockchain_tests_engine\for_bogota\cancun\eip4844_blobs\blob_txs\invalid_tx_blob_count.json
tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_tx_max_fee_per_blob_gas[fork_Bogota-insufficient_max_fee_per_blob_gas-blockchain_test_engine_inclusion_list-account_balance_modifier_1000000000] from blockchain_tests_engine\for_bogota\cancun\eip4844_blobs\blob_txs\invalid_tx_max_fee_per_blob_gas.json
tests/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_tx_max_fee_per_blob_gas[fork_Bogota-invalid_max_fee_per_blob_gas-blockchain_test_engine_inclusion_list-account_balance_modifier_1000000000] from blockchain_tests_engine\for_bogota\cancun\eip4844_blobs\blob_txs\invalid_tx_max_fee_per_blob_gas.json
```

Contributor guide

Open the contributing guide

Research direction

Start with tests/cancun/eip4844_blobs/test_blob_txs.py and the referenced JSON fixtures under blockchain_tests_engine/for_bogota/cancun/eip4844_blobs/blob_txs/. Run the listed inclusion-list cases and compare transaction placement with the reported blobGasUsed values. Done means the affected blockchain_test_engine inclusion-list tests consistently report the expected blobGasUsed for transactions in the block and in the inclusion list.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
blockchain, testing-qa
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.