From: Jakub Narebski <jnareb@gmail.com>
Date: Sun, 11 Jul 2010 08:33:33 GMT
Subject: Re: [PATCH] t3000 (ls-files -o): modernize style
Message-ID: <m3d3uul8pd.fsf@localhost.localdomain>
In-Reply-To: <20100711042025.GA7424@burratino>

Jonathan Nieder <jrnieder@gmail.com> writes:

> This script is part of the second batch of tests, from the same day
> the test infrastructure was added to git.  Update it to use a more
> modern style in the spirit of v1.6.4-rc0~45^2~2 (2009-05-22).
> In particular:
> 
>  - Put setup code inside test assertions, to avoid unexpected
>    breakages and avoid stray output without -v (as t/README
>    recommends); and

O.K., but...
 
> -cat >expected1 <<EOF
> -expected1
> -expected2
> -expected3
> -output
> -path0
> -path1
> -path2-junk
> -path2/file2
> -EOF
> -sed -e 's|path2/file2|path2/|' <expected1 >expected2
> -cat <expected2 >expected3
> -echo path4/ >>expected2
> -
> +test_expect_success 'setup: expected output' '
> +	cat >expected1 <<-\EOF &&
> +	expected1
> +	expected2
> +	expected3
> +	output
> +	path0
> +	path1
> +	path2-junk
> +	path2/file2
> +	EOF
> +
> +	sed -e "s|path2/file2|path2/|" <expected1 >expected2 &&
> +	cp expected2 expected3 &&
> +	echo path4/ >>expected2
> +'

... I don't think it is really needed.  There isn't anything there
that can fail, and there isn't anything there that produces any
output.

BTW. do all sane shells that can be used to run test suite support
'<<-' here-doc redirection operators?  Is it in POSIX?

-- 
Jakub Narebski
Poland
ShadeHawk on #git