test: make NormalizeDir/WriteRootConfig/Regenerate tests Windows-portable

After NormalizeDir was canonicalized to forward slashes (and WriteRootConfig
already runs all paths through toGitPath), the corresponding test
expectations — built via filepath.Join, which uses OS-native separators —
no longer matched on Windows. Run those expectations through the same
slash-conversion as the production code.

POSIX-rooted absolute paths like /Users/x/projects/work are not absolute
on Windows (filepath.Abs resolves them against the cwd's drive), so the
two NormalizeDir rows that use them are skipped on Windows. Equivalent
semantics are still covered by the tilde and relative-path rows.

Signed-off-by: Andre Nogueira <aanogueira@protonmail.com>

Author: aanogueira

internal/config/dirs_test.go

@@ -3,10 +3,16 @@ package config
 import (
 	"os"
 	"path/filepath"
+	"runtime"
 	"strings"
 	"testing"
 )
 
+// toFwd mirrors NormalizeDir's backslash-to-slash canonicalization so test
+// expectations built via filepath.Join (which uses OS-native separators)
+// stay comparable on Windows.
+func toFwd(p string) string { return strings.ReplaceAll(p, `\`, `/`) }
+
 func TestNormalizeDir(t *testing.T) {
 	t.Parallel()
 
@@ -20,23 +26,35 @@ func TestNormalizeDir(t *testing.T) {
 		t.Fatalf("Getwd failed: %v", err)
 	}
 
-	tests := []struct {
+	type tcase struct {
 		name string
 		in   string
 		want string
-	}{
-		{"absolute path gets trailing slash", "/Users/x/projects/work", "/Users/x/projects/work/"},
-		{
-			"absolute path keeps existing trailing slash",
-			"/Users/x/projects/work/",
-			"/Users/x/projects/work/",
-		},
-		{"tilde expands to home", "~/projects/work", filepath.Join(home, "projects", "work") + "/"},
-		{"relative resolves against cwd", "./foo", filepath.Join(cwd, "foo") + "/"},
+	}
+
+	tests := []tcase{
+		{"tilde expands to home", "~/projects/work", toFwd(filepath.Join(home, "projects", "work")) + "/"},
+		{"relative resolves against cwd", "./foo", toFwd(filepath.Join(cwd, "foo")) + "/"},
 		{"single-star glob passes through unchanged", "~/work/*/repo", "~/work/*/repo"},
 		{"double-star glob passes through unchanged", "~/work/**", "~/work/**"},
 	}
 
+	// POSIX-rooted absolute paths (`/Users/x/...`) are not absolute on
+	// Windows — `filepath.Abs` resolves them against the cwd's drive,
+	// producing `D:/Users/x/...`. Skip those rows on Windows; the
+	// equivalent semantics are exercised by the tilde + relative cases
+	// above, which build absolute paths via filepath.Join.
+	if runtime.GOOS != "windows" {
+		tests = append(tests,
+			tcase{"absolute path gets trailing slash", "/Users/x/projects/work", "/Users/x/projects/work/"},
+			tcase{
+				"absolute path keeps existing trailing slash",
+				"/Users/x/projects/work/",
+				"/Users/x/projects/work/",
+			},
+		)
+	}
+
 	for _, tc := range tests {
 		t.Run(tc.name, func(t *testing.T) {
 			t.Parallel()

internal/git/git_test.go

@@ -331,7 +331,7 @@ func TestWriteRootConfigDefaultAndAssignments(t *testing.T) {
 		t.Errorf("missing [include] block:\n%s", content)
 	}
 
-	if !strings.Contains(content, "path = "+defaultProfilePath) {
+	if !strings.Contains(content, "path = "+toGitPath(defaultProfilePath)) {
 		t.Errorf("missing default profile path:\n%s", content)
 	}
 
@@ -464,7 +464,7 @@ func TestRegenerate(t *testing.T) {
 	}
 
 	rootStr := string(root)
-	wantInclude := filepath.Join(profilesDir, "work.gitconfig")
+	wantInclude := toGitPath(filepath.Join(profilesDir, "work.gitconfig"))
 
 	if !strings.Contains(rootStr, "path = "+wantInclude) {
 		t.Errorf("root missing default include for %q:\n%s", wantInclude, rootStr)