Skip to content
This repository has been archived by the owner on Apr 4, 2024. It is now read-only.

Setup more sophisticated testing environment in unit test suite #323

Closed
yihuang opened this issue Jul 20, 2021 · 0 comments · Fixed by #324
Closed

Setup more sophisticated testing environment in unit test suite #323

yihuang opened this issue Jul 20, 2021 · 0 comments · Fixed by #324
Assignees
@yihuang

Comments

@yihuang
Copy link
Contributor

yihuang commented Jul 20, 2021

  1. With the default KeeperTestSuite, GetCoinbaseAddress will raise an error, because of proposer address and validators not initialized
  2. We need to allocate some funds in genesis.

Originally posted by @yihuang in #272 (comment)
@yihuang yihuang self-assigned this Jul 20, 2021
@yihuang yihuang mentioned this issue Jul 20, 2021
Merged
11 tasks
yihuang referenced this issue in yihuang/ethermint Jul 23, 2021
@yihuang
add some unit tests for estimateGas
cad2e88 
Also add some test environment setup, Closes #323

test estimateGas of erc20 token transfer

fix failed test case

the trick is to keep a clean transient store, by doing a commit

put artifacts to external file
fedekunze added a commit that referenced this issue Jul 23, 2021
@yihuang @fedekunze
evm: estimate gas unit tests (#324)
f6dc80d 
* add some unit tests for estimateGas

Also add some test environment setup, Closes #323

test estimateGas of erc20 token transfer

fix failed test case

the trick is to keep a clean transient store, by doing a commit

put artifacts to external file

* fix test failure

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@yihuang yihuang

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add some unit tests for estimateGas yihuang/ethermint
1 participant
@yihuang