On 6/7/19 7:06 AM, Peter Krempa wrote: > On Fri, Jun 07, 2019 at 10:13:04 +0200, Peter Krempa wrote: >> On Thu, Jun 06, 2019 at 08:40:14 -0500, Eric Blake wrote: >>> On 6/6/19 7:53 AM, Peter Krempa wrote: > > [...] > >> In this case I feel it's more useful to do the check against the schema >> rather than to see that the resposne is the same. >> >> Alternatively I can see whether it's reasonably feasible to do schema >> checking also in qemuMonitorTestAddItemVerbatim. > > https://www.redhat.com/archives/libvir-list/2019-June/msg00210.html > > So we can keep using qemuMonitorTestAddItemVerbatim here. AddItemVerbatim is a pain to maintain; I'd rather stick with AddItem + schema checks. But in doing that, I found that a lot of existing code in the test did not do schema tests; hence I'm planning on pushing these four patches (amended per your review) only after a prerequisite fix of the existing tests: https://www.redhat.com/archives/libvir-list/2019-June/msg00283.html -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3226 Virtualization: qemu.org | libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list