t/lib-httpd: add the strip_access_log() helper function
Four tests in three httpd-related test scripts check the contents of Apache's 'access.log', and they all do so by running 'sed' with the exact same script consisting of four s/// commands to strip uninteresting log fields and to vertically align the requested URLs. Extract this into a common helper function 'strip_access_log' in 'lib-httpd.sh', and use it in all of those tests. Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:

committed by
Junio C Hamano

parent
a704c6439a
commit
6940a06022
@ -287,3 +287,12 @@ expect_askpass() {
|
|||||||
test_cmp "$TRASH_DIRECTORY/askpass-expect" \
|
test_cmp "$TRASH_DIRECTORY/askpass-expect" \
|
||||||
"$TRASH_DIRECTORY/askpass-query"
|
"$TRASH_DIRECTORY/askpass-query"
|
||||||
}
|
}
|
||||||
|
|
||||||
|
strip_access_log() {
|
||||||
|
sed -e "
|
||||||
|
s/^.* \"//
|
||||||
|
s/\"//
|
||||||
|
s/ [1-9][0-9]*\$//
|
||||||
|
s/^GET /GET /
|
||||||
|
" "$HTTPD_ROOT_PATH"/access.log
|
||||||
|
}
|
||||||
|
@ -47,12 +47,7 @@ test_expect_success 'no empty path components' '
|
|||||||
cd "$ROOT_PATH" &&
|
cd "$ROOT_PATH" &&
|
||||||
git clone $HTTPD_URL/smart/test_repo.git/ test_repo_clone &&
|
git clone $HTTPD_URL/smart/test_repo.git/ test_repo_clone &&
|
||||||
|
|
||||||
sed -e "
|
strip_access_log >act &&
|
||||||
s/^.* \"//
|
|
||||||
s/\"//
|
|
||||||
s/ [1-9][0-9]*\$//
|
|
||||||
s/^GET /GET /
|
|
||||||
" >act <"$HTTPD_ROOT_PATH"/access.log &&
|
|
||||||
test_cmp exp act
|
test_cmp exp act
|
||||||
'
|
'
|
||||||
|
|
||||||
@ -134,12 +129,7 @@ GET /smart/test_repo.git/info/refs?service=git-receive-pack HTTP/1.1 200
|
|||||||
POST /smart/test_repo.git/git-receive-pack HTTP/1.1 200
|
POST /smart/test_repo.git/git-receive-pack HTTP/1.1 200
|
||||||
EOF
|
EOF
|
||||||
test_expect_success 'used receive-pack service' '
|
test_expect_success 'used receive-pack service' '
|
||||||
sed -e "
|
strip_access_log >act &&
|
||||||
s/^.* \"//
|
|
||||||
s/\"//
|
|
||||||
s/ [1-9][0-9]*\$//
|
|
||||||
s/^GET /GET /
|
|
||||||
" >act <"$HTTPD_ROOT_PATH"/access.log &&
|
|
||||||
test_cmp exp act
|
test_cmp exp act
|
||||||
'
|
'
|
||||||
|
|
||||||
|
@ -98,12 +98,7 @@ GET /smart/repo.git/info/refs?service=git-upload-pack HTTP/1.1 200
|
|||||||
POST /smart/repo.git/git-upload-pack HTTP/1.1 200
|
POST /smart/repo.git/git-upload-pack HTTP/1.1 200
|
||||||
EOF
|
EOF
|
||||||
test_expect_success 'used upload-pack service' '
|
test_expect_success 'used upload-pack service' '
|
||||||
sed -e "
|
strip_access_log >act &&
|
||||||
s/^.* \"//
|
|
||||||
s/\"//
|
|
||||||
s/ [1-9][0-9]*\$//
|
|
||||||
s/^GET /GET /
|
|
||||||
" >act <"$HTTPD_ROOT_PATH"/access.log &&
|
|
||||||
test_cmp exp act
|
test_cmp exp act
|
||||||
'
|
'
|
||||||
|
|
||||||
|
@ -123,12 +123,7 @@ GET /smart/repo.git/info/refs?service=git-receive-pack HTTP/1.1 403 -
|
|||||||
POST /smart/repo.git/git-receive-pack HTTP/1.1 403 -
|
POST /smart/repo.git/git-receive-pack HTTP/1.1 403 -
|
||||||
EOF
|
EOF
|
||||||
test_expect_success 'server request log matches test results' '
|
test_expect_success 'server request log matches test results' '
|
||||||
sed -e "
|
strip_access_log >act &&
|
||||||
s/^.* \"//
|
|
||||||
s/\"//
|
|
||||||
s/ [1-9][0-9]*\$//
|
|
||||||
s/^GET /GET /
|
|
||||||
" >act <"$HTTPD_ROOT_PATH"/access.log &&
|
|
||||||
test_cmp exp act
|
test_cmp exp act
|
||||||
'
|
'
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user