t5516: clean up cloned and new-wt in denyCurrentBranch and worktrees test

pabloosabaterr · gitster · commit b310755ecaf4 · 2026-03-30T12:52:40.000-07:00
The 'denyCurrentBranch and worktrees' test creates a 'cloned' and a 'new-wt'
but it doesn't clean them after the test. This makes other tests that use
the same name after this one to fail.

Add test_when_finished to clean them at the end.

Signed-off-by: Pablo Sabater &lt;pabloosabaterr@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/t/t5516-fetch-push.sh b/t/t5516-fetch-push.sh
@@ -1792,6 +1792,7 @@ test_expect_success 'updateInstead with push-to-checkout hook' '
 '
 
 test_expect_success 'denyCurrentBranch and worktrees' '
+	test_when_finished "rm -fr cloned && git worktree remove --force new-wt" &&
 	git worktree add new-wt &&
 	git clone . cloned &&
 	test_commit -C cloned first &&

Original file line number	Diff line number	Diff line change
`@@ -1792,6 +1792,7 @@ test_expect_success 'updateInstead with push-to-checkout hook' '`
`1792`	`1792`	`'`
`1793`	`1793`
`1794`	`1794`	`test_expect_success 'denyCurrentBranch and worktrees' '`
	`1795`	`+ test_when_finished "rm -fr cloned && git worktree remove --force new-wt" &&`
`1795`	`1796`	`git worktree add new-wt &&`
`1796`	`1797`	`git clone . cloned &&`
`1797`	`1798`	`test_commit -C cloned first &&`