Fix --report-file not created due to cross-device link (#4675)

Author: Resonance1584

internal/report/writer.go

@@ -10,6 +10,7 @@ import (
 	"strings"
 	"time"
 
+	"github.com/gruntwork-io/terragrunt/util"
 	"github.com/invopop/jsonschema"
 )
 
@@ -61,7 +62,7 @@ func (r *Report) WriteToFile(path string) error {
 		path = filepath.Join(r.workingDir, path)
 	}
 
-	return os.Rename(tmpFile.Name(), path)
+	return util.MoveFile(tmpFile.Name(), path)
 }
 
 // WriteCSV writes the report to a writer in CSV format.
@@ -192,7 +193,7 @@ func (r *Report) WriteSchemaToFile(path string) error {
 		path = filepath.Join(r.workingDir, path)
 	}
 
-	return os.Rename(tmpFile.Name(), path)
+	return util.MoveFile(tmpFile.Name(), path)
 }
 
 // WriteSchema writes a JSON schema for the report to a writer.

util/file.go

@@ -11,6 +11,7 @@ import (
 	"path/filepath"
 	"regexp"
 	"strings"
+	"syscall"
 
 	urlhelper "github.com/hashicorp/go-getter/helper/url"
 
@@ -1020,3 +1021,23 @@ func SanitizePath(baseDir string, file string) (string, error) {
 
 	return fullPath, nil
 }
+
+// MoveFile attempts to rename a file from source to destination, if this fails
+// due to invalid cross-device link it falls back to copying the file contents
+// and deleting the original file.
+func MoveFile(source string, destination string) error {
+	if renameErr := os.Rename(source, destination); renameErr != nil {
+		var sysErr syscall.Errno
+		if errors.As(renameErr, &sysErr) && sysErr == syscall.EXDEV {
+			if moveErr := CopyFile(source, destination); moveErr != nil {
+				return moveErr
+			}
+
+			return os.Remove(source)
+		}
+
+		return renameErr
+	}
+
+	return nil
+}

util/file_test.go

@@ -724,3 +724,21 @@ func Test_sanitizePath(t *testing.T) {
 		})
 	}
 }
+
+func TestMoveFile(t *testing.T) {
+	t.Parallel()
+	tempDir := t.TempDir()
+
+	src := filepath.Join(tempDir, "src.txt")
+	dst := filepath.Join(tempDir, "dst.txt")
+
+	require.NoError(t, os.WriteFile(src, []byte("test"), 0644))
+	require.NoError(t, util.MoveFile(src, dst))
+
+	// Verify the file was moved
+	_, err := os.Stat(src)
+	require.True(t, os.IsNotExist(err))
+	contents, err := os.ReadFile(dst)
+	require.NoError(t, err)
+	assert.Equal(t, "test", string(contents))
+}