Re: [PATCH v4 3/6] tests: new test for orderfile options
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Apr 23, 2014, 17:38 UTC
- Message-ID
- <xmqq8uqw0ww0.fsf@gitster.dls.corp.google.com>
- In-Reply-To
- <1398255277-26303-3-git-send-email-mst@redhat.com>
"Michael S. Tsirkin" <mst@redhat.com> writes:
Show 30 quoted lines
> The test is very basic and can be extended. > Couldn't find a good existing place to put it, > so created a new file. > > Signed-off-by: Michael S. Tsirkin <mst@redhat.com> > --- > t/t4056-diff-order.sh | 63 +++++++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 63 insertions(+) > create mode 100755 t/t4056-diff-order.sh > > diff --git a/t/t4056-diff-order.sh b/t/t4056-diff-order.sh > new file mode 100755 > index 0000000..a247b7a > --- /dev/null > +++ b/t/t4056-diff-order.sh > @@ -0,0 +1,63 @@ > +#!/bin/sh > + > +test_description='diff orderfile' > + > +. ./test-lib.sh > + > +test_expect_success 'setup' ' > + as="a a a a a a a a" && # eight a > + test_write_lines $as >foo && > + test_write_lines $as >bar && > + git add foo bar && > + git commit -a -m initial && > + test_write_lines $as b >foo && > + test_write_lines $as b >bar &&
These lines are curiously jaggy; are you indenting with spaces? Please don't.
My running "git am -s3c" may auto-fix these indentation issues, so please wait before seeing what may be pushed out on 'pu'. It may turn out that there is no other need for rerolling this patch in the series.
Thanks.
Show 49 quoted lines
> + git commit -a -m first &&
> + test_write_lines bar foo >bar-then-foo &&
> + test_write_lines foo bar >foo-then-bar &&
> + git diff -Ofoo-then-bar HEAD~1..HEAD >diff-foo-then-bar &&
> + git diff -Obar-then-foo HEAD~1..HEAD >diff-bar-then-foo
> +'
> +
> +test_diff_well_formed () {
> + grep ^+b "$1" >added
> + grep ^-b "$1" >removed
> + grep ^+++ "$1" >oldfiles
> + grep ^--- "$1" >newfiles
> + test_line_count = 2 added &&
> + test_line_count = 0 removed &&
> + test_line_count = 2 oldfiles &&
> + test_line_count = 2 newfiles
> +}
> +
> +test_expect_success 'diff output with -O is well-formed' '
> + test_diff_well_formed diff-foo-then-bar &&
> + test_diff_well_formed diff-bar-then-foo
> +'
> +
> +test_expect_success 'flag -O affects diff output' '
> + ! test_cmp diff-foo-then-bar diff-bar-then-foo
> +'
> +
> +test_expect_success 'orderfile is same as -O' '
> + test_config diff.orderfile foo-then-bar &&
> + git diff HEAD~1..HEAD >diff-foo-then-bar-config &&
> + test_config diff.orderfile bar-then-foo &&
> + git diff HEAD~1..HEAD >diff-bar-then-foo-config &&
> + test_cmp diff-foo-then-bar diff-foo-then-bar-config &&
> + test_cmp diff-bar-then-foo diff-bar-then-foo-config
> +'
> +
> +test_expect_success '-O overrides orderfile' '
> + test_config diff.orderfile foo-then-bar &&
> + git diff -Obar-then-foo HEAD~1..HEAD >diff-bar-then-foo-flag &&
> + test_cmp diff-bar-then-foo diff-bar-then-foo-flag
> +'
> +
> +test_expect_success '/dev/null is same as no orderfile' '
> + git diff -O/dev/null HEAD~1..HEAD>diff-null-orderfile &&
> + git diff HEAD~1..HEAD >diff-default &&
> + test_cmp diff-null-orderfile diff-default
> +'
> +
> +test_done