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

[TA-3268]: Poa module simulations #36

Merged
merged 27 commits into from
Nov 6, 2024
Merged

[TA-3268]: Poa module simulations #36

merged 27 commits into from
Nov 6, 2024

Conversation

GuillemGarciaDev
Copy link
Contributor

@GuillemGarciaDev GuillemGarciaDev commented Sep 12, 2024 •
edited
Loading

[TA-3268]: Poa module simulations

Dependencies

#35

Motivation 💡

  • This PR aims to provide simulation benchmarks and full app tests for the poa module.

Changes 🛠

poa

  • Added poa/simulation/proposals messages for MsgAddValidator and MsgRemoveValidator
  • Removed deprecated ProposalContents method

general

  • Add test-sim-benchmark-simulation and test-sim-full-app-fast rules
  • Added tests/sim/params.json simulation config

GuillemGarciaDev and others added 23 commits September 6, 2024 16:13
@GuillemGarciaDev
refactor(poa): decouple evmos staking keeper from poa keeper
6b73e70
@GuillemGarciaDev
chore: add mock makefile rule
2535658
@GuillemGarciaDev
feat(poa): update expected_keepers for poa module
20f0dd0
@GuillemGarciaDev
feat(poa): generate PubKey, StakingMsgServer and Tx mocks
47480f1
@GuillemGarciaDev
feat(scripts): add mockgen entries
1eb422a
@GuillemGarciaDev
feat(poa/ante): add unit testing support
decffdb
@GuillemGarciaDev
feat(poa/keeper): provide hooks, keeper and msg server unit tests
2e7dea9
@GuillemGarciaDev
chore: add mockgen pkg dependency
210f364
@GuillemGarciaDev
chore: add test_poa Makefile rule
a788778
@GuillemGarciaDev
feat(poa/ante): unify setup poa decorator
c6031a6
@GuillemGarciaDev
feat(poa/simulation): add MsgAddValidator and MsgRemoveValidator …
8f9cbec 
…for `ProposalMsg` simulation
@GuillemGarciaDev
fix(poa/simulation): remove unused files
6721444
@GuillemGarciaDev
fix(poa/simulation): proposals linting
7a341aa
@GuillemGarciaDev
Merge branch 'feat/poa/module-testing' into feat/poa/simulation
030cf2a
@AdriaCarrera
simulation changes
24af247
@GuillemGarciaDev
Merge branch 'feat/poa/simulation' of https://github.com/Peersyst/node …
aabdfe5 
…into feat/poa/simulation
@GuillemGarciaDev
fix(chore): restore go.mod & go.sum
54bee4c
@GuillemGarciaDev
feat(app): add TestFullAppSimulation
3797af9
@GuillemGarciaDev
fix: lint modules
d6a8f6d
@GuillemGarciaDev
feat(poa/simulation): add ProposalMsgs testing
d7c294b
@GuillemGarciaDev
feat(poa/simulation): adjust msgs weight
09fbab1
@GuillemGarciaDev
feat(sim): add default simulation params to override staking msg defa…
860d0da 
…ult weights
@GuillemGarciaDev
fix(poa/simulation): fix proposals exports
64499fe
@GuillemGarciaDev GuillemGarciaDev marked this pull request as ready for review September 12, 2024 11:08
@GuillemGarciaDev GuillemGarciaDev mentioned this pull request Sep 16, 2024
Merged
@AdriaCarrera AdriaCarrera merged commit 1a2d009 into xrplevm:main Nov 6, 2024
1 check passed
@AdriaCarrera AdriaCarrera deleted the feat/poa/simulation branch November 6, 2024 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AdriaCarrera AdriaCarrera AdriaCarrera approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@GuillemGarciaDev @AdriaCarrera