Put HelmCommand in Help, not run.Config [#15]

internal/helm/plan.go

@@ -29,7 +29,6 @@ func NewPlan(cfg Config) (*Plan, error) {
 	p := Plan{
 		cfg: cfg,
 		runCfg: run.Config{
-			HelmCommand:  string(cfg.Command),
 			Debug:        cfg.Debug,
 			Values:       cfg.Values,
 			StringValues: cfg.StringValues,
@@ -132,7 +131,9 @@ var lint = func(cfg Config) []Step {
 }
 
 var help = func(cfg Config) []Step {
-	help := &run.Help{}
+	help := &run.Help{
+		HelmCommand: cfg.Command,
+	}
 	return []Step{help}
 }
 

internal/helm/plan_test.go

@@ -43,7 +43,6 @@ func (suite *PlanTestSuite) TestNewPlan() {
 	}
 
 	runCfg := run.Config{
-		HelmCommand:  "help",
 		Debug:        false,
 		Values:       "steadfastness,forthrightness",
 		StringValues: "tensile_strength,flexibility",

internal/run/config.go

@@ -6,7 +6,6 @@ import (
 
 // Config contains configuration applicable to all helm commands
 type Config struct {
-	HelmCommand  string
 	Debug        bool
 	Values       string
 	StringValues string

internal/run/help.go

@@ -6,6 +6,7 @@ import (
 
 // Help is a step in a helm Plan that calls `helm help`.
 type Help struct {
+	HelmCommand string
 	cmd         cmd
 }
 
@@ -15,10 +16,10 @@ func (h *Help) Execute(cfg Config) error {
 		return fmt.Errorf("while running '%s': %w", h.cmd.String(), err)
 	}
 
-	if cfg.HelmCommand == "help" {
+	if h.HelmCommand == "help" {
 		return nil
 	}
-	return fmt.Errorf("unknown command '%s'", cfg.HelmCommand)
+	return fmt.Errorf("unknown command '%s'", h.HelmCommand)
 }
 
 // Prepare gets the Help ready to execute.

internal/run/help_test.go

@@ -63,15 +63,14 @@ func (suite *HelpTestSuite) TestExecute() {
 		Run().
 		Times(2)
 
-	cfg := Config{
-		HelmCommand: "help",
-	}
+	cfg := Config{}
 	help := Help{
+		HelmCommand: "help",
 		cmd:         mCmd,
 	}
 	suite.NoError(help.Execute(cfg))
 
-	cfg.HelmCommand = "get down on friday"
+	help.HelmCommand = "get down on friday"
 	suite.EqualError(help.Execute(cfg), "unknown command 'get down on friday'")
 }