Skip to content

Commit

Permalink
fix: skip non relevant tests
Browse files Browse the repository at this point in the history
  • Loading branch information
marcello33 committed Jan 15, 2025
1 parent cbcc483 commit f44ce76
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 8 deletions.
8 changes: 5 additions & 3 deletions tests/integration/rapidgen/rapidgen.go
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@ import (
mintapi "cosmossdk.io/api/cosmos/mint/v1beta1"
paramsapi "cosmossdk.io/api/cosmos/params/v1beta1"
slashingapi "cosmossdk.io/api/cosmos/slashing/v1beta1"
stakingapi "cosmossdk.io/api/cosmos/staking/v1beta1"
upgradeapi "cosmossdk.io/api/cosmos/upgrade/v1beta1"
vestingapi "cosmossdk.io/api/cosmos/vesting/v1beta1"
evidencetypes "cosmossdk.io/x/evidence/types"
Expand All @@ -44,7 +43,6 @@ import (
minttypes "github.com/cosmos/cosmos-sdk/x/mint/types"
"github.com/cosmos/cosmos-sdk/x/params/types/proposal"
slashingtypes "github.com/cosmos/cosmos-sdk/x/slashing/types"
stakingtypes "github.com/cosmos/cosmos-sdk/x/staking/types"
)

type GeneratedType struct {
Expand Down Expand Up @@ -207,7 +205,8 @@ var (
GenType(&slashingtypes.MsgUpdateParams{}, &slashingapi.MsgUpdateParams{}, GenOpts.WithDisallowNil()),

// staking
GenType(&stakingtypes.MsgCreateValidator{}, &stakingapi.MsgCreateValidator{},
/* HV2 we have our own staking module
GenType(&stakingtypes.MsgCreateValidator{}, &stakingapi.MsgCreateValidator{},
GenOpts.WithDisallowNil().
WithAnyTypes(&ed25519.PubKey{}).
WithInterfaceHint("cosmos.crypto.PubKey", &ed25519.PubKey{}),
Expand All @@ -217,6 +216,7 @@ var (
GenType(&stakingtypes.MsgUndelegate{}, &stakingapi.MsgUndelegate{}, GenOpts.WithDisallowNil()),
GenType(&stakingtypes.MsgBeginRedelegate{}, &stakingapi.MsgBeginRedelegate{}, GenOpts.WithDisallowNil()),
GenType(&stakingtypes.MsgUpdateParams{}, &stakingapi.MsgUpdateParams{}, GenOpts.WithDisallowNil()),
*/

// upgrade
GenType(&upgradetypes.MsgSoftwareUpgrade{}, &upgradeapi.MsgSoftwareUpgrade{}, GenOpts.WithDisallowNil()),
Expand Down Expand Up @@ -272,7 +272,9 @@ var (

GenType(&slashingtypes.Params{}, &slashingapi.Params{}, GenOpts.WithDisallowNil()),

/* HV2 we have our own staking modules
GenType(&stakingtypes.StakeAuthorization{}, &stakingapi.StakeAuthorization{}, GenOpts),
*/

GenType(&upgradetypes.CancelSoftwareUpgradeProposal{}, &upgradeapi.CancelSoftwareUpgradeProposal{}, GenOpts), // nolint:staticcheck // testing legacy code path
GenType(&upgradetypes.SoftwareUpgradeProposal{}, &upgradeapi.SoftwareUpgradeProposal{}, GenOpts.WithDisallowNil()), // nolint:staticcheck // testing legacy code path
Expand Down
12 changes: 7 additions & 5 deletions tests/integration/tx/aminojson/aminojson_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -196,10 +196,10 @@ func TestAminoJSON_LegacyParity(t *testing.T) {
legacytx.RegressionTestingAminoCodec = encCfg.Amino

aj := aminojson.NewEncoder(aminojson.EncoderOptions{DoNotSortFields: true})
ac := address.NewHexCodec()
addr1Str := "0x000000000000000000000000000000000000dead"
addr1, err := ac.StringToBytes(addr1Str)
require.NoError(t, err)
// ac := address.NewHexCodec()
// addr1Str := "0x000000000000000000000000000000000000dead"
// addr1, err := ac.StringToBytes(addr1Str)
// require.NoError(t, err)
now := time.Now()

genericAuth, _ := codectypes.NewAnyWithValue(&authztypes.GenericAuthorization{Msg: "foo"})
Expand All @@ -224,6 +224,7 @@ func TestAminoJSON_LegacyParity(t *testing.T) {
sortJSON bool
}{
"auth/params": {gogo: &authtypes.Params{TxSigLimit: 10}, pulsar: &authapi.Params{TxSigLimit: 10}},
/* TODO HV2: https://polygon.atlassian.net/browse/POS-2756
"auth/module_account": {
gogo: &authtypes.ModuleAccount{
BaseAccount: authtypes.NewBaseAccountWithAddress(addr1),
Expand All @@ -234,7 +235,6 @@ func TestAminoJSON_LegacyParity(t *testing.T) {
roundTripUnequal: true,
sortJSON: true,
},
/* TODO HV2: https://polygon.atlassian.net/browse/POS-2756
"auth/base_account": {
gogo: &authtypes.BaseAccount{Address: addr1Str, PubKey: pubkeyAny},
pulsar: &authapi.BaseAccount{Address: addr1Str, PubKey: pubkeyAnyPulsar},
Expand All @@ -260,10 +260,12 @@ func TestAminoJSON_LegacyParity(t *testing.T) {
gogo: &disttypes.DelegatorStartingInfo{},
pulsar: &distapi.DelegatorStartingInfo{},
},
/* TODO HV2: https://polygon.atlassian.net/browse/POS-2756
"distribution/delegator_starting_info/non_zero_dec": {
gogo: &disttypes.DelegatorStartingInfo{Stake: math.LegacyNewDec(10)},
pulsar: &distapi.DelegatorStartingInfo{Stake: string(dec10bz)},
},
*/
"distribution/delegation_delegator_reward": {
gogo: &disttypes.DelegationDelegatorReward{},
pulsar: &distapi.DelegationDelegatorReward{},
Expand Down

0 comments on commit f44ce76

Please sign in to comment.