Fix broken cluster_test

cmd/minikube/main.go

@@ -28,6 +28,9 @@ import (
 	// Fix for https://github.com/kubernetes/minikube/issues/4866
 	_ "k8s.io/minikube/pkg/initflag"
 
+	// Register drivers
+	_ "k8s.io/minikube/pkg/minikube/registry/drvs"
+
 	mlog "github.com/docker/machine/libmachine/log"
 
 	"github.com/golang/glog"

pkg/minikube/cluster/cluster.go

@@ -51,7 +51,6 @@ import (
 	"k8s.io/minikube/pkg/minikube/localpath"
 	"k8s.io/minikube/pkg/minikube/out"
 	"k8s.io/minikube/pkg/minikube/registry"
-	_ "k8s.io/minikube/pkg/minikube/registry/drvs"
 	pkgutil "k8s.io/minikube/pkg/util"
 	"k8s.io/minikube/pkg/util/retry"
 )

pkg/minikube/cluster/cluster_test.go

@@ -22,6 +22,9 @@ import (
 	"testing"
 	"time"
 
+	// Register drivers
+	_ "k8s.io/minikube/pkg/minikube/registry/drvs"
+
 	"github.com/docker/machine/libmachine/drivers"
 	"github.com/docker/machine/libmachine/host"
 	"github.com/docker/machine/libmachine/provision"

pkg/minikube/cluster/machine.go

@@ -21,6 +21,7 @@ import (
 	"path/filepath"
 
 	"github.com/docker/machine/libmachine/host"
+	"github.com/golang/glog"
 	"github.com/pkg/errors"
 	"k8s.io/minikube/pkg/minikube/localpath"
 	"k8s.io/minikube/pkg/minikube/machine"
@@ -68,6 +69,7 @@ func ListMachines(miniHome ...string) (validMachines []*Machine, inValidMachines
 	for _, n := range pDirs {
 		p, err := LoadMachine(n)
 		if err != nil {
+			glog.Infof("%s not valid: %v", n, err)
 			inValidMachines = append(inValidMachines, p)
 			continue
 		}

pkg/minikube/driver/constants.go

@@ -1,30 +0,0 @@
-/*
-Copyright 2019 The Kubernetes Authors All rights reserved.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-package driver
-
-const (
-	// Driver names
-	Mock         = "mock-driver"
-	None         = "none"
-	KVM2         = "kvm2"
-	VirtualBox   = "virtualbox"
-	HyperKit     = "hyperkit"
-	VMware       = "vmware"
-	VMwareFusion = "vmwarefusion"
-	HyperV       = "hyperv"
-	Parallels    = "parallels"
-)

pkg/minikube/driver/driver.go

@@ -21,6 +21,18 @@ import (
 	"os"
 )
 
+const (
+	Mock         = "mock"
+	None         = "none"
+	KVM2         = "kvm2"
+	VirtualBox   = "virtualbox"
+	HyperKit     = "hyperkit"
+	VMware       = "vmware"
+	VMwareFusion = "vmwarefusion"
+	HyperV       = "hyperv"
+	Parallels    = "parallels"
+)
+
 // SupportedDrivers returns a list of supported drivers
 func SupportedDrivers() []string {
 	return supportedDrivers