Skip to content
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

tests: EVM: Exact tx gas usage and gas estimation tests #2597

Merged
merged 4 commits into from
Oct 18, 2023

Conversation

sieniven
Copy link
Contributor

Summary

  • Add new feature_evm_gas test into the e2e tests suite to ensure gas usage functionality and specifying exact gas usage inside gas limit tx attribute.
  • Test EVM balance transfer transactions should succeed when specifying exact gas usage (21_000)
  • Test EVM contract call transactions should suceed when specifying the exact gas usage, obtained with gas estimation RPC

Implications

  • Storage

    • Database reindex required
    • Database reindex optional
    • Database reindex not required
    • None
  • Consensus

    • Network upgrade required
    • Includes backward compatible changes
    • Includes consensus workarounds
    • Includes consensus refactors
    • None

@prasannavl prasannavl changed the title EVM: Add exact tx gas usage and gas estimation tests tests: EVM: Exact tx gas usage and gas estimation tests Oct 18, 2023
@prasannavl prasannavl merged commit 08ecdd7 into master Oct 18, 2023
@prasannavl prasannavl deleted the evm/add-gas-test branch October 18, 2023 05:13
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.

2 participants