tests: add tests on fsutils.ShortestRelPath

Author: ldez

pkg/fsutils/fsutils.go

@@ -34,13 +34,13 @@ func Getwd() (string, error) {
 			return
 		}
 
-		evaledWd, err := EvalSymlinks(cachedWd)
+		evaluatedWd, err := EvalSymlinks(cachedWd)
 		if err != nil {
 			cachedWd, cachedWdError = "", fmt.Errorf("can't eval symlinks on wd %s: %w", cachedWd, err)
 			return
 		}
 
-		cachedWd = evaledWd
+		cachedWd = evaluatedWd
 	})
 
 	return cachedWd, cachedWdError
@@ -83,8 +83,8 @@ func ShortestRelPath(path, wd string) (string, error) {
 	path = evaledPath
 
 	// make path absolute and then relative to be able to fix this case:
-	// we are in /test dir, we want to normalize ../test, and have file file.go in this dir;
-	// it must have normalized path file.go, not ../test/file.go,
+	// we are in `/test` dir, we want to normalize `../test`, and have file `file.go` in this dir;
+	// it must have normalized path `file.go`, not `../test/file.go`,
 	var absPath string
 	if filepath.IsAbs(path) {
 		absPath = path

pkg/fsutils/fsutils_test.go

@@ -0,0 +1,42 @@
+package fsutils
+
+import (
+	"path/filepath"
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+)
+
+func TestShortestRelPath(t *testing.T) {
+	testCases := []struct {
+		desc     string
+		path     string
+		wd       string
+		expected string
+	}{
+		{
+			desc:     "based on parent path",
+			path:     "fsutils_test.go",
+			wd:       filepath.Join("..", "fsutils"),
+			expected: "fsutils_test.go",
+		},
+		{
+			desc:     "based on current working directory",
+			path:     "fsutils_test.go",
+			wd:       "",
+			expected: "fsutils_test.go",
+		},
+	}
+
+	for _, test := range testCases {
+		t.Run(test.desc, func(t *testing.T) {
+			t.Parallel()
+
+			rel, err := ShortestRelPath("fsutils_test.go", filepath.Join("..", "fsutils"))
+			require.NoError(t, err)
+
+			assert.Equal(t, test.expected, rel)
+		})
+	}
+}