Re: [PATCHv4 02/15] t4017 (diff-retval): replace manual exit code check with test_expect_code
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Sep 29, 2010, 18:07 UTC
- Message-ID
- <7vd3rwo22t.fsf@alter.siamese.dyndns.org>
- In-Reply-To
- <1285542879-16381-3-git-send-email-newren@gmail.com>
Elijah Newren <newren@gmail.com> writes:
Show 24 quoted lines
> Acked-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com> > Signed-off-by: Elijah Newren <newren@gmail.com> > --- > t/t4017-diff-retval.sh | 30 ++++++++++-------------------- > 1 files changed, 10 insertions(+), 20 deletions(-) > > diff --git a/t/t4017-diff-retval.sh b/t/t4017-diff-retval.sh > index 6158985..6605e12 100755 > --- a/t/t4017-diff-retval.sh > +++ b/t/t4017-diff-retval.sh > @@ -28,37 +28,29 @@ test_expect_success 'git diff --quiet -w HEAD^ HEAD' ' > test_must_fail git diff --quiet -w HEAD^ HEAD > ' > > -test_expect_success 'git diff-tree HEAD^ HEAD' ' > +test_expect_code 1 'git diff-tree HEAD^ HEAD' ' > git diff-tree --exit-code HEAD^ HEAD > - test $? = 1 > ' > -test_expect_success 'git diff-tree HEAD^ HEAD -- a' ' > +test_expect_code 0 'git diff-tree HEAD^ HEAD -- a' ' > git diff-tree --exit-code HEAD^ HEAD -- a > - test $? = 0 > '
It probably is better to simply drop "test $? = 0" and keep the expect-success, no?