On 8/7/19 10:12 AM, Vladimir Sementsov-Ogievskiy wrote:
> Signed-off-by: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com>
> ---
> tests/qemu-iotests/iotests.py | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/tests/qemu-iotests/iotests.py b/tests/qemu-iotests/iotests.py
> index ce74177ab1..4ad265f140 100644
> --- a/tests/qemu-iotests/iotests.py
> +++ b/tests/qemu-iotests/iotests.py
> @@ -540,6 +540,16 @@ class VM(qtest.QEMUQtestMachine):
> log(result, filters, indent=indent)
> return result
>
> + def event_wait_log(self, name, **kwargs):
> + event = self.event_wait(name, **kwargs)
> + log(event, filters=[filter_qmp_event])
> + return event
> +
> + def events_wait_log(self, events, **kwargs):
> + event = self.events_wait(events, **kwargs)
> + log(event, filters=[filter_qmp_event])
> + return event
> +
> # Returns None on success, and an error string on failure
> def run_job(self, job, auto_finalize=True, auto_dismiss=False,
> pre_finalize=None, use_log=True, wait=60.0):
>
I'm not sure these are really needed, since you can just log the event
you get after calling either of these methods anyway. There's nothing
stopping you from:
```
event = event_wait_log(...)
log(filter_qmp_event(event))
```
