diff options
author | Eric Sunshine <sunshine@sunshineco.com> | 2022-08-22 18:26:41 +0000 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2022-08-22 12:53:02 -0700 |
commit | 625ff5c3201186cedb856678907a3f4bd88406a6 (patch) | |
tree | 65667972256f05725ed0ef7a8f9e15f551facd17 /builtin/count-objects.c | |
parent | 308cbaa0829f6c039728b521153e63928934681e (diff) |
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>
Diffstat (limited to 'builtin/count-objects.c')
0 files changed, 0 insertions, 0 deletions