From: Jeff King <peff@peff.net>
Date: Mon, 03 Dec 2007 07:35:53 GMT
Subject: Re: make test failure with latest master
Message-ID: <20071203073553.GA3614@sigill.intra.peff.net>
In-Reply-To: <4753B147.2050404@viscovery.net>

On Mon, Dec 03, 2007 at 08:33:27AM +0100, Johannes Sixt wrote:

>> -	test_expect_success 'skipping cvsimport tests, cvs/cvsps not found' ''
>> +	test_expect_success 'skipping cvsimport tests, cvs not found' ''
>
> FWIW, the idiom to provide informative messages in test cases is
>
> 	say 'skipping cvsimport tests, cvs not found'
>
> This gives a distinguishing color, too.

Ah, I adapted the test from t9400, which uses expect_success. Thanks for
the tip.

-Peff