[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 06/34] tests/test-qmp-cmds: Check responses more thoroughl
From: |
Marc-André Lureau |
Subject: |
Re: [PATCH v3 06/34] tests/test-qmp-cmds: Check responses more thoroughly |
Date: |
Mon, 16 Mar 2020 16:08:08 +0100 |
On Sun, Mar 15, 2020 at 3:51 PM Markus Armbruster <address@hidden> wrote:
>
> Signed-off-by: Markus Armbruster <address@hidden>
Reviewed-by: Marc-André Lureau <address@hidden>
> ---
> tests/test-qmp-cmds.c | 23 +++++++++++++++++------
> 1 file changed, 17 insertions(+), 6 deletions(-)
>
> diff --git a/tests/test-qmp-cmds.c b/tests/test-qmp-cmds.c
> index b31064b064..464b370189 100644
> --- a/tests/test-qmp-cmds.c
> +++ b/tests/test-qmp-cmds.c
> @@ -151,9 +151,10 @@ static QObject *do_qmp_dispatch(QDict *req, bool
> allow_oob)
> QObject *ret;
>
> resp = qmp_dispatch(&qmp_commands, QOBJECT(req), allow_oob);
> - g_assert(resp && !qdict_haskey(resp, "error"));
> + g_assert(resp);
> ret = qdict_get(resp, "return");
> g_assert(ret);
> + g_assert(qdict_size(resp) == 1);
>
> qobject_ref(ret);
> qobject_unref(resp);
> @@ -163,9 +164,17 @@ static QObject *do_qmp_dispatch(QDict *req, bool
> allow_oob)
> static void do_qmp_dispatch_error(QDict *req, bool allow_oob, ErrorClass cls)
> {
> QDict *resp;
> + QDict *error;
>
> resp = qmp_dispatch(&qmp_commands, QOBJECT(req), allow_oob);
> - g_assert(resp && qdict_haskey(resp, "error"));
> + g_assert(resp);
> + error = qdict_get_qdict(resp, "error");
> + g_assert(error);
> + g_assert_cmpstr(qdict_get_try_str(error, "class"),
> + ==, QapiErrorClass_str(cls));
> + g_assert(qdict_get_try_str(error, "desc"));
> + g_assert(qdict_size(error) == 2);
> + g_assert(qdict_size(resp) == 1);
>
> qobject_unref(resp);
> }
> @@ -174,11 +183,12 @@ static void do_qmp_dispatch_error(QDict *req, bool
> allow_oob, ErrorClass cls)
> static void test_dispatch_cmd(void)
> {
> QDict *req = qdict_new();
> - QObject *ret;
> + QDict *ret;
>
> qdict_put_str(req, "execute", "user_def_cmd");
>
> - ret = do_qmp_dispatch(req, false);
> + ret = qobject_to(QDict, do_qmp_dispatch(req, false));
> + assert(ret && qdict_size(ret) == 0);
>
> qobject_unref(ret);
> qobject_unref(req);
> @@ -187,11 +197,12 @@ static void test_dispatch_cmd(void)
> static void test_dispatch_cmd_oob(void)
> {
> QDict *req = qdict_new();
> - QObject *ret;
> + QDict *ret;
>
> qdict_put_str(req, "exec-oob", "test-flags-command");
>
> - ret = do_qmp_dispatch(req, true);
> + ret = qobject_to(QDict, do_qmp_dispatch(req, true));
> + assert(ret && qdict_size(ret) == 0);
>
> qobject_unref(ret);
> qobject_unref(req);
> --
> 2.21.1
>
>
--
Marc-André Lureau
- [PATCH v3 00/34] Configurable policy for handling deprecated interfaces, Markus Armbruster, 2020/03/15
- [PATCH v3 09/34] tests/test-qmp-event: Use qobject_is_equal(), Markus Armbruster, 2020/03/15
- [PATCH v3 03/34] docs/devel/qapi-code-gen: Clarify allow-oob introspection, Markus Armbruster, 2020/03/15
- [PATCH v3 06/34] tests/test-qmp-cmds: Check responses more thoroughly, Markus Armbruster, 2020/03/15
- [PATCH v3 10/34] tests/test-qmp-event: Check event is actually emitted, Markus Armbruster, 2020/03/15
- [PATCH v3 05/34] tests/test-qmp-cmds: Factor out qmp_dispatch() test helpers, Markus Armbruster, 2020/03/15
- [PATCH v3 23/34] qapi: Simplify how qmp_dispatch() gets the request ID, Markus Armbruster, 2020/03/15