update tests

pkg.go

@@ -34,11 +34,12 @@ func GetPkgRoot() (string, error) {
 	}
 
 	for {
+		if currentDir == "/" {
+			return "", errors.New("go.mod not found")
+		}
+
 		if _, err := os.Stat(path.Join(currentDir, "go.mod")); !os.IsNotExist(err) {
 			absPath, err := filepath.Abs(currentDir)
-			if absPath == "/" {
-				return "", errors.New("go.mod not found")
-			}
 			if err != nil {
 				return "", err
 			}
@@ -66,7 +67,7 @@ func GetCommandVersionedPkgPath(pkgRoot, binName string) (string, error) {
 	}
 
 	if binModulePath == "" {
-		return "", errors.New("can not find bin in tools file")
+		return "", errors.New("cant find bin in tools file")
 	}
 
 	gomodPath := path.Join(pkgRoot, "go.mod")
@@ -88,7 +89,7 @@ func GetCommandVersionedPkgPath(pkgRoot, binName string) (string, error) {
 	}
 
 	if cmdPath == "" {
-		return "", fmt.Errorf("can not find require for module %s in go.mod", binModulePath)
+		return "", fmt.Errorf("cant find require for module %s in go.mod", binModulePath)
 	}
 
 	return cmdPath, nil

pkg_test.go

@@ -21,11 +21,27 @@ var _ = Describe("pkg", func() {
 	goVersion := strings.Split(string(goVersionOutput), " ")[2]
 
 	Context("GetPkgRoot", func() {
-		It("it should return the current working directory that contains a go.mod", func() {
+		It("should return the current working directory that contains a go.mod", func() {
 			dir, err := gomodrun.GetPkgRoot()
 			Expect(err).To(BeNil())
 			Expect(dir).To(Equal(cwd))
 		})
+
+		It("should return an error when it can not find a go.mod", func() {
+			err := os.Chdir("../")
+			if err != nil {
+				panic(err)
+			}
+
+			dir, err := gomodrun.GetPkgRoot()
+			Expect(err).ToNot(BeNil())
+			Expect(dir).To(Equal(""))
+
+			err = os.Chdir(cwd)
+			if err != nil {
+				panic(err)
+			}
+		})
 	})
 
 	Context("GetCommandVersionedPkgPath", func() {
@@ -34,6 +50,38 @@ var _ = Describe("pkg", func() {
 			Expect(err).To(BeNil())
 			Expect(cmdPath).To(Equal(testPackage))
 		})
+
+		It("should throw an error when it cant find tools imports", func() {
+			cmdPath, err := gomodrun.GetCommandVersionedPkgPath(path.Join(cwd, "../"), "hello-world")
+			Expect(err).ToNot(BeNil())
+			Expect(cmdPath).To(Equal(""))
+		})
+
+		It("should throw an error when it cant find specified bin in imports", func() {
+			cmdPath, err := gomodrun.GetCommandVersionedPkgPath(cwd, "not-real")
+			Expect(err).ToNot(BeNil())
+			Expect(strings.Contains(err.Error(), "cant find bin in tools file")).To(BeTrue())
+			Expect(cmdPath).To(Equal(""))
+		})
+
+		It("should throw an error when go.mod cant be found", func() {
+			cmdPath, err := gomodrun.GetCommandVersionedPkgPath(path.Join(cwd, "./tests/missing-go-mod"), "hello-world")
+			Expect(err).ToNot(BeNil())
+			Expect(cmdPath).To(Equal(""))
+		})
+
+		It("should throw an error when go.mod is corrupted", func() {
+			cmdPath, err := gomodrun.GetCommandVersionedPkgPath(path.Join(cwd, "./tests/corrupted-go-mod"), "hello-world")
+			Expect(err).ToNot(BeNil())
+			Expect(cmdPath).To(Equal(""))
+		})
+
+		It("should throw an error when go.mod is missing the requested dependency", func() {
+			cmdPath, err := gomodrun.GetCommandVersionedPkgPath(path.Join(cwd, "./tests/incomplete-go-mod"), "hello-world")
+			Expect(err).ToNot(BeNil())
+			Expect(strings.Contains(err.Error(), "cant find require")).To(BeTrue())
+			Expect(cmdPath).To(Equal(""))
+		})
 	})
 
 	Context("GetCachedBin", func() {

tests/corrupted-go-mod/go.mod

@@ -0,0 +1 @@
+not-a-real-go-mod

tests/corrupted-go-mod/go.tools.go

@@ -0,0 +1,7 @@
+// +build tools
+
+package gomodrun
+
+import (
+	_ "github.com/dustinblackman/go-hello-world-test/hello-world"
+)

tests/incomplete-go-mod/go.mod

@@ -0,0 +1,3 @@
+module github.com/dustinblackman/gomodrun-test
+
+go 1.13

tests/incomplete-go-mod/go.tools.go

@@ -0,0 +1,7 @@
+// +build tools
+
+package gomodrun
+
+import (
+	_ "github.com/dustinblackman/go-hello-world-test/hello-world"
+)

tests/missing-go-mod/go.tools.go

@@ -0,0 +1,7 @@
+// +build tools
+
+package gomodrun
+
+import (
+	_ "github.com/dustinblackman/go-hello-world-test/hello-world"
+)