Stefan Beller <sbeller@xxxxxxxxxx> writes: >> +test_expect_success '--rebase fails with multiple branches' ' >> + git reset --hard before-rebase && >> + test_must_fail git pull --rebase . copy master 2>out && >> + test_when_finished "rm -f out" && > > I think it would make sense to switch the previous 2 lines, because the > test_when_finished should also be run if the test actually fails. Excellent. Thanks for carefully reading patches. -- 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