test_must_fail: 129 is a valid error code from usage()

gitster · gitster · commit 8ee002fd3d87 · 2008-03-27T12:13:39.000-07:00
When a git command is run under test_must_fail to make sure that
the argument parser catches bogus command line, it exits with 129.
We need to catch it as a valid "graceful error exit".

Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/t/test-lib.sh b/t/test-lib.sh
@@ -300,7 +300,7 @@ test_expect_code () {
 
 test_must_fail () {
 	"$@"
-	test $? -gt 0 -a $? -le 128
+	test $? -gt 0 -a $? -le 129
 }
 
 # test_cmp is a helper function to compare actual and expected output.

Original file line number	Diff line number	Diff line change
`@@ -300,7 +300,7 @@ test_expect_code () {`
`300`	`300`
`301`	`301`	`test_must_fail () {`
`302`	`302`	`"$@"`
`303`		`- test $? -gt 0 -a $? -le 128`
	`303`	`+ test $? -gt 0 -a $? -le 129`
`304`	`304`	`}`
`305`	`305`
`306`	`306`	`# test_cmp is a helper function to compare actual and expected output.`