On Fri, Oct 6, 2017 at 3:01 PM, Jonathan Nieder <[email protected]> wrote:
> Stefan Beller wrote:
>
>> Fix the argument order for test_cmp. When given the expected
>> result first the diff shows the actual output with '+' and the
>> expectation with '-', which is the convention for our tests.
>>
>> Signed-off-by: Stefan Beller <[email protected]>
>> ---
>
> Yes, this should make the output from failing tests easier to take in
> at a glance.
>
> Reviewed-by: Jonathan Nieder <[email protected]>
>
> How did you find these? E.g. is there a grep pattern that reviewers
> can use to repeat your results?
The grunge work was done via scrolling through
git -C t grep test_cmp
However it occurred to me that checking for the completeness could be done
via
git -C t grep test_cmp | \
awk '{$1=""; print } | \ # remove file name from output
sort | uniq
There are some cases that are still pretty obvious what the
"actual" and the "expect" is, but they are not included in this patch
as the code (and file names) looked hairy.
