[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 6/9] tests: add a few qmp tests
From: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [PATCH v3 6/9] tests: add a few qmp tests |
Date: |
Mon, 27 Aug 2018 09:47:41 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 |
On 2018-08-25 15:57, Marc-André Lureau wrote:
> test_object_add_without_props() tests a bug in qmp_object_add() we
> fixed in commit e64c75a975. Sadly, we don't have systematic
> object-add tests. This lone test can go into qmp-cmd-test for want of
> a better home.
>
> test_qom_set_without_value() is about a bug in infrastructure used by
> the QMP core, fixed in commit c489780203. We covered the bug in
> infrastructure unit tests (commit bce3035a44). I wrote that test
> earlier, to cover QMP level as well, the test could go into qmp-test.
>
> Signed-off-by: Marc-André Lureau <address@hidden>
> ---
> tests/qmp-cmd-test.c | 31 +++++++++++++++++++++++++++++++
> tests/qmp-test.c | 18 ++++++++++++++++++
> 2 files changed, 49 insertions(+)
Reviewed-by: Thomas Huth <address@hidden>
- [Qemu-devel] [PATCH v3 1/9] monitor: consitify qmp_send_response() QDict argument, (continued)
- [Qemu-devel] [PATCH v3 1/9] monitor: consitify qmp_send_response() QDict argument, Marc-André Lureau, 2018/08/25
- [Qemu-devel] [PATCH v3 2/9] qmp: constify qmp_is_oob(), Marc-André Lureau, 2018/08/25
- [Qemu-devel] [PATCH v3 3/9] Revert "qmp: isolate responses into io thread", Marc-André Lureau, 2018/08/25
- [Qemu-devel] [PATCH v3 4/9] monitor: no need to save need_resume, Marc-André Lureau, 2018/08/25
- [Qemu-devel] [PATCH v3 5/9] json-lexer: make it safe to call destroy multiple times, Marc-André Lureau, 2018/08/25
- [Qemu-devel] [PATCH v3 6/9] tests: add a few qmp tests, Marc-André Lureau, 2018/08/25
- Re: [Qemu-devel] [PATCH v3 6/9] tests: add a few qmp tests,
Thomas Huth <=
- [Qemu-devel] [PATCH v3 8/9] qga: process_event() simplification, Marc-André Lureau, 2018/08/25
- [Qemu-devel] [PATCH v3 7/9] tests: add a qmp success-response test, Marc-André Lureau, 2018/08/25
- [Qemu-devel] [PATCH v3 9/9] qmp: common 'id' handling & make QGA conform to QMP spec, Marc-André Lureau, 2018/08/25
- Re: [Qemu-devel] [PATCH v3 0/9] monitor: various code simplification and fixes, Markus Armbruster, 2018/08/28