[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] qemu-iotests/iotests.py: improve assert_qmp message
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
Re: [PATCH] qemu-iotests/iotests.py: improve assert_qmp message |
Date: |
Sat, 26 Oct 2019 16:58:17 +0000 |
26.10.2019 14:29, Philippe Mathieu-Daudé wrote:
> On 10/26/19 12:12 PM, Vladimir Sementsov-Ogievskiy wrote:
>> Make it obvious, from the two values which is found at path and which
>> is expected.
>
> Maybe:
>
> "From the two values compared, make it obvious which is found at path, and
> which is expected."
>
> Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Thanks for review and better wording!
>
>>
>> Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
>> ---
>>
>> It's a pain, I can never remember it, and checking each time in source
>> code who is who is boring.
>>
>> tests/qemu-iotests/iotests.py | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/tests/qemu-iotests/iotests.py b/tests/qemu-iotests/iotests.py
>> index 709def4d5d..e805b9ab14 100644
>> --- a/tests/qemu-iotests/iotests.py
>> +++ b/tests/qemu-iotests/iotests.py
>> @@ -729,8 +729,8 @@ class QMPTestCase(unittest.TestCase):
>> self.fail('no match for "%s" in %s' % (str(result),
>> str(value)))
>> else:
>> self.assertEqual(result, value,
>> - 'values not equal "%s" and "%s"'
>> - % (str(result), str(value)))
>> + '"%s" is "%s", expected "%s"'
>> + % (path, str(result), str(value)))
>> def assert_no_active_block_jobs(self):
>> result = self.vm.qmp('query-block-jobs')
>>
--
Best regards,
Vladimir