From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41911) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bsTxw-0000Ou-3H for qemu-devel@nongnu.org; Fri, 07 Oct 2016 08:09:29 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bsTxq-0006NG-7p for qemu-devel@nongnu.org; Fri, 07 Oct 2016 08:09:26 -0400 Received: from mx1.redhat.com ([209.132.183.28]:48756) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bsTxp-0006N1-Tc for qemu-devel@nongnu.org; Fri, 07 Oct 2016 08:09:22 -0400 Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 82E16345B59 for ; Fri, 7 Oct 2016 12:09:21 +0000 (UTC) From: Markus Armbruster Date: Fri, 7 Oct 2016 14:09:11 +0200 Message-Id: <1475842159-26039-5-git-send-email-armbru@redhat.com> In-Reply-To: <1475842159-26039-1-git-send-email-armbru@redhat.com> References: <1475842159-26039-1-git-send-email-armbru@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: [Qemu-devel] [PULL 04/12] tests: start generic qemu-qmp tests List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= From: Marc-Andr=C3=A9 Lureau These 2 tests exhibit two qmp bugs fixed by the previous patches. Signed-off-by: Marc-Andr=C3=A9 Lureau Reviewed-by: Daniel P. Berrange Reviewed-by: Eric Blake Message-Id: <20160922203927.28241-4-marcandre.lureau@redhat.com> [Rename tests/test-qemu-qmp.c to tests/qmp-test.c, cover it in MAINTAINERS, add a file comment] Reviewed-by: Markus Armbruster Signed-off-by: Markus Armbruster --- MAINTAINERS | 1 + tests/Makefile.include | 2 ++ tests/qmp-test.c | 79 ++++++++++++++++++++++++++++++++++++++++++++= ++++++ 3 files changed, 82 insertions(+) create mode 100644 tests/qmp-test.c diff --git a/MAINTAINERS b/MAINTAINERS index 76a0fdb..7851d0f 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1284,6 +1284,7 @@ F: qmp.c F: monitor.c F: docs/*qmp-* F: scripts/qmp/ +F: tests/qmp-test.c T: git git://repo.or.cz/qemu/armbru.git qapi-next =20 Register API diff --git a/tests/Makefile.include b/tests/Makefile.include index 8162f6f..fcacd06 100644 --- a/tests/Makefile.include +++ b/tests/Makefile.include @@ -300,6 +300,7 @@ check-qtest-s390x-y =3D tests/boot-serial-test$(EXESU= F) =20 check-qtest-generic-y +=3D tests/qom-test$(EXESUF) check-qtest-generic-y +=3D tests/ptimer-test$(EXESUF) +check-qtest-generic-y +=3D tests/qmp-test$(EXESUF) =20 qapi-schema +=3D alternate-any.json qapi-schema +=3D alternate-array.json @@ -642,6 +643,7 @@ tests/tpci200-test$(EXESUF): tests/tpci200-test.o tests/display-vga-test$(EXESUF): tests/display-vga-test.o tests/ipoctal232-test$(EXESUF): tests/ipoctal232-test.o tests/qom-test$(EXESUF): tests/qom-test.o +tests/qmp-test$(EXESUF): tests/qmp-test.o tests/drive_del-test$(EXESUF): tests/drive_del-test.o $(libqos-pc-obj-y) tests/qdev-monitor-test$(EXESUF): tests/qdev-monitor-test.o $(libqos-pc-= obj-y) tests/nvme-test$(EXESUF): tests/nvme-test.o diff --git a/tests/qmp-test.c b/tests/qmp-test.c new file mode 100644 index 0000000..2b88bff --- /dev/null +++ b/tests/qmp-test.c @@ -0,0 +1,79 @@ +/* + * QTest testcase for QMP + * + * Copyright (c) 2016 Red Hat, Inc. + * + * This work is licensed under the terms of the GNU GPL, version 2 or la= ter. + * See the COPYING file in the top-level directory. + */ + +/* + * This program tests QMP commands maintained with the QMP core. + * These are defined in qmp.c. Tests for QMP commands defined in + * another subsystem should go into a test program maintained with + * that subsystem. + * + * TODO Actually cover the commands. The tests we got so far only + * demonstrate specific bugs we've fixed. + */ + +#include "qemu/osdep.h" +#include "libqtest.h" + +static void test_object_add_without_props(void) +{ + QDict *ret, *error; + const gchar *klass, *desc; + + ret =3D qmp("{'execute': 'object-add'," + " 'arguments': { 'qom-type': 'memory-backend-ram', 'id': '= ram1' } }"); + g_assert_nonnull(ret); + + error =3D qdict_get_qdict(ret, "error"); + klass =3D qdict_get_try_str(error, "class"); + desc =3D qdict_get_try_str(error, "desc"); + + g_assert_cmpstr(klass, =3D=3D, "GenericError"); + g_assert_cmpstr(desc, =3D=3D, "can't create backend with size 0"); + + QDECREF(ret); +} + +static void test_qom_set_without_value(void) +{ + QDict *ret, *error; + const gchar *klass, *desc; + + ret =3D qmp("{'execute': 'qom-set'," + " 'arguments': { 'path': '/machine', 'property': 'rtc-time= ' } }"); + g_assert_nonnull(ret); + + error =3D qdict_get_qdict(ret, "error"); + klass =3D qdict_get_try_str(error, "class"); + desc =3D qdict_get_try_str(error, "desc"); + + g_assert_cmpstr(klass, =3D=3D, "GenericError"); + g_assert_cmpstr(desc, =3D=3D, "Parameter 'value' is missing"); + + QDECREF(ret); +} + +int main(int argc, char **argv) +{ + int ret; + + g_test_init(&argc, &argv, NULL); + + qtest_start(""); + + qtest_add_func("/qemu-qmp/object-add-without-props", + test_object_add_without_props); + qtest_add_func("/qemu-qmp/qom-set-without-value", + test_qom_set_without_value); + + ret =3D g_test_run(); + + qtest_end(); + + return ret; +} --=20 2.5.5