[libvirt] [PATCH 2/3] tests: assume FMT_COMPAT for qemu-img tests

Ján Tomko jtomko at redhat.com
Tue Apr 17 21:43:34 UTC 2018


No point in testing outdated command lines.

Signed-off-by: Ján Tomko <jtomko at redhat.com>
---
 .../qcow2-nocapacity-convert-prealloc.argv         |  2 +-
 tests/storagevolxml2argvdata/qcow2-nocapacity.argv |  2 +-
 tests/storagevolxml2argvtest.c                     | 32 +++++++++++-----------
 3 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/tests/storagevolxml2argvdata/qcow2-nocapacity-convert-prealloc.argv b/tests/storagevolxml2argvdata/qcow2-nocapacity-convert-prealloc.argv
index b151b9401..73499178e 100644
--- a/tests/storagevolxml2argvdata/qcow2-nocapacity-convert-prealloc.argv
+++ b/tests/storagevolxml2argvdata/qcow2-nocapacity-convert-prealloc.argv
@@ -1,4 +1,4 @@
 qemu-img convert -f raw -O qcow2 \
--o encryption=on,preallocation=falloc \
+-o encryption=on,preallocation=falloc,compat=0.10 \
 /var/lib/libvirt/images/sparse.img \
 /var/lib/libvirt/images/OtherDemo.img
diff --git a/tests/storagevolxml2argvdata/qcow2-nocapacity.argv b/tests/storagevolxml2argvdata/qcow2-nocapacity.argv
index 1198cbaf2..fd8805589 100644
--- a/tests/storagevolxml2argvdata/qcow2-nocapacity.argv
+++ b/tests/storagevolxml2argvdata/qcow2-nocapacity.argv
@@ -1,5 +1,5 @@
 qemu-img create \
 -f qcow2 \
 -b /dev/null \
--o backing_fmt=raw,encryption=on \
+-o backing_fmt=raw,encryption=on,compat=0.10 \
 /var/lib/libvirt/images/OtherDemo.img
diff --git a/tests/storagevolxml2argvtest.c b/tests/storagevolxml2argvtest.c
index 4353ad467..68ee9c3d8 100644
--- a/tests/storagevolxml2argvtest.c
+++ b/tests/storagevolxml2argvtest.c
@@ -180,10 +180,10 @@ mymain(void)
     unsigned int flags = VIR_STORAGE_VOL_CREATE_PREALLOC_METADATA;
 
 #define DO_TEST_FULL(shouldFail, parseflags, pool, vol, inputpool, inputvol, \
-                     cmdline, flags, imgformat) \
+                     cmdline, flags) \
     do { \
         struct testInfo info = { shouldFail, pool, vol, inputpool, inputvol, \
-                                 cmdline, flags, imgformat, parseflags }; \
+                                 cmdline, flags, FMT_COMPAT, parseflags }; \
         if (virTestRun("Storage Vol XML-2-argv " cmdline, \
                        testCompareXMLToArgvHelper, &info) < 0) \
             ret = -1; \
@@ -198,47 +198,47 @@ mymain(void)
 
     DO_TEST("pool-dir", "vol-qcow2",
             NULL, NULL,
-            "qcow2-compat", 0, FMT_COMPAT);
+            "qcow2-compat", 0);
     DO_TEST("pool-dir", "vol-qcow2-nobacking",
             NULL, NULL,
-            "qcow2-nobacking-prealloc-compat", flags, FMT_COMPAT);
+            "qcow2-nobacking-prealloc-compat", flags);
     DO_TEST("pool-dir", "vol-qcow2-nobacking",
             "pool-dir", "vol-file",
-            "qcow2-nobacking-convert-prealloc-compat", flags, FMT_COMPAT);
+            "qcow2-nobacking-convert-prealloc-compat", flags);
     DO_TEST("pool-dir", "vol-qcow2-lazy",
             NULL, NULL,
-            "qcow2-lazy", 0, FMT_COMPAT);
+            "qcow2-lazy", 0);
     DO_TEST("pool-dir", "vol-qcow2-1.1",
             NULL, NULL,
-            "qcow2-1.1", 0, FMT_COMPAT);
+            "qcow2-1.1", 0);
     DO_TEST_FAIL("pool-dir", "vol-qcow2-0.10-lazy",
                  NULL, NULL,
-                 "qcow2-0.10-lazy", 0, FMT_COMPAT);
+                 "qcow2-0.10-lazy", 0);
     DO_TEST("pool-dir", "vol-qcow2-nobacking",
             "pool-logical", "vol-logical",
-            "qcow2-from-logical-compat", 0, FMT_COMPAT);
+            "qcow2-from-logical-compat", 0);
     DO_TEST("pool-logical", "vol-logical",
             "pool-dir", "vol-qcow2-nobacking",
-            "logical-from-qcow2", 0, FMT_COMPAT);
+            "logical-from-qcow2", 0);
     DO_TEST("pool-dir", "vol-qcow2-nocow",
             NULL, NULL,
-            "qcow2-nocow-compat", 0, FMT_COMPAT);
+            "qcow2-nocow-compat", 0);
     DO_TEST("pool-dir", "vol-qcow2-nocapacity",
             "pool-dir", "vol-file",
-            "qcow2-nocapacity-convert-prealloc", flags, FMT_OPTIONS);
+            "qcow2-nocapacity-convert-prealloc", flags);
     DO_TEST("pool-dir", "vol-qcow2-zerocapacity",
             NULL, NULL,
-            "qcow2-zerocapacity", 0, FMT_COMPAT);
+            "qcow2-zerocapacity", 0);
     DO_TEST_FULL(false, VIR_VOL_XML_PARSE_OPT_CAPACITY,
                  "pool-dir", "vol-qcow2-nocapacity-backing", NULL, NULL,
-                 "qcow2-nocapacity", 0, FMT_OPTIONS);
+                 "qcow2-nocapacity", 0);
 
     DO_TEST("pool-dir", "vol-file-iso",
             NULL, NULL,
-            "iso", 0, FMT_OPTIONS);
+            "iso", 0);
     DO_TEST("pool-dir", "vol-file",
             "pool-dir", "vol-file-iso",
-            "iso-input", 0, FMT_OPTIONS);
+            "iso-input", 0);
 
     return ret == 0 ? EXIT_SUCCESS : EXIT_FAILURE;
 }
-- 
2.13.6




More information about the libvir-list mailing list