From a21d6ad61f414a4600718a45d4796b4501ebc764 Mon Sep 17 00:00:00 2001
From: Erin Call <erin@liffft.com>
Date: Wed, 18 Dec 2019 11:47:15 -0800
Subject: [PATCH] Implement a Delete Step [#4]

---
 internal/run/delete.go      |  52 +++++++++++++
 internal/run/delete_test.go | 141 ++++++++++++++++++++++++++++++++++++
 2 files changed, 193 insertions(+)
 create mode 100644 internal/run/delete.go
 create mode 100644 internal/run/delete_test.go

diff --git a/internal/run/delete.go b/internal/run/delete.go
new file mode 100644
index 0000000..7ddc495
--- /dev/null
+++ b/internal/run/delete.go
@@ -0,0 +1,52 @@
+package run
+
+import (
+	"fmt"
+)
+
+// Delete is an execution step that calls `helm upgrade` when executed.
+type Delete struct {
+	Release string
+	DryRun  bool
+
+	cmd cmd
+}
+
+// Execute executes the `helm upgrade` command.
+func (d *Delete) Execute(_ Config) error {
+	return d.cmd.Run()
+}
+
+// Prepare gets the Delete ready to execute.
+func (d *Delete) Prepare(cfg Config) error {
+	if d.Release == "" {
+		return fmt.Errorf("release is required")
+	}
+
+	args := []string{"--kubeconfig", cfg.KubeConfig}
+
+	if cfg.Namespace != "" {
+		args = append(args, "--namespace", cfg.Namespace)
+	}
+	if cfg.Debug {
+		args = append(args, "--debug")
+	}
+
+	args = append(args, "delete")
+
+	if d.DryRun {
+		args = append(args, "--dry-run")
+	}
+
+	args = append(args, d.Release)
+
+	d.cmd = command(helmBin, args...)
+	d.cmd.Stdout(cfg.Stdout)
+	d.cmd.Stderr(cfg.Stderr)
+
+	if cfg.Debug {
+		fmt.Fprintf(cfg.Stderr, "Generated command: '%s'\n", d.cmd.String())
+	}
+
+	return nil
+}
diff --git a/internal/run/delete_test.go b/internal/run/delete_test.go
new file mode 100644
index 0000000..f9e547d
--- /dev/null
+++ b/internal/run/delete_test.go
@@ -0,0 +1,141 @@
+package run
+
+import (
+	"fmt"
+	"github.com/golang/mock/gomock"
+	"github.com/stretchr/testify/suite"
+	"strings"
+	"testing"
+)
+
+type DeleteTestSuite struct {
+	suite.Suite
+	ctrl            *gomock.Controller
+	mockCmd         *Mockcmd
+	actualArgs      []string
+	originalCommand func(string, ...string) cmd
+}
+
+func (suite *DeleteTestSuite) BeforeTest(_, _ string) {
+	suite.ctrl = gomock.NewController(suite.T())
+	suite.mockCmd = NewMockcmd(suite.ctrl)
+
+	suite.originalCommand = command
+	command = func(path string, args ...string) cmd {
+		suite.actualArgs = args
+		return suite.mockCmd
+	}
+}
+
+func (suite *DeleteTestSuite) AfterTest(_, _ string) {
+	command = suite.originalCommand
+}
+
+func TestDeleteTestSuite(t *testing.T) {
+	suite.Run(t, new(DeleteTestSuite))
+}
+
+func (suite *DeleteTestSuite) TestPrepareAndExecute() {
+	defer suite.ctrl.Finish()
+
+	d := Delete{
+		Release: "zayde_wølf_king",
+	}
+
+	actual := []string{}
+	command = func(path string, args ...string) cmd {
+		suite.Equal(helmBin, path)
+		actual = args
+
+		return suite.mockCmd
+	}
+
+	suite.mockCmd.EXPECT().
+		Stdout(gomock.Any())
+	suite.mockCmd.EXPECT().
+		Stderr(gomock.Any())
+	suite.mockCmd.EXPECT().
+		Run().
+		Times(1)
+
+	cfg := Config{
+		KubeConfig: "/root/.kube/config",
+	}
+	suite.NoError(d.Prepare(cfg))
+	expected := []string{"--kubeconfig", "/root/.kube/config", "delete", "zayde_wølf_king"}
+	suite.Equal(expected, actual)
+
+	d.Execute(cfg)
+}
+
+func (suite *DeleteTestSuite) TestPrepareDryRunFlag() {
+	d := Delete{
+		Release: "firefox_ak_wildfire",
+		DryRun:  true,
+	}
+	cfg := Config{
+		KubeConfig: "/root/.kube/config",
+	}
+
+	suite.mockCmd.EXPECT().Stdout(gomock.Any()).AnyTimes()
+	suite.mockCmd.EXPECT().Stderr(gomock.Any()).AnyTimes()
+
+	suite.NoError(d.Prepare(cfg))
+	expected := []string{"--kubeconfig", "/root/.kube/config", "delete", "--dry-run", "firefox_ak_wildfire"}
+	suite.Equal(expected, suite.actualArgs)
+}
+
+func (suite *DeleteTestSuite) TestPrepareNamespaceFlag() {
+	d := Delete{
+		Release: "carly_simon_run_away_with_me",
+	}
+	cfg := Config{
+		KubeConfig: "/root/.kube/config",
+		Namespace:  "emotion",
+	}
+
+	suite.mockCmd.EXPECT().Stdout(gomock.Any()).AnyTimes()
+	suite.mockCmd.EXPECT().Stderr(gomock.Any()).AnyTimes()
+
+	suite.NoError(d.Prepare(cfg))
+	expected := []string{"--kubeconfig", "/root/.kube/config",
+		"--namespace", "emotion", "delete", "carly_simon_run_away_with_me"}
+	suite.Equal(expected, suite.actualArgs)
+}
+
+func (suite *DeleteTestSuite) TestPrepareDebugFlag() {
+	d := Delete{
+		Release: "just_a_band_huff_and_puff",
+	}
+	stderr := strings.Builder{}
+	cfg := Config{
+		KubeConfig: "/root/.kube/config",
+		Debug:      true,
+		Stderr:     &stderr,
+	}
+
+	command = func(path string, args ...string) cmd {
+		suite.mockCmd.EXPECT().
+			String().
+			Return(fmt.Sprintf("%s %s", path, strings.Join(args, " ")))
+
+		return suite.mockCmd
+	}
+
+	suite.mockCmd.EXPECT().Stdout(gomock.Any()).AnyTimes()
+	suite.mockCmd.EXPECT().Stderr(&stderr).AnyTimes()
+
+	suite.NoError(d.Prepare(cfg))
+	suite.Equal(fmt.Sprintf("Generated command: '%s --kubeconfig /root/.kube/config "+
+		"--debug delete just_a_band_huff_and_puff'\n", helmBin), stderr.String())
+}
+
+func (suite *DeleteTestSuite) TestPrepareRequiresRelease() {
+	// These aren't really expected, but allowing them gives clearer test-failure messages
+	suite.mockCmd.EXPECT().Stdout(gomock.Any()).AnyTimes()
+	suite.mockCmd.EXPECT().Stderr(gomock.Any()).AnyTimes()
+
+	d := Delete{}
+	err := d.Prepare(Config{})
+	suite.EqualError(err, "release is required", "Delete.Release should be mandatory")
+}