On 3/3/2014 7:11 AM, Philip Oakley wrote:
From: "Ilya Bobyr" <ilya.bobyr@xxxxxxxxx>
We used to show "(missing )" next to tests skipped because they are
specified in GIT_SKIP_TESTS. Use "(matched by GIT_SKIP_TESTS)" instead.
The message below forgets the "by".
I'll fix the commit message. I think the output is long enough, while
"by" does not add any information.
Otherwise looks sensible.
Thanks for looking at it :)
---
t/test-lib.sh | 13 ++++++++-----
1 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/t/test-lib.sh b/t/test-lib.sh
index 1531c24..89a405b 100644
--- a/t/test-lib.sh
+++ b/t/test-lib.sh
@@ -446,25 +446,28 @@ test_finish_ () {
test_skip () {
to_skip=
+ skipped_reason=
if match_pattern_list $this_test.$test_count $GIT_SKIP_TESTS
then
to_skip=t
+ skipped_reason="matched GIT_SKIP_TESTS"
s/matched GIT_SKIP_TESTS/matched by GIT_SKIP_TESTS/
[...]
--
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