Re: [PATCH 1/2] diff --no-index tests: add test for --exit-code
flat view
From: Ramsay Jones <hidden>
Date: 2021-03-21 18:42:34
Possibly related (same subject, not in this thread)
- 2021-03-21 · Re: [PATCH 1/2] diff --no-index tests: add test for --exit-code · Ævar Arnfjörð Bjarmason <hidden>
- 2021-03-21 · Re: [PATCH 1/2] diff --no-index tests: add test for --exit-code · Junio C Hamano <hidden>
On 21/03/2021 12:39, Ævar Arnfjörð Bjarmason wrote:
quoted hunk ↗ jump to hunk
Add a test for --exit-code working with --no-index. There's no reason to suppose it wouldn't, but we weren't testing for it anywhere in our tests. Let's fix that blind spot. Signed-off-by: Ævar Arnfjörð Bjarmason <redacted> --- t/t4053-diff-no-index.sh | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)diff --git a/t/t4053-diff-no-index.sh b/t/t4053-diff-no-index.sh index 0168946b639..9b7a8ebfd3f 100755 --- a/t/t4053-diff-no-index.sh +++ b/t/t4053-diff-no-index.sh@@ -16,7 +16,12 @@ test_expect_success 'setup' ' echo 1 >non/git/b ' -test_expect_success 'git diff --no-index directories' ' +test_expect_success 'git diff --no-index --exit-code' ' + git diff --no-index --exit-code a/1 non/git/a && + test_expect_code 1 git diff --no-index --exit-code a/1 a/2 +' + +Test_expect_success 'git diff --no-index directories' '
I assume that s/test/Test/ was not intended. ;-) ATB, Ramsay Jones
test_expect_code 1 git diff --no-index a b >cnt && test_line_count = 14 cnt '