[libvirt] [PATCH v2 03/10] virsh: Add support for list -reason

John Ferlan jferlan at redhat.com
Fri Dec 11 11:44:17 UTC 2015



On 12/11/2015 05:40 AM, Martin Kletzander wrote:
[...]
>>> +static int testCompareListReason(const void *data ATTRIBUTE_UNUSED)
>>> +{
>>> +    const char *const argv[] = { VIRSH_CUSTOM, "list", "--reason",
>>> NULL };
>>> +  const char *exp = "\
>>> + Id    Name                           State      Reason    \n\
>>> +------------------------------------------------------------\n\
>>> + 1     fv0                            running    unknown   \n\
>>> + 2     fc4                            running    unknown   \n\
>>> +\n";
>>> +  return testCompareOutputLit(exp, NULL, argv);
>>> +}
>>> +
>>
>> Nice to have this test... Would be better to have one with Table too...
>>
> 
> You mean Title, not Table, right?  This is Table output (default).  With
> Title I would find out that there is extra newline, I'll include that as
> well.
> 

Yeah Title not Table

John

[...]




More information about the libvir-list mailing list