Re: [libvirt] [PATCH 3/6] conf: rename some test fields

A later patch will be adding some new fields to virStorageFileMetadata; to minimize confusion, renaming the test fields now will make it more obvious which fields are being tested later. * tests/virstoragetest.c (_testFileData): Alter names. (testStorageChain, mymain): Adjust clients. Signed-off-by: Eric Blake <eblake(a)redhat.com&gt; --- tests/virstoragetest.c | 78 +++++++++++++++++++++++++------------------------- 1 file changed, 39 insertions(+), 39 deletions(-) diff --git a/tests/virstoragetest.c b/tests/virstoragetest.c index 74ad15c..093053a 100644 --- a/tests/virstoragetest.c +++ b/tests/virstoragetest.c @@ -212,9 +212,9 @@ struct _testFileData { const char *expBackingStore; const char *expBackingStoreRaw; - const char *expDirectoryRel; - const char *expDirectoryAbs; - enum virStorageFileFormat expFormat; + const char *expBackingDirRel; + const char *expBackingDirAbs; + enum virStorageFileFormat expBackingFormat; bool expIsFile; unsigned long long expCapacity; bool expEncrypted; @@ -285,21 +285,21 @@ testStorageChain(const void *args) while (elt) { char *expect = NULL; char *actual = NULL; - const char *expDirectory; + const char *expBackingDirectory; if (i == data->nfiles) { fprintf(stderr, "probed chain was too long\n"); goto cleanup; } - expDirectory = abs ? data->files[i]->expDirectoryAbs - : data->files[i]->expDirectoryRel; + expBackingDirectory = abs ? data->files[i]->expBackingDirAbs + : data->files[i]->expBackingDirRel;

if (virAsprintf(&expect, "store:%s\nraw:%s\ndirectory:%s\nother:%d %d %lld %d", NULLSTR(data->files[i]->expBackingStore), NULLSTR(data->files[i]->expBackingStoreRaw), - NULLSTR(expDirectory), - data->files[i]->expFormat, + NULLSTR(expBackingDirectory), + data->files[i]->expBackingFormat, data->files[i]->expIsFile, data->files[i]->expCapacity, data->files[i]->expEncrypted) < 0 ||