fix debugging #445
Merged
fix debugging #445
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gakonst
approved these changes
Jan 13, 2022
gakonst
added a commit
that referenced
this pull request
Jan 13, 2022
* feat(fuzz): expose function to get internal evm * refactor(evm): move EvmOpts from cli to evm-adapters * feat(evm): add helper for creating sputnik backend * feat(forge): add base evm opts for test usage * feat(evm): derive default for EvmOpts * test(forge): add utils for instantiating backend * feat(forge): instantiate runner with EvmOpts instead of an EVM This allows us to instantiate as many EVMs as we want inside of the runner, which in turn will enable running tests in parallel * feat(forge): pass evm by reference instead of using self.evm * feat(forge): run unit tests with unique evm instantiation previously we'd reuse the same EVM, now, we use a different EVM per test, allowing us to get rid of the mutable reference on self * feat(forge): run fuzz tests with unique evm instantiations * test(forge): adjust tests to new instantiation style * feat(forge): run tests in parallel with rayon * feat(evm-adapters): put backend behind enum to avoid trait object * chore(forge): move fuzzer instead of ref * feat(forge): make multi contract runner compatible with new runner * feat(forge): parallelize multi contract runner by file * chore(cli): remove unused helper functions * fix(cli/run): use new contract runner initialization There's a TODO here around how we should do the evm.debug_calls check which we should figure out * fix(cli/test): use evm_opts instead of directly passing evm * chore: formatting fixes * chore: update lockfile * fix(evm-adapters): correctly init test caller and origin fixes #249 fixes #253 * chore: clippy lint on unreachable code w disabled features * fix: instantiate evm cfg without contract size limit * fix debugging (#445) * merge cleanup Co-authored-by: brockelmore <[email protected]> Co-authored-by: Brock <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
add
DebugArenatoTestResult, use the updatedevm_optsthat forces verbosity