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

[RFC PATCHv2 05/16] t4002 (diff-basic): use test_might_fail for commands that might fail

From
Elijah Newren <newren@gmail.com>
Date
Sep 24, 2010, 22:22 UTC
Message-ID
<1285366976-22216-6-git-send-email-newren@gmail.com>
In-Reply-To
<1285366976-22216-1-git-send-email-newren@gmail.com>

Also replace '|| return 1' by '&&' to allow chain of operations to be checked for proper return status.

Signed-off-by: Elijah Newren <newren@gmail.com>
---
 t/t4002-diff-basic.sh |   12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/t/t4002-diff-basic.sh b/t/t4002-diff-basic.sh
index 73441a5..340aa7d 100755
--- a/t/t4002-diff-basic.sh
+++ b/t/t4002-diff-basic.sh
@@ -205,8 +205,8 @@ test_expect_success \
     'rm -fr Z [A-Z][A-Z] &&
      git read-tree $tree_A &&
      git checkout-index -f -a &&
-     git read-tree --reset $tree_O || return 1
-     git update-index --refresh >/dev/null ;# this can exit non-zero
+     git read-tree --reset $tree_O &&
+     test_might_fail git update-index --refresh >/dev/null &&
      git diff-files >.test-a &&
      cmp_diff_files_output .test-a .test-recursive-OA'
 
@@ -215,8 +215,8 @@ test_expect_success \
     'rm -fr Z [A-Z][A-Z] &&
      git read-tree $tree_B &&
      git checkout-index -f -a &&
-     git read-tree --reset $tree_O || return 1
-     git update-index --refresh >/dev/null ;# this can exit non-zero
+     git read-tree --reset $tree_O &&
+     test_might_fail git update-index --refresh >/dev/null &&
      git diff-files >.test-a &&
      cmp_diff_files_output .test-a .test-recursive-OB'
 
@@ -225,8 +225,8 @@ test_expect_success \
     'rm -fr Z [A-Z][A-Z] &&
      git read-tree $tree_B &&
      git checkout-index -f -a &&
-     git read-tree --reset $tree_A || return 1
-     git update-index --refresh >/dev/null ;# this can exit non-zero
+     git read-tree --reset $tree_A &&
+     test_might_fail git update-index --refresh >/dev/null &&
      git diff-files >.test-a &&
      cmp_diff_files_output .test-a .test-recursive-AB'
 
-- 
1.7.3.95.g14291
Previous: Elijah NewrenNext: Ævar Arnfjörð Bjarmason
Message 8 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.