On 06/03/14 17:44, Eric Blake wrote:
> On 06/03/2014 09:21 AM, Peter Krempa wrote:
>> The qemu JSON monitor test allows to test also expected command
>> arguments. As the error from the monitor simulator is returned as a
>> simulated qemu error (in JSON) all other JSON contained in the error
>> message needs to be escaped. This will happen if the monitor command
>> under test receives a JSON array as an argument.
>>
>> This will improve the error message from:
>> libvirt: error : internal error: cannot parse json { "error": { "desc":
>> "Invalid value of argument 'keys' of command 'send-key': expected 'ble'
>> got '[{"type":"number","data":43},{"type":"number","data":26},
>> {"type":"number","data":46},{"type":"number","data":32}]'",
>> "class": "UnexpectedCommand" } }: lexical error: invalid string in json text.
>>
>> To:
>> libvirt: QEMU Driver error : internal error: unable to execute QEMU
>> command 'send-key': Invalid value of argument 'keys' of command
>> 'send-key': expected 'ble' got '[{"type":"number","data":43},
>> {"type":"number","data":26},{"type":"number","data":46},
>> {"type":"number","data":32}]'
>>
>> This improvement will not have any effect on tests executing as
>> expected, but it will help test development.
>> ---
>> tests/qemumonitortestutils.c | 6 ++++++
>> 1 file changed, 6 insertions(+)
>
> ACK.
>
Pushed; Thanks.
Peter
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list