tests: Add tests for QEMU migration parameters

This is an enhanced replacement for the original test from
qemumonitorjsontest which was dropped earlier in this series. More data
files with some real data will be added in the following patches.

Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>

diff --git a/tests/Makefile.am b/tests/Makefile.am
index a4f231d2a7fb1f3a409bf239541df51657a4109c..2c0d1311c2cc6127edf3b0ab720b5fea1be4ad32 100644 (file)
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -124,6 +124,7 @@ EXTRA_DIST = \
        qemuhotplugtestcpus \
        qemuhotplugtestdevices \
        qemuhotplugtestdomains \
+       qemumigparamsdata \
        qemumonitorjsondata \
        qemuxml2argvdata \
        qemuxml2startupxmloutdata \
@@ -284,6 +285,7 @@ test_programs += qemuxml2argvtest qemuxml2xmltest \
        qemumemlocktest \
        qemucommandutiltest \
        qemublocktest \
+       qemumigparamstest \
        $(NULL)
 test_helpers += qemucapsprobe
 test_libraries += libqemumonitortestutils.la \
@@ -670,6 +672,15 @@ qemumemlocktest_SOURCES = \
        testutilsqemu.c testutilsqemu.h \
        testutils.c testutils.h
 qemumemlocktest_LDADD = $(qemu_LDADDS) $(LDADDS)
+
+qemumigparamstest_SOURCES = \
+       qemumigparamstest.c \
+       testutils.c testutils.h \
+       testutilsqemu.c testutilsqemu.h \
+       $(NULL)
+qemumigparamstest_LDADD = libqemumonitortestutils.la \
+       $(qemu_LDADDS) $(LDADDS)
+
 else ! WITH_QEMU
 EXTRA_DIST += qemuxml2argvtest.c qemuxml2xmltest.c qemuargv2xmltest.c \
        domainsnapshotxml2xmltest.c \
@@ -680,6 +691,7 @@ EXTRA_DIST += qemuxml2argvtest.c qemuxml2xmltest.c qemuargv2xmltest.c \
        qemucaps2xmltest.c qemucommandutiltest.c \
        qemumemlocktest.c qemucpumock.c testutilshostcpus.h \
        qemublocktest.c \
+       qemumigparamstest.c \
        $(QEMUMONITORTESTUTILS_SOURCES)
 endif ! WITH_QEMU
 

diff --git a/tests/qemumigparamsdata/unsupported.json b/tests/qemumigparamsdata/unsupported.json
new file mode 100644 (file)
index 0000000..0db3279
--- /dev/null
+++ b/tests/qemumigparamsdata/unsupported.json
@@ -0,0 +1,3 @@
+{
+
+}

diff --git a/tests/qemumigparamsdata/unsupported.reply b/tests/qemumigparamsdata/unsupported.reply
new file mode 100644 (file)
index 0000000..2b88ba1
--- /dev/null
+++ b/tests/qemumigparamsdata/unsupported.reply
@@ -0,0 +1,7 @@
+{
+  "id": "libvirt-1",
+  "error": {
+    "class": "CommandNotFound",
+    "desc": "The command query-migrate-parameters has not been found"
+  }
+}

diff --git a/tests/qemumigparamsdata/unsupported.xml b/tests/qemumigparamsdata/unsupported.xml
new file mode 100644 (file)
index 0000000..8aa3abe
--- /dev/null
+++ b/tests/qemumigparamsdata/unsupported.xml
@@ -0,0 +1,4 @@
+<test>
+  <migParams>
+  </migParams>
+</test>

diff --git a/tests/qemumigparamstest.c b/tests/qemumigparamstest.c
new file mode 100644 (file)
index 0000000..4cc27bc
--- /dev/null
+++ b/tests/qemumigparamstest.c
@@ -0,0 +1,237 @@
+/*
+ * Copyright (C) 2011-2013 Red Hat, Inc.
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library.  If not, see
+ * <http://www.gnu.org/licenses/>.
+ *
+ */
+
+#include <config.h>
+
+#include "virjson.h"
+#include "virbuffer.h"
+#include "virxml.h"
+#include "testutils.h"
+#include "testutilsqemu.h"
+#include "qemumonitortestutils.h"
+#include "qemu/qemu_migration_params.h"
+#include "qemu/qemu_migration_paramspriv.h"
+#include "qemu/qemu_monitor.h"
+
+#define VIR_FROM_THIS VIR_FROM_NONE
+
+typedef struct _qemuMigParamsData qemuMigParamsData;
+struct _qemuMigParamsData {
+    virDomainXMLOptionPtr xmlopt;
+    const char *name;
+};
+
+
+static void
+qemuMigParamsTestFormatXML(virBufferPtr buf,
+                           qemuMigrationParamsPtr migParams)
+{
+    virBufferAddLit(buf, "<test>\n");
+    virBufferAdjustIndent(buf, 2);
+
+    if (migParams)
+        qemuMigrationParamsFormat(buf, migParams);
+
+    virBufferAdjustIndent(buf, -2);
+    virBufferAddLit(buf, "</test>\n");
+}
+
+
+static int
+qemuMigParamsTestXML2XML(const void *opaque)
+{
+    const qemuMigParamsData *data = opaque;
+    virBuffer buf = VIR_BUFFER_INITIALIZER;
+    char *xmlFile = NULL;
+    xmlDocPtr doc = NULL;
+    xmlXPathContextPtr ctxt = NULL;
+    qemuMigrationParamsPtr migParams = NULL;
+    char *actualXML = NULL;
+    int ret = -1;
+
+    if (virAsprintf(&xmlFile, "%s/qemumigparamsdata/%s.xml",
+                    abs_srcdir, data->name) < 0)
+        goto cleanup;
+
+    if (!(doc = virXMLParseFileCtxt(xmlFile, &ctxt)))
+        goto cleanup;
+
+    if (qemuMigrationParamsParse(ctxt, &migParams) < 0)
+        goto cleanup;
+
+    qemuMigParamsTestFormatXML(&buf, migParams);
+
+    if (!(actualXML = virBufferContentAndReset(&buf)))
+        goto cleanup;
+
+    if (virTestCompareToFile(actualXML, xmlFile) < 0)
+        goto cleanup;
+
+    ret = 0;
+
+ cleanup:
+    VIR_FREE(xmlFile);
+    VIR_FREE(actualXML);
+    qemuMigrationParamsFree(migParams);
+    virBufferFreeAndReset(&buf);
+    xmlXPathFreeContext(ctxt);
+    xmlFreeDoc(doc);
+    return ret;
+}
+
+
+static int
+qemuMigParamsTestXML(const void *opaque)
+{
+    const qemuMigParamsData *data = opaque;
+    virBuffer buf = VIR_BUFFER_INITIALIZER;
+    char *replyFile = NULL;
+    char *xmlFile = NULL;
+    qemuMonitorTestPtr mon = NULL;
+    virJSONValuePtr params = NULL;
+    qemuMigrationParamsPtr migParams = NULL;
+    char *actualXML = NULL;
+    int ret = -1;
+
+    if (virAsprintf(&replyFile, "%s/qemumigparamsdata/%s.reply",
+                    abs_srcdir, data->name) < 0 ||
+        virAsprintf(&xmlFile, "%s/qemumigparamsdata/%s.xml",
+                    abs_srcdir, data->name) < 0)
+        goto cleanup;
+
+    if (!(mon = qemuMonitorTestNewFromFile(replyFile, data->xmlopt, true)))
+        goto cleanup;
+
+    if (qemuMonitorGetMigrationParams(qemuMonitorTestGetMonitor(mon),
+                                      &params) < 0)
+        goto cleanup;
+
+    if (!(migParams = qemuMigrationParamsFromJSON(params)))
+        goto cleanup;
+
+    qemuMigParamsTestFormatXML(&buf, migParams);
+
+    if (!(actualXML = virBufferContentAndReset(&buf)))
+        goto cleanup;
+
+    if (virTestCompareToFile(actualXML, xmlFile) < 0)
+        goto cleanup;
+
+    ret = 0;
+
+ cleanup:
+    VIR_FREE(replyFile);
+    VIR_FREE(xmlFile);
+    VIR_FREE(actualXML);
+    virJSONValueFree(params);
+    qemuMigrationParamsFree(migParams);
+    virBufferFreeAndReset(&buf);
+    qemuMonitorTestFree(mon);
+    return ret;
+}
+
+
+static int
+qemuMigParamsTestJSON(const void *opaque)
+{
+    const qemuMigParamsData *data = opaque;
+    char *replyFile = NULL;
+    char *jsonFile = NULL;
+    qemuMonitorTestPtr mon = NULL;
+    virJSONValuePtr paramsIn = NULL;
+    virJSONValuePtr paramsOut = NULL;
+    qemuMigrationParamsPtr migParams = NULL;
+    char *actualJSON = NULL;
+    int ret = -1;
+
+    if (virAsprintf(&replyFile, "%s/qemumigparamsdata/%s.reply",
+                    abs_srcdir, data->name) < 0 ||
+        virAsprintf(&jsonFile, "%s/qemumigparamsdata/%s.json",
+                    abs_srcdir, data->name) < 0)
+        goto cleanup;
+
+    if (!(mon = qemuMonitorTestNewFromFile(replyFile, data->xmlopt, true)))
+        goto cleanup;
+
+    if (qemuMonitorGetMigrationParams(qemuMonitorTestGetMonitor(mon),
+                                      &paramsIn) < 0)
+        goto cleanup;
+
+    if (!(migParams = qemuMigrationParamsFromJSON(paramsIn)))
+        goto cleanup;
+
+    if (!(paramsOut = qemuMigrationParamsToJSON(migParams)) ||
+        !(actualJSON = virJSONValueToString(paramsOut, true)))
+        goto cleanup;
+
+    if (virTestCompareToFile(actualJSON, jsonFile) < 0)
+        goto cleanup;
+
+    ret = 0;
+
+ cleanup:
+    VIR_FREE(replyFile);
+    VIR_FREE(jsonFile);
+    VIR_FREE(actualJSON);
+    virJSONValueFree(paramsIn);
+    virJSONValueFree(paramsOut);
+    qemuMigrationParamsFree(migParams);
+    qemuMonitorTestFree(mon);
+    return ret;
+}
+
+
+static int
+mymain(void)
+{
+    virQEMUDriver driver;
+    int ret = 0;
+
+#if !WITH_YAJL
+    fputs("libvirt not compiled with yajl, skipping this test\n", stderr);
+    return EXIT_AM_SKIP;
+#endif
+
+    if (virThreadInitialize() < 0 ||
+        qemuTestDriverInit(&driver) < 0)
+        return EXIT_FAILURE;
+
+    virEventRegisterDefaultImpl();
+
+#define DO_TEST(name) \
+    do { \
+        qemuMigParamsData data = { \
+            driver.xmlopt, name \
+        }; \
+        if (virTestRun(name " (xml)", qemuMigParamsTestXML, &data) < 0) \
+            ret = -1; \
+        if (virTestRun(name " (json)", qemuMigParamsTestJSON, &data) < 0) \
+            ret = -1; \
+        if (virTestRun(name " (xml2xml)", qemuMigParamsTestXML2XML, &data) < 0) \
+            ret = -1; \
+    } while (0)
+
+    DO_TEST("unsupported");
+
+    qemuTestDriverFree(&driver);
+
+    return (ret == 0) ? EXIT_SUCCESS : EXIT_FAILURE;
+}
+
+VIR_TEST_MAIN(mymain)