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

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

 




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

[...]

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]