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

[PATCH] Fix tests when using bash_completion

From
RPRon Parker <ron.parker@mobot.org>
Date
Apr 23, 2007, 20:51 UTC
Message-ID
<769697AE3E25EF4FBC0763CD91AB1B0201D496D0@MBGMail01.mobot.org>

When using bash_completion, 'cd' echos the target directory name to standard out. If 'cd' participates in a pipe whose output is compared against an expected value, this causes a false failure report. Pipe cd's stdout to /dev/null in these cases.

Signed-off-by: Ron Parker <ron.parker@mobot.org>
---
	I was going to look into some of the preliminary subproject
	support, since I have a need for it.  But a bunch of the tests
	failed because I run the bash_completion package.
	I don't think I've sent any patches for git before.  Please be
	gentle.  If there is a problem with this patch or the way I've
	sent it, I'll gladly redo it.
 t/t3040-subprojects-basic.sh |    2 +-
 t/t5000-tar-tree.sh          |   10 +++++-----
 t/t5500-fetch-pack.sh        |    8 ++++----
 t/t5502-quickfetch.sh        |    8 ++++----
 t/t7002-grep.sh              |    4 ++--
 5 files changed, 16 insertions(+), 16 deletions(-)
diff --git a/t/t3040-subprojects-basic.sh b/t/t3040-subprojects-basic.sh
index 79b9f23..9543921 100755
--- a/t/t3040-subprojects-basic.sh
+++ b/t/t3040-subprojects-basic.sh
@@ -61,7 +61,7 @@ test_expect_success 'check if git diff works for
subproject elements' \
 test_expect_success 'check if clone works' \
     'git ls-files -s >expected &&
     git clone -l -s . cloned &&
-    ( cd cloned && git ls-files -s ) >current &&
+    ( cd cloned >/dev/null && git ls-files -s ) >current &&
     git diff expected current'
 
 test_expect_success 'removing and adding subproject' \
diff --git a/t/t5000-tar-tree.sh b/t/t5000-tar-tree.sh
index e223c07..a6654a3 100755
--- a/t/t5000-tar-tree.sh
+++ b/t/t5000-tar-tree.sh
@@ -38,7 +38,7 @@ test_expect_success \
      (p=long_path_to_a_file && cd a &&
       for depth in 1 2 3 4 5; do mkdir $p && cd $p; done &&
       echo text >file_with_long_path) &&
-     (cd a && find .) | sort >a.lst'
+     (cd a >/dev/null && find .) | sort >a.lst'
 
 test_expect_success \
     'add files to repository' \
@@ -80,7 +80,7 @@ test_expect_success \
 
 test_expect_success \
     'validate filenames' \
-    '(cd b/a && find .) | sort >b.lst &&
+    '(cd b/a >/dev/null && find .) | sort >b.lst &&
      diff a.lst b.lst'
 
 test_expect_success \
@@ -97,7 +97,7 @@ test_expect_success \
 
 test_expect_success \
     'validate filenames with prefix' \
-    '(cd c/prefix/a && find .) | sort >c.lst &&
+    '(cd c/prefix/a >/dev/null && find .) | sort >c.lst &&
      diff a.lst c.lst'
 
 test_expect_success \
@@ -114,7 +114,7 @@ test_expect_success \
 
 test_expect_success \
     'validate filenames' \
-    '(cd d/a && find .) | sort >d.lst &&
+    '(cd d/a >/dev/null && find .) | sort >d.lst &&
      diff a.lst d.lst'
 
 test_expect_success \
@@ -131,7 +131,7 @@ test_expect_success \
 
 test_expect_success \
     'validate filenames with prefix' \
-    '(cd e/prefix/a && find .) | sort >e.lst &&
+    '(cd e/prefix/a >/dev/null && find .) | sort >e.lst &&
      diff a.lst e.lst'
 
 test_expect_success \
diff --git a/t/t5500-fetch-pack.sh b/t/t5500-fetch-pack.sh
index 48e3d17..d8ed508 100755
--- a/t/t5500-fetch-pack.sh
+++ b/t/t5500-fetch-pack.sh
@@ -131,7 +131,7 @@ pull_to_client 3rd "A" $((1*3)) # old fails
 
 test_expect_success "clone shallow" "git-clone --depth 2 . shallow"
 
-(cd shallow; git-count-objects -v) > count.shallow
+(cd shallow >/dev/null; git-count-objects -v) > count.shallow
 
 test_expect_success "clone shallow object count" \
 	"test \"in-pack: 18\" = \"$(grep in-pack count.shallow)\""
@@ -155,7 +155,7 @@ add B67 $B66
 test_expect_success "pull in shallow repo" \
 	"(cd shallow; git pull .. B)"
 
-(cd shallow; git-count-objects -v) > count.shallow
+(cd shallow >/dev/null; git-count-objects -v) > count.shallow
 test_expect_success "clone shallow object count" \
 	"test \"count: 6\" = \"$(grep count count.shallow)\""
 
@@ -165,14 +165,14 @@ add B69 $B68
 test_expect_success "deepening pull in shallow repo" \
 	"(cd shallow; git pull --depth 4 .. B)"
 
-(cd shallow; git-count-objects -v) > count.shallow
+(cd shallow >/dev/null; git-count-objects -v) > count.shallow
 test_expect_success "clone shallow object count" \
 	"test \"count: 12\" = \"$(grep count count.shallow)\""
 
 test_expect_success "deepening fetch in shallow repo" \
 	"(cd shallow; git fetch --depth 4 .. A:A)"
 
-(cd shallow; git-count-objects -v) > count.shallow
+(cd shallow >/dev/null; git-count-objects -v) > count.shallow
 test_expect_success "clone shallow object count" \
 	"test \"count: 18\" = \"$(grep count count.shallow)\""
 
diff --git a/t/t5502-quickfetch.sh b/t/t5502-quickfetch.sh
index b4760f2..b33e414 100755
--- a/t/t5502-quickfetch.sh
+++ b/t/t5502-quickfetch.sh
@@ -26,7 +26,7 @@ test_expect_success 'clone without alternate' '
 		git remote add -f origin ..
 	) &&
 	cnt=$( (
-		cd cloned &&
+		cd cloned >/dev/null &&
 		git count-objects | sed -e "s/ *objects,.*//"
 	) ) &&
 	test $cnt -eq 3
@@ -54,7 +54,7 @@ test_expect_success 'copy commit and tree but not blob
by hand' '
 	) &&
 
 	cnt=$( (
-		cd cloned &&
+		cd cloned >/dev/null &&
 		git count-objects | sed -e "s/ *objects,.*//"
 	) ) &&
 	test $cnt -eq 6
@@ -64,7 +64,7 @@ test_expect_success 'copy commit and tree but not blob
by hand' '
 	rm -f "cloned/.git/objects/$blob" &&
 
 	cnt=$( (
-		cd cloned &&
+		cd cloned >/dev/null &&
 		git count-objects | sed -e "s/ *objects,.*//"
 	) ) &&
 	test $cnt -eq 5
@@ -79,7 +79,7 @@ test_expect_success 'quickfetch should not leave a
corrupted repository' '
 	) &&
 
 	cnt=$( (
-		cd cloned &&
+		cd cloned >/dev/null &&
 		git count-objects | sed -e "s/ *objects,.*//"
 	) ) &&
 	test $cnt -eq 6
diff --git a/t/t7002-grep.sh b/t/t7002-grep.sh
index 6bfb899..5d2fc4a 100755
--- a/t/t7002-grep.sh
+++ b/t/t7002-grep.sh
@@ -92,7 +92,7 @@ do
 	test_expect_success "grep $L (t-2)" '
 		echo "${HC}t:1:test" >expected &&
 		(
-			cd t &&
+			cd t >/dev/null &&
 			git grep -n -e test $H
 		) >actual &&
 		diff expected actual
@@ -101,7 +101,7 @@ do
 	test_expect_success "grep $L (t-3)" '
 		echo "${HC}t/t:1:test" >expected &&
 		(
-			cd t &&
+			cd t >/dev/null &&
 			git grep --full-name -n -e test $H
 		) >actual &&
 		diff expected actual
-- 
1.5.2.rc0.1.g2cc31
Next: Junio C Hamano
Message 1 of 8 in “Fix tests when using bash_completion”
  1. Fix tests when using bash_completionRon Parker, Apr 23, 2007
  2. Junio C HamanoApr 23, 2007
  3. Julian PhillipsApr 23, 2007
  4. Junio C HamanoApr 23, 2007
  5. [OT] CDPATH (was: Re: [PATCH] Fix tests when using bash_completion)Julian Phillips, Apr 23, 2007
  6. Ron ParkerApr 24, 2007
  7. Junio C HamanoApr 24, 2007
  8. Unset CDPATH in the test libraryRon Parker, Apr 24, 2007

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.