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.

Sign up for GitHub

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

Jump to bottom

SimulateMsgUpdateTotalSupply missing in WeightedOperation in simulation #718

Closed
lumtis opened this issue Apr 13, 2022 · 1 comment · Fixed by #722
Closed

SimulateMsgUpdateTotalSupply missing in WeightedOperation in simulation #718

lumtis opened this issue Apr 13, 2022 · 1 comment · Fixed by #722
Assignees
@giunatale
Labels
campaign Related to campaign module test Improve tests

Comments

@lumtis
Copy link
Contributor

lumtis commented Apr 13, 2022

SimulateMsgUpdateTotalSupply is not part of the WeightedOperation array in campaign/module_simulation

We should add it to simulate the message

Since the message operation call GetCoordSimAccountWithCampaignID with arg requireNoMainnetInitialize is set to true, we may have the same issue as in #717
@lumtis lumtis added test Improve tests campaign Related to campaign module labels Apr 13, 2022
@giunatale giunatale self-assigned this Apr 13, 2022
@giunatale
Copy link
Contributor

As per my comment to #717, fyi with make test-sim-ci we get 5 failures and 6 successes for this msg simulation

@giunatale giunatale mentioned this issue Apr 13, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@giunatale giunatale

Labels
campaign Related to campaign module test Improve tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test(campaign): add SimulateMsgUpdateTotalSupply to WeightedOperation in simulation tendermint/spn
2 participants
@lumtis @giunatale