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

[RFC PATCHv2 10/16] t5602 (clone-remote-exec): add missing &&

From
Elijah Newren <newren@gmail.com>
Date
Sep 24, 2010, 22:22 UTC
Message-ID
<1285366976-22216-11-git-send-email-newren@gmail.com>
In-Reply-To
<1285366976-22216-1-git-send-email-newren@gmail.com>
I also had to add a couple test_must_fail invocations as well.
Signed-off-by: Elijah Newren <newren@gmail.com>
---
 t/t5602-clone-remote-exec.sh |   14 +++++++-------
 1 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/t/t5602-clone-remote-exec.sh b/t/t5602-clone-remote-exec.sh
index deffdae..6aa6e0d 100755
--- a/t/t5602-clone-remote-exec.sh
+++ b/t/t5602-clone-remote-exec.sh
@@ -5,21 +5,21 @@ test_description=clone
 . ./test-lib.sh
 
 test_expect_success setup '
-	echo "#!/bin/sh" > not_ssh
-	echo "echo \"\$*\" > not_ssh_output" >> not_ssh
-	echo "exit 1" >> not_ssh
+	echo "#!/bin/sh" > not_ssh &&
+	echo "echo \"\$*\" > not_ssh_output" >> not_ssh &&
+	echo "exit 1" >> not_ssh &&
 	chmod +x not_ssh
 '
 
 test_expect_success 'clone calls git upload-pack unqualified with no -u option' '
-	GIT_SSH=./not_ssh git clone localhost:/path/to/repo junk
-	echo "localhost git-upload-pack '\''/path/to/repo'\''" >expected
+	GIT_SSH=./not_ssh test_must_fail git clone localhost:/path/to/repo junk &&
+	echo "localhost git-upload-pack '\''/path/to/repo'\''" >expected &&
 	test_cmp expected not_ssh_output
 '
 
 test_expect_success 'clone calls specified git upload-pack with -u option' '
-	GIT_SSH=./not_ssh git clone -u ./something/bin/git-upload-pack localhost:/path/to/repo junk
-	echo "localhost ./something/bin/git-upload-pack '\''/path/to/repo'\''" >expected
+	GIT_SSH=./not_ssh test_must_fail git clone -u ./something/bin/git-upload-pack localhost:/path/to/repo junk &&
+	echo "localhost ./something/bin/git-upload-pack '\''/path/to/repo'\''" >expected &&
 	test_cmp expected not_ssh_output
 '
 
-- 
1.7.3.95.g14291
Previous: Jakub NarebskiNext: Elijah Newren
Message 18 of 31 in “[RFC PATCHv2 00/16] Add missing &&'s in the testsuite”
  1. Elijah NewrenSep 24, 2010
  2. 01/16 t3020 (ls-files-error-unmatch): remove stray '1' from end of fileElijah Newren, Sep 24, 2010
  3. 02/16 t4017 (diff-retval): replace manual exit code check with test_expect_codeElijah Newren, Sep 24, 2010
  4. Ævar Arnfjörð BjarmasonSep 24, 2010
  5. 03/16 t1001 (read-tree-m-2way): add missing &&Elijah Newren, Sep 24, 2010
  6. Ævar Arnfjörð BjarmasonSep 24, 2010
  7. 04/16 t1002 (read-tree-m-u-2way): add missing &&Elijah Newren, Sep 24, 2010
  8. 05/16 t4002 (diff-basic): use test_might_fail for commands that might failElijah Newren, Sep 24, 2010
  9. Ævar Arnfjörð BjarmasonSep 24, 2010
  10. 06/16 t4202 (log): Replace '<git-command> || :' with test_might_failElijah Newren, Sep 24, 2010
  11. 07/16 t3600 (rm): add lots of missing &&Elijah Newren, Sep 24, 2010
  12. Ævar Arnfjörð BjarmasonSep 24, 2010
  13. 08/16 t4019 (diff-wserror): add lots of missing &&Elijah Newren, Sep 24, 2010
  14. 09/16 t4026 (color): add missing &&Elijah Newren, Sep 24, 2010
  15. Ævar Arnfjörð BjarmasonSep 24, 2010
  16. Jeff KingSep 25, 2010
  17. Jakub NarebskiSep 25, 2010
  18. 10/16 t5602 (clone-remote-exec): add missing &&Elijah Newren, Sep 24, 2010
  19. 11/16 t6016 (rev-list-graph-simplify-history): add missing &&Elijah Newren, Sep 24, 2010
  20. Ævar Arnfjörð BjarmasonSep 24, 2010
  21. 12/16 t6050 (replace): add missing &&Elijah Newren, Sep 24, 2010
  22. Ævar Arnfjörð BjarmasonSep 24, 2010
  23. 13/16 t7001 (mv): add missing &&Elijah Newren, Sep 24, 2010
  24. Ævar Arnfjörð BjarmasonSep 24, 2010
  25. 14/16 t7601 (merge-pull-config): add missing &&Elijah Newren, Sep 24, 2010
  26. 15/16 t7800 (difftool): add missing &&Elijah Newren, Sep 24, 2010
  27. Ævar Arnfjörð BjarmasonSep 24, 2010
  28. 16/16 Add missing &&'s throughout the testsuiteElijah Newren, Sep 24, 2010
  29. Ævar Arnfjörð BjarmasonSep 24, 2010
  30. Elijah NewrenSep 25, 2010
  31. Ævar Arnfjörð BjarmasonSep 25, 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.