extend history WriteString

pkg/cmd/history_writer.go

@@ -20,17 +20,28 @@ import (
 )
 
 // Write writes history to given path
-func (w *GardenctlHistoryWriter) Write(historyPath string, history map[string]string) error {
+func (w *GardenctlHistoryWriter) Write(historyPath string, m map[string]string) error {
 	f, err := os.OpenFile(historyPath, os.O_WRONLY|os.O_APPEND|os.O_CREATE, 0666)
 	if err != nil {
 		return err
 	}
 
-	j, err := json.Marshal(history)
+	j, err := json.Marshal(m)
 	if err != nil {
 		return err
 	}
 
 	_, err = f.WriteString(string(j) + "\n")
 	return err
 }
+
+// WriteString writes history to given path
+func (w *GardenctlHistoryWriter) WriteString(historyPath string, s string) error {
+	f, err := os.OpenFile(historyPath, os.O_WRONLY|os.O_APPEND|os.O_CREATE, 0666)
+	if err != nil {
+		return err
+	}
+
+	_, err = f.WriteString(s + "\n")
+	return err
+}

pkg/cmd/root.go

@@ -162,7 +162,7 @@ func init() {
 	RootCmd.AddCommand(NewInfoCmd(targetReader, ioStreams))
 	RootCmd.AddCommand(NewVersionCmd(), NewUpdateCheckCmd())
 	RootCmd.AddCommand(NewDiagCmd(targetReader, ioStreams))
-	RootCmd.AddCommand(NewHistoryCmd(targetWriter))
+	RootCmd.AddCommand(NewHistoryCmd(targetWriter, historyWriter))
 
 	RootCmd.SuggestionsMinimumDistance = suggestionsMinimumDistance
 	RootCmd.BashCompletionFunction = bashCompletionFunc

pkg/cmd/target_history.go

@@ -32,7 +32,7 @@ const (
 )
 
 //NewHistoryCmd use for list/search targting history
-func NewHistoryCmd(targetWriter TargetWriter) *cobra.Command {
+func NewHistoryCmd(targetWriter TargetWriter, historyWriter HistoryWriter) *cobra.Command {
 	cmd := &cobra.Command{
 		Use:          "history",
 		Short:        "List/Search targeting history, e.g. \"gardenctl x\"",
@@ -46,9 +46,9 @@ func NewHistoryCmd(targetWriter TargetWriter) *cobra.Command {
 				os.Exit(0)
 			}
 
-			item := h.Load().Reverse().Select().PromptItem
+			items := h.Load().Reverse().Select()
 
-			m, err := toMap(item)
+			m, err := toMap(items.PromptItem)
 			if err != nil {
 				return err
 			}
@@ -85,6 +85,12 @@ func NewHistoryCmd(targetWriter TargetWriter) *cobra.Command {
 			}
 
 			kubeconfigPathOutput(&target)
+
+			err = historyWriter.WriteString(pathHistory, items.Item)
+			if err != nil {
+				return fmt.Errorf("error write history %s", err)
+			}
+
 			return nil
 		},
 	}

pkg/cmd/target_history_test.go

@@ -23,7 +23,6 @@ import (
 var _ = Describe("History", func() {
 	history := History{
 		ConfigPath: "A",
-		Binary:     "gardenctl",
 		Items:      []string{"A", "B"},
 	}
 

pkg/cmd/types.go

@@ -42,6 +42,7 @@ type KubeconfigWriter interface {
 // HistoryWriter writes history to given path.
 type HistoryWriter interface {
 	Write(path string, history map[string]string) error
+	WriteString(path string, history string) error
 }
 
 // GardenctlTargetReader implements TargetReader.

pkg/internal/history/history.go

@@ -30,8 +30,8 @@ import (
 //History contains the history path, binary name and history items.
 type History struct {
 	ConfigPath string
-	Binary     string
 	Items      []string
+	Item       string
 	Prompt     []PromptItem
 	PromptItem PromptItem
 }
@@ -126,6 +126,7 @@ func (h *History) Select() *History {
 	}
 
 	h.PromptItem = h.Prompt[i]
+	h.Item = h.Items[i]
 	return h
 }