On Mon, Sep 20, 2010 at 3:24 AM, Johannes Sixt <j.sixt@xxxxxxxxxxxxx> wrote: > Am 9/20/2010 10:28, schrieb Elijah Newren: >> -test_expect_code 1 'Merge with d/f conflicts' 'git merge "merge msg" B master' >> +test_expect_success 'Merge with d/f conflicts' ' >> + Â Â test_must_fail git merge master >> +' > > The old version requested a particular kind of failure. Are you saying > that with modern 'git merge' all non-zero exit codes mean the same kind of > failure? No, I'm saying that I don't think the test meant to try to distinguish between exit codes. I think the test only used test_expect_code because that's all that existed in 2005 (test_must_fail postdates this test by over 2 years). There's really only one exit code from git merge[1], so there's really not much point in trying to distinguish between kinds of failure. [1] Okay, technically you can get code 128 if you hit a die(), but we don't in general try to differentiate between failure due to hitting die() and failure for other reasons when using test_must fail, so I didn't think it was worth checking here either. -- 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