[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 10/11] pytest: add pytest to the meson build system
From: |
Michael S. Tsirkin |
Subject: |
Re: [PATCH v2 10/11] pytest: add pytest to the meson build system |
Date: |
Tue, 27 Sep 2022 17:26:06 -0400 |
On Tue, Sep 06, 2022 at 02:10:56PM +0100, Daniel P. Berrangé wrote:
> On Tue, Jul 12, 2022 at 12:22:10PM +0530, Ani Sinha wrote:
> >
> >
> > On Mon, 11 Jul 2022, John Snow wrote:
> >
> > > On Sun, Jul 10, 2022 at 1:01 PM Ani Sinha <ani@anisinha.ca> wrote:
> > > >
> > > > Integrate the pytest framework with the meson build system. This will
> > > > make meson
> > > > run all the pytests under the pytest directory.
> > > >
> > > > Signed-off-by: Ani Sinha <ani@anisinha.ca>
> > > > ---
> > > > tests/Makefile.include | 4 +++-
> > > > tests/meson.build | 1 +
> > > > tests/pytest/meson.build | 49 ++++++++++++++++++++++++++++++++++++++++
> > > > 3 files changed, 53 insertions(+), 1 deletion(-)
> > > > create mode 100644 tests/pytest/meson.build
> > > >
> > > > diff --git a/tests/Makefile.include b/tests/Makefile.include
> > > > index 3accb83b13..40755a6bd1 100644
> > > > --- a/tests/Makefile.include
> > > > +++ b/tests/Makefile.include
> > > > @@ -3,12 +3,14 @@
> > > > .PHONY: check-help
> > > > check-help:
> > > > @echo "Regression testing targets:"
> > > > - @echo " $(MAKE) check Run block, qapi-schema,
> > > > unit, softfloat, qtest and decodetree tests"
> > > > + @echo " $(MAKE) check Run block, qapi-schema,
> > > > unit, softfloat, qtest, pytest and decodetree tests"
> > >
> > > Does this mean that "make check" *requires* an internet connection?
> >
> > No. My test will be skipped if it is unable to download the artifacts it
> > requires due to lack of Internet connectivity.
>
> That's not the only concern, there are also people who have metered
> internet connections, or whose connections are slow and thus have
> long download times. Any test that downloads should be opt-in only.
>
>
> With regards,
> Daniel
<rant>
This is why I wanted git submodules. A well understood decentralized
model. Now we are reinventing them badly.
I asked on the maintainers summit what issues people have with
submodules, no one volunteered any information.
It might make sense to figure out if there's a way to
use submodules sanely.
</rant>
Anyway, download should just be done separately,
make check should just verify it has the correct binary
and if not fail.
And I'd like to have a target that fails if it can not
run the tests a opposed to skipping.
> --
> |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
> |: https://libvirt.org -o- https://fstop138.berrange.com :|
> |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|