8:49 a.m.
From: "Daniel P. Berrange" <berrange(a)redhat.com>
Take advantage of the previously added monitor helpers to
create a test suite for the QEMU JSON monitor impl. As a
proof of concept, this tests the 'qemuMonitorGetStatus'
implementation
---
.gitignore | 1 +
tests/Makefile.am | 11 +++-
tests/qemumonitorjsontest.c | 150 ++++++++++++++++++++++++++++++++++++++++++++
3 files changed, 161 insertions(+), 1 deletion(-)
create mode 100644 tests/qemumonitorjsontest.c
diff --git a/.gitignore b/.gitignore
index 17dcae4..661509e 100644
--- a/.gitignore
+++ b/.gitignore
@@ -140,6 +140,7 @@
/tests/openvzutilstest
/tests/qemuargv2xmltest
/tests/qemuhelptest
+/tests/qemumonitorjsontest
/tests/qemumonitortest
/tests/qemuxmlnstest
/tests/qparamtest
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 22a7a26..884bb7d 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -109,7 +109,7 @@ endif
if WITH_QEMU
test_programs += qemuxml2argvtest qemuxml2xmltest qemuxmlnstest \
qemuargv2xmltest qemuhelptest domainsnapshotxml2xmltest \
- qemumonitortest
+ qemumonitortest qemumonitorjsontest
endif
if WITH_LXC
@@ -346,6 +346,14 @@ qemuhelptest_LDADD = $(qemu_LDADDS)
qemumonitortest_SOURCES = qemumonitortest.c testutils.c testutils.h
qemumonitortest_LDADD = $(qemu_LDADDS)
+qemumonitorjsontest_SOURCES = \
+ qemumonitorjsontest.c \
+ testutils.c testutils.h \
+ testutilsqemu.c testutilsqemu.h \
+ $(NULL)
+qemumonitorjsontest_LDADD = $(qemu_LDADDS) libqemumonitortestutils.la
+qemumonitorjsontest_CFLAGS = -Dabs_builddir="\"$(abs_builddir)\""
$(AM_CFLAGS)
+
domainsnapshotxml2xmltest_SOURCES = \
domainsnapshotxml2xmltest.c testutilsqemu.c testutilsqemu.h \
testutils.c testutils.h
@@ -354,6 +362,7 @@ else
EXTRA_DIST += qemuxml2argvtest.c qemuxml2xmltest.c qemuargv2xmltest.c \
qemuxmlnstest.c qemuhelptest.c domainsnapshotxml2xmltest.c \
qemumonitortest.c testutilsqemu.c testutilsqemu.h \
+ qemumonitorjsontest.c \
$(QEMUMONITORTESTUTILS_SOURCES)
endif
diff --git a/tests/qemumonitorjsontest.c b/tests/qemumonitorjsontest.c
new file mode 100644
index 0000000..ea69feb
--- /dev/null
+++ b/tests/qemumonitorjsontest.c
@@ -0,0 +1,150 @@
+/*
+ * Copyright (C) 2011-2012 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 "testutils.h"
+#include "testutilsqemu.h"
+#include "qemumonitortestutils.h"
+#include "threads.h"
+#include "virterror_internal.h"
+
+
+#define VIR_FROM_THIS VIR_FROM_NONE
+
+static int
+testQemuMonitorJSONGetStatus(const void *data)
+{
+ virCapsPtr caps = (virCapsPtr)data;
+ qemuMonitorTestPtr test = qemuMonitorTestNew(true, caps);
+ int ret = -1;
+ bool running = false;
+ virDomainPausedReason reason = 0;
+
+ if (!test)
+ return -1;
+
+ if (qemuMonitorTestAddItem(test, "query-status",
+ "{ "
+ " \"return\": { "
+ " \"status\": \"running\",
"
+ " \"singlestep\": false, "
+ " \"running\": true "
+ " } "
+ "}") < 0)
+ goto cleanup;
+ if (qemuMonitorTestAddItem(test, "query-status",
+ "{ "
+ " \"return\": { "
+ " \"singlestep\": false, "
+ " \"running\": false "
+ " } "
+ "}") < 0)
+ goto cleanup;
+ if (qemuMonitorTestAddItem(test, "query-status",
+ "{ "
+ " \"return\": { "
+ " \"status\":
\"inmigrate\", "
+ " \"singlestep\": false, "
+ " \"running\": false "
+ " } "
+ "}") < 0)
+ goto cleanup;
+
+ if (qemuMonitorGetStatus(qemuMonitorTestGetMonitor(test),
+ &running, &reason) < 0)
+ goto cleanup;
+
+ if (!running) {
+ virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
+ "Running was not true");
+ goto cleanup;
+ }
+
+ if (reason != VIR_DOMAIN_PAUSED_UNKNOWN) {
+ virReportError(VIR_ERR_INTERNAL_ERROR,
+ "Reason was unexpectedly set to %d", reason);
+ goto cleanup;
+ }
+
+ if (qemuMonitorGetStatus(qemuMonitorTestGetMonitor(test),
+ &running, &reason) < 0)
+ goto cleanup;
+
+ if (running) {
+ virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
+ "Running was not false");
+ goto cleanup;
+ }
+
+ if (reason != VIR_DOMAIN_PAUSED_UNKNOWN) {
+ virReportError(VIR_ERR_INTERNAL_ERROR,
+ "Reason was unexpectedly set to %d", reason);
+ goto cleanup;
+ }
+
+ if (qemuMonitorGetStatus(qemuMonitorTestGetMonitor(test),
+ &running, &reason) < 0)
+ goto cleanup;
+
+ if (running) {
+ virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
+ "Running was not false");
+ goto cleanup;
+ }
+
+ if (reason != VIR_DOMAIN_PAUSED_MIGRATION) {
+ virReportError(VIR_ERR_INTERNAL_ERROR,
+ "Reason was unexpectedly set to %d", reason);
+ goto cleanup;
+ }
+
+ ret = 0;
+
+cleanup:
+ qemuMonitorTestFree(test);
+ return ret;
+}
+
+static int
+mymain(void)
+{
+ int ret = 0;
+ virCapsPtr caps;
+
+ if (virThreadInitialize() < 0)
+ exit(EXIT_FAILURE);
+
+ if (!(caps = testQemuCapsInit()))
+ exit(EXIT_FAILURE);
+
+ virEventRegisterDefaultImpl();
+
+#define DO_TEST(name) \
+ if (virtTestRun(# name, 1, testQemuMonitorJSON ## name, caps) < 0) \
+ ret = -1
+
+ DO_TEST(GetStatus);
+
+ virCapabilitiesFree(caps);
+
+ return (ret == 0) ? EXIT_SUCCESS : EXIT_FAILURE;
+}
+
+VIRT_TEST_MAIN(mymain)
--
1.7.11.2