Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[HQC-50] Increase test coverage for erc20/cli and evm/cli #271

Merged
merged 4 commits into from
Jan 5, 2024

Conversation

ivanovpetr
Copy link
Contributor

No description provided.

@kioqq kioqq changed the title [WIP][HQC-50] Increase test coverage for erc20/cli and evm/cli [HQC-50] Increase test coverage for erc20/cli and evm/cli Jan 5, 2024
@kioqq kioqq merged commit d63697b into sdk-upgrade Jan 5, 2024
12 checks passed
kodiakhq bot pushed a commit that referenced this pull request Jan 8, 2024
…7.3.1 (#262)

* chore: remove old upgrades

* build(upgrade)!: upgrade cosmos sdk, cometbft and ibc versions

BREAKING CHANGE:
Some modules have been changed due to SDK upgrade (v0.47) as old initialization and structure is not compatible with new SDK.
All references to Tendermint have been replaced with CometBFT. Cosmos SDK v0.47+ requires CometBFT v0.37+ and does not support Tendermint anymore.
IBC version has been bumped due to SDK upgrade.

* chore: update buf dependencies

* fix: fixes due to recent rebase

* fix: remove default value for chain-id flag

* test: add 2 txs erc20 spend test for bank keeper

* fix(test): fix tx sequence in bank keeper test

* chore: update swagger

* chore: bump go version

* chore: clean errors

* [HQC-50] Increase test coverage for erc20/cli and evm/cli (#271)

* add query tests
* add tx tests

---------

Co-authored-by: Petr Ivanov <petr.ivanov@tendermint.com>

* fix: keys migration on upgrade

---------

Co-authored-by: Evgeniy Abramov <i@kio.ninja>
Co-authored-by: Petr Ivanov <krest2005@yandex.ru>
Co-authored-by: Petr Ivanov <petr.ivanov@tendermint.com>
Co-authored-by: Evgeniy Abramov <5331145+kioqq@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants