diff options
| author | Lars Hjemli | 2008-02-24 19:43:46 +0100 |
|---|---|---|
| committer | Lars Hjemli | 2008-02-24 19:53:08 +0100 |
| commit | 20cdffd5e086ebfb8ba6a82ab98ab27276657111 (patch) | |
| tree | 3d6b33855468682275a2580d09ba5c6c24726fa9 /tests | |
| parent | Make output from `make test` more readable (diff) | |
| download | cgit-20cdffd5e086ebfb8ba6a82ab98ab27276657111.tar.gz 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>
Diffstat (limited to 'tests')
| -rwxr-xr-x | tests/t0010-validate-html.sh | 5 |
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 |