Skip to content

feat: implemented initialization of evm_opts and env in TestRunnerCon… #10542

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 3 commits into
base: master
Choose a base branch
from

Conversation

MozirDmitriy
Copy link

…fig and their use in configure_executor

Motivation

TODO

Solution

  • Updated evm_opts and env fields in reconfigure_with method based on new configuration (Config), which allows to correctly apply inline-config and update EVM environment.
  • The configure_executor method now explicitly applies env and evm_opts to the Executor: environment, gas limit, create2_deployer and other parameters are set.
  • Removed the TODO comments associated with these fields and implemented the actual logic.
  • This ensures that the test runner works correctly when changing the configuration on the fly and eliminates possible inconsistencies between the settings and the state of the EVM.

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

1 participant