test: add simulation test for cancel-unbond

x/staking/client/cli/tx.go

@@ -282,14 +282,14 @@ func NewCancelUnBondDelegation() *cobra.Command {
 	bech32PrefixValAddr := sdk.GetConfig().GetBech32ValidatorAddrPrefix()
 
 	cmd := &cobra.Command{
-		Use:   "cancel-unbonding-delegation [validator-addr] [amount]",
-		Short: "Cancel Unbonding Delegation and delegate back to validator",
+		Use:   "cancel-unbond [validator-addr] [amount] [creation-height]",
+		Short: "Cancel unbonding delegation and delegate back to validator",
 		Args:  cobra.ExactArgs(3),
 		Long: strings.TrimSpace(
 			fmt.Sprintf(`Cancel Unbonding Delegation and delegate back to validator.
 
 Example:
-$ %s tx staking cancel-unbonding-delegation %s1gghjut3ccd8ay0zduzj64hwre2fxs9ldmqhffj 100stake --from mykey
+$ %s tx staking cancel-unbond %s1gghjut3ccd8ay0zduzj64hwre2fxs9ldmqhffj 100stake 2 --from mykey
 `,
 				version.AppName, bech32PrefixValAddr,
 			),
@@ -312,7 +312,7 @@ $ %s tx staking cancel-unbonding-delegation %s1gghjut3ccd8ay0zduzj64hwre2fxs9ldm
 
 			creationHeight, ok := sdk.NewIntFromString(args[2])
 			if !ok {
-				return sdkerrors.ErrInvalidCoins
+				return sdkerrors.Wrap(fmt.Errorf("invalid height: %d", creationHeight), "invalid height")
 			}
 
 			msg := types.NewMsgCancelUnbondingDelegation(delAddr, valAddr, creationHeight.Uint64(), amount)

x/staking/simulation/operations.go

@@ -35,7 +35,7 @@ func WeightedOperations(
 		weightMsgDelegate                  int
 		weightMsgUndelegate                int
 		weightMsgBeginRedelegate           int
-		//weightMsgCancelUnbondingDelegation int
+		weightMsgCancelUnbondingDelegation int
 	)
 
 	appParams.GetOrGenerate(cdc, OpWeightMsgCreateValidator, &weightMsgCreateValidator, nil,
@@ -68,11 +68,11 @@ func WeightedOperations(
 		},
 	)
 
-	//appParams.GetOrGenerate(cdc, OpWeightMsgCancelUnbondingDelegation, &weightMsgCancelUnbondingDelegation, nil,
-	//	func(_ *rand.Rand) {
-	//		weightMsgCancelUnbondingDelegation = simappparams.DefaultWeightMsgCancelUnbondingDelegation
-	//	},
-	//)
+	appParams.GetOrGenerate(cdc, OpWeightMsgCancelUnbondingDelegation, &weightMsgCancelUnbondingDelegation, nil,
+		func(_ *rand.Rand) {
+			weightMsgCancelUnbondingDelegation = simappparams.DefaultWeightMsgCancelUnbondingDelegation
+		},
+	)
 
 	return simulation.WeightedOperations{
 		simulation.NewWeightedOperation(
@@ -95,10 +95,10 @@ func WeightedOperations(
 			weightMsgBeginRedelegate,
 			SimulateMsgBeginRedelegate(ak, bk, k),
 		),
-		//simulation.NewWeightedOperation(
-		//	weightMsgCancelUnbondingDelegation,
-		//	SimulateMsgCancelUnbondingDelegate(ak, bk, k),
-		//),
+		simulation.NewWeightedOperation(
+			weightMsgCancelUnbondingDelegation,
+			SimulateMsgCancelUnbondingDelegate(ak, bk, k),
+		),
 	}
 }