t1092: fix buggy sparse "blame" test
commit625ff5c3201186cedb856678907a3f4bd88406a6
authorEric Sunshine <sunshine@sunshineco.com>
Mon, 22 Aug 2022 18:26:41 +0000 (22 18:26 +0000)
committerJunio C Hamano <gitster@pobox.com>
Mon, 22 Aug 2022 19:53:02 +0000 (22 12:53 -0700)
tree65667972256f05725ed0ef7a8f9e15f551facd17
parent308cbaa0829f6c039728b521153e63928934681e
t1092: fix buggy sparse "blame" test

This test wants to verify that `git blame` errors out when asked to
blame a file _not_ in the sparse checkout. However, the very first file
it asks to blame _is_ present in the checkout, thus `test_must_fail git
blame $file` gives an unexpected result (the "blame" succeeds). This
problem went unnoticed because the test invokes `test_must_fail git
blame $file` in loop but forgets to break out of the loop early upon
failure, thus the failure gets swallowed.

Fix the test by having it not ask to blame a file present in the sparse
checkout, and instead only blame files not present, as intended. While
at it, also add the missing `|| return 1` which allowed this bug to go
unnoticed.

Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t1092-sparse-checkout-compatibility.sh