# [PATCH] t7501: always use test_cmp instead of diff 5 messages from 2008-09-09 to 2008-09-10. Participants: Miklos Vajna, Jeff King, Junio C Hamano. Thread: https://gitlist.dev/t/15467 ## Miklos Vajna, 2008-09-09 23:41 Subject: [PATCH] t7501: always use test_cmp instead of diff Message-ID: <1221003666-17115-1-git-send-email-vmiklos@frugalware.org> URL: https://gitlist.dev/e/1221003666-17115-1-git-send-email-vmiklos%40frugalware.org ``` This should make the output more readable (by default using diff -u) when some tests fail. Signed-off-by: Miklos Vajna --- t/t7501-commit.sh | 12 ++++++------ 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/t/t7501-commit.sh b/t/t7501-commit.sh index 0edd9dd..6f8c038 100755 --- a/t/t7501-commit.sh +++ b/t/t7501-commit.sh @@ -141,7 +141,7 @@ EOF test_expect_success \ 'validate git-rev-list output.' \ - 'diff current expected' + 'test_cmp current expected' test_expect_success 'partial commit that involves removal (1)' ' @@ -151,7 +151,7 @@ test_expect_success 'partial commit that involves removal (1)' ' git commit -m "Partial: add elif" elif && git diff-tree --name-status HEAD^ HEAD >current && echo "A elif" >expected && - diff expected current + test_cmp expected current ' @@ -160,7 +160,7 @@ test_expect_success 'partial commit that involves removal (2)' ' git commit -m "Partial: remove file" file && git diff-tree --name-status HEAD^ HEAD >current && echo "D file" >expected && - diff expected current + test_cmp expected current ' @@ -171,7 +171,7 @@ test_expect_success 'partial commit that involves removal (3)' ' git commit -m "Partial: modify elif" elif && git diff-tree --name-status HEAD^ HEAD >current && echo "M elif" >expected && - diff expected current + test_cmp expected current ' @@ -187,7 +187,7 @@ test_expect_success 'amend commit to fix author' ' expected && git commit --amend --author="$author" && git cat-file -p HEAD > current && - diff expected current + test_cmp expected current ' @@ -256,7 +256,7 @@ test_expect_success 'amend commit to fix author' ' expected && git commit --amend --author="$author" && git cat-file -p HEAD > current && - diff expected current + test_cmp expected current ' -- 1.6.0.1 ``` ## Miklos Vajna, 2008-09-09 23:49 Subject: Re: [PATCH] t7501: always use test_cmp instead of diff Message-ID: <20080909234924.GN4829@genesis.frugalware.org> URL: https://gitlist.dev/e/20080909234924.GN4829%40genesis.frugalware.org In-Reply-To: <1221003666-17115-1-git-send-email-vmiklos@frugalware.org> ``` On Wed, Sep 10, 2008 at 01:41:06AM +0200, Miklos Vajna wrote: > This should make the output more readable (by default using diff -u) > when some tests fail. Sorry for the duplication, I didn't want to send it twice. I also just realised after sending that I forgot to rebase it again current master, where git-rev-list is replaced by git rev-list, but I guess git am -3 handles it. ;-) ``` ## Jeff King, 2008-09-09 23:54 Subject: Re: [PATCH] t7501: always use test_cmp instead of diff Message-ID: <20080909235414.GA4768@coredump.intra.peff.net> URL: https://gitlist.dev/e/20080909235414.GA4768%40coredump.intra.peff.net In-Reply-To: <1221003666-17115-1-git-send-email-vmiklos@frugalware.org> ``` On Wed, Sep 10, 2008 at 01:41:06AM +0200, Miklos Vajna wrote: > test_expect_success \ > 'validate git-rev-list output.' \ > - 'diff current expected' > + 'test_cmp current expected' We seem to use the convention of test_cmp elsewhere, rather than test_cmp as you have here. Most noticeably, that means the diff will show deviations from expected, rather "what should be done to make this as expected". But it is also possible that in the future test_cmp could be enhanced to do something more interesting. I don't think it is worth it to go fix all such instances, but while you are here... -Peff ``` ## Junio C Hamano, 2008-09-10 00:02 Subject: Re: [PATCH] t7501: always use test_cmp instead of diff Message-ID: <7vljy03m2m.fsf@gitster.siamese.dyndns.org> URL: https://gitlist.dev/e/7vljy03m2m.fsf%40gitster.siamese.dyndns.org In-Reply-To: <20080909235414.GA4768@coredump.intra.peff.net> ``` Jeff King writes: > On Wed, Sep 10, 2008 at 01:41:06AM +0200, Miklos Vajna wrote: > >> test_expect_success \ >> 'validate git-rev-list output.' \ >> - 'diff current expected' >> + 'test_cmp current expected' > > We seem to use the convention of > > test_cmp > > elsewhere, rather than > > test_cmp > > as you have here. Most noticeably, that means the diff will show > deviations from expected, rather "what should be done to make this as > expected". Yes, please. I am glad to see somebody noticed it (I've been fixing these whenever I touched vicinity of the ones that had comparisons swapped). ``` ## Miklos Vajna, 2008-09-10 17:32 Subject: [PATCH] t7501: always use test_cmp instead of diff Message-ID: <1221067964-12287-1-git-send-email-vmiklos@frugalware.org> URL: https://gitlist.dev/e/1221067964-12287-1-git-send-email-vmiklos%40frugalware.org In-Reply-To: <7vljy03m2m.fsf@gitster.siamese.dyndns.org> ``` This should make the output more readable (by default using diff -u) when some tests fail. Also changed the diff order from "current expected" to "expected current". Signed-off-by: Miklos Vajna --- On Tue, Sep 09, 2008 at 05:02:09PM -0700, Junio C Hamano wrote: > > We seem to use the convention of > > > > test_cmp > > > > elsewhere, rather than > > > > test_cmp > > > > as you have here. Most noticeably, that means the diff will show > > deviations from expected, rather "what should be done to make this > > as expected". > > Yes, please. Here is an updated patch that does this as well. t/t7501-commit.sh | 12 ++++++------ 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/t/t7501-commit.sh b/t/t7501-commit.sh index 469bff8..63bfc6d 100755 --- a/t/t7501-commit.sh +++ b/t/t7501-commit.sh @@ -141,7 +141,7 @@ EOF test_expect_success \ 'validate git rev-list output.' \ - 'diff current expected' + 'test_cmp expected current' test_expect_success 'partial commit that involves removal (1)' ' @@ -151,7 +151,7 @@ test_expect_success 'partial commit that involves removal (1)' ' git commit -m "Partial: add elif" elif && git diff-tree --name-status HEAD^ HEAD >current && echo "A elif" >expected && - diff expected current + test_cmp expected current ' @@ -160,7 +160,7 @@ test_expect_success 'partial commit that involves removal (2)' ' git commit -m "Partial: remove file" file && git diff-tree --name-status HEAD^ HEAD >current && echo "D file" >expected && - diff expected current + test_cmp expected current ' @@ -171,7 +171,7 @@ test_expect_success 'partial commit that involves removal (3)' ' git commit -m "Partial: modify elif" elif && git diff-tree --name-status HEAD^ HEAD >current && echo "M elif" >expected && - diff expected current + test_cmp expected current ' @@ -187,7 +187,7 @@ test_expect_success 'amend commit to fix author' ' expected && git commit --amend --author="$author" && git cat-file -p HEAD > current && - diff expected current + test_cmp expected current ' @@ -256,7 +256,7 @@ test_expect_success 'amend commit to fix author' ' expected && git commit --amend --author="$author" && git cat-file -p HEAD > current && - diff expected current + test_cmp expected current ' -- 1.6.0.1 ```