Skip to content

Commit

Permalink
Fix bogus tests on rev-list output.
Browse files Browse the repository at this point in the history
These tests seem to mean checking the output with expected
result, but was not doing its handrolled test helper function.

Also fix the guard to workaround wc output that have whitespace
padding, which was broken but not exposed because the test was
not testing it ;-).

Signed-off-by: Junio C Hamano <junkio@cox.net>
  • Loading branch information
Junio C Hamano committed Dec 27, 2005
1 parent 8eafa3d commit e9add36
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion t/t6001-rev-list-merge-order.sh
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@ save_tag g4 unique_commit g6 tree -p g3 -p h2

git-update-ref HEAD $(tag l5)

test_expect_success 'rev-list has correct number of entries' 'git-rev-list HEAD | wc -l | tr -s " "' <<EOF
test_output_expect_success 'rev-list has correct number of entries' 'git-rev-list HEAD | wc -l | tr -d \" \"' <<EOF
19
EOF

Expand Down
2 changes: 1 addition & 1 deletion t/t6003-rev-list-topo-order.sh
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ save_tag g4 unique_commit g6 tree -p g3 -p h2

git-update-ref HEAD $(tag l5)

test_expect_success 'rev-list has correct number of entries' 'git-rev-list HEAD | wc -l | tr -s " "' <<EOF
test_output_expect_success 'rev-list has correct number of entries' 'git-rev-list HEAD | wc -l | tr -d \" \"' <<EOF
19
EOF

Expand Down

0 comments on commit e9add36

Please sign in to comment.