[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] tests: fix intermittent failure in test/du/

From: Pádraig Brady
Subject: Re: [PATCH] tests: fix intermittent failure in test/du/
Date: Sun, 28 Jul 2013 17:17:31 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2

On 07/28/2013 04:09 PM, Bernhard Voelker wrote:
> On 07/28/2013 01:29 AM, Pádraig Brady wrote:
>> Prompted by the continuous integration build failure at:
>> * test/du/ Due to undefined order in returned directory
>> entries, the expected output might not match, so sort both expected
>> and actual output when returning --all directory entries.
> Thanks for fixing this.
> I wondered myself while prototyping why the order was different,
> but didn't get back to that issue.  Sorry.
>> Also use a simpler test for ensuring no errors are output.
> ...
>> -compare /dev/null err || fail=1
>> +test -s err && fail=1
> This is admittedly simpler, but will prevent an actual error
> message to show up automatically in the case of a test failure
> report: compare() calls compare_dev_null_() which would do that
> for free.
> While we're at it, I think the following could be changed for
> a similar reason:
> -  compare exp out || { cat out; fail=1; }
> +  compare exp out || fail=1
> The diff would be shown automatically there, too, so we could
> leave out the "cat out"s.

Agreed on both points.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]