negodiy/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

test: trim pre-commit hook integration setup

Browse Source
This commit is contained in:
Peter Steinberger
2026-02-16 05:15:55 +00:00
parent 9684ae4c6d
commit 2b6f8548c9
1 changed files with 2 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
test/git-hooks-pre-commit.integration.test.ts
View File

@@ -13,8 +13,6 @@ describe("git-hooks/pre-commit (integration)", () => {
it("does not treat staged filenames as git-add flags (e.g. --all)", async () => { it("does not treat staged filenames as git-add flags (e.g. --all)", async () => {
const dir = await mkdtemp(path.join(os.tmpdir(), "openclaw-pre-commit-")); const dir = await mkdtemp(path.join(os.tmpdir(), "openclaw-pre-commit-"));
run(dir, "git", ["init", "-q"]); run(dir, "git", ["init", "-q"]);
run(dir, "git", ["config", "user.email", "test@example.com"]);
run(dir, "git", ["config", "user.name", "Test"]);
// Copy the hook + helpers so the test exercises real on-disk wiring. // Copy the hook + helpers so the test exercises real on-disk wiring.
await mkdir(path.join(dir, "git-hooks"), { recursive: true }); await mkdir(path.join(dir, "git-hooks"), { recursive: true });
@@ -34,13 +32,8 @@ describe("git-hooks/pre-commit (integration)", () => {
chmodSync(path.join(dir, "git-hooks", "pre-commit"), 0o755); chmodSync(path.join(dir, "git-hooks", "pre-commit"), 0o755);
chmodSync(path.join(dir, "scripts", "pre-commit", "run-node-tool.sh"), 0o755); chmodSync(path.join(dir, "scripts", "pre-commit", "run-node-tool.sh"), 0o755);
await writeFile(path.join(dir, "tracked.txt"), "initial\n"); // Create an untracked file that should NOT be staged by the hook.
run(dir, "git", ["add", "--", "tracked.txt"]); await writeFile(path.join(dir, "secret.txt"), "do-not-stage\n");
run(dir, "git", ["commit", "-qm", "init"]);
// Create changes that should NOT be staged by the hook.
await writeFile(path.join(dir, "secret.txt"), "do-not-stage\n"); // untracked, not ignored
await writeFile(path.join(dir, "tracked.txt"), "changed\n"); // tracked, but not staged
// Stage a maliciously-named file. Older hooks using `xargs git add` could run `git add --all`. // Stage a maliciously-named file. Older hooks using `xargs git add` could run `git add --all`.
await writeFile(path.join(dir, "--all"), "flag\n"); await writeFile(path.join(dir, "--all"), "flag\n");