[PATCHv2] test-lib: TAP compliance for skipping tests on request
- From
Michael J Gruber <git@drmicha.warpmail.net>
- Date
- Jul 12, 2010, 10:33 UTC
- Message-ID
- <82aab66bffd86b562c76a13a4666d3cf22a64cca.1278930335.git.git@drmicha.warpmail.net>
- In-Reply-To
- <AANLkTiml1lQlSBDGgUg9scsvsaffDT2n405USmsSIm1n@mail.gmail.com>
Make the output TAP compliant for tests skipped on request (GIT_SKIP_TESTS).
Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net> --- v2 has a typo fix in the commit message and the spelling of "skip" as suggested by AAB.
I think I've read it differently in the RFC, but then I was also confused about the "#" in the RFC, and following the "reference" implementation (as per AAB) should be the best approach.
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 bc06564..db8371c 100644 --- a/t/test-lib.sh +++ b/t/test-lib.sh @@ -396,7 +396,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" : true ;; *) @@ -833,7 +833,7 @@ do case "$this_test" in $skp) 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 done
-- 1.7.2.rc1.212.g850a