Re: [PATCHv5 05/19] tests: virstoragetest: Remove "expBackingStore" field

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

 



On 06/19/2014 07:59 AM, Peter Krempa wrote:
> Now that we changed ordering of the stored metadata so that the backing
> store is described by the child element the test should reflect this
> change too.
> 
> Remove the expected backing store field as it's actually described by
> the next element in the backing chain, so there's no need for
> duplication.
> ---
>  tests/virstoragetest.c | 19 +++----------------
>  1 file changed, 3 insertions(+), 16 deletions(-)
> 

> @@ -315,12 +314,11 @@ struct testChainData
> 
>  static const char testStorageChainFormat[] =
>      "chain member: %zu\n"
> -    "store: %s\n"
> +    "path:%s\n"
>      "backingStoreRaw: %s\n"

Why no space after ':'?

>      "capacity: %lld\n"
>      "encryption: %d\n"
>      "relPath:%s\n"
> -    "path:%s\n"
>      "relDir:%s\n"

Oh - we have a mix of formats.  It might be nice to be consistent; but
as this format string is only ever used on failing tests, it doesn't
matter in the normal case of all tests passing.

ACK. Up to you if you want to tweak that string format.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
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]