Re: [PATCH] tests: monitor: json: Fix error message when returning json in json

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]