summaryrefslogtreecommitdiffstatshomepage
diff options
context:
space:
mode:
authorJohn Keeping2013-05-18 18:46:38 +0100
committerJason A. Donenfeld2013-05-22 12:53:06 +0200
commit75bfec6448b44e26c06736253f8ab0197a6652f7 (patch)
treedf0fff85104702712c6d569a14f2749a8461970e
parent1e9f1ee64e5f8f75a361260a2165996bc137b961 (diff)
downloadcgit-75bfec6448b44e26c06736253f8ab0197a6652f7.tar.gz
tests: introduce strip_header() helper function
This means that we can avoid hardcoding the number of headers we expect CGit to generate in test cases and simply remove whatever headers happen to by there when we are checking body content. Signed-off-by: John Keeping <john@keeping.me.uk>
-rwxr-xr-xtests/setup.sh8
-rwxr-xr-xtests/t0107-snapshot.sh4
2 files changed, 10 insertions, 2 deletions
diff --git a/tests/setup.sh b/tests/setup.sh
index a573444..1d8677a 100755
--- a/tests/setup.sh
+++ b/tests/setup.sh
@@ -98,4 +98,12 @@ cgit_url()
CGIT_CONFIG="$PWD/cgitrc" QUERY_STRING="url=$1" cgit
}
+strip_headers () {
+ while read -r line
+ do
+ test -z "$line" && break
+ done
+ cat
+}
+
test -z "$CGIT_TEST_NO_CREATE_REPOS" && setup_repos
diff --git a/tests/t0107-snapshot.sh b/tests/t0107-snapshot.sh
index 053062c..6cf7aaa 100755
--- a/tests/t0107-snapshot.sh
+++ b/tests/t0107-snapshot.sh
@@ -16,7 +16,7 @@ test_expect_success 'check html headers' '
'
test_expect_success 'strip off the header lines' '
- tail -n +6 tmp > master.tar.gz
+ strip_headers <tmp >master.tar.gz
'
test_expect_success 'verify gzip format' '
@@ -51,7 +51,7 @@ test_expect_success 'check HTML headers (zip)' '
'
test_expect_success 'strip off the header lines (zip)' '
- tail -n +6 tmp >master.zip
+ strip_headers <tmp >master.zip
'
if test -n "$(which unzip 2>/dev/null)"; then