You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We should also include some tests of commonly used cosmos rpcs via CLI
Cosmos SDK has unit and integration tests to verify its functionality, but we should have some tests to confirm that we are using it correctly.
The simulation tests interact with all the messages, but they do not go through the CLI
For example, the recent change to staking tx, which modified the cli command to accept a json file instead of the individual parameters, should have been caught by us.
Although this is minor, as the message itself is being tested. It would be good to have
The text was updated successfully, but these errors were encountered:
We should also include some tests of commonly used cosmos rpcs via CLI
Cosmos SDK has unit and integration tests to verify its functionality, but we should have some tests to confirm that we are using it correctly.
The simulation tests interact with all the messages, but they do not go through the CLI
For example, the recent change to staking tx, which modified the cli command to accept a json file instead of the individual parameters, should have been caught by us.
Although this is minor, as the message itself is being tested. It would be good to have
The text was updated successfully, but these errors were encountered: