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

Add operations to run with Simulations #797

Closed
alpe opened this issue Apr 5, 2022 · 3 comments · Fixed by #1055
Closed

Add operations to run with Simulations #797

alpe opened this issue Apr 5, 2022 · 3 comments · Fixed by #1055
Labels
help wanted Extra attention is needed

Comments

@alpe
Copy link
Contributor

alpe commented Apr 5, 2022

Current simulation setup does mutate the genesis state but does not provide any "WeightedOperations" (see staking)

Would be nice to have some examples to run

@alpe
Copy link
Contributor Author

alpe commented Aug 26, 2022

We have the basic create, instantiate, execute operations covered already.
Missing are:

  • migrate
  • update admin
  • clear admin

and all gov proposals

There was no strong demand from the community to have more. Let's move this to some future release

@ethanfrey
Copy link
Member

I would move this to v0.30 to trim down the v0.29 release

@ethanfrey ethanfrey removed this from the v0.29.0 milestone Sep 6, 2022
@alpe alpe moved this to 🆕 New in wasmd backlog Sep 23, 2022
@alpe alpe moved this from 🆕 New to 📋 Unsorted Backlog in wasmd backlog Sep 23, 2022
@GNaD13
Copy link
Contributor

GNaD13 commented Oct 19, 2022

Hi @alpe , in the simulation for instantiate contract, I see that Admin is a random account

Admin: simtypes.RandomAccounts(r, 1)[0].Address.String(),

The problem is, if Admin is a random account like that, I can't find the way to (migrate, update or clear admin) because the sender of these 3 msg must be admin.
Can you check about this?

@alpe alpe closed this as completed in #1055 Nov 2, 2022
Repository owner moved this from 📋 Unsorted Backlog to ✅ Done in wasmd backlog Nov 2, 2022
@alpe alpe removed this from wasmd backlog Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants