On Fri, Jul 9, 2010 at 11:15, Michael J Gruber <git@xxxxxxxxxxxxxxxxxxxx> wrote:
> Make the output TAP compliant for tests skipped on request (GI_SKIP_TESTS).
That "GI_SKIP_TESTS" has a T-deficit.
> Signed-off-by: Michael J Gruber <git@xxxxxxxxxxxxxxxxxxxx>
> ---
> We may want to better spell out the reasons, but I think
> it's good enough like this for *explicitly* skipped tests.
>
> t/test-lib.sh | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/t/test-lib.sh b/t/test-lib.sh
> index 8e3de53..2076271 100644
> --- a/t/test-lib.sh
> +++ b/t/test-lib.sh
> @@ -395,7 +395,7 @@ test_skip () {
> case "$to_skip" in
> t)
> say_color skip >&3 "skipping test: $@"
> - say_color skip "ok $test_count: # skip $1"
> + say_color skip "ok $test_count # SKIP $1"
Good catch with the s/://, but I think the lower-case "skip" should
stay that way. At least that's what Test::More does:
$ perl -MTest::More=tests,2 -E 'SKIP: { skip "no can do", 1 } pass "can do"'
1..2
ok 1 # skip no can do
ok 2 - can do
It only uses upper-case on skip-all. I.e. when part of the plan line:
$ perl -MTest::More=skip_all,"this platform sucks" -e1
1..0 # SKIP this platform sucks
Maybe I've missed something, or Test::More has a bug.
> : true
> ;;
> *)
> @@ -838,7 +838,7 @@ done
> case "$to_skip" in
> t)
> say_color skip >&3 "skipping test $this_test altogether"
> - say_color skip "skip all tests in $this_test"
> + skip_all="skip all tests in $this_test"
> test_done
> esac
Good catch. I missed that (again) in my "tests: Use skip_all=<reason>
to skip tests" patch.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html