[libvirt PATCH] tests: Don't advertise VIR_TEST_EXPENSIVE to users

Pavel Hrdina phrdina at redhat.com
Wed Sep 23 08:01:52 UTC 2020


On Tue, Sep 22, 2020 at 03:18:49PM +0200, Andrea Bolognani wrote:
> Right now, the logic that takes care of deciding whether expensive
> tests should be run or not is not working correctly: more
> specifically, it's not possible to use something like
> 
>   $ VIR_TEST_EXPENSIVE=1 ninja test
> 
> to override the default choice, because in meson.build we always
> pass an explicit value that overrides whatever is present in the
> environment.
> 
> We could implement logic to make this work properly, but that
> would require some refactoring of our test infrastructure and is
> arguably of little value given that running
> 
>   $ meson build -Dexpensive_tests=enabled
> 
> is very fast, so let's just stop telling users about the variable
> instead and call it a day.
> 
> Signed-off-by: Andrea Bolognani <abologna at redhat.com>
> ---
>  docs/advanced-tests.rst | 8 +-------
>  meson_options.txt       | 2 +-
>  tests/test-lib.sh       | 2 +-
>  3 files changed, 3 insertions(+), 9 deletions(-)

Reviewed-by: Pavel Hrdina <phrdina at redhat.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20200923/804cf004/attachment-0001.sig>


More information about the libvir-list mailing list