mirror of
https://github.com/numtide/treefmt.git
synced 2024-10-05 21:17:37 +03:00
d4ab015bc6
Signed-off-by: Brian McGee <brian@bmcgee.ie>
607 lines
15 KiB
Go
607 lines
15 KiB
Go
package cli
|
|
|
|
import (
|
|
"fmt"
|
|
"os"
|
|
"os/exec"
|
|
"path"
|
|
"path/filepath"
|
|
"testing"
|
|
|
|
config2 "git.numtide.com/numtide/treefmt/config"
|
|
"git.numtide.com/numtide/treefmt/format"
|
|
"git.numtide.com/numtide/treefmt/test"
|
|
|
|
"github.com/go-git/go-billy/v5/osfs"
|
|
"github.com/go-git/go-git/v5"
|
|
"github.com/go-git/go-git/v5/plumbing/cache"
|
|
"github.com/go-git/go-git/v5/storage/filesystem"
|
|
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
func TestAllowMissingFormatter(t *testing.T) {
|
|
as := require.New(t)
|
|
|
|
tempDir := t.TempDir()
|
|
configPath := tempDir + "/treefmt.toml"
|
|
|
|
test.WriteConfig(t, configPath, config2.Config{
|
|
Formatters: map[string]*config2.Formatter{
|
|
"foo-fmt": {
|
|
Command: "foo-fmt",
|
|
},
|
|
},
|
|
})
|
|
|
|
_, err := cmd(t, "--config-file", configPath, "--tree-root", tempDir)
|
|
as.ErrorIs(err, format.ErrCommandNotFound)
|
|
|
|
_, err = cmd(t, "--config-file", configPath, "--tree-root", tempDir, "--allow-missing-formatter")
|
|
as.NoError(err)
|
|
}
|
|
|
|
func TestDependencyCycle(t *testing.T) {
|
|
as := require.New(t)
|
|
|
|
tempDir := t.TempDir()
|
|
configPath := tempDir + "/treefmt.toml"
|
|
|
|
test.WriteConfig(t, configPath, config2.Config{
|
|
Formatters: map[string]*config2.Formatter{
|
|
"a": {Command: "echo", Before: "b"},
|
|
"b": {Command: "echo", Before: "c"},
|
|
"c": {Command: "echo", Before: "a"},
|
|
"d": {Command: "echo", Before: "e"},
|
|
"e": {Command: "echo", Before: "f"},
|
|
"f": {Command: "echo"},
|
|
},
|
|
})
|
|
|
|
_, err := cmd(t, "--config-file", configPath, "--tree-root", tempDir)
|
|
as.ErrorContains(err, "formatter cycle detected")
|
|
}
|
|
|
|
func TestSpecifyingFormatters(t *testing.T) {
|
|
as := require.New(t)
|
|
|
|
tempDir := test.TempExamples(t)
|
|
configPath := tempDir + "/treefmt.toml"
|
|
|
|
test.WriteConfig(t, configPath, config2.Config{
|
|
Formatters: map[string]*config2.Formatter{
|
|
"elm": {
|
|
Command: "echo",
|
|
Includes: []string{"*.elm"},
|
|
},
|
|
"nix": {
|
|
Command: "echo",
|
|
Includes: []string{"*.nix"},
|
|
},
|
|
"ruby": {
|
|
Command: "echo",
|
|
Includes: []string{"*.rb"},
|
|
},
|
|
},
|
|
})
|
|
|
|
out, err := cmd(t, "-c", "--config-file", configPath, "--tree-root", tempDir)
|
|
as.NoError(err)
|
|
as.Contains(string(out), "3 files changed")
|
|
|
|
out, err = cmd(t, "-c", "--config-file", configPath, "--tree-root", tempDir, "--formatters", "elm,nix")
|
|
as.NoError(err)
|
|
as.Contains(string(out), "2 files changed")
|
|
|
|
out, err = cmd(t, "-c", "--config-file", configPath, "--tree-root", tempDir, "--formatters", "ruby,nix")
|
|
as.NoError(err)
|
|
as.Contains(string(out), "2 files changed")
|
|
|
|
out, err = cmd(t, "-c", "--config-file", configPath, "--tree-root", tempDir, "--formatters", "nix")
|
|
as.NoError(err)
|
|
as.Contains(string(out), "1 files changed")
|
|
|
|
// test bad names
|
|
|
|
out, err = cmd(t, "-c", "--config-file", configPath, "--tree-root", tempDir, "--formatters", "foo")
|
|
as.Errorf(err, "formatter not found in config: foo")
|
|
|
|
out, err = cmd(t, "-c", "--config-file", configPath, "--tree-root", tempDir, "--formatters", "bar,foo")
|
|
as.Errorf(err, "formatter not found in config: bar")
|
|
}
|
|
|
|
func TestIncludesAndExcludes(t *testing.T) {
|
|
as := require.New(t)
|
|
|
|
tempDir := test.TempExamples(t)
|
|
configPath := tempDir + "/echo.toml"
|
|
|
|
// test without any excludes
|
|
cfg := config2.Config{
|
|
Formatters: map[string]*config2.Formatter{
|
|
"echo": {
|
|
Command: "echo",
|
|
Includes: []string{"*"},
|
|
},
|
|
},
|
|
}
|
|
|
|
test.WriteConfig(t, configPath, cfg)
|
|
out, err := cmd(t, "-c", "--config-file", configPath, "--tree-root", tempDir)
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 30))
|
|
|
|
// globally exclude nix files
|
|
cfg.Global.Excludes = []string{"*.nix"}
|
|
|
|
test.WriteConfig(t, configPath, cfg)
|
|
out, err = cmd(t, "-c", "--config-file", configPath, "--tree-root", tempDir)
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 29))
|
|
|
|
// add haskell files to the global exclude
|
|
cfg.Global.Excludes = []string{"*.nix", "*.hs"}
|
|
|
|
test.WriteConfig(t, configPath, cfg)
|
|
out, err = cmd(t, "-c", "--config-file", configPath, "--tree-root", tempDir)
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 23))
|
|
|
|
echo := cfg.Formatters["echo"]
|
|
|
|
// remove python files from the echo formatter
|
|
echo.Excludes = []string{"*.py"}
|
|
|
|
test.WriteConfig(t, configPath, cfg)
|
|
out, err = cmd(t, "-c", "--config-file", configPath, "--tree-root", tempDir)
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 21))
|
|
|
|
// remove go files from the echo formatter
|
|
echo.Excludes = []string{"*.py", "*.go"}
|
|
|
|
test.WriteConfig(t, configPath, cfg)
|
|
out, err = cmd(t, "-c", "--config-file", configPath, "--tree-root", tempDir)
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 20))
|
|
|
|
// adjust the includes for echo to only include elm files
|
|
echo.Includes = []string{"*.elm"}
|
|
|
|
test.WriteConfig(t, configPath, cfg)
|
|
out, err = cmd(t, "-c", "--config-file", configPath, "--tree-root", tempDir)
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 1))
|
|
|
|
// add js files to echo formatter
|
|
echo.Includes = []string{"*.elm", "*.js"}
|
|
|
|
test.WriteConfig(t, configPath, cfg)
|
|
out, err = cmd(t, "-c", "--config-file", configPath, "--tree-root", tempDir)
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 2))
|
|
}
|
|
|
|
func TestCache(t *testing.T) {
|
|
as := require.New(t)
|
|
|
|
tempDir := test.TempExamples(t)
|
|
configPath := tempDir + "/echo.toml"
|
|
|
|
// test without any excludes
|
|
cfg := config2.Config{
|
|
Formatters: map[string]*config2.Formatter{
|
|
"echo": {
|
|
Command: "echo",
|
|
Includes: []string{"*"},
|
|
},
|
|
},
|
|
}
|
|
|
|
test.WriteConfig(t, configPath, cfg)
|
|
out, err := cmd(t, "--config-file", configPath, "--tree-root", tempDir)
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 30))
|
|
|
|
out, err = cmd(t, "--config-file", configPath, "--tree-root", tempDir)
|
|
as.NoError(err)
|
|
as.Contains(string(out), "0 files changed")
|
|
|
|
// clear cache
|
|
out, err = cmd(t, "--config-file", configPath, "--tree-root", tempDir, "-c")
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 30))
|
|
|
|
out, err = cmd(t, "--config-file", configPath, "--tree-root", tempDir)
|
|
as.NoError(err)
|
|
as.Contains(string(out), "0 files changed")
|
|
|
|
// clear cache
|
|
out, err = cmd(t, "--config-file", configPath, "--tree-root", tempDir, "-c")
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 30))
|
|
|
|
out, err = cmd(t, "--config-file", configPath, "--tree-root", tempDir)
|
|
as.NoError(err)
|
|
as.Contains(string(out), "0 files changed")
|
|
|
|
// no cache
|
|
out, err = cmd(t, "--config-file", configPath, "--tree-root", tempDir, "--no-cache")
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 30))
|
|
}
|
|
|
|
func TestChangeWorkingDirectory(t *testing.T) {
|
|
as := require.New(t)
|
|
|
|
// capture current cwd, so we can replace it after the test is finished
|
|
cwd, err := os.Getwd()
|
|
as.NoError(err)
|
|
|
|
t.Cleanup(func() {
|
|
// return to the previous working directory
|
|
as.NoError(os.Chdir(cwd))
|
|
})
|
|
|
|
tempDir := test.TempExamples(t)
|
|
configPath := tempDir + "/treefmt.toml"
|
|
|
|
// test without any excludes
|
|
cfg := config2.Config{
|
|
Formatters: map[string]*config2.Formatter{
|
|
"echo": {
|
|
Command: "echo",
|
|
Includes: []string{"*"},
|
|
},
|
|
},
|
|
}
|
|
|
|
test.WriteConfig(t, configPath, cfg)
|
|
|
|
// by default, we look for ./treefmt.toml and use the cwd for the tree root
|
|
// this should fail if the working directory hasn't been changed first
|
|
out, err := cmd(t, "-C", tempDir)
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 30))
|
|
}
|
|
|
|
func TestFailOnChange(t *testing.T) {
|
|
as := require.New(t)
|
|
|
|
tempDir := test.TempExamples(t)
|
|
configPath := tempDir + "/echo.toml"
|
|
|
|
// test without any excludes
|
|
cfg := config2.Config{
|
|
Formatters: map[string]*config2.Formatter{
|
|
"echo": {
|
|
Command: "echo",
|
|
Includes: []string{"*"},
|
|
},
|
|
},
|
|
}
|
|
|
|
test.WriteConfig(t, configPath, cfg)
|
|
_, err := cmd(t, "--fail-on-change", "--config-file", configPath, "--tree-root", tempDir)
|
|
as.ErrorIs(err, ErrFailOnChange)
|
|
}
|
|
|
|
func TestBustCacheOnFormatterChange(t *testing.T) {
|
|
as := require.New(t)
|
|
|
|
tempDir := test.TempExamples(t)
|
|
configPath := tempDir + "/echo.toml"
|
|
|
|
// symlink some formatters into temp dir, so we can mess with their mod times
|
|
binPath := tempDir + "/bin"
|
|
as.NoError(os.Mkdir(binPath, 0o755))
|
|
|
|
binaries := []string{"black", "elm-format", "gofmt"}
|
|
|
|
for _, name := range binaries {
|
|
src, err := exec.LookPath(name)
|
|
as.NoError(err)
|
|
as.NoError(os.Symlink(src, binPath+"/"+name))
|
|
}
|
|
|
|
// prepend our test bin directory to PATH
|
|
as.NoError(os.Setenv("PATH", binPath+":"+os.Getenv("PATH")))
|
|
|
|
// start with 2 formatters
|
|
cfg := config2.Config{
|
|
Formatters: map[string]*config2.Formatter{
|
|
"python": {
|
|
Command: "black",
|
|
Includes: []string{"*.py"},
|
|
},
|
|
"elm": {
|
|
Command: "elm-format",
|
|
Options: []string{"--yes"},
|
|
Includes: []string{"*.elm"},
|
|
},
|
|
},
|
|
}
|
|
|
|
test.WriteConfig(t, configPath, cfg)
|
|
args := []string{"--config-file", configPath, "--tree-root", tempDir}
|
|
out, err := cmd(t, args...)
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 3))
|
|
|
|
// tweak mod time of elm formatter
|
|
as.NoError(test.RecreateSymlink(t, binPath+"/"+"elm-format"))
|
|
|
|
out, err = cmd(t, args...)
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 3))
|
|
|
|
// check cache is working
|
|
out, err = cmd(t, args...)
|
|
as.NoError(err)
|
|
as.Contains(string(out), "0 files changed")
|
|
|
|
// tweak mod time of python formatter
|
|
as.NoError(test.RecreateSymlink(t, binPath+"/"+"black"))
|
|
|
|
out, err = cmd(t, args...)
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 3))
|
|
|
|
// check cache is working
|
|
out, err = cmd(t, args...)
|
|
as.NoError(err)
|
|
as.Contains(string(out), "0 files changed")
|
|
|
|
// add go formatter
|
|
cfg.Formatters["go"] = &config2.Formatter{
|
|
Command: "gofmt",
|
|
Options: []string{"-w"},
|
|
Includes: []string{"*.go"},
|
|
}
|
|
test.WriteConfig(t, configPath, cfg)
|
|
|
|
out, err = cmd(t, args...)
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 4))
|
|
|
|
// check cache is working
|
|
out, err = cmd(t, args...)
|
|
as.NoError(err)
|
|
as.Contains(string(out), "0 files changed")
|
|
|
|
// remove python formatter
|
|
delete(cfg.Formatters, "python")
|
|
test.WriteConfig(t, configPath, cfg)
|
|
|
|
out, err = cmd(t, args...)
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 2))
|
|
|
|
// check cache is working
|
|
out, err = cmd(t, args...)
|
|
as.NoError(err)
|
|
as.Contains(string(out), "0 files changed")
|
|
|
|
// remove elm formatter
|
|
delete(cfg.Formatters, "elm")
|
|
test.WriteConfig(t, configPath, cfg)
|
|
|
|
out, err = cmd(t, args...)
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 1))
|
|
|
|
// check cache is working
|
|
out, err = cmd(t, args...)
|
|
as.NoError(err)
|
|
as.Contains(string(out), "0 files changed")
|
|
}
|
|
|
|
func TestGitWorktree(t *testing.T) {
|
|
as := require.New(t)
|
|
|
|
tempDir := test.TempExamples(t)
|
|
configPath := filepath.Join(tempDir, "/treefmt.toml")
|
|
|
|
// basic config
|
|
cfg := config2.Config{
|
|
Formatters: map[string]*config2.Formatter{
|
|
"echo": {
|
|
Command: "echo",
|
|
Includes: []string{"*"},
|
|
},
|
|
},
|
|
}
|
|
test.WriteConfig(t, configPath, cfg)
|
|
|
|
// init a git repo
|
|
repo, err := git.Init(
|
|
filesystem.NewStorage(
|
|
osfs.New(path.Join(tempDir, ".git")),
|
|
cache.NewObjectLRUDefault(),
|
|
),
|
|
osfs.New(tempDir),
|
|
)
|
|
as.NoError(err, "failed to init git repository")
|
|
|
|
// get worktree
|
|
wt, err := repo.Worktree()
|
|
as.NoError(err, "failed to get git worktree")
|
|
|
|
run := func(changed int) {
|
|
out, err := cmd(t, "-c", "--config-file", configPath, "--tree-root", tempDir)
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", changed))
|
|
}
|
|
|
|
// run before adding anything to the worktree
|
|
run(0)
|
|
|
|
// add everything to the worktree
|
|
as.NoError(wt.AddGlob("."))
|
|
as.NoError(err)
|
|
run(30)
|
|
|
|
// remove python directory
|
|
as.NoError(wt.RemoveGlob("python/*"))
|
|
run(27)
|
|
|
|
// walk with filesystem instead of git
|
|
out, err := cmd(t, "-c", "--config-file", configPath, "--tree-root", tempDir, "--walk", "filesystem")
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 57))
|
|
}
|
|
|
|
func TestOrderingFormatters(t *testing.T) {
|
|
as := require.New(t)
|
|
|
|
tempDir := test.TempExamples(t)
|
|
configPath := path.Join(tempDir, "treefmt.toml")
|
|
|
|
// missing child
|
|
test.WriteConfig(t, configPath, config2.Config{
|
|
Formatters: map[string]*config2.Formatter{
|
|
"hs-a": {
|
|
Command: "echo",
|
|
Includes: []string{"*.hs"},
|
|
Before: "hs-b",
|
|
},
|
|
},
|
|
})
|
|
|
|
out, err := cmd(t, "--config-file", configPath, "--tree-root", tempDir)
|
|
as.ErrorContains(err, "formatter hs-a is before hs-b but config for hs-b was not found")
|
|
|
|
// multiple roots
|
|
test.WriteConfig(t, configPath, config2.Config{
|
|
Formatters: map[string]*config2.Formatter{
|
|
"hs-a": {
|
|
Command: "echo",
|
|
Includes: []string{"*.hs"},
|
|
Before: "hs-b",
|
|
},
|
|
"hs-b": {
|
|
Command: "echo",
|
|
Includes: []string{"*.hs"},
|
|
Before: "hs-c",
|
|
},
|
|
"hs-c": {
|
|
Command: "echo",
|
|
Includes: []string{"*.hs"},
|
|
},
|
|
"py-a": {
|
|
Command: "echo",
|
|
Includes: []string{"*.py"},
|
|
Before: "py-b",
|
|
},
|
|
"py-b": {
|
|
Command: "echo",
|
|
Includes: []string{"*.py"},
|
|
},
|
|
},
|
|
})
|
|
|
|
out, err = cmd(t, "--config-file", configPath, "--tree-root", tempDir)
|
|
as.NoError(err)
|
|
as.Contains(string(out), "8 files changed")
|
|
}
|
|
|
|
func TestPathsArg(t *testing.T) {
|
|
as := require.New(t)
|
|
|
|
// capture current cwd, so we can replace it after the test is finished
|
|
cwd, err := os.Getwd()
|
|
as.NoError(err)
|
|
|
|
t.Cleanup(func() {
|
|
// return to the previous working directory
|
|
as.NoError(os.Chdir(cwd))
|
|
})
|
|
|
|
tempDir := test.TempExamples(t)
|
|
configPath := filepath.Join(tempDir, "/treefmt.toml")
|
|
|
|
// change working directory to temp root
|
|
as.NoError(os.Chdir(tempDir))
|
|
|
|
// basic config
|
|
cfg := config2.Config{
|
|
Formatters: map[string]*config2.Formatter{
|
|
"echo": {
|
|
Command: "echo",
|
|
Includes: []string{"*"},
|
|
},
|
|
},
|
|
}
|
|
test.WriteConfig(t, configPath, cfg)
|
|
|
|
// without any path args
|
|
out, err := cmd(t, "-C", tempDir)
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 30))
|
|
|
|
// specify some explicit paths
|
|
out, err = cmd(t, "-C", tempDir, "-c", "elm/elm.json", "haskell/Nested/Foo.hs")
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 2))
|
|
|
|
// specify a bad path
|
|
out, err = cmd(t, "-C", tempDir, "-c", "elm/elm.json", "haskell/Nested/Bar.hs")
|
|
as.ErrorContains(err, "no such file or directory")
|
|
}
|
|
|
|
func TestStdIn(t *testing.T) {
|
|
as := require.New(t)
|
|
|
|
// capture current cwd, so we can replace it after the test is finished
|
|
cwd, err := os.Getwd()
|
|
as.NoError(err)
|
|
|
|
t.Cleanup(func() {
|
|
// return to the previous working directory
|
|
as.NoError(os.Chdir(cwd))
|
|
})
|
|
|
|
tempDir := test.TempExamples(t)
|
|
configPath := filepath.Join(tempDir, "/treefmt.toml")
|
|
|
|
// change working directory to temp root
|
|
as.NoError(os.Chdir(tempDir))
|
|
|
|
// basic config
|
|
cfg := config2.Config{
|
|
Formatters: map[string]*config2.Formatter{
|
|
"echo": {
|
|
Command: "echo",
|
|
Includes: []string{"*"},
|
|
},
|
|
},
|
|
}
|
|
test.WriteConfig(t, configPath, cfg)
|
|
|
|
// swap out stdin
|
|
prevStdIn := os.Stdin
|
|
stdin, err := os.CreateTemp("", "stdin")
|
|
as.NoError(err)
|
|
|
|
os.Stdin = stdin
|
|
|
|
t.Cleanup(func() {
|
|
os.Stdin = prevStdIn
|
|
_ = os.Remove(stdin.Name())
|
|
})
|
|
|
|
go func() {
|
|
_, err := stdin.WriteString(`treefmt.toml
|
|
elm/elm.json
|
|
go/main.go
|
|
`)
|
|
as.NoError(err, "failed to write to stdin")
|
|
as.NoError(stdin.Sync())
|
|
_, _ = stdin.Seek(0, 0)
|
|
}()
|
|
|
|
out, err := cmd(t, "-C", tempDir, "--stdin")
|
|
as.NoError(err)
|
|
as.Contains(string(out), fmt.Sprintf("%d files changed", 3))
|
|
}
|