threads / patch / 7515

patchFix t4200-rerere for white-space from "wc -l"

Subject: [PATCH] Fix t4200-rerere for white-space from "wc -l"

## tl;dr

One message between Apr 4, 2007 and Apr 4, 2007. Diffs are folded; open one to read it.

replies: 0people: 1as markdown or json

Brian Gernhardt· Apr 4, 2007, 19:39 UTC · lore

On OS X, wc outputs 6 spaces before the number of lines, so the test expecting the string "10" failed. Changing the test to use -eq for numerical equality ignores the problematic whitespace.

Also fix the grammar of the test name while making changes to it.
There's only one preimage, so it's "has", not "have".
---
 t/t4200-rerere.sh |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
Show changes to t/t4200-rerere.sh +2 −2
diff --git a/t/t4200-rerere.sh b/t/t4200-rerere.sh
index 8b611bb..cd939ac 100755
--- a/t/t4200-rerere.sh
+++ b/t/t4200-rerere.sh
@@ -50,10 +50,10 @@ test_expect_success 'recorded preimage' "grep ======= $rr/preimage"
 test_expect_success 'no postimage or thisimage yet' \
 	"test ! -f $rr/postimage -a ! -f $rr/thisimage"
 
-test_expect_success 'preimage have right number of lines' '
+test_expect_success 'preimage has right number of lines' '
 
 	cnt=$(sed -ne "/^<<<<<<</,/^>>>>>>>/p" $rr/preimage | wc -l) &&
-	test "$cnt" = 10
+	test "$cnt" -eq 10
 
 '
 
-- 
1.5.1.rc3.30.ga8f4

← back to recent threads