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>

diff --git a/tests/setup.sh b/tests/setup.sh
index 9f66d899103cd9d9161c8639b2cbd5c6afd37f23..b2f1169c789846abc912c7429dee19dfb3c1487c 100755 (executable)
--- a/tests/setup.sh
+++ b/tests/setup.sh
@@ -20,10 +20,10 @@ mkrepo() {
        name=$1
        count=$2
        dir=$PWD
        name=$1
        count=$2
        dir=$PWD

-       test -d $name && return
+       test -d "$name" && return

        printf "Creating testrepo %s\n" $name
        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
        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/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
        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=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
 }
 
 EOF
 }
 

diff --git a/tests/t0101-index.sh b/tests/t0101-index.sh
index 07e39f9139de1324013677e58b34ac01ecdc4bba..573a351b8078193563f26f22888b6f1a90f7ab78 100755 (executable)
--- a/tests/t0101-index.sh
+++ b/tests/t0101-index.sh
@@ -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 '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'
 
 run_test 'no tree-link' '! grep -e "foo/tree" trash/tmp'
 run_test 'no log-link' '! grep -e "foo/log" trash/tmp'
 

diff --git a/tests/t0103-log.sh b/tests/t0103-log.sh
index b08cd299f37f298887b586cac10384ae793e7d9d..def5c1812ac82017193a8d2a1be4aa22b8e39ca5 100755 (executable)
--- a/tests/t0103-log.sh
+++ b/tests/t0103-log.sh
@@ -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 '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
 tests_done