[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 10/38] tests: add a few qemu-qmp tests
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v3 10/38] tests: add a few qemu-qmp tests |
Date: |
Thu, 05 Jul 2018 13:55:51 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
Marc-André Lureau <address@hidden> writes:
> These 2 tests exhibited two qmp bugs that were fixed in 2.7
> (series from commit e64c75a9752c5d0fd64eb2e684c656a5ea7d03c6 to
> commit 1382d4abdf9619985e4078e37e49e487cea9935e)
>
> Signed-off-by: Marc-André Lureau <address@hidden>
Thanks for holding on to the test cases :)
Needs a rebase.
> ---
> tests/qmp-test.c | 49 ++++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 49 insertions(+)
>
> diff --git a/tests/qmp-test.c b/tests/qmp-test.c
> index 07c0b87e27..642f46a332 100644
> --- a/tests/qmp-test.c
> +++ b/tests/qmp-test.c
> @@ -224,7 +224,50 @@ static void test_qmp_oob(void)
> }
> QDECREF(resp);
> }
> + qtest_end();
> +}
> +
> +static void test_object_add_without_props(void)
> +{
> + QDict *ret, *error;
> + const gchar *klass, *desc;
> +
> + qtest_start("-machine none");
>
> + ret = qmp("{'execute': 'object-add',"
> + " 'arguments': { 'qom-type': 'memory-backend-ram', 'id':
> 'ram1' } }");
> + g_assert_nonnull(ret);
> +
> + error = qdict_get_qdict(ret, "error");
> + klass = qdict_get_try_str(error, "class");
> + desc = qdict_get_try_str(error, "desc");
> +
> + g_assert_cmpstr(klass, ==, "GenericError");
> + g_assert_cmpstr(desc, ==, "can't create backend with size 0");
Sure checking "desc" is worthwhile? We don't do that elsewhere in this
test program, and only rarely in other test programs.
> +
> + QDECREF(ret);
> + qtest_end();
> +}
> +
> +static void test_qom_set_without_value(void)
> +{
> + QDict *ret, *error;
> + const gchar *klass, *desc;
> +
> + qtest_start("-machine none");
> +
> + ret = qmp("{'execute': 'qom-set',"
> + " 'arguments': { 'path': '/machine', 'property': 'rtc-time' }
> }");
> + g_assert_nonnull(ret);
> +
> + error = qdict_get_qdict(ret, "error");
> + klass = qdict_get_try_str(error, "class");
> + desc = qdict_get_try_str(error, "desc");
> +
> + g_assert_cmpstr(klass, ==, "GenericError");
> + g_assert_cmpstr(desc, ==, "Parameter 'value' is missing");
Likewise.
> +
> + QDECREF(ret);
> qtest_end();
> }
>
> @@ -411,13 +454,19 @@ int main(int argc, char *argv[])
>
> g_test_init(&argc, &argv, NULL);
>
> + qtest_add_func("qmp/object-add-without-props",
> + test_object_add_without_props);
> + qtest_add_func("qmp/qom-set-without-value",
> + test_qom_set_without_value);
> qtest_add_func("qmp/protocol", test_qmp_protocol);
> qtest_add_func("qmp/oob", test_qmp_oob);
> +
> qmp_schema_init(&schema);
> add_query_tests(&schema);
>
> ret = g_test_run();
>
> qmp_schema_cleanup(&schema);
> +
> return ret;
> }
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v3 10/38] tests: add a few qemu-qmp tests,
Markus Armbruster <=