Re: [PATCH] t/README: test_must_fail is for testing Git

From: Junio C Hamano <gitster@pobox.com>
Date: Jun 4, 2013, 20:49 UTC
Message-ID: <7v1u8h7gac.fsf@alter.siamese.dyndns.org>
In-Reply-To: <AAD729AEE8D044B89388799A7CC7552A@PhilipOakley>

"Philip Oakley" <philipoakley@iee.org> writes:

Show 12 quoted lines

> From: "Junio C Hamano" <gitster@pobox.com>
> Sent: Tuesday, June 04, 2013 5:50 PM
>> When a test wants to make sure there is no <string> in an output
>> file, we should just say "! grep string output";
>
> Small nit: It took me two readings of the commit message to correctly
> parse this break point. The flowing together of the two parts with the
> semicolon fooled me. Separate them?
>
>>      "test_must_fail"
>> is there only to test Git command and catch unusual deaths we know
>> about (e.g. segv) as an error, not as an expected failure.

Thanks.  Does this read better?

    t/README: test_must_fail is for testing Git
    
    When a test wants to make sure there is no <string> in an output
    file, we should just say "! grep string output".
    
    "test_must_fail" is there only to test Git command and catch unusual
    deaths we know about (e.g. segv) as an error, not as an expected
    failure.  "test_must_fail grep string output" is unnecessary, as
    we are not making sure the system binaries do not dump core or
    anything like that.
    
    Signed-off-by: Junio C Hamano <gitster@pobox.com>