diff options
| author | Daniel P. Berrangé <berrange@redhat.com> | 2022-03-10 17:18:07 +0000 |
|---|---|---|
| committer | Dr. David Alan Gilbert <dgilbert@redhat.com> | 2022-04-21 19:36:46 +0100 |
| commit | 0c2b6c85c99d8d2a957e6637c7cc0fd33f2b0540 (patch) | |
| tree | 5614402f4028ba35682f66511a072344ed3bc973 | |
| parent | dcd23e9cae61f3156155db93ae699d30340158d9 (diff) | |
| download | qemu-0c2b6c85c99d8d2a957e6637c7cc0fd33f2b0540.zip qemu-0c2b6c85c99d8d2a957e6637c7cc0fd33f2b0540.tar.gz qemu-0c2b6c85c99d8d2a957e6637c7cc0fd33f2b0540.tar.bz2 | |
tests: print newline after QMP response in qtest logs
The QMP commands have a trailing newline, but the response does not.
This makes the qtest logs hard to follow as the next QMP command
appears in the same line as the previous QMP response.
Reviewed-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Peter Xu <peterx@redhat.com>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
Message-Id: <20220310171821.3724080-5-berrange@redhat.com>
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
| -rw-r--r-- | tests/qtest/libqtest.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/tests/qtest/libqtest.c b/tests/qtest/libqtest.c index 7db7ef3..5098d71 100644 --- a/tests/qtest/libqtest.c +++ b/tests/qtest/libqtest.c @@ -612,10 +612,13 @@ QDict *qmp_fd_receive(int fd) } if (log) { - len = write(2, &c, 1); + g_assert(write(2, &c, 1) == 1); } json_message_parser_feed(&qmp.parser, &c, 1); } + if (log) { + g_assert(write(2, "\n", 1) == 1); + } json_message_parser_destroy(&qmp.parser); return qmp.response; |