t0005: fix broken &&-chains

peff · gitster · commit 635ce72fae23 · 2015-03-20T11:35:57.000-07:00
The ":" noop command always returns true, so it is fine to
include these lines in an &amp;&amp;-chain (and it appeases
--chain-lint).

Signed-off-by: Jeff King &lt;peff@peff.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/t/t0005-signals.sh b/t/t0005-signals.sh
@@ -40,12 +40,12 @@ test_expect_success 'create blob' '
 '
 
 test_expect_success !MINGW 'a constipated git dies with SIGPIPE' '
-	OUT=$( ((large_git; echo $? 1>&3) | :) 3>&1 )
+	OUT=$( ((large_git; echo $? 1>&3) | :) 3>&1 ) &&
 	test "$OUT" -eq 141
 '
 
 test_expect_success !MINGW 'a constipated git dies with SIGPIPE even if parent ignores it' '
-	OUT=$( ((trap "" PIPE; large_git; echo $? 1>&3) | :) 3>&1 )
+	OUT=$( ((trap "" PIPE; large_git; echo $? 1>&3) | :) 3>&1 ) &&
 	test "$OUT" -eq 141
 '
 

Original file line number	Diff line number	Diff line change
`@@ -40,12 +40,12 @@ test_expect_success 'create blob' '`
`40`	`40`	`'`
`41`	`41`
`42`	`42`	`test_expect_success !MINGW 'a constipated git dies with SIGPIPE' '`
`43`		`- OUT=$( ((large_git; echo $? 1>&3) \| :) 3>&1 )`
	`43`	`+ OUT=$( ((large_git; echo $? 1>&3) \| :) 3>&1 ) &&`
`44`	`44`	`test "$OUT" -eq 141`
`45`	`45`	`'`
`46`	`46`
`47`	`47`	`test_expect_success !MINGW 'a constipated git dies with SIGPIPE even if parent ignores it' '`
`48`		`- OUT=$( ((trap "" PIPE; large_git; echo $? 1>&3) \| :) 3>&1 )`
	`48`	`+ OUT=$( ((trap "" PIPE; large_git; echo $? 1>&3) \| :) 3>&1 ) &&`
`49`	`49`	`test "$OUT" -eq 141`
`50`	`50`	`'`
`51`	`51`