Return an error on unknown commands [#15]

I'm probably overthinking this--explicitly calling help is a strange and
unusual case--but it doesn't really hurt, so I'm going for it.

internal/helm/plan.go

@@ -26,6 +26,7 @@ func NewPlan(cfg Config) (*Plan, error) {
 	p := Plan{
 		cfg: cfg,
 		runCfg: run.Config{
+			HelmCommand:  string(cfg.Command),
 			Debug:        cfg.Debug,
 			KubeConfig:   cfg.KubeConfig,
 			Values:       cfg.Values,

internal/helm/plan_test.go

@@ -40,6 +40,7 @@ func (suite *PlanTestSuite) TestNewPlan() {
 	}
 
 	runCfg := run.Config{
+		HelmCommand:  "help",
 		Debug:        false,
 		KubeConfig:   "/branch/.sfere/profig",
 		Values:       "steadfastness,forthrightness",
@@ -255,10 +256,3 @@ func (suite *PlanTestSuite) TestDeterminePlanHelpCommand() {
 	stepsMaker := determineSteps(cfg)
 	suite.Same(&help, stepsMaker)
 }
-
-func (suite *PlanTestSuite) TestDeterminePlanHelpOnUnknown() {
-	cfg := Config{}
-
-	stepsMaker := determineSteps(cfg)
-	suite.Same(&help, stepsMaker)
-}

internal/run/config.go

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

internal/run/help.go

@@ -10,8 +10,15 @@ type Help struct {
 }
 
 // Execute executes the `helm help` command.
-func (h *Help) Execute(_ Config) error {
+func (h *Help) Execute(cfg Config) error {
-	return h.cmd.Run()
+	if err := h.cmd.Run(); err != nil {
+		return fmt.Errorf("while running '%s': %w", h.cmd.String(), err)
+	}
+
+	if cfg.HelmCommand == "help" {
+		return nil
+	}
+	return fmt.Errorf("unknown command '%s'", cfg.HelmCommand)
 }
 
 // Prepare gets the Help ready to execute.

internal/run/help_test.go

@@ -38,9 +38,6 @@ func (suite *HelpTestSuite) TestPrepare() {
 		Stdout(&stdout)
 	mCmd.EXPECT().
 		Stderr(&stderr)
-	mCmd.EXPECT().
-		Run().
-		Times(1)
 
 	cfg := Config{
 		Stdout: &stdout,
@@ -49,8 +46,33 @@ func (suite *HelpTestSuite) TestPrepare() {
 
 	h := Help{}
 	err := h.Prepare(cfg)
-	suite.Require().Nil(err)
+	suite.NoError(err)
-	h.Execute(cfg)
+}
+
+func (suite *HelpTestSuite) TestExecute() {
+	ctrl := gomock.NewController(suite.T())
+	defer ctrl.Finish()
+	mCmd := NewMockcmd(ctrl)
+	originalCommand := command
+	command = func(_ string, _ ...string) cmd {
+		return mCmd
+	}
+	defer func() { command = originalCommand }()
+
+	mCmd.EXPECT().
+		Run().
+		Times(2)
+
+	cfg := Config{
+		HelmCommand: "help",
+	}
+	help := Help{
+		cmd: mCmd,
+	}
+	suite.NoError(help.Execute(cfg))
+
+	cfg.HelmCommand = "get down on friday"
+	suite.EqualError(help.Execute(cfg), "unknown command 'get down on friday'")
 }
 
 func (suite *HelpTestSuite) TestPrepareDebugFlag() {