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

[GSoC PATCH v2] t2000: modernize path checks with test_path_is_* helpers

From
Zakariyah Ali <zakariyahali100@gmail.com>
Date
Mar 26, 2026, 19:26 UTC
Message-ID
<20260326192603.23961-1-zakariyahali100@gmail.com>
In-Reply-To
<CAPAKoce5BeXMPtok+dni+WDTkokCAq0o4uMHGg1KZto9-UniXQ@mail.gmail.com>

Replace bare 'test -f/-d' and 'test ! -h' assertions with dedicated helpers. These helpers report loudly what expectation wasn't met, therefore making debugging easier.

Signed-off-by: Zakariyah Ali <zakariyahali100@gmail.com>
---
 t/t2000-conflict-when-checking-files-out.sh | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/t/t2000-conflict-when-checking-files-out.sh b/t/t2000-conflict-when-checking-files-out.sh
index f18616ad2b..96bae6c53d 100755
--- a/t/t2000-conflict-when-checking-files-out.sh
+++ b/t/t2000-conflict-when-checking-files-out.sh
@@ -58,7 +58,9 @@ test_expect_success \
 
 test_expect_success \
     'git checkout-index conflicting paths.' \
-    'test -f path0 && test -d path1 && test -f path1/file1'
+    'test_path_is_file path0 &&
+     test_path_is_dir path1 &&
+     test_path_is_file path1/file1'
 
 test_expect_success SYMLINKS 'checkout-index -f twice with --prefix' '
 	mkdir -p tar/get &&
@@ -127,9 +129,9 @@ test_debug 'show_files $tree2'
 
 test_expect_success \
     'checking out conflicting path with -f' \
-    'test ! -h path2 && test -d path2 &&
-     test ! -h path3 && test -d path3 &&
-     test ! -h path2/file0 && test -f path2/file0 &&
-     test ! -h path3/file1 && test -f path3/file1'
+    'test_path_is_dir_not_symlink path2 &&
+     test_path_is_dir_not_symlink path3 &&
+     test_path_is_file_not_symlink path2/file0 &&
+     test_path_is_file_not_symlink path3/file1'
 
 test_done
-- 
2.43.0
Previous: PabloNext: Junio C Hamano
Message 3 of 17 in “Github Patch”
  1. Zakariyah AliMar 26, 2026
  2. PabloMar 26, 2026
  3. t2000: modernize path checks with test_path_is_* helpersZakariyah Ali, Mar 26, 2026
  4. Junio C HamanoMar 26, 2026
  5. [GSoC][PATCH v3] t2000: modernise overall structureZakariyah Ali, Mar 27, 2026
  6. Zakariyah AliMar 30, 2026
  7. Tian YuchenApr 1, 2026
  8. 1/1 t2000: modernize overall structure and path checksZakariyah Ali, Apr 5, 2026
  9. Karthik NayakApr 5, 2026
  10. Tian YuchenApr 6, 2026
  11. t2000: modernize overall structure and path checksZakariyah Ali, Apr 7, 2026
  12. Junio C HamanoApr 7, 2026
  13. Junio C HamanoApr 7, 2026
  14. t2000: consolidate second scenario into a single test blockZakariyah Ali, Apr 29, 2026
  15. Zakariyah AliMay 5, 2026
  16. Junio C HamanoMay 12, 2026
  17. Zakariyah AliMay 12, 2026

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.