Don't run scripts in diff mode

Author: twpayne

cmd/diff.go

@@ -20,6 +20,7 @@ func init() {
 }
 
 func (c *Config) runDiffCmd(fs vfs.FS, args []string) error {
+	c.DryRun = true
 	mutator := chezmoi.NewLoggingMutator(c.Stdout(), chezmoi.NullMutator{}, c.colored)
 	return c.applyArgs(fs, args, mutator)
 }

cmd/diff_posix_test.go

@@ -0,0 +1,41 @@
+// +build !windows
+
+package cmd
+
+import (
+	"io/ioutil"
+	"os"
+	"path/filepath"
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+	vfs "github.com/twpayne/go-vfs"
+	"github.com/twpayne/go-vfs/vfst"
+)
+
+func TestDiffDoesNotRunScript(t *testing.T) {
+	tempDir, err := ioutil.TempDir("", "chezmoi")
+	require.NoError(t, err)
+	defer func() {
+		require.NoError(t, os.RemoveAll(tempDir))
+	}()
+	fs := vfs.NewPathFS(vfs.OSFS, tempDir)
+	require.NoError(t, vfst.NewBuilder().Build(
+		fs,
+		map[string]interface{}{
+			"/home/user/.local/share/chezmoi/run_true": "#!/bin/sh\necho foo >>" + filepath.Join(tempDir, "evidence") + "\n",
+		},
+	))
+	c := &Config{
+		SourceDir: "/home/user/.local/share/chezmoi",
+		DestDir:   "/",
+		Umask:     022,
+	}
+	assert.NoError(t, c.runDiffCmd(fs, nil))
+	vfst.RunTests(t, vfs.OSFS, "",
+		vfst.TestPath(filepath.Join(tempDir, "evidence"),
+			vfst.TestDoesNotExist,
+		),
+	)
+}