Improve UnsetTest

cmd/minikube/cmd/config/unset_test.go

@@ -16,7 +16,11 @@ limitations under the License.
 
 package config
 
-import "testing"
+import (
+	"testing"
+
+	"k8s.io/minikube/pkg/minikube/config"
+)
 
 func TestUnsetConfig(t *testing.T) {
 	propName := "cpus"
@@ -25,8 +29,23 @@ func TestUnsetConfig(t *testing.T) {
 	if err != nil {
 		t.Errorf("Failed to set the property %q", propName)
 	}
+
+	cpus, err := config.Get("cpus")
+	if err != nil {
+		t.Errorf("Failed to read config %q", err)
+	}
+
+	if cpus != propValue {
+		t.Errorf("Expected cpus to be %s but got %s", propValue, cpus)
+	}
+
 	err = Unset(propName)
 	if err != nil {
 		t.Errorf("Failed to unset property %q", err)
 	}
+
+	_, err = config.Get("cpus")
+	if err != config.ErrKeyNotFound {
+		t.Errorf("Expected error %q but got %q", config.ErrKeyNotFound, err)
+	}
 }

pkg/minikube/config/config.go

@@ -50,6 +50,11 @@ const (
 	ShowBootstrapperDeprecationNotification = "ShowBootstrapperDeprecationNotification"
 )
 
+var (
+	// ErrKeyNotFound is the error returned when a key doesn't exist in the config file
+	ErrKeyNotFound = errors.New("specified key could not be found in config")
+)
+
 // MinikubeConfig represents minikube config
 type MinikubeConfig map[string]interface{}
 
@@ -66,7 +71,7 @@ func get(name string, config MinikubeConfig) (string, error) {
 	if val, ok := config[name]; ok {
 		return fmt.Sprintf("%v", val), nil
 	}
-	return "", errors.New("specified key could not be found in config")
+	return "", ErrKeyNotFound
 }
 
 // ReadConfig reads in the JSON minikube config