Browse Source

[PATCH] Demonstrate broken t6001 test case function

Junio discovered a problem where an actual test case break was hidden
because pipelines are not handled properly by the test infrastructure in
t6001.

This patch fixes the broken infrastructure (and demonstrates the break
explicitly).

A subsequent patch in this series will fix the test case so that it
doesn't fail.

Signed-off-by: Jon Seymour <jon.seymour@gmail.com>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
maint
Jon Seymour 20 years ago committed by Linus Torvalds
parent
commit
2ba1beebd3
  1. 5
      t/t6001-rev-list-merge-order.sh

5
t/t6001-rev-list-merge-order.sh

@ -85,13 +85,12 @@ check_output() @@ -85,13 +85,12 @@ check_output()
{
_name=$1
shift 1
if "$@" | entag > $_name.actual
if eval "$*" | entag > $_name.actual
then
diff $_name.expected $_name.actual
else
return 1;
fi
}

# Turn a reasonable test description into a reasonable test name.
@ -114,7 +113,7 @@ test_output_expect_success() @@ -114,7 +113,7 @@ test_output_expect_success()
[ $# -eq 2 ] || error "usage: test_output_expect_success description test <<EOF ... EOF"
_name=$(echo $_description | name_from_description)
cat > $_name.expected
test_expect_success "$_description" "check_output $_name $_test"
test_expect_success "$_description" "check_output $_name \"$_test\""
}

# --- end of stuff to move ---

Loading…
Cancel
Save