return error for AskToConfirm

ioctl/client.go

@@ -53,7 +53,7 @@ type (
 		// SelectTranslation select a translation based on UILanguage
 		SelectTranslation(map[config.Language]string) (string, config.Language)
 		// AskToConfirm asks user to confirm from terminal, true to continue
-		AskToConfirm(string) bool
+		AskToConfirm(string) (bool, error)
 		// ReadSecret reads password from terminal
 		ReadSecret() (string, error)
 		// Execute a bash command
@@ -173,15 +173,14 @@ func (c *client) SetInsecureWithFlag(cb func(*bool, string, bool, string)) {
 	cb(&c.insecure, "insecure", !c.cfg.SecureConnect, usage)
 }
 
-func (c *client) AskToConfirm(info string) bool {
+func (c *client) AskToConfirm(info string) (bool, error) {
 	message := ConfirmationMessage{Info: info, Options: []string{"yes"}}
 	fmt.Println(message.String())
 	var confirm string
 	if _, err := fmt.Scanf("%s", &confirm); err != nil {
-		fmt.Println(err)
-		return false
+		return false, err
 	}
-	return strings.EqualFold(confirm, "yes")
+	return strings.EqualFold(confirm, "yes"), nil
 }
 
 func (c *client) SelectTranslation(trls map[config.Language]string) (string, config.Language) {

ioctl/client_test.go

@@ -38,9 +38,10 @@ func TestAskToConfirm(t *testing.T) {
 	r := require.New(t)
 	c := NewClient(config.Config{}, "")
 	defer c.Stop(context.Background())
-	blang := c.AskToConfirm("test")
+	confirmed, err := c.AskToConfirm("test")
 	// no input
-	r.False(blang)
+	r.Equal("EOF", err.Error())
+	r.False(confirmed)
 }
 
 func TestAPIServiceClient(t *testing.T) {

ioctl/newcmd/account/accountcreateadd.go

@@ -71,7 +71,11 @@ func NewAccountCreateAdd(client ioctl.Client) *cobra.Command {
 			}
 
 			if addr, ok := client.Config().Aliases[args[0]]; ok {
-				if !client.AskToConfirm(fmt.Sprintf(_aliasHasAlreadyUsed, args[0], addr)) {
+				confirmed, err := client.AskToConfirm(fmt.Sprintf(_aliasHasAlreadyUsed, args[0], addr))
+				if err != nil {
+					return errors.Wrap(err, "failed to ask confirm")
+				}
+				if !confirmed {
 					cmd.Println(infoQuit)
 					return nil
 				}

ioctl/newcmd/account/accountcreateadd_test.go

@@ -29,7 +29,7 @@ func TestNewAccountCreateAdd(t *testing.T) {
 	require.NoError(err)
 
 	client.EXPECT().ReadSecret().Return(pwd, nil).Times(4)
-	client.EXPECT().AskToConfirm(gomock.Any()).Return(true).Times(2)
+	client.EXPECT().AskToConfirm(gomock.Any()).Return(true, nil).Times(2)
 	client.EXPECT().Config().Return(config.Config{
 		Wallet: testWallet,
 		Aliases: map[string]string{
@@ -57,7 +57,7 @@ func TestNewAccountCreateAdd(t *testing.T) {
 	})
 
 	t.Run("failed to confirm", func(t *testing.T) {
-		client.EXPECT().AskToConfirm(gomock.Any()).Return(false)
+		client.EXPECT().AskToConfirm(gomock.Any()).Return(false, nil)
 
 		cmd := NewAccountCreateAdd(client)
 		_, err := util.ExecuteCmd(cmd, "aaa")

ioctl/newcmd/account/accountdelete.go

@@ -120,14 +120,18 @@ func NewAccountDelete(client ioctl.Client) *cobra.Command {
 			if _, err = os.Stat(filePath); err != nil {
 				return errors.Wrapf(err, _failToFindAccount, addr)
 			}
-			if !client.AskToConfirm(_infoWarn) {
+			confirmed, err := client.AskToConfirm(_infoWarn)
+			if err != nil {
+				return errors.Wrap(err, "failed to ask confirm")
+			}
+			if !confirmed {
 				cmd.Println(_infoQuit)
 				return nil
 			}
-			if err := os.Remove(filePath); err != nil {
+			if err = os.Remove(filePath); err != nil {
 				return errors.Wrap(err, _failToRemoveKeystoreFile)
 			}
-			if err := client.DeleteAlias(client.AliasMap()[addr]); err != nil {
+			if err = client.DeleteAlias(client.AliasMap()[addr]); err != nil {
 				return errors.Wrap(err, _failToWriteToConfigFile)
 			}
 			cmd.Println(fmt.Sprintf(_resultSuccess, addr))

ioctl/newcmd/account/accountdelete_test.go

@@ -44,12 +44,12 @@ func TestNewAccountDelete(t *testing.T) {
 			"io1uwnr55vqmhf3xeg5phgurlyl702af6eju542s1": "ccc",
 		})
 
-		client.EXPECT().AskToConfirm(gomock.Any()).Return(false)
+		client.EXPECT().AskToConfirm(gomock.Any()).Return(false, nil)
 		cmd := NewAccountDelete(client)
 		_, err := util.ExecuteCmd(cmd)
 		require.NoError(err)
 
-		client.EXPECT().AskToConfirm(gomock.Any()).Return(true)
+		client.EXPECT().AskToConfirm(gomock.Any()).Return(true, nil)
 		client.EXPECT().DeleteAlias("aaa").Return(nil)
 		cmd = NewAccountDelete(client)
 		result, err := util.ExecuteCmd(cmd)
@@ -82,7 +82,7 @@ func TestNewAccountDelete(t *testing.T) {
 		crypto.WritePrivateKeyToPem(pemFilePath, priKey2.(*crypto.P256sm2PrvKey), "test")
 		client.EXPECT().AddressWithDefaultIfNotExist(gomock.Any()).Return(addr2.String(), nil)
 
-		client.EXPECT().AskToConfirm(gomock.Any()).Return(true)
+		client.EXPECT().AskToConfirm(gomock.Any()).Return(true, nil)
 		client.EXPECT().DeleteAlias("aaa").Return(nil)
 		cmd := NewAccountDelete(client)
 		result, err := util.ExecuteCmd(cmd)

ioctl/newcmd/action/action.go

@@ -396,10 +396,14 @@ func SendAction(client ioctl.Client, cmd *cobra.Command, elp action.Envelope, si
 	if !getAssumeYesFlagValue(cmd) {
 		infoWarn := selectTranslation(client, _infoWarn)
 		infoQuit := selectTranslation(client, _infoQuit)
-		if !client.AskToConfirm(infoWarn) {
+		confirmed, err := client.AskToConfirm(infoWarn)
+		if err != nil {
+			return errors.Wrap(err, "failed to ask confirm")
+		}
+		if !confirmed {
 			cmd.Println(infoQuit)
+			return nil
 		}
-		return nil
 	}
 
 	return SendRaw(client, cmd, selp)

ioctl/newcmd/hdwallet/hdwalletdelete.go

@@ -9,6 +9,7 @@ package hdwallet
 import (
 	"fmt"
 
+	"github.com/pkg/errors"
 	"github.com/spf13/cobra"
 
 	"github.com/iotexproject/iotex-core/ioctl"
@@ -36,7 +37,11 @@ func NewHdwalletDeleteCmd(client ioctl.Client) *cobra.Command {
 			info := fmt.Sprintf("** This is an irreversible action!\n" +
 				"Once an hdwallet is deleted, all the assets under this hdwallet may be lost!\n" +
 				"Type 'YES' to continue, quit for anything else.")
-			if !client.AskToConfirm(info) {
+			confirmed, err := client.AskToConfirm(info)
+			if err != nil {
+				return errors.Wrap(err, "failed to ask confirm")
+			}
+			if !confirmed {
 				cmd.Println("quit")
 				return nil
 			}

ioctl/newcmd/hdwallet/hdwalletdelete_test.go

@@ -25,7 +25,7 @@ func TestNewHdwalletDeleteCmd(t *testing.T) {
 	client.EXPECT().SelectTranslation(gomock.Any()).Return("mockTranslationString", config.English).Times(4)
 
 	t.Run("delete hdwallet", func(t *testing.T) {
-		client.EXPECT().AskToConfirm(gomock.Any()).Return(true)
+		client.EXPECT().AskToConfirm(gomock.Any()).Return(true, nil)
 		client.EXPECT().RemoveHdWalletConfigFile().Return(nil)
 
 		cmd := NewHdwalletDeleteCmd(client)
@@ -35,7 +35,7 @@ func TestNewHdwalletDeleteCmd(t *testing.T) {
 	})
 
 	t.Run("quit hdwallet delete command", func(t *testing.T) {
-		client.EXPECT().AskToConfirm(gomock.Any()).Return(false)
+		client.EXPECT().AskToConfirm(gomock.Any()).Return(false, nil)
 
 		cmd := NewHdwalletDeleteCmd(client)
 		result, err := util.ExecuteCmd(cmd)

ioctl/newcmd/update/update.go

@@ -86,13 +86,17 @@ func NewUpdateCmd(c ioctl.Client) *cobra.Command {
 				return errors.New(fmt.Sprintf(invalidVersionType, versionType))
 			}
 
-			if !c.AskToConfirm(_infoWarn) {
+			confirmed, err := c.AskToConfirm(_infoWarn)
+			if err != nil {
+				return errors.Wrap(err, "failed to ask confirm")
+			}
+			if !confirmed {
 				cmd.Println(_infoQuit)
 				return nil
 			}
 			cmd.Printf(info, versionType)
 
-			if err := c.Execute(cmdString); err != nil {
+			if err = c.Execute(cmdString); err != nil {
 				return errors.Wrap(err, fail)
 			}
 			cmd.Println(success)

ioctl/newcmd/update/update_test.go

@@ -26,7 +26,7 @@ func TestNewUpdateCmd(t *testing.T) {
 	expectedValue := "ioctl is up-to-date now."
 	client.EXPECT().SelectTranslation(gomock.Any()).Return(expectedValue,
 		config.English).Times(18)
-	client.EXPECT().AskToConfirm(gomock.Any()).Return(true).Times(2)
+	client.EXPECT().AskToConfirm(gomock.Any()).Return(true, nil).Times(2)
 	client.EXPECT().Execute(gomock.Any()).Return(nil).Times(2)
 
 	t.Run("update cli with stable", func(t *testing.T) {
@@ -47,7 +47,7 @@ func TestNewUpdateCmd(t *testing.T) {
 		expectedError := errors.New("failed to execute bash command")
 		client.EXPECT().SelectTranslation(gomock.Any()).Return("mockTranslationResult",
 			config.English).Times(9)
-		client.EXPECT().AskToConfirm(gomock.Any()).Return(true).Times(1)
+		client.EXPECT().AskToConfirm(gomock.Any()).Return(true, nil).Times(1)
 		client.EXPECT().Execute(gomock.Any()).Return(expectedError).Times(1)
 
 		cmd := NewUpdateCmd(client)