ethereum / ethereum/tests

`expectException`: Blob count vs blob gas allowance

Open
#1,456 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
626
Forks
364
PR merge metrics
No merged PRs in 30d

Description

To me, the blocks in these two tests seem invalid for the same reason, but they expect different exceptions:
- [blobhashListBounds7_d0g0v0_Cancun](https://github.com/ethereum/tests/blob/b4060bf59d71b7f8169b8f759235da72eec2d59f/BlockchainTests/GeneralStateTests/Cancun/stEIP4844-blobtransactions/blobhashListBounds7.json#L2)
- [src/GeneralStateTestsFiller/Pyspecs/cancun/eip4844_blobs/test_blob_txs.py::test_invalid_block_blob_count[fork_Cancun-blobs_per_tx_(7,)-blockchain_test-]](https://github.com/ethereum/tests/blob/b4060bf59d71b7f8169b8f759235da72eec2d59f/BlockchainTests/GeneralStateTests/Pyspecs/cancun/eip4844_blobs/invalid_block_blob_count.json#L5546)

Each has one transaction with 7 `blobVersionedHashes` so I think they should have `TYPE_3_TX_BLOB_COUNT_EXCEEDED` as the `expectException`, but the second one expects a `TYPE_3_TX_MAX_BLOB_GAS_ALLOWANCE_EXCEEDED` instead.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.