mirror of
https://git.zx2c4.com/cgit
synced 2024-11-09 18:18:42 +00:00
tests: add tests for links with space in path and/or args
These tests tries to detect bad links in various pages. On the log page, there currently exists links which are not properly escaped due to the use of cgit_fileurl() when building the link. For now, this bug is simply tagged as such. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
This commit is contained in:
parent
afe04daa33
commit
084ca50972
@ -20,10 +20,10 @@ mkrepo() {
|
||||
name=$1
|
||||
count=$2
|
||||
dir=$PWD
|
||||
test -d $name && return
|
||||
test -d "$name" && return
|
||||
printf "Creating testrepo %s\n" $name
|
||||
mkdir -p $name
|
||||
cd $name
|
||||
mkdir -p "$name"
|
||||
cd "$name"
|
||||
git init
|
||||
n=1
|
||||
while test $n -le $count
|
||||
@ -50,6 +50,7 @@ setup_repos()
|
||||
mkrepo trash/repos/foo 5 >/dev/null
|
||||
mkrepo trash/repos/bar 50 >/dev/null
|
||||
mkrepo trash/repos/foo+bar 10 testplus >/dev/null
|
||||
mkrepo "trash/repos/with space" 2 >/dev/null
|
||||
cat >trash/cgitrc <<EOF
|
||||
virtual-root=/
|
||||
cache-root=$PWD/trash/cache
|
||||
@ -75,6 +76,10 @@ repo.desc=the bar repo
|
||||
repo.url=foo+bar
|
||||
repo.path=$PWD/trash/repos/foo+bar/.git
|
||||
repo.desc=the foo+bar repo
|
||||
|
||||
repo.url=with space
|
||||
repo.path=$PWD/trash/repos/with space/.git
|
||||
repo.desc=spaced repo
|
||||
EOF
|
||||
}
|
||||
|
||||
|
@ -11,6 +11,7 @@ run_test 'find bar repo' 'grep -e "bar" trash/tmp'
|
||||
run_test 'find bar description' 'grep -e "the bar repo" trash/tmp'
|
||||
run_test 'find foo+bar repo' 'grep -e ">foo+bar<" trash/tmp'
|
||||
run_test 'verify foo+bar link' 'grep -e "/foo+bar/" trash/tmp'
|
||||
run_test 'verify "with%20space" link' 'grep -e "/with%20space/" trash/tmp'
|
||||
run_test 'no tree-link' '! grep -e "foo/tree" trash/tmp'
|
||||
run_test 'no log-link' '! grep -e "foo/log" trash/tmp'
|
||||
|
||||
|
@ -12,4 +12,14 @@ run_test 'generate bar/log' 'cgit_url "bar/log" >trash/tmp'
|
||||
run_test 'find commit 1' 'grep -e "commit 1" trash/tmp'
|
||||
run_test 'find commit 50' 'grep -e "commit 50" trash/tmp'
|
||||
|
||||
run_test 'generate "with%20space/log?qt=grep&q=commit+1"' '
|
||||
cgit_url "with+space/log&qt=grep&q=commit+1" >trash/tmp
|
||||
'
|
||||
run_test 'find commit 1' 'grep -e "commit 1" trash/tmp'
|
||||
run_test 'find link with %20 in path' 'grep -e "/with%20space/log/?qt=grep" trash/tmp'
|
||||
run_test 'find link with + in arg' 'grep -e "/log/?qt=grep&q=commit+1" trash/tmp'
|
||||
run_test BUG 'no links with space in path' '! grep -e "href=./with space/" trash/tmp'
|
||||
run_test 'no links with space in arg' '! grep -e "q=commit 1" trash/tmp'
|
||||
run_test 'commit 2 is not visible' '! grep -e "commit 2" trash/tmp'
|
||||
|
||||
tests_done
|
||||
|
Loading…
Reference in New Issue
Block a user