fix: fixed top level panic from invalid command, uninstantiated logger (

#23)

cmd/root.go

@@ -100,7 +100,7 @@ func Execute() {
 		ImageTools: images.GetClingyImageCapture(),
 	}
 	if err := RootCmd(rootConfig).Execute(); err != nil {
-		logger.Println("Error when trying to execute", err)
+		fmt.Println("Error when trying to execute", err)
 		rootConfig.ExitTools.Exit(1)
 	}
 }

cmd/root_test.go

@@ -1,6 +1,8 @@
 package cmd
 
 import (
+	"bytes"
+	"io/ioutil"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
@@ -21,7 +23,22 @@ func TestRootCmdExecuteHelp(t *testing.T) {
 	mockTools := internal.GenerateMockInterfacesForClingy(t)
 	defer mockTools.Ctrl.Finish()
 
-	cmd := RootCmd(&RootConfig{ImageTools: mockTools.MagickClientImpl})
+	cmd := RootCmd(&RootConfig{ExitTools: mockTools.ExitClientsImpl, ImageTools: mockTools.MagickClientImpl})
 	output := internal.ExecCobraCmdAndReturnString(t, cmd, []string{"--help"})
 	assert.Contains(t, output, "clingy is a tool to test and capture CLI flows")
 }
+
+func TestRootCmdExecuteInvalidCommand(t *testing.T) {
+	mockTools := internal.GenerateMockInterfacesForClingy(t)
+	defer mockTools.Ctrl.Finish()
+
+	cmd := RootCmd(&RootConfig{ExitTools: mockTools.ExitClientsImpl, ImageTools: mockTools.MagickClientImpl})
+	b := new(bytes.Buffer)
+	cmd.SetOut(b)
+	cmd.SetErr(b)
+	cmd.SetArgs([]string{"will-exit-1-does-not-exist"})
+	cmdErr := cmd.Execute()
+	out, _ := ioutil.ReadAll(b)
+	assert.NotNil(t, cmdErr)
+	assert.Contains(t, string(out), "Error: unknown command")
+}