[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 04/10] iotests.py: add event_wait_log and eve
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
Re: [Qemu-devel] [PATCH v4 04/10] iotests.py: add event_wait_log and events_wait_log helpers |
Date: |
Fri, 27 Sep 2019 07:31:21 +0000 |
27.09.2019 2:05, John Snow wrote:
>
>
> On 8/7/19 10:12 AM, Vladimir Sementsov-Ogievskiy wrote:
>> Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
>> ---
>> 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))
> ```
two lines vs one
Hm, just simple wrappers like qmp_log(), to make test a bit more readable, why
not..
--
Best regards,
Vladimir