[libvirt] [PATCH] tests: Remove caps from DO_TEST_PARSE_ERROR() calls
Andrea Bolognani
abologna at redhat.com
Wed Jan 31 09:08:05 UTC 2018
On Wed, 2018-01-31 at 09:38 +0100, Peter Krempa wrote:
> > @@ -1037,17 +1034,10 @@ mymain(void)
> > QEMU_CAPS_DRIVE_DISCARD,
> > QEMU_CAPS_DRIVE_DETECT_ZEROES);
> > DO_TEST("disk-snapshot", NONE);
> > - DO_TEST_PARSE_ERROR("disk-same-targets",
> > - QEMU_CAPS_SCSI_LSI,
> > - QEMU_CAPS_DEVICE_USB_STORAGE, QEMU_CAPS_NODEFCONFIG);
>
> So this is not quite equivalent. Here's a diff of error messages prior
> and after your patch:
>
> $ diff -u stderr stderr2
> --- stderr 2018-01-31 09:34:37.295273384 +0100
> +++ stderr2 2018-01-31 09:34:39.241273208 +0100
[...]
> @@ -896,7 +895,7 @@
> OK
> 560) QEMU XML-2-ARGV s390-panic-missing ... OK
> 561) QEMU XML-2-ARGV s390-no-parallel ... Got expected error:
> - error : qemuDomainChrDefValidate:3588 : unsupported configuration: parallel ports are not supported
> + error : virDomainPCIAddressGetNextAddr:805 : XML error: No PCI buses available
> OK
> 562) QEMU XML-2-ARGV s390-serial ... OK
> 563) QEMU XML-2-ARGV s390-serial-2 ... OK
Yeah, I thought about going through this exercise of comparing the
output from before and after my patch - but only after leaving the
office for the day ;)
I could go through the list and only remove the capabilities that
are really unused, but it sounds like an awful lot of work just to
shave less than 100 lines off the test suite. Let's just drop it.
--
Andrea Bolognani / Red Hat / Virtualization
More information about the libvir-list
mailing list