mirror of
https://github.com/patriceckhart/zot.git
synced 2026-06-26 21:36:31 +02:00
Replace the static recursiveSkipDirs list (which would inevitably drift as new tools appear) with the project's root .gitignore. Most caches that bloat a recursive walk \u2014 build outputs, dependency dirs, and IaC caches like .terraform/.terragrunt-cache \u2014 are already gitignored in real projects. - Extract the existing .gitignore matcher from agent/extcmd.go into a new leaf package, packages/ignore, so packages/agent/modes can share it without an import cycle. extcmd keeps thin aliases for its tests. - scanRecursive now loads the root .gitignore and prunes ignored entries, plus an unconditional .git skip (rarely self-listed). - Tests: gitignore-driven pruning in the picker, plus unit tests for the extracted matcher. No new dependencies.
126 lines
3.8 KiB
Go
126 lines
3.8 KiB
Go
package agent
|
|
|
|
import (
|
|
"os"
|
|
"path/filepath"
|
|
"testing"
|
|
)
|
|
|
|
// TestExtInstallDotSource verifies that `zot ext install .` derives the
|
|
// extension name from the resolved directory name rather than collapsing
|
|
// to the extensions/ parent directory (the false "already exists" bug).
|
|
func TestExtInstallDotSource(t *testing.T) {
|
|
home := t.TempDir()
|
|
t.Setenv("ZOT_HOME", home)
|
|
|
|
// Pre-create extensions/ to mimic a normal first run.
|
|
if err := os.MkdirAll(filepath.Join(home, "extensions"), 0o755); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
srcParent := t.TempDir()
|
|
src := filepath.Join(srcParent, "kagi")
|
|
if err := os.MkdirAll(src, 0o755); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
if err := os.WriteFile(filepath.Join(src, "extension.json"), []byte(`{"name":"kagi"}`), 0o644); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
cwd, err := os.Getwd()
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
defer os.Chdir(cwd)
|
|
if err := os.Chdir(src); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
if err := extInstall([]string{"."}); err != nil {
|
|
t.Fatalf("install with '.' failed: %v", err)
|
|
}
|
|
|
|
out := filepath.Join(home, "extensions", "kagi")
|
|
if _, err := os.Stat(filepath.Join(out, "extension.json")); err != nil {
|
|
t.Fatalf("expected installed extension at %s: %v", out, err)
|
|
}
|
|
}
|
|
|
|
// TestExtInstallRejectsParentName guards against deriving a name of ".."
|
|
// from a source that resolves to a filesystem root edge case. A normal
|
|
// directory always yields a real basename, so this just ensures the
|
|
// guard logic does not crash for well-formed input.
|
|
func TestExtInstallNamedDir(t *testing.T) {
|
|
home := t.TempDir()
|
|
t.Setenv("ZOT_HOME", home)
|
|
|
|
src := filepath.Join(t.TempDir(), "myext")
|
|
if err := os.MkdirAll(src, 0o755); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
if err := os.WriteFile(filepath.Join(src, "extension.json"), []byte(`{"name":"myext"}`), 0o644); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
if err := extInstall([]string{src}); err != nil {
|
|
t.Fatalf("install failed: %v", err)
|
|
}
|
|
if _, err := os.Stat(filepath.Join(home, "extensions", "myext", "extension.json")); err != nil {
|
|
t.Fatalf("expected installed extension: %v", err)
|
|
}
|
|
}
|
|
|
|
// TestCopyDirRespectsGitignore verifies that non-portable directories
|
|
// listed in the source .gitignore (e.g. .venv, node_modules) are not
|
|
// copied during install, while tracked files are.
|
|
func TestCopyDirRespectsGitignore(t *testing.T) {
|
|
src := t.TempDir()
|
|
dst := filepath.Join(t.TempDir(), "out")
|
|
|
|
mustWrite := func(rel, content string) {
|
|
p := filepath.Join(src, filepath.FromSlash(rel))
|
|
if err := os.MkdirAll(filepath.Dir(p), 0o755); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
if err := os.WriteFile(p, []byte(content), 0o644); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
}
|
|
|
|
mustWrite("extension.json", `{"name":"x"}`)
|
|
mustWrite("main.py", "print('hi')")
|
|
mustWrite(".gitignore", ".venv/\nnode_modules/\n*.log\n")
|
|
mustWrite(".venv/bin/python", "binary")
|
|
mustWrite("node_modules/pkg/index.js", "module")
|
|
mustWrite("debug.log", "noise")
|
|
mustWrite("src/app.py", "code")
|
|
mustWrite(".git/config", "gitdir")
|
|
|
|
if err := copyDir(src, dst); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
wantPresent := []string{"extension.json", "main.py", "src/app.py", ".gitignore"}
|
|
for _, rel := range wantPresent {
|
|
if _, err := os.Stat(filepath.Join(dst, filepath.FromSlash(rel))); err != nil {
|
|
t.Fatalf("expected %s to be copied: %v", rel, err)
|
|
}
|
|
}
|
|
|
|
wantAbsent := []string{".venv", "node_modules", "debug.log", ".git"}
|
|
for _, rel := range wantAbsent {
|
|
if _, err := os.Stat(filepath.Join(dst, filepath.FromSlash(rel))); err == nil {
|
|
t.Fatalf("expected %s to be skipped, but it was copied", rel)
|
|
}
|
|
}
|
|
}
|
|
|
|
func TestGitignoreNegation(t *testing.T) {
|
|
g := loadGitignoreFromString("build/\n!build/keep.txt\n")
|
|
if !g.Match("build", true) {
|
|
t.Fatal("expected build/ dir to be ignored")
|
|
}
|
|
if g.Match("build/keep.txt", false) {
|
|
t.Fatal("expected build/keep.txt to be re-included by negation")
|
|
}
|
|
}
|