cmd/dep: Move importers under internal/importers

Author: carolynvs

cmd/dep/base_importer_test.go

@@ -619,7 +619,7 @@ func (cmd *ensureCommand) runAdd(ctx *dep.Ctx, args []string, p *dep.Project, sm
 				// could have been converted into a source by the solver.
 				if proj.Ident().ProjectRoot == pr {
 					found = true
-					pp = getProjectPropertiesFromVersion(proj.Version())
+					pp = gps.GetProjectPropertiesFromVersion(proj.Version())
 					break
 				}
 			}

cmd/dep/ensure.go

@@ -150,33 +150,6 @@ func contains(a []string, b string) bool {
 	return false
 }
 
-// getProjectPropertiesFromVersion takes a gps.Version and returns a proper
-// gps.ProjectProperties with Constraint value based on the provided version.
-func getProjectPropertiesFromVersion(v gps.Version) gps.ProjectProperties {
-	pp := gps.ProjectProperties{}
-
-	// extract version and ignore if it's revision only
-	switch tv := v.(type) {
-	case gps.PairedVersion:
-		v = tv.Unpair()
-	case gps.Revision:
-		return pp
-	}
-
-	switch v.Type() {
-	case gps.IsBranch, gps.IsVersion:
-		pp.Constraint = v
-	case gps.IsSemver:
-		c, err := gps.NewSemverConstraintIC(v.String())
-		if err != nil {
-			panic(err)
-		}
-		pp.Constraint = c
-	}
-
-	return pp
-}
-
 type projectData struct {
 	constraints  gps.ProjectConstraints          // constraints that could be found
 	dependencies map[gps.ProjectRoot][]string    // all dependencies (imports) found by project root
@@ -230,7 +203,7 @@ func (g *gopathScanner) scanGopathForDependencies() (projectData, error) {
 		}
 
 		ondisk[pr] = v
-		pp := getProjectPropertiesFromVersion(v)
+		pp := gps.GetProjectPropertiesFromVersion(v)
 		if pp.Constraint != nil || pp.Source != "" {
 			constraints[pr] = pp
 		}

cmd/dep/glide_importer_test.go

@@ -5,6 +5,8 @@
 package main
 
 import (
+	"io/ioutil"
+	"log"
 	"reflect"
 	"testing"
 
@@ -16,11 +18,24 @@ import (
 const testProject1 string = "github.com/sdboyer/deptest"
 const testProject2 string = "github.com/sdboyer/deptestdos"
 
+// NewTestContext creates a unique context with its own GOPATH for a single test.
+func NewTestContext(h *test.Helper) *dep.Ctx {
+	h.TempDir("src")
+	pwd := h.Path(".")
+	discardLogger := log.New(ioutil.Discard, "", 0)
+
+	return &dep.Ctx{
+		GOPATH: pwd,
+		Out:    discardLogger,
+		Err:    discardLogger,
+	}
+}
+
 func TestGopathScanner_OverlayManifestConstraints(t *testing.T) {
 	t.Parallel()
 
 	h := test.NewHelper(t)
-	ctx := newTestContext(h)
+	ctx := NewTestContext(h)
 
 	pi1 := gps.ProjectIdentifier{ProjectRoot: gps.ProjectRoot(testProject1)}
 	pi2 := gps.ProjectIdentifier{ProjectRoot: gps.ProjectRoot(testProject2)}
@@ -72,7 +87,7 @@ func TestGopathScanner_OverlayLockProjects(t *testing.T) {
 	t.Parallel()
 
 	h := test.NewHelper(t)
-	ctx := newTestContext(h)
+	ctx := NewTestContext(h)
 
 	rootM := dep.NewManifest()
 	pi1 := gps.ProjectIdentifier{ProjectRoot: gps.ProjectRoot(testProject1)}
@@ -164,7 +179,7 @@ func TestGetProjectPropertiesFromVersion(t *testing.T) {
 	}
 
 	for _, c := range cases {
-		actualProp := getProjectPropertiesFromVersion(c.version.(gps.Version))
+		actualProp := gps.GetProjectPropertiesFromVersion(c.version.(gps.Version))
 		if !reflect.DeepEqual(c.want, actualProp.Constraint) {
 			t.Fatalf("Constraints are not as expected: \n\t(GOT) %v\n\t(WNT) %v", actualProp.Constraint, c.want)
 		}

cmd/dep/gopath_scanner.go

@@ -5,9 +5,8 @@
 package main
 
 import (
-	"testing"
-
 	"path/filepath"
+	"testing"
 
 	"github.com/golang/dep"
 	"github.com/golang/dep/internal/gps"
@@ -18,7 +17,7 @@ func TestGetDirectDependencies_ConsolidatesRootProjects(t *testing.T) {
 	h := test.NewHelper(t)
 	defer h.Cleanup()
 
-	ctx := newTestContext(h)
+	ctx := NewTestContext(h)
 	sm, err := ctx.SourceManager()
 	h.Must(err)
 	defer sm.Release()

cmd/dep/gopath_scanner_test.go

@@ -5,6 +5,8 @@
 package main
 
 import (
+	"io/ioutil"
+	"log"
 	"path/filepath"
 	"reflect"
 	"sort"
@@ -27,6 +29,8 @@ func TestCalculatePrune(t *testing.T) {
 		filepath.FromSlash("github.com/keep/pkg/sub"),
 	}
 
+	discardLogger := log.New(ioutil.Discard, "", 0)
+
 	got, err := calculatePrune(h.Path(vendorDir), toKeep, discardLogger)
 	if err != nil {
 		t.Fatal(err)