Simplify temp dir creation in tests

If the tempdir is evaluated on creation, there is no need for special
handling of "darwin" (OSX) in NewTestProject()

internal/test/integration_testproj.go

@@ -11,7 +11,6 @@ import (
 	"os"
 	"os/exec"
 	"path/filepath"
-	"runtime"
 	"sort"
 	"strings"
 	"testing"
@@ -50,16 +49,7 @@ func NewTestProject(t *testing.T, initPath, wd string, externalProc bool, run Ru
 	new.TempDir(ProjectRoot, "vendor")
 	new.CopyTree(initPath)
 
-	// Note that the Travis darwin platform, directories with certain roots such
-	// as /var are actually links to a dirtree under /private.  Without the patch
-	// below the wd, and therefore the GOPATH, is recorded as "/var/..." but the
-	// actual process runs in "/private/var/..." and dies due to not being in the
-	// GOPATH because the roots don't line up.
-	if externalProc && runtime.GOOS == "darwin" && needsPrivateLeader(new.tempdir) {
-		new.Setenv("GOPATH", filepath.Join("/private", new.tempdir))
-	} else {
-		new.Setenv("GOPATH", new.tempdir)
-	}
+	new.Setenv("GOPATH", new.tempdir)
 
 	return new
 }
@@ -283,6 +273,10 @@ func (p *IntegrationTestProject) makeRootTempDir() {
 		var err error
 		p.tempdir, err = ioutil.TempDir("", "gotest")
 		p.Must(err)
+
+		// Fox for OSX where the tempdir is a symlink:
+		p.tempdir, err = filepath.EvalSymlinks(p.tempdir)
+		p.Must(err)
 	}
 }
 
@@ -298,15 +292,3 @@ func (p *IntegrationTestProject) Must(err error) {
 		p.t.Fatalf("%+v", err)
 	}
 }
-
-// Checks for filepath beginnings that result in the "/private" leader
-// on Mac platforms
-func needsPrivateLeader(path string) bool {
-	var roots = []string{"/var", "/tmp", "/etc"}
-	for _, root := range roots {
-		if strings.HasPrefix(path, root) {
-			return true
-		}
-	}
-	return false
-}