aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLars Hjemli2008-02-24 19:43:46 +0100
committerLars Hjemli2008-02-24 19:53:08 +0100
commit20cdffd5e086ebfb8ba6a82ab98ab27276657111 (patch)
tree3d6b33855468682275a2580d09ba5c6c24726fa9
parentb44b02a98253e78334f7fd13d9c4e1eb59562392 (diff)
downloadcgit-20cdffd5e086ebfb8ba6a82ab98ab27276657111.tar.gz
cgit-20cdffd5e086ebfb8ba6a82ab98ab27276657111.tar.bz2
cgit-20cdffd5e086ebfb8ba6a82ab98ab27276657111.zip
t0010-validate.sh: return on cgit errors
The earlier segfault in cgit passed by unnoticed by this test-script due to the pipe between cgit and sed. There might be a more elegant solution to this problem, but my shellscript- foo is weak. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
-rwxr-xr-xtests/t0010-validate-html.sh5
1 files changed, 4 insertions, 1 deletions
diff --git a/tests/t0010-validate-html.sh b/tests/t0010-validate-html.sh
index 907a415..94aa52b 100755
--- a/tests/t0010-validate-html.sh
+++ b/tests/t0010-validate-html.sh
@@ -7,9 +7,12 @@ test_url()
7{ 7{
8 tidy_opt="-eq" 8 tidy_opt="-eq"
9 test -z "$NO_TIDY_WARNINGS" || tidy_opt+=" --show-warnings no" 9 test -z "$NO_TIDY_WARNINGS" || tidy_opt+=" --show-warnings no"
10 cgit_url "$1" | sed -e "1,4d" >trash/tidy-$test_count 10 cgit_url "$1" >trash/tidy-$test_count || return
11 sed -ie "1,4d" trash/tidy-$test_count || return
11 tidy $tidy_opt trash/tidy-$test_count 12 tidy $tidy_opt trash/tidy-$test_count
12 rc=$? 13 rc=$?
14
15 # tidy returns with exitcode 1 on warnings, 2 on error
13 if test $rc = 2 16 if test $rc = 2
14 then 17 then
15 false 18 false