improve: command 'configure' can automatically configure the default profile when no profile name is specified (#192)

Author: night556

config/configure.go

@@ -34,7 +34,6 @@ func NewConfigureCommand() *cli.Command {
 			}
 			profileName, _ := ProfileFlag(ctx.Flags()).GetValue()
 			mode, _ := ModeFlag(ctx.Flags()).GetValue()
-
 			return doConfigure(ctx, profileName, mode)
 		},
 	}
@@ -54,6 +53,22 @@ func doConfigure(ctx *cli.Context, profileName string, mode string) error {
 		return err
 	}
 
+	if profileName == "" {
+		if conf.CurrentProfile == "" {
+			profileName = "default"
+		} else {
+			profileName = conf.CurrentProfile
+			originMode := string(conf.GetCurrentProfile(ctx).Mode)
+			if mode == "" {
+				mode = originMode
+			} else if mode != originMode {
+				cli.Printf(w, "Warning: You are changing the authentication type of profile '%s' from '%s' to '%s'\n", profileName, originMode, mode)
+			}
+		}
+	}
+	if mode == "" {
+		mode = "AK"
+	}
 	cp, ok := conf.GetProfile(profileName)
 	if !ok {
 		cp = conf.NewProfile(profileName)

config/configure_test.go

@@ -7,6 +7,7 @@ import (
 	"bytes"
 	"io"
 	"runtime"
+	"strings"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
@@ -91,16 +92,7 @@ func TestNewConfigureCommand(t *testing.T) {
 	w.Reset()
 	err = cmd.Run(ctx, []string{})
 	assert.Nil(t, err)
-	assert.Equal(t, "Configuring profile '' in '' authenticate mode...\n"+
-		"Access Key Id []: Access Key Secret []: Default Region Id []: Default Output Format [json]: json (Only support json)\n"+
-		"Default Language [zh|en] en: Saving profile[] ...Done.\n"+
-		"-----------------------------------------------\n"+
-		"!!! Configure Failed please configure again !!!\n"+
-		"-----------------------------------------------\n"+
-		"AccessKeyId/AccessKeySecret is empty! run `aliyun configure` first\n"+
-		"-----------------------------------------------\n"+
-		"!!! Configure Failed please configure again !!!\n"+
-		"-----------------------------------------------\n", w.String())
+	assert.Equal(t, "Configuring profile 'default' in 'AK' authenticate mode...\nAccess Key Id [*************************_id]: Access Key Secret [*****************************ret]: Default Region Id []: Default Output Format [json]: json (Only support json)\nDefault Language [zh|en] : Saving profile[default] ...Done.\n-----------------------------------------------\n!!! Configure Failed please configure again !!!\n-----------------------------------------------\ndefault RegionId is empty! run `aliyun configure` first\n-----------------------------------------------\n!!! Configure Failed please configure again !!!\n-----------------------------------------------\n", w.String())
 }
 
 func TestDoConfigure(t *testing.T) {
@@ -126,6 +118,17 @@ func TestDoConfigure(t *testing.T) {
 	err := doConfigure(ctx, "profile", "AK")
 	assert.Nil(t, err)
 	assert.Equal(t, "Configuring profile 'profile' in 'AK' authenticate mode...\nAccess Key Id []: Access Key Secret []: Default Region Id []: Default Output Format [json]: json (Only support json)\nDefault Language [zh|en] en: Saving profile[profile] ...Done.\n-----------------------------------------------\n!!! Configure Failed please configure again !!!\n-----------------------------------------------\nAccessKeyId/AccessKeySecret is empty! run `aliyun configure` first\n-----------------------------------------------\n!!! Configure Failed please configure again !!!\n-----------------------------------------------\n", w.String())
+	w.Reset()
+
+	err = doConfigure(ctx, "", "")
+	assert.Nil(t, err)
+	assert.Equal(t, "Configuring profile 'default' in 'AK' authenticate mode...\nAccess Key Id [*************************_id]: Access Key Secret [*****************************ret]: Default Region Id []: Default Output Format [json]: json (Only support json)\nDefault Language [zh|en] : Saving profile[default] ...Done.\n-----------------------------------------------\n!!! Configure Failed please configure again !!!\n-----------------------------------------------\ndefault RegionId is empty! run `aliyun configure` first\n-----------------------------------------------\n!!! Configure Failed please configure again !!!\n-----------------------------------------------\n", w.String())
+	w.Reset()
+
+	err = doConfigure(ctx, "", "StsToken")
+	assert.Nil(t, err)
+	assert.True(t, strings.Contains(w.String(), "Warning: You are changing the authentication type of profile 'default' from 'AK' to 'StsToken'\nConfiguring profile 'default' in 'StsToken' authenticate mode...\nAccess Key Id [*************************_id]: Access Key Secret [*****************************ret]: Sts Token []: Default Region Id []: Default Output Format [json]: json (Only support json)\nDefault Language [zh|en] : Saving profile[default] ...Done.\n-----------------------------------------------\n!!! Configure Failed please configure again !!!\n-----------------------------------------------\n"))
+	w.Reset()
 }
 
 func TestConfigureAK(t *testing.T) {