chore: fix tests after the gas changes #341
Merged
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
* changed evm_setNextBlockTimestamp argument type to U64 * fixed caller of evm_setNextBlockTimestamp * setting the timestamp of the next block in evm_setNextBlockTimestamp * formatted * updated comment
* not panicking in is_write_initial_internal * not panicking in fork_network_and_client * not panicking in get_earlier_transactions_in_same_block * not panicking in get_block_gas_details * Update src/fork.rs Co-authored-by: AnastasiiaVashchuk <72273339+AnastasiiaVashchuk@users.noreply.github.com> * more specific error message * more specific error message * added miniblock number to error message --------- Co-authored-by: AnastasiiaVashchuk <72273339+AnastasiiaVashchuk@users.noreply.github.com>
* added evm_setAccountNonce alias * added anvil_setNonce alias * added anvil_impersonateAccount & anvil_stopImpersonatingAccount aliases * added anvil_mine alias * added anvil_reset alias * added anvil_setBalance alias * added anvil_setCode alias * added anvil_setStorageAt alias * added anvil_* aliases * added anvil_mine test * modified hardhat_impersonateAccount & hardhat_stopImpersonatingAccount to actually call hardhat_stopImpersonatingAccount, interfere less with other tests * added anvil_impersonateAccount & anvil_stopImpersonatingAccount test * added anvil_setCode test * added hardhat_setStorageAt and anvil_setStorageAt tests * formatted * removed links to anvil namespace * split tables * Revert "split tables" This reverts commit 4a54fe8. * renamed AnvilNamespaceT::hardhat_mine to anvil_mine * Update SUPPORTED_APIS.md Co-authored-by: AnastasiiaVashchuk <72273339+AnastasiiaVashchuk@users.noreply.github.com> * added section for evm_setAccountNonce * changed var to let to satisfy linter * using random address in anvil_setCode test * added semicolon * named test account address * changed var to let to satisfy linter * Update src/namespaces/anvil.rs Co-authored-by: AnastasiiaVashchuk <72273339+AnastasiiaVashchuk@users.noreply.github.com> * removed to.be.within checks * supressing TypeScript errors * supressing TypeScript errors * changed anvil_setCode code type * using find instead of an explicit loop --------- Co-authored-by: AnastasiiaVashchuk <72273339+AnastasiiaVashchuk@users.noreply.github.com>
* add test and do follow up fixes * run lint * fix nits
* fix: fork gas estimation * Additional fixes
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.
What 💻
Fix tests after gas change
Why ✋
Because 30 tests fail