author | Lars Hjemli <hjemli@gmail.com> | 2008-02-24 14:35:52 (UTC) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2008-02-24 14:55:01 (UTC) |
commit | e5ca46bf4b974ce3b51d09da4c7ad741aaa70ede (patch) (unidiff) | |
tree | c4595cba7c332eb9281afc20439c1a25dc0a07db | |
parent | 19134112bf942c0492f2095b668ea31c1a498f1a (diff) | |
download | cgit-e5ca46bf4b974ce3b51d09da4c7ad741aaa70ede.zip cgit-e5ca46bf4b974ce3b51d09da4c7ad741aaa70ede.tar.gz cgit-e5ca46bf4b974ce3b51d09da4c7ad741aaa70ede.tar.bz2 |
Brown paper bag: don't use `grep -v`
For some inexplicable reason I'd gotten the semantics of `grep -v` totally
backwards, thinking it somehow would make the exitcode from grep indicate
the non-match of the specified pattern.
This fixes the broken tests and gives me a valuable lession about shell
programming at the same time.
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
-rwxr-xr-x | tests/t0101-index.sh | 4 | ||||
-rwxr-xr-x | tests/t0102-summary.sh | 6 |
2 files changed, 5 insertions, 5 deletions
diff --git a/tests/t0101-index.sh b/tests/t0101-index.sh index 59edc59..445af6a 100755 --- a/tests/t0101-index.sh +++ b/tests/t0101-index.sh | |||
@@ -1,15 +1,15 @@ | |||
1 | #!/bin/sh | 1 | #!/bin/sh |
2 | 2 | ||
3 | . ./setup.sh | 3 | . ./setup.sh |
4 | 4 | ||
5 | prepare_tests "Check content on index page" | 5 | prepare_tests "Check content on index page" |
6 | 6 | ||
7 | run_test 'generate index page' 'cgit_url "" >trash/tmp' | 7 | run_test 'generate index page' 'cgit_url "" >trash/tmp' |
8 | run_test 'find foo repo' 'grep -e "foo" trash/tmp' | 8 | run_test 'find foo repo' 'grep -e "foo" trash/tmp' |
9 | run_test 'find foo description' 'grep -e "\[no description\]" trash/tmp' | 9 | run_test 'find foo description' 'grep -e "\[no description\]" trash/tmp' |
10 | run_test 'find bar repo' 'grep -e "bar" trash/tmp' | 10 | run_test 'find bar repo' 'grep -e "bar" trash/tmp' |
11 | run_test 'find bar description' 'grep -e "the bar repo" trash/tmp' | 11 | run_test 'find bar description' 'grep -e "the bar repo" trash/tmp' |
12 | run_test 'no tree-link' 'grep -ve "foo/tree" trash/tmp' | 12 | run_test 'no tree-link' '! grep -e "foo/tree" trash/tmp' |
13 | run_test 'no log-link' 'grep -ve "foo/log" trash/tmp' | 13 | run_test 'no log-link' '! grep -e "foo/log" trash/tmp' |
14 | 14 | ||
15 | tests_done | 15 | tests_done |
diff --git a/tests/t0102-summary.sh b/tests/t0102-summary.sh index 7edd675..f0b0d9a 100755 --- a/tests/t0102-summary.sh +++ b/tests/t0102-summary.sh | |||
@@ -1,20 +1,20 @@ | |||
1 | #!/bin/sh | 1 | #!/bin/sh |
2 | 2 | ||
3 | . ./setup.sh | 3 | . ./setup.sh |
4 | 4 | ||
5 | prepare_tests "Check content on summary page" | 5 | prepare_tests "Check content on summary page" |
6 | 6 | ||
7 | run_test 'generate foo summary' 'cgit_url "foo" >trash/tmp' | 7 | run_test 'generate foo summary' 'cgit_url "foo" >trash/tmp' |
8 | run_test 'find commit 1' 'grep -e "commit 1" trash/tmp' | 8 | run_test 'find commit 1' 'grep -e "commit 1" trash/tmp' |
9 | run_test 'find commit 5' 'grep -e "commit 5" trash/tmp' | 9 | run_test 'find commit 5' 'grep -e "commit 5" trash/tmp' |
10 | run_test 'find branch master' 'grep -e "master" trash/tmp' | 10 | run_test 'find branch master' 'grep -e "master" trash/tmp' |
11 | run_test 'no tags' 'grep -ve "tags" trash/tmp' | 11 | run_test 'no tags' '! grep -e "tags" trash/tmp' |
12 | 12 | ||
13 | run_test 'generate bar summary' 'cgit_url "bar" >trash/tmp' | 13 | run_test 'generate bar summary' 'cgit_url "bar" >trash/tmp' |
14 | run_test 'no commit 45' 'grep -ve "commit 45" trash/tmp' | 14 | run_test 'no commit 45' '! grep -e "commit 45" trash/tmp' |
15 | run_test 'find commit 46' 'grep -e "commit 46" trash/tmp' | 15 | run_test 'find commit 46' 'grep -e "commit 46" trash/tmp' |
16 | run_test 'find commit 50' 'grep -e "commit 50" trash/tmp' | 16 | run_test 'find commit 50' 'grep -e "commit 50" trash/tmp' |
17 | run_test 'find branch master' 'grep -e "master" trash/tmp' | 17 | run_test 'find branch master' 'grep -e "master" trash/tmp' |
18 | run_test 'no tags' 'grep -ve "tags" trash/tmp' | 18 | run_test 'no tags' '! grep -e "tags" trash/tmp' |
19 | 19 | ||
20 | tests_done | 20 | tests_done |