From: Junio C Hamano <gitster@pobox.com>
Date: Wed, 29 Apr 2020 22:06:56 GMT
Subject: Re: Re* [PATCH 0/4] t: replace incorrect test_must_fail usage (part 5)
Message-ID: <xmqq368muetb.fsf@gitster.c.googlers.com>
In-Reply-To: <CAPig+cT5DH2KW4nnQxUMaZzLR0JSLovGbpGC=3vYOWY2QWyKsw@mail.gmail.com>

Eric Sunshine <sunshine@sunshineco.com> writes:

> This seems like a good addition and perhaps may help reduce reviewer
> burden (not that this comes up very often, but I've recommended it
> here and there when reviewing patches). Here's a possible alternate
> wording:
>
>     Similar to test_must_fail, but tolerate success, too. Use this
>     instead of "<git-command> || :" to catch failures due to segv.
>     To ignore failure of non-git commands, however, use
>     "{ <non-git-command> || :; }".

That is certainly more succinct.  "git grep -e '|| :' t/" gives some
hits that are candidates to convert to test_might_fail, by the way.