[libvirt] [PATCH] tests: Fix detection of expected errors
Jiri Denemark
jdenemar at redhat.com
Mon Nov 29 15:47:53 UTC 2010
> > tests/nwfilterxml2xmltest.c | 15 +++++----------
> > tests/qemuxml2argvtest.c | 8 +++-----
> > 2 files changed, 8 insertions(+), 15 deletions(-)
>
> ACK to the patch as is. It fixes the problem
Thanks, I pushed it.
> qemudBuildCommandLine returns 0 and reports and error. Actually it
> doesn't report an error, because then it should return non-zero. What
> it does in the 3 cases where the test expects and "error" it reports
> actually a warning. For example this one:
>
> qemuReportError(VIR_ERR_CONFIG_UNSUPPORTED,
> _("the QEMU binary %s does not support kvm"),
> emulator);
>
> This is currently reported at error level (VIR_ERR_ERROR) but it
> should actually be reported at warning level (VIR_ERR_WARNING),
> because it's treated as non-fatal.
Yeah, there's a whole bunch of such errors in qemudBuildCommandLine which
don't really result in returning an error code. These should all be changed to
warnings.
I'll prepare a patch for this after the release.
Jirka
More information about the libvir-list
mailing list