Sanitize snapshots

tests/cli_test.go

@@ -16,6 +16,7 @@ import (
 
 	"github.com/charmbracelet/lipgloss"
 	"github.com/creack/pty"
+	"github.com/go-git/go-git/v5"
 	"github.com/hexops/gotextdiff"
 	"github.com/hexops/gotextdiff/myers"
 	"github.com/hexops/gotextdiff/span"
@@ -26,9 +27,11 @@ import (
 )
 
 // Command-line flag for regenerating snapshots
-var regenerateSnapshots = flag.Bool("regenerate-snapshots", false, "Regenerate all golden snapshots")
-var startingDir string
-var snapshotBaseDir string
+var (
+	regenerateSnapshots = flag.Bool("regenerate-snapshots", false, "Regenerate all golden snapshots")
+	startingDir         string
+	snapshotBaseDir     string
+)
 
 // Define styles using lipgloss
 var (
@@ -175,6 +178,21 @@ func isCIEnvironment() bool {
 	return os.Getenv("CI") != ""
 }
 
+// sanitizeOutput replaces the absolute repository root path in the provided output
+// with the placeholder "/absolute/path/to/repo".
+// It returns an error if the repository root cannot be determined.
+func sanitizeOutput(output string) (string, error) {
+	repoRoot, err := findGitRepoRoot(startingDir)
+	if err != nil {
+		return "", err
+	}
+	if repoRoot == "" {
+		return "", errors.New("failed to determine repository root")
+	}
+	// Replace all instances of the repo root with the placeholder.
+	return strings.ReplaceAll(output, repoRoot, "/absolute/path/to/repo"), nil
+}
+
 // sanitizeTestName converts t.Name() into a valid filename.
 func sanitizeTestName(name string) string {
 	// Replace slashes with underscores
@@ -561,11 +579,11 @@ func verifyFileContains(t *testing.T, filePatterns map[string][]MatchPattern) bo
 }
 
 func updateSnapshot(fullPath, output string) {
-	err := os.MkdirAll(filepath.Dir(fullPath), 0755) // Ensure parent directories exist
+	err := os.MkdirAll(filepath.Dir(fullPath), 0o755) // Ensure parent directories exist
 	if err != nil {
 		panic(fmt.Sprintf("Failed to create snapshot directory: %v", err))
 	}
-	err = os.WriteFile(fullPath, []byte(output), 0644) // Write snapshot
+	err = os.WriteFile(fullPath, []byte(output), 0o644) // Write snapshot
 	if err != nil {
 		panic(fmt.Sprintf("Failed to write snapshot file: %v", err))
 	}
@@ -645,6 +663,17 @@ func verifySnapshot(t *testing.T, tc TestCase, stdoutOutput, stderrOutput string
 		return true
 	}
 
+	// Sanitize outputs and fail the test if sanitization fails.
+	var err error
+	stdoutOutput, err = sanitizeOutput(stdoutOutput)
+	if err != nil {
+		t.Fatalf("failed to sanitize stdout output: %v", err)
+	}
+	stderrOutput, err = sanitizeOutput(stderrOutput)
+	if err != nil {
+		t.Fatalf("failed to sanitize stderr output: %v", err)
+	}
+
 	testName := sanitizeTestName(t.Name())
 	stdoutFileName := fmt.Sprintf("%s.stdout.golden", testName)
 	stderrFileName := fmt.Sprintf("%s.stderr.golden", testName)
@@ -675,7 +704,6 @@ $ go test -run=%q -regenerate-snapshots`, stdoutPath, t.Name())
 		if isCIEnvironment() || !term.IsTerminal(int(os.Stdout.Fd())) {
 			// Generate a colorized diff for better readability
 			diff = generateUnifiedDiff(filteredStdoutActual, filteredStdoutExpected)
-
 		} else {
 			diff = colorizeDiffWithThreshold(filteredStdoutActual, filteredStdoutExpected, 10)
 		}
@@ -706,6 +734,29 @@ $ go test -run=%q -regenerate-snapshots`, stderrPath, t.Name())
 	return true
 }
 
+// findGitRepo finds the Git repository root
+func findGitRepoRoot(path string) (string, error) {
+	// Open the Git repository starting from the given path
+	repo, err := git.PlainOpenWithOptions(path, &git.PlainOpenOptions{DetectDotGit: true})
+	if err != nil {
+		return "", fmt.Errorf("failed to find git repository: %w", err)
+	}
+
+	// Get the repository's working tree
+	worktree, err := repo.Worktree()
+	if err != nil {
+		return "", fmt.Errorf("failed to get worktree: %w", err)
+	}
+
+	// Return the absolute path to the root of the working tree
+	root, err := filepath.Abs(worktree.Filesystem.Root())
+	if err != nil {
+		return "", fmt.Errorf("failed to get absolute path of repository root: %w", err)
+	}
+
+	return root, nil
+}
+
 func TestUnmarshalMatchPattern(t *testing.T) {
 	yamlData := `
 expect:

tests/test-cases/demo-stacks.yaml

@@ -96,12 +96,7 @@ tests:
       - "-f"
       - "yaml"
     expect:
-      diff:
-        - "stacksBaseAbsolutePath"
-        - "terraformDirAbsolutePath"
-        - "helmfileDirAbsolutePath"
-        - 'examples[/\\]+demo-stacks[/\\]+stacks[/\\]+\*\*[/\\]+_defaults.yaml'
-        - 'examples[/\\]+demo-stacks[/\\]+stacks[/\\]deploy[/\\]+\*\*[/\\]+\*'
+      diff: []
       stdout:
         - 'append_user_agent: Atmos/(\d+\.\d+\.\d+|test) \(Cloud Posse; \+https:\/\/atmos\.tools\)'
       stderr:
@@ -120,11 +115,6 @@ tests:
     expect:
       diff:
         - '"append_user_agent": "Atmos/(\d+\.\d+\.\d+|test) \(Cloud Posse; \+https:\/\/atmos\.tools\)"'
-        - "stacksBaseAbsolutePath"
-        - "terraformDirAbsolutePath"
-        - "helmfileDirAbsolutePath"
-        - 'examples[/\\]+demo-stacks[/\\]+stacks[/\\]+\*\*[/\\]+_defaults.yaml'
-        - 'examples[/\\]+demo-stacks[/\\]+stacks[/\\]+deploy[/\\]+\*\*[/\\]+\*'
       stdout:
         - '"append_user_agent": "Atmos/(\d+\.\d+\.\d+|test) \(Cloud Posse; \+https:\/\/atmos\.tools\)"'
       stderr: