hoverctl will now overwrite the default target if the default is not found

Author: benjih

functional-tests/hoverctl/logs_test.go

@@ -23,7 +23,7 @@ var _ = Describe("When I use hoverctl", func() {
 	)
 
 	BeforeEach(func() {
-		functional_tests.Run(hoverctlBinary, "targets", "create", "localm", "--admin-port", adminPort, "--proxy-port", proxyPort)
+		functional_tests.Run(hoverctlBinary, "targets", "create", "local", "--admin-port", adminPort, "--proxy-port", proxyPort)
 	})
 
 	AfterEach(func() {

functional-tests/hoverctl/targets_test.go

@@ -207,14 +207,14 @@ var _ = Describe("When using the `targets` command", func() {
 			}))
 		})
 
-		It("should error when the default points to a non-existing  targets", func() {
+		It("the default should automatically get updated to local if the default does not exist", func() {
 			functional_tests.Run(hoverctlBinary, "targets", "create", "newdefault")
 			functional_tests.Run(hoverctlBinary, "targets", "default", "newdefault")
 			functional_tests.Run(hoverctlBinary, "targets", "delete", "newdefault", "--force")
 
 			output := functional_tests.Run(hoverctlBinary, "targets", "default")
 
-			Expect(output).To(ContainSubstring("No targets registered"))
+			Expect(output).To(ContainSubstring("Default target `newdefault` not found, changing default target to `local`"))
 		})
 
 		It("should error when given an invalid target name", func() {

hoverctl/cmd/root.go

@@ -77,6 +77,12 @@ func initConfig() {
 
 	config = configuration.GetConfig()
 
+	if config.GetTarget(config.DefaultTarget) == nil {
+		fmt.Printf("Default target `%v` not found, changing default target to `local`", config.DefaultTarget)
+		config.DefaultTarget = "local"
+
+	}
+
 	target = config.GetTarget(targetNameFlag)
 	if targetNameFlag == "" && target == nil {
 		target = configuration.NewDefaultTarget()