wip

Author: koba1t

api/internal/loader/fileloader.go

@@ -211,8 +211,9 @@ func newLoaderAtGitClone(
 	// check for this after cloning repo.
 	if !root.HasPrefix(repoSpec.CloneDir()) {
 		_ = cleaner()
-		return nil, fmt.Errorf("%q refers to directory outside of repo %q", repoSpec.AbsPath(),
-			repoSpec.CloneDir())
+		return nil, fmt.Errorf("%q refers to directory outside of repo %q",
+			filepath.ToSlash(repoSpec.AbsPath()),
+			filepath.ToSlash(repoSpec.CloneDir().String()))
 	}
 	return &FileLoader{
 		// Clones never allowed to escape root.

api/internal/loader/fileloader_test.go

@@ -208,7 +208,7 @@ func TestLoaderLocalScheme(t *testing.T) {
 	t.Run("file", func(t *testing.T) {
 		fSys, dir := setupOnDisk(t)
 		parts := []string{
-			"ssh:",
+			"ssh-scheme",
 			"resource.yaml",
 		}
 		require.NoError(t, fSys.Mkdir(dir.Join(parts[0])))
@@ -226,8 +226,9 @@ func TestLoaderLocalScheme(t *testing.T) {
 	})
 	t.Run("directory", func(t *testing.T) {
 		fSys, dir := setupOnDisk(t)
+		// Use scheme-like name without colon for Windows compatibility
 		parts := []string{
-			"https:",
+			"https-scheme",
 			"root",
 		}
 		require.NoError(t, fSys.MkdirAll(dir.Join(filepath.Join(parts...))))

api/internal/loader/loadrestrictions_test.go

@@ -4,6 +4,7 @@
 package loader
 
 import (
+	"fmt"
 	"path/filepath"
 	"strings"
 	"testing"
@@ -59,9 +60,11 @@ func TestRestrictionRootOnly(t *testing.T) {
 	if err == nil {
 		t.Fatal("should have an error")
 	}
-	if !strings.Contains(
-		err.Error(),
-		"file '/tmp/illegal' is not in or below '/tmp/foo'") {
+	// Normalize paths to forward slashes for cross-platform comparison
+	expectedErr := fmt.Sprintf("file '%s' is not in or below '%s'",
+		filepath.ToSlash(filepath.Join(filesys.Separator+"tmp", "illegal")),
+		filepath.ToSlash(filepath.Join(filesys.Separator+"tmp", "foo")))
+	if !strings.Contains(err.Error(), expectedErr) {
 		t.Fatalf("unexpected err: %s", err)
 	}
 }

api/internal/localizer/util_test.go

@@ -14,6 +14,7 @@ import (
 	"sigs.k8s.io/kustomize/api/ifc"
 	"sigs.k8s.io/kustomize/api/internal/git"
 	"sigs.k8s.io/kustomize/kyaml/filesys"
+	"sigs.k8s.io/kustomize/kyaml/testutil"
 )
 
 func TestDefaultNewDirRepo(t *testing.T) {
@@ -139,6 +140,9 @@ func TestLocFilePathColon(t *testing.T) {
 }
 
 func TestLocFilePath_SpecialChar(t *testing.T) {
+	// Skip on Windows as asterisk is invalid in Windows paths
+	testutil.SkipWindows(t)
+
 	req := require.New(t)
 
 	// The wild card character is one of the legal uri characters with more meaning
@@ -163,19 +167,26 @@ func TestLocFilePath_SpecialFiles(t *testing.T) {
 	for name, tFSys := range map[string]struct {
 		urlPath           string
 		pathDir, pathFile string
+		skipWindows       bool // Skip this test case on Windows
 	}{
 		"windows_reserved_name": {
 			urlPath:  "/aux/file",
 			pathDir:  "aux",
 			pathFile: "file",
 		},
 		"hidden_files": {
-			urlPath:  "/.../.file",
-			pathDir:  "...",
-			pathFile: ".file",
+			urlPath:     "/.../.file",
+			pathDir:     "...",
+			pathFile:    ".file",
+			skipWindows: true, // Windows treats "..." specially
 		},
 	} {
 		t.Run(name, func(t *testing.T) {
+			// Skip Windows-incompatible tests
+			if tFSys.skipWindows {
+				testutil.SkipWindows(t)
+			}
+
 			req := require.New(t)
 
 			expectedPath := simpleJoin(t, LocalizeDir, "host", tFSys.pathDir, tFSys.pathFile)
@@ -304,23 +315,27 @@ func TestLocRootPath_SymlinkPath(t *testing.T) {
 
 func TestCleanedRelativePath(t *testing.T) {
 	fSys := filesys.MakeFsInMemory()
-	require.NoError(t, fSys.MkdirAll("/root/test"))
-	require.NoError(t, fSys.WriteFile("/root/test/file.yaml", []byte("")))
-	require.NoError(t, fSys.WriteFile("/root/filetwo.yaml", []byte("")))
+	// Use platform-appropriate root path
+	rootPath := string(filepath.Separator) + "root"
+	testPath := rootPath + string(filepath.Separator) + "test"
+	
+	require.NoError(t, fSys.MkdirAll(testPath))
+	require.NoError(t, fSys.WriteFile(testPath+string(filepath.Separator)+"file.yaml", []byte("")))
+	require.NoError(t, fSys.WriteFile(rootPath+string(filepath.Separator)+"filetwo.yaml", []byte("")))
 
 	// Absolute path is cleaned to relative path
-	cleanedPath := cleanedRelativePath(fSys, "/root/", "/root/test/file.yaml")
-	require.Equal(t, "test/file.yaml", cleanedPath)
+	cleanedPath := cleanedRelativePath(fSys, filesys.ConfirmedDir(rootPath+string(filepath.Separator)), testPath+string(filepath.Separator)+"file.yaml")
+	require.Equal(t, filepath.Join("test", "file.yaml"), cleanedPath)
 
 	// Winding absolute path is cleaned to relative path
-	cleanedPath = cleanedRelativePath(fSys, "/root/", "/root/test/../filetwo.yaml")
+	cleanedPath = cleanedRelativePath(fSys, filesys.ConfirmedDir(rootPath+string(filepath.Separator)), rootPath+string(filepath.Separator)+"test"+string(filepath.Separator)+".."+string(filepath.Separator)+"filetwo.yaml")
 	require.Equal(t, "filetwo.yaml", cleanedPath)
 
 	// Already clean relative path stays the same
-	cleanedPath = cleanedRelativePath(fSys, "/root/", "test/file.yaml")
-	require.Equal(t, "test/file.yaml", cleanedPath)
+	cleanedPath = cleanedRelativePath(fSys, filesys.ConfirmedDir(rootPath+string(filepath.Separator)), "test"+string(filepath.Separator)+"file.yaml")
+	require.Equal(t, filepath.Join("test", "file.yaml"), cleanedPath)
 
 	// Winding relative path is cleaned
-	cleanedPath = cleanedRelativePath(fSys, "/root/", "test/../filetwo.yaml")
+	cleanedPath = cleanedRelativePath(fSys, filesys.ConfirmedDir(rootPath+string(filepath.Separator)), "test"+string(filepath.Separator)+".."+string(filepath.Separator)+"filetwo.yaml")
 	require.Equal(t, "filetwo.yaml", cleanedPath)
 }

api/internal/target/kusttarget_test.go

@@ -122,11 +122,13 @@ commonLabels:
 		},
 	}
 
+	// Use platform-appropriate root directory
+	rootDir := string(filepath.Separator)
 	kt := makeKustTargetWithRf(
-		t, th.GetFSys(), "/", provider.NewDefaultDepProvider())
+		t, th.GetFSys(), rootDir, provider.NewDefaultDepProvider())
 	for tn, tc := range testCases {
 		t.Run(tn, func(t *testing.T) {
-			th.WriteK("/", tc.content)
+			th.WriteK(rootDir, tc.content)
 			err := kt.Load()
 			if tc.errContains != "" {
 				require.NotNilf(t, err, "expected error containing: `%s`", tc.errContains)

api/resource/origin.go

@@ -54,6 +54,8 @@ func (origin *Origin) Append(inputPath string) *Origin {
 	if err == nil {
 		originCopy.Repo = repoSpec.CloneSpec()
 		absPath := repoSpec.AbsPath()
+		// Normalize to forward slashes for cross-platform compatibility
+		absPath = strings.ReplaceAll(absPath, "\\", "/")
 		inputPath = absPath[strings.Index(absPath[1:], "/")+1:][1:]
 		originCopy.Path = ""
 		originCopy.Ref = repoSpec.Ref