On 3/3/2014 7:11 AM, Philip Oakley wrote:
From: "Ilya Bobyr" <ilya.bo...@gmail.com>
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 () {
+ skipped_reason=
 if match_pattern_list $this_test.$test_count $GIT_SKIP_TESTS
+ 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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to