diff options
| author | Markus Armbruster <armbru@redhat.com> | 2018-08-06 08:53:25 +0200 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2018-08-16 08:42:06 +0200 |
| commit | 055a1efc7c5a30ca0993720da57ba70179d28c7b (patch) | |
| tree | a64a16a0cf3cc6e3f1f8dff8289b20d6b10549d6 /tests/ide-test.c | |
| parent | be62e1724f383c9e5012732af6c4dce587a917ee (diff) | |
libqtest: Remove qtest_qmp_discard_response() & friends
qtest_qmp_discard_response(...) is shorthand for
qobject_unref(qtest_qmp(...), except it's not actually shorter.
Moreover, the presence of these functions encourage sloppy testing.
Remove them from libqtest. Add them as macros to the tests that use
them, with a TODO comment asking for cleanup.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20180806065344.7103-5-armbru@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Diffstat (limited to 'tests/ide-test.c')
| -rw-r--r-- | tests/ide-test.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/tests/ide-test.c b/tests/ide-test.c index 2384c2c3e2..7ab09c3448 100644 --- a/tests/ide-test.c +++ b/tests/ide-test.c @@ -29,12 +29,15 @@ #include "libqos/libqos.h" #include "libqos/pci-pc.h" #include "libqos/malloc-pc.h" - +#include "qapi/qmp/qdict.h" #include "qemu-common.h" #include "qemu/bswap.h" #include "hw/pci/pci_ids.h" #include "hw/pci/pci_regs.h" +/* TODO actually test the results and get rid of this */ +#define qmp_discard_response(...) qobject_unref(qmp(__VA_ARGS__)) + #define TEST_IMAGE_SIZE 64 * 1024 * 1024 #define IDE_PCI_DEV 1 |