diff --git a/client/tx/tx.go b/client/tx/tx.go index 29a5483e47..d9a82c49fc 100644 --- a/client/tx/tx.go +++ b/client/tx/tx.go @@ -361,14 +361,10 @@ func makeAuxSignerData(clientCtx client.Context, f Factory, msgs ...sdk.Msg) (tx } if f.tip != nil { - if f.tip.Tipper == "" { - return tx.AuxSignerData{}, sdkerrors.Wrap(errors.New("tipper flag required"), "tipper") - } else { - if _, err := sdk.AccAddressFromBech32(f.tip.Tipper); err != nil { - return tx.AuxSignerData{}, sdkerrors.ErrInvalidAddress.Wrap("tipper must be a bech32 address") - } - b.SetTip(f.tip) + if _, err := sdk.AccAddressFromBech32(f.tip.Tipper); err != nil { + return tx.AuxSignerData{}, sdkerrors.ErrInvalidAddress.Wrap("tipper must be a bech32 address") } + b.SetTip(f.tip) } err = b.SetSignMode(f.SignMode()) diff --git a/x/auth/client/testutil/suite.go b/x/auth/client/testutil/suite.go index 8e5d25d652..4bf61146b7 100644 --- a/x/auth/client/testutil/suite.go +++ b/x/auth/client/testutil/suite.go @@ -1596,24 +1596,6 @@ func (s *IntegrationTestSuite) TestAuxToFee() { tipperArgs []string feePayerArgs []string }{ - { - name: "when --aux and --sign-mode = direct set: error", - tipper: tipper, - feePayer: feePayer, - tip: tip, - tipperArgs: []string{ - fmt.Sprintf("--%s=%s", flags.FlagSignMode, flags.SignModeDirect), - fmt.Sprintf("--%s=%s", flags.FlagTip, tip), - fmt.Sprintf("--%s=true", flags.FlagAux), - }, - expectErrAux: true, - feePayerArgs: []string{ - fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation), - fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync), - fmt.Sprintf("--%s=%s", flags.FlagFrom, feePayer), - fmt.Sprintf("--%s=%s", flags.FlagFees, fee.String()), - }, - }, { name: "when --aux and --sign-mode = direct set: error", tipper: tipper,