evmhooks and tests #68
Merged
Conversation
This file contains hidden or 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
…on TestRegisterTokenPairWithContract
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
fedekunze
reviewed
Oct 27, 2021
fedekunze
reviewed
Oct 27, 2021
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
fedekunze
added a commit
that referenced
this pull request
Nov 17, 2021
* irm * events and comments * tests wip * comments & genesis * grpc queries * mint test * abi added * Add TestGetTokenPairID * testing: irm type tests (#57) * Validate genesis fix. genesis and params tests * initial structure for msg and proposal tests * testing: proposal tests addeds * testing: update token pair erc20 with invalid addresses * token pair and msgs test Co-authored-by: ramacarlucho <ramirocarlucho@gmail.com> * WIP TestGetAllTokenPairs * Testing (TokenPairs): Add tests for full coverage and fix GrpcQuery test * WIP Keepertests(TestRegisterTokenPair) * add missing fields for metadata * Add Keepertests(TestRegisterTokenPair) * add Keepertests(TestEnabledRelay) * add Keepertests(UpdateTokenPairERC20) * WIP goc_hooks tests * gov hooks tests wip * WIP Keeper tests(AfterProposalDeposit) * wip: refactor 2 * hook wip * fix ABI and integrations (#66) * merge irms branchs * Fix pack method * removed valid test from TestRegisterTokenPair, this should be tested on TestRegisterTokenPairWithContract * using ethermint CompiledContract * removed unused import * unpacking using interfaces * Update x/intrarelayer/types/contracts/erc20.go Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * moved evm types to types folder * feemarket enabled by default on test suite * contracts function and test setup moved to keeper_test Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * WIP(Keeper tests): gov_hooks_test, msg_server_test, proposals_test * vbank module verification after token pair creation * EnableRelay test added * wip update token function * WIP (Keepertests) msg_server tests * fix panic on test * Add SetDenomMap and SetERC20Map to RegisterTokenPair * force dynamic fee * wip update token pair * keeper: evm hooks and tests (#68) * merge irms branchs * Fix pack method * removed valid test from TestRegisterTokenPair, this should be tested on TestRegisterTokenPairWithContract * using ethermint CompiledContract * removed unused import * unpacking using interfaces * wip evmhooks tests * evmhooks working * Update x/intrarelayer/types/contracts/erc20.go Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * moved evm types to types folder * feemarket enabled by default on test suite * contracts function and test setup moved to keeper_test * check correct execution * more test cases * pr suggestions * pr suggestions * Apply suggestions from code review Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Hanchon <guillermo.paoletti@gmail.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * force dynamic fee * merge rama changes * proposal tests green * proposal tests updated * add setupNewTokenPair to msg_server_test * dynamic fee fix * WIP(Intrarelayer): TestUpdateTokenPairERC20 * WIP(Intrarelayer): TestUpdateTokenPairERC20 delete non TDD test * WIP(Intrarelayer): Refactor TestRegisterTokenPair to use TDD and add all missing cases * WIP(Intrarelayer): Refactor TestToggleRelay to use TDD and add all missing cases * share * fixes to minting from module * cleanup keeper test suite * tests: irm proposal tests (#69) * vbank module verification after token pair creation * EnableRelay test added * wip update token function * force dynamic fee * wip update token pair * force dynamic fee * merge rama changes * proposal tests green * proposal tests updated * dynamic fee fix * WIP(Intrarelayer): TestUpdateTokenPairERC20 * WIP(Intrarelayer): TestUpdateTokenPairERC20 delete non TDD test * WIP(Intrarelayer): Refactor TestRegisterTokenPair to use TDD and add all missing cases * pr changes * Apply suggestions from code review Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * enable to toggle rename Co-authored-by: ramacarlucho <ramirocarlucho@gmail.com> Co-authored-by: Daniel Burckhardt <daniel.m.burckhardt@gmail.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> * fail early * burn and mint fixes. Send tx module address as burn test * calling evm directly * sending evm transactions using module address * transfering tokens using evmcall * Wip: Refactor Coin/Token proposal * wip: tests compiling, not working * Wip: register coin and register token proposals * wip: missing contract deployment and tests * wip evmhooks working * escrow fix on converterc20 * initial msg_server_tests working * tests fixes * changed erc20 metadata. RegisterErc20 token tests working * wip deploy contract * IRM(new Specs): Split EventRegiesterTokenPair into two Events Coin & ERC20 * IRM(new Specs): Refactor with isNativeCoin() and isNavtiveERC20() methods * IRM(new Specs): Add tests for isNativeCoin() and isNavtiveERC20() methods * deployment first approach working * Intrarelayer(Tests): Add TestRegisterCoin * moduleAccount nonce. ConvertToken fixes * cleanup * rm nonce key * rm nonce key.go * validate genesis state * err * IRM(new Specs): Add tests for token_pairs delete methods * SDK fork * fix log read on hook. Use simple call to deploy contract * delete unnecessary functions. cleanup * convert coins/tokens cases * evm hooks register coin case * Intrarelayer(new Specs): Add more types tests * Intrarelayer(new Specs): Add EnableEVMHook Parameter with default true * Intrarelayer(new Specs): Add types msg tests * Intrarelayer(new Specs): Add evm & genesis tests * Intrarelayer(new Specs): Add last types/proposal tests * gov hooks fixes and testing * minor changes * linter fixes * Intrarelayer(new Specs): Refactor Msg_server methods * nolint * Intrarelayer(new Specs): Refactor msg_server tests * convert erc20 event * missing Evm executions checks * Intrarelayer(new Specs): Add keeper/evm tests * function comments added * use ERC20 with custom burn Co-authored-by: Federico Kunze <federico.kunze94@gmail.com> Co-authored-by: jolube <64335177+jolube@users.noreply.github.com> Co-authored-by: Daniel Burckhardt <daniel.m.burckhardt@gmail.com> Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Guillermo Paoletti <guillermo.paoletti@gmail.com>
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.
Closes: #XXX
Description
Evm hooks fixes and initial test.
For contributor use:
docs/) or specification (x/<module>/spec/)godoccomments.Unreleasedsection inCHANGELOG.mdFiles changedin the Github PR explorerFor admin use:
WIP,R4R,docs, etc)