forge's DAPP_TEST_ADDRESS == dapp's DAPP_TEST_CALLER
#253
Labels
T-bug
Type: bug
Comments
brockelmore
changed the title
forge's DAPP_TEST_ADDRESS != dapp's DAPP_TEST_ADDRESSforge's DAPP_TEST_ADDRESS == dapp's DAPP_TEST_CALLER
This was referenced Jan 12, 2022
gakonst
added a commit
that referenced
this issue
Jan 13, 2022
|
@onbjerg confirming that this is correct to be closed right? |
|
Yes, this is correct |
charisma98
added a commit
to charisma98/foundry
that referenced
this issue
Mar 4, 2023
* 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 foundry-rs/foundry#249 fixes foundry-rs/foundry#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]>
0129general
added a commit
to 0129general/FoundryProject
that referenced
this issue
May 8, 2024
* 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 foundry-rs/foundry#249 fixes foundry-rs/foundry#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
Related to #249,
forge'sDAPP_TEST_ADDRESSis reallydapp'sDAPP_TEST_ORIGIN/DAPP_TEST_CALLER.dapp'sDAPP_TEST_ADDRESSis the address of the test contract not the caller.Likely the fix for #249 should include this fix as well
The text was updated successfully, but these errors were encountered: