Skip to content

Conversation

@marioevz
Copy link
Member

@marioevz marioevz commented Dec 6, 2023

🗒️ Description

Fixed tests/cancun/eip4844_blobs/test_blob_txs.py:test_invalid_tx_max_fee_per_blob_gas to account for extra gas required in the case where the account is incorrectly deduced the balance as if it had the correct block blob gas fee

🔗 Related Issues

Partially addresses #343

✅ Checklist

  • All: Set appropriate labels for the changes.
  • All: Considered squashing commits to improve commit history.
  • All: Added an entry to CHANGELOG.md.
  • All: Considered updating the online docs in the ./docs/ directory.
  • Tests: Included the type and version of evm t8n tool used to locally execute test cases: e.g., ref with commit hash or geth 1.13.1-stable-3f40e65.
  • Tests: Ran mkdocs serve locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.

@marioevz marioevz added the scope:tests Scope: Changes EL client test cases in `./tests` label Dec 6, 2023
@marioevz
Copy link
Member Author

marioevz commented Jan 6, 2024

Added to #370.

@marioevz marioevz closed this Jan 6, 2024
@marioevz marioevz deleted the fix-invalid-blob-gas-fee-test branch January 6, 2024 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope:tests Scope: Changes EL client test cases in `./tests`

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant