Re: [PATCH 17/18] t4202: mark bogus head hash test with REFFILES
From: Ævar Arnfjörð Bjarmason <hidden>
Date: 2021-04-21 07:08:21
On Mon, Apr 19 2021, Han-Wen Nienhuys via GitGitGadget wrote:
From: Han-Wen Nienhuys <redacted> In reftable, hashes are correctly formed by design
With the file backend this hits this in revision.c:
if (!refname || !(flags & REF_ISSYMREF) || (flags & REF_ISBROKEN))
die(_("your current branch appears to be broken"));
There's a REF_ISBROKEN is the reftable code, so is hitting this codepath
impossible under reftable and therefore we won't need this test at all?
Maybe, and that would be cool, but re [1]'s 3rd item ("[...]what we
*don't* cover[...]") shouldn't revision.c have a "if (reftable) BUG()"
condition there then?
1. https://lore.kernel.org/git/87wnt2th1v.fsf@evledraar.gmail.com/ (local)
quoted hunk ↗ jump to hunk
Signed-off-by: Han-Wen Nienhuys <redacted> --- t/t4202-log.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/t/t4202-log.sh b/t/t4202-log.sh index c575deaad4fb..ed6d4ecd3a28 100755 --- a/t/t4202-log.sh +++ b/t/t4202-log.sh@@ -1834,7 +1834,7 @@ test_expect_success 'log --graph --no-walk is forbidden' ' test_must_fail git log --graph --no-walk ' -test_expect_success 'log diagnoses bogus HEAD hash' ' +test_expect_success REFFILES 'log diagnoses bogus HEAD hash' ' git init empty && test_must_fail git -C empty log 2>stderr && test_i18ngrep does.not.have.any.commits stderr &&