git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 1/6] Modernize t5530-upload-pack-error.

From
Johannes Sixt <j6t@kdbg.org>
Date
Mar 6, 2010, 15:40 UTC
Message-ID
<40f4b185c3661e5826ec2e58124137cd735e7199.1267889703.git.j6t@kdbg.org>
In-Reply-To
<cover.1267889703.git.j6t@kdbg.org>
Some tests did not use test_must_fail.
Signed-off-by: Johannes Sixt <j6t@kdbg.org>
---
 t/t5530-upload-pack-error.sh |   18 +++++++++---------
 1 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/t/t5530-upload-pack-error.sh b/t/t5530-upload-pack-error.sh
index a696b87..044603c 100755
--- a/t/t5530-upload-pack-error.sh
+++ b/t/t5530-upload-pack-error.sh
@@ -32,9 +32,9 @@ test_expect_success 'fsck fails' '
 
 test_expect_success 'upload-pack fails due to error in pack-objects packing' '
 
-	! echo "0032want $(git rev-parse HEAD)
-00000009done
-0000" | git upload-pack . > /dev/null 2> output.err &&
+	printf "0032want %s\n00000009done\n0000" \
+		$(git rev-parse HEAD) >input &&
+	test_must_fail git upload-pack . <input >/dev/null 2>output.err &&
 	grep "unable to read" output.err &&
 	grep "pack-objects died" output.err
 '
@@ -51,9 +51,9 @@ test_expect_success 'fsck fails' '
 '
 test_expect_success 'upload-pack fails due to error in rev-list' '
 
-	! echo "0032want $(git rev-parse HEAD)
-0034shallow $(git rev-parse HEAD^)00000009done
-0000" | git upload-pack . > /dev/null 2> output.err &&
+	printf "0032want %s\n0034shallow %s00000009done\n0000" \
+		$(git rev-parse HEAD) $(git rev-parse HEAD^) >input &&
+	test_must_fail git upload-pack . <input >/dev/null 2>output.err &&
 	# pack-objects survived
 	grep "Total.*, reused" output.err &&
 	# but there was an error, which must have been in rev-list
@@ -62,9 +62,9 @@ test_expect_success 'upload-pack fails due to error in rev-list' '
 
 test_expect_success 'upload-pack fails due to error in pack-objects enumeration' '
 
-	! echo "0032want $(git rev-parse HEAD)
-00000009done
-0000" | git upload-pack . > /dev/null 2> output.err &&
+	printf "0032want %s\n00000009done\n0000" \
+		$(git rev-parse HEAD) >input &&
+	test_must_fail git upload-pack . <input >/dev/null 2>output.err &&
 	grep "bad tree object" output.err &&
 	grep "pack-objects died" output.err
 '
-- 
1.7.0.rc2.65.g7b13a
Next: Johannes Sixt
Message 1 of 6 in “Modernize t5530-upload-pack-error.”
  1. 1/6 Modernize t5530-upload-pack-error.Johannes Sixt, Mar 6, 2010
  2. 2/6 Make report() from usage.c public as vreportf() and use it.Johannes Sixt, Mar 6, 2010
  3. 3/6 Fix signature of fcntl() compatibility dummyJohannes Sixt, Mar 6, 2010
  4. 4/6 Windows: more pthreads functionsJohannes Sixt, Mar 6, 2010
  5. 5/6 Reimplement async procedures using pthreadsJohannes Sixt, Mar 6, 2010
  6. 6/6 Dying in an async procedure should only exit the thread, not the process.Johannes Sixt, Mar 6, 2010

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.