node icon indicating copy to clipboard operation
node copied to clipboard

test: all messages from cosmos sdk that are used by zetachain frequently

Open kingpinXD opened this issue 4 months ago • 3 comments

The following is a complete list of messages used. These include messages used by ZetaClient and for operations.

This task should verify that these messages are already being tested, and if they are not, add tests to include them as part of the e2e suite.

Cosmos SDK Modules and Messages

Authz

  • [x] MsgExec
  • [ ] MsgGrant

Bank

  • [ ] MsgSend

Consensus

  • [x] MsgUpdateParams (Through a gov proposal)

Distribution

  • [ ] MsgWithdrawValidatorCommission
  • [ ] MsgWithdrawDelegatorReward
  • [ ] MsgUpdateParams (Through a gov proposal)

Gov

  • [x] MsgVote
  • [x] MsgSubmitProposal

Group

  • [ ] MsgCreateGroupWithPolicy
  • [ ] MsgUpdateGroupMembers
  • [ ] MsgSubmitProposal
  • [ ] MsgExec

Slashing

  • [ ] MsgUnjail
  • [ ] MsgUpdateParams

Staking

  • [ ] MsgDelegate
  • [ ] MsgBeginRedelegate
  • [ ] MsgUndelegate
  • [ ] MsgUpdateParams (Through a gov proposal)

Upgrade

  • [x] MsgSoftwareUpgrade
  • [ ] MsgCancelUpgrade

FeeMarket

  • [x] MsgUpdateParams (Through a gov proposal)

VM (EVM)

  • [ ] MsgUpdateParams (Through a gov proposal)
  • [ ] MsgEthereumTx
  • [ ] MsgRegisterPreinstalls

kingpinXD avatar Aug 20 '25 18:08 kingpinXD

Tests for athens v34

Cosmos SDK Modules and Messages

Authz

  • [x] MsgExec zetaclient trasnactions are all executed through MsgExec
  • [x] MsgGrant zqt tx 44789A665B3461D7B9050938D3D892BCC9DB975C8EB08E590FE8F5F3A365673E

Bank

  • [x] MsgSend zqt tx 7887A50DD92A8B97410A7A810EA34FE4A6B83DB46C3199CC73179F63E654653E

Consensus

  • [x] MsgUpdateParams zqt gov proposal 114

Distribution

  • [ ] MsgWithdrawValidatorCommission Needs to be tested via a validator
  • [x] MsgWithdrawAllRewards zqt tx 778736D34A021EA11D931D0BD2E79675B5A94C58ED560F68A9927DFDA8782505
  • [x] MsgUpdateParams zqt gov proposal 114

Gov

  • [x] MsgVote tested already through govops repo
  • [x] MsgSubmitProposal zqt tx 637F9D93D0051D339E6696469F7A3632A31153079B032216B1FFAFC2F3685DCD

Group

  • [x] MsgCreateGroupWithPolicy zqt tx 06C52ED0E9985F602BD8102E249FDA714011FD675D48F16F0A92ADB284F58E91
  • [ ] MsgUpdateGroupMembers
  • [x] MsgSubmitProposal zqt tx D24D5E3A20E75C4201498F61E0DBC8D21060BC3315522A23D6E7F011EBE4084F
  • [x] MsgVote zqt tx 1598222D3818E0680817E0DA24668037583C5E24F58F05D177A66C44C1FB914F
  • [x] MsgExec zqt tx F64CDF91191811D9B5EBBC531CFD6DBB22B8A05246CA0003F988BAC4772DE2D0

Slashing

  • [ ] MsgUnjail Needs to be tested via a validator
  • [x] MsgUpdateParams zqt gov proposal 114

Staking

  • [x] MsgDelegate zqt tx B460CC2EB2D3E3D67E7233333B6050D0B12CD6D253BDA2DB7A2D9F5F93E6278A
  • [x] MsgBeginRedelegate zqt tx 62C347272687E730CB69B95358692CC02694CFBBF760B12179B7658A54665F91
  • [x] MsgBond zqt tx 95B2A8325EE960B64285F64FA12FFA1159A371FC96C36D9291277F39EC71D97D
  • [x] MsgUpdateParams zqt gov proposal 114

Upgrade

  • [x] MsgSoftwareUpgrade
  • [ ] MsgCancelUpgrade

FeeMarket

  • [x] MsgUpdateParams zqt gov proposal 114

VM (EVM)

  • [x] MsgUpdateParams already tested through govops repo
  • [ ] MsgEthereumTx
  • [x] MsgRegisterPreinstalls needs gov proposal

kingpinXD avatar Aug 23 '25 00:08 kingpinXD

Updated the title Based on some investigation, I feel like the main cause of things breaking post upgrade seems to be the divergence in types. These tests should be separate from the existing e2e test suite (Which uses the proto types) . We should ideally try to test using something similar to what wallets, explorers, and indexers use so that we can catch issues before any of these applications break doe to a change in rpc

kingpinXD avatar Aug 26 '25 16:08 kingpinXD

Updated the title Based on some investigation, I feel like the main cause of things breaking post upgrade seems to be the divergence in types. These tests should be separate from the existing e2e test suite (Which uses the proto types) . We should ideally try to test using something similar to what wallets, explorers, and indexers use so that we can catch issues before any of these applications break doe to a change in rpc

Could we merge the issue with https://github.com/zeta-chain/node/issues/3514?

lumtis avatar Sep 01 '25 13:09 lumtis