Re: [PATCH v2 3/3] format-patch: check that header line has expected format

From: Torsten Bögershausen <tboegi@web.de>
Date: Dec 14, 2015, 07:16 UTC
Message-ID: <566E6CDF.3020004@web.de>
In-Reply-To: <1450027638-788102-4-git-send-email-sandals@crustytoothpaste.net>

On 13.12.15 18:27, brian m. carlson wrote:

Show 21 quoted lines

> The format of the "From " header line is very specific to allow
> utilities to detect Git-style patches.  Add a test that the patches
> created are in the expected format.
> 
> Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
> ---
>  t/t4014-format-patch.sh | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/t/t4014-format-patch.sh b/t/t4014-format-patch.sh
> index b740e3da..362bc228 100755
> --- a/t/t4014-format-patch.sh
> +++ b/t/t4014-format-patch.sh
> @@ -1437,4 +1437,10 @@ test_expect_success 'format-patch --zero-commit' '
>  	test $cnt = 3
>  '
>  
> +test_expect_success 'From line has expected format' '
> +	git format-patch --stdout v2..v1 >patch2 &&
> +	cnt=$(egrep "^From [0-9a-f]{40} Mon Sep 17 00:00:00 2001" patch2 | wc -l) &&
> +	test $cnt = 3

For these kind of things: test_line_count() is your friend