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