Finschia / finschia-sdk

A framework for building blockchains based Finschia Mainnet that is forked from cosmos-sdk
Apache License 2.0
61 stars 31 forks source link

test: add cli_test for fswap module #1391

Closed 170210 closed 6 months ago

170210 commented 6 months ago

Description

closes: #XXXX

Motivation and context

How has this been tested?

Screenshots (if appropriate):

Checklist:

codecov[bot] commented 6 months ago

Codecov Report

Attention: Patch coverage is 99.57082% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 70.05%. Comparing base (e7c6501) to head (4a4938c). Report is 2 commits behind head on main.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1391/graphs/tree.svg?width=650&height=150&src=pr&token=m16qfzIPO7&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia)](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1391?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia) ```diff @@ Coverage Diff @@ ## main #1391 +/- ## ========================================== + Coverage 69.57% 70.05% +0.47% ========================================== Files 668 672 +4 Lines 56231 56941 +710 ========================================== + Hits 39125 39890 +765 + Misses 14835 14776 -59 - Partials 2271 2275 +4 ``` | [Files](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1391?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia) | Coverage Δ | | |---|---|---| | [x/fswap/client/testutil/query.go](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1391?src=pr&el=tree&filepath=x%2Ffswap%2Fclient%2Ftestutil%2Fquery.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia#diff-eC9mc3dhcC9jbGllbnQvdGVzdHV0aWwvcXVlcnkuZ28=) | `100.00% <100.00%> (ø)` | | | [x/fswap/client/testutil/suite.go](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1391?src=pr&el=tree&filepath=x%2Ffswap%2Fclient%2Ftestutil%2Fsuite.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia#diff-eC9mc3dhcC9jbGllbnQvdGVzdHV0aWwvc3VpdGUuZ28=) | `100.00% <100.00%> (ø)` | | | [x/fswap/client/testutil/tx.go](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1391?src=pr&el=tree&filepath=x%2Ffswap%2Fclient%2Ftestutil%2Ftx.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia#diff-eC9mc3dhcC9jbGllbnQvdGVzdHV0aWwvdHguZ28=) | `100.00% <100.00%> (ø)` | | | [x/fswap/client/testutil/grpc.go](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1391?src=pr&el=tree&filepath=x%2Ffswap%2Fclient%2Ftestutil%2Fgrpc.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia#diff-eC9mc3dhcC9jbGllbnQvdGVzdHV0aWwvZ3JwYy5nbw==) | `98.15% <98.15%> (ø)` | | ... and [4 files with indirect coverage changes](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1391/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia)
codecov-commenter commented 6 months ago

Codecov Report

Attention: Patch coverage is 99.51691% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 70.01%. Comparing base (6ca08db) to head (e4a51f9). Report is 1 commits behind head on main.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1391/graphs/tree.svg?width=650&height=150&src=pr&token=m16qfzIPO7&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia)](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1391?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia) ```diff @@ Coverage Diff @@ ## main #1391 +/- ## ========================================== + Coverage 69.68% 70.01% +0.32% ========================================== Files 668 672 +4 Lines 56242 56863 +621 ========================================== + Hits 39194 39813 +619 Misses 14775 14775 - Partials 2273 2275 +2 ``` | [Files](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1391?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia) | Coverage Δ | | |---|---|---| | [x/fswap/client/testutil/query.go](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1391?src=pr&el=tree&filepath=x%2Ffswap%2Fclient%2Ftestutil%2Fquery.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia#diff-eC9mc3dhcC9jbGllbnQvdGVzdHV0aWwvcXVlcnkuZ28=) | `100.00% <100.00%> (ø)` | | | [x/fswap/client/testutil/suite.go](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1391?src=pr&el=tree&filepath=x%2Ffswap%2Fclient%2Ftestutil%2Fsuite.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia#diff-eC9mc3dhcC9jbGllbnQvdGVzdHV0aWwvc3VpdGUuZ28=) | `100.00% <100.00%> (ø)` | | | [x/fswap/client/testutil/tx.go](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1391?src=pr&el=tree&filepath=x%2Ffswap%2Fclient%2Ftestutil%2Ftx.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia#diff-eC9mc3dhcC9jbGllbnQvdGVzdHV0aWwvdHguZ28=) | `100.00% <100.00%> (ø)` | | | [x/fswap/client/testutil/grpc.go](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1391?src=pr&el=tree&filepath=x%2Ffswap%2Fclient%2Ftestutil%2Fgrpc.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia#diff-eC9mc3dhcC9jbGllbnQvdGVzdHV0aWwvZ3JwYy5nbw==) | `98.15% <98.15%> (ø)` | | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1391/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia)