Change the tests in t/t5400-send-pack.sh to use test_when_finished to unset git config variables that were set in the tests, e.g.: git config --unset $variable_they_set This doesn't change the test behavior, but makes each test more self-contained. Signed-off-by: Ãvar ArnfjÃrà Bjarmason <avarab@xxxxxxxxx> --- t/t5400-send-pack.sh | 9 ++++++--- 1 files changed, 6 insertions(+), 3 deletions(-) diff --git a/t/t5400-send-pack.sh b/t/t5400-send-pack.sh index 656ab1a..0357610 100755 --- a/t/t5400-send-pack.sh +++ b/t/t5400-send-pack.sh @@ -90,7 +90,8 @@ test_expect_success 'refuse deleting push with denyDeletes' ' git config receive.denyDeletes true && git branch extra master ) && - test_must_fail git send-pack ./victim :extra master + test_must_fail git send-pack ./victim :extra master && + test_when_finished "(cd victim && git config --unset receive.denyDeletes)" ' test_expect_success 'cannot override denyDeletes with git -c send-pack' ' @@ -101,7 +102,8 @@ test_expect_success 'cannot override denyDeletes with git -c send-pack' ' git branch extra master ) && test_must_fail git -c receive.denyDeletes=false \ - send-pack ./victim :extra master + send-pack ./victim :extra master && + test_when_finished "(cd victim && git config --unset receive.denyDeletes)" ' test_expect_success 'override denyDeletes with git -c receive-pack' ' @@ -125,7 +127,8 @@ test_expect_success 'denyNonFastforwards trumps --force' ' victim_orig=$(cd victim && git rev-parse --verify master) && test_must_fail git send-pack --force ./victim master^:master && victim_head=$(cd victim && git rev-parse --verify master) && - test "$victim_orig" = "$victim_head" + test "$victim_orig" = "$victim_head" && + test_when_finished "(cd victim && git config --unset receive.denyNonFastforwards)" ' test_expect_success 'push --all excludes remote-tracking hierarchy' ' -- 1.7.2.3 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html