Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> writes: > Add a test for one of the error conditions added in > 938a60d64f (mailmap: clean up read_mailmap error handling, > 2012-12-12). > > Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> > --- > t/t4203-mailmap.sh | 9 +++++++++ > 1 file changed, 9 insertions(+) Nice to see a patch that tries to be careful, like this one. > diff --git a/t/t4203-mailmap.sh b/t/t4203-mailmap.sh > index 03a98d9635..78d56e0566 100755 > --- a/t/t4203-mailmap.sh > +++ b/t/t4203-mailmap.sh > @@ -333,6 +333,15 @@ test_expect_success 'mailmap.blob can be missing' ' > test_cmp expect actual > ' > > +test_expect_success 'mailmap.blob might be the wrong type' ' > + test_when_finished "rm .mailmap" && > + cp default.map .mailmap && > + > + git -c mailmap.blob=HEAD: shortlog HEAD >actual 2>err && > + test_i18ngrep "mailmap is not a blob" err && > + test_cmp expect actual > +' > + > test_expect_success 'mailmap.blob defaults to off in non-bare repo' ' > git init non-bare && > (