Refactor OS Specific code

And introduce Per-OS testing

Signed-off-by: Jean-Laurent de Morlhon <jeanlaurent@morlhon.net>

drivers/virtualbox/virtualbox.go

@@ -372,18 +372,7 @@ func (d *Driver) Create() error {
 		return err
 	}
 
-	var shareName, shareDir string // TODO configurable at some point
-	switch runtime.GOOS {
-	case "windows":
-		shareName = "c/Users"
-		shareDir = "c:\\Users"
-	case "darwin":
-		shareName = "Users"
-		shareDir = "/Users"
-	case "linux":
-		shareName = "hosthome"
-		shareDir = "/home"
-	}
+	shareName, shareDir := getShareDriveAndName()
 
 	if shareDir != "" && !d.NoShare {
 		log.Debugf("setting up shareDir")

drivers/virtualbox/virtualbox_darwin.go

@@ -24,3 +24,7 @@ func (d *Driver) IsVTXDisabled() bool {
 func detectVBoxManageCmd() string {
 	return detectVBoxManageCmdInPath()
 }
+
+func getShareDriveAndName() (string, string) {
+	return "Users", "/Users"
+}

drivers/virtualbox/virtualbox_darwin_test.go

@@ -0,0 +1,14 @@
+package virtualbox
+
+import (
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+)
+func TestShareName(t *testing.T) {
+	name, dir := getShareDriveAndName()
+
+	assert.Equal(t, name, "Users")
+	assert.Equal(t, dir, "/Users")
+
+}

drivers/virtualbox/virtualbox_linux.go

@@ -34,3 +34,7 @@ func (d *Driver) IsVTXDisabled() bool {
 func detectVBoxManageCmd() string {
 	return detectVBoxManageCmdInPath()
 }
+
+func getShareDriveAndName() (string, string) {
+	return "hosthome", "/home"
+}

drivers/virtualbox/virtualbox_linux_test.go

@@ -0,0 +1,13 @@
+package virtualbox
+
+import (
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+)
+func TestShareName(t *testing.T) {
+	name, dir := getShareDriveAndName()
+
+	assert.Equal(t, name, "hosthome")
+	assert.Equal(t, dir, "/home")
+}

drivers/virtualbox/virtualbox_windows.go

@@ -89,3 +89,7 @@ func findVBoxInstallDirInRegistry() (string, error) {
 
 	return installDir, nil
 }
+
+func getShareDriveAndName() (string, string) {
+	return "c/Users", "c:\\Users"
+}