Re: [PATCH 12/12] tests/domaincaps: Use testQemuCapsIterate()

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

 



On Wed, Oct 23, 2019 at 12:43:02 +0200, Andrea Bolognani wrote:
> Now that the only data we need for fully testing a QEMU binary is
> the (version, arch) combo, we can stop providing that information
> ourselves and instead rely on testQemuCapsIterate() automatically
> picking up new input files as they are added to the repository,
> the same way the qemucapabilities and qemucaps2xml tests already
> behave.
> 
> Unsurprisingly, this change results in a bunch of extra output
> files being created, significantly expanding our test coverage.
> 
> Signed-off-by: Andrea Bolognani <abologna@xxxxxxxxxx>
...
> +    /*
> +     * Run "tests/qemucapsprobe /path/to/qemu/binary >foo.replies"
> +     * to generate updated or new *.replies data files.
> +     *
> +     * If you manually edit replies files you can run
> +     * "tests/qemucapsfixreplies foo.replies" to fix the replies ids.
> +     *
> +     * Once a replies file has been generated and tweaked if necessary,
> +     * you can drop it into tests/qemucapabilitiesdata/ (with a sensible
> +     * name - look at what's already there for inspiration) and test
> +     * programs will automatically pick it up.
> +     *
> +     * To generate the corresponding output files after a new replies
> +     * file has been added, run "VIR_TEST_REGENERATE_OUTPUT=1 make check".
> +     */

I'd just suggest running

    VIR_TEST_REGENERATE_OUTPUT=1 tests/qemucapabilitiestest

followed by

    VIR_TEST_REGENERATE_OUTPUT=1 tests/domaincapstest

For the whole series:
Reviewed-by: Jiri Denemark <jdenemar@xxxxxxxxxx>

--
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]

  Powered by Linux