diff options
author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2022-03-07 15:48:57 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2022-03-08 00:27:40 +0300 |
commit | 1ff543acd01b9e8cfe9af7c262d118ab0bf4d5e7 (patch) | |
tree | eb90c71e9ff7a5f8bdb8505ba708e67cfde9c4e5 /t/t4123-apply-shrink.sh | |
parent | 5476bdf0e86272c47a93ae5c10486d2de31415bd (diff) |
apply tests: use "test_must_fail" instead of ad-hoc pattern
Change a fragile test pattern introduced in 6b763c424e4 (git-apply: do
not read past the end of buffer, 2007-09-05). Before this we wouldn't
distinguish normal "git apply" failures from segfaults or abort().
I'd previously marked this test as passing under SANITIZE=leak in
f54f48fc074 (leak tests: mark some apply tests as passing with
SANITIZE=leak, 2021-10-31). Let's remove that annotation as this test
will no longer pass.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t4123-apply-shrink.sh')
-rwxr-xr-x | t/t4123-apply-shrink.sh | 18 |
1 files changed, 2 insertions, 16 deletions
diff --git a/t/t4123-apply-shrink.sh b/t/t4123-apply-shrink.sh index dfa053ff28..3ef84619f5 100755 --- a/t/t4123-apply-shrink.sh +++ b/t/t4123-apply-shrink.sh @@ -2,8 +2,6 @@ test_description='apply a patch that is larger than the preimage' - -TEST_PASSES_SANITIZE_LEAK=true . ./test-lib.sh cat >F <<\EOF @@ -41,20 +39,8 @@ test_expect_success setup ' ' test_expect_success 'apply should fail gracefully' ' - - if git apply --index patch - then - echo Oops, should not have succeeded - false - else - status=$? && - echo "Status was $status" && - if test -f .git/index.lock - then - echo Oops, should not have crashed - false - fi - fi + test_must_fail git apply --index patch && + test_path_is_missing .git/index.lock ' test_done |