refactor: use same code path and config for Solidity test runner integration tests #637
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR achieves to goal set out in #607 to use same code path and config for Solidity test runner integration tests. It does this by
MultiContractRunner::new
).SolidityTestRunnerConfig
that previously resided inedr_napi
, instead offoundry_config::Config
.foundry_config::Config
has been renamed tofoundry_config::IntegrationTestConfig
as some fields of it are still needed to build the Rust integration tests as a Foundry project. The rest of its functionality has been removed. I kept thefoundry_config
crate for now to be able to track the changes.The PR doesn't contain any non-trivial code changes, so for me the most helpful would be to review it to see if anything has been removed that shouldn't be removed.