Johannes Schindelin venit, vidit, dixit 06.02.2009 16:36: > Hi, > > On Fri, 6 Feb 2009, Michael J Gruber wrote: > >> Add a test for the handling of empty notes by "git notes show". >> --- >> t/t3301-notes.sh | 5 +++++ >> 1 files changed, 5 insertions(+), 0 deletions(-) >> >> diff --git a/t/t3301-notes.sh b/t/t3301-notes.sh >> index 9393a25..4900dca 100755 >> --- a/t/t3301-notes.sh >> +++ b/t/t3301-notes.sh >> @@ -35,6 +35,11 @@ test_expect_success 'need valid notes ref' ' >> ! MSG=2 GIT_NOTES_REF='/' git notes show >> ' >> >> +# 1 indicates caught gracefully by die, 128 means git-show barfed >> +test_expect_failure 'handle empty notes gracefully' ' >> + git notes show || test 1 = $? >> +' > > That test would succeed if the exit status is 0. Yes. If "git notes show" returns 0 then even better. It does neither before nor after the patch. Or should we always expect 1 and test for that? I thought about grepping the output for "fatal" (which appears now) but that seemed ugly. Michael -- 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