From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [RFC] testing: add python-cryptography tests
Date: Wed, 6 Sep 2017 10:06:56 +0200 [thread overview]
Message-ID: <20170906100656.5f5de18d@windsurf.lan> (raw)
In-Reply-To: <59af5dfae6364_331f1091a18551a0@ultri3.mail>
Hello,
On Tue, 05 Sep 2017 23:31:22 -0300, Ricardo Martincoski wrote:
> 1) I propose to merge this file into the existing test_python.py
>
> The rationale is:
> For the long term:
> As I see, the test infra is used to detect regressions. It is important for both
> the basic features and also for the complex scenarios (like the dependencies of
> this package). But I don't expect to see tests added to every single package or
> python package (it would take a very long time to run all those tests), so this
> file probably will not grow too much and we can keep the tests together.
Actually, I am not sure I agree with this last part. I even think we
should probably have a minimal test for each and every Python package,
because the only way to verify that their dependencies are correct is
to have a runtime test for them. So it would be nice to have a very
simple way to write Python related tests, like a list of tuples
(Buildroot package, Python package to import or Python command to run),
and have the test infrastructure "generate" tests for all entries in
this list.
Yes, it's going to be a long test suite, with lots of tests, but the
test suite is anyway going to be long enough that it won't make sense
for anyone to run the full test suite. I really see two use cases for
the test suite:
- You're doing tests in a specific area, and want to validate that
you're not breaking things. For this use case, you'll just run the
few test cases you're interested in.
- We regularly run in a CI environment all the tests. For this
situation, we don't really care how long it takes. Even if it takes
24 hours to run the full test suite, I don't think it's a big
problem.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
next prev parent reply other threads:[~2017-09-06 8:06 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-04 7:15 [Buildroot] [RFC] testing: add python-cryptography tests yegorslists at googlemail.com
2017-09-06 2:31 ` Ricardo Martincoski
2017-09-06 8:06 ` Thomas Petazzoni [this message]
2017-09-06 11:31 ` Ricardo Martincoski
2017-09-06 11:49 ` Thomas Petazzoni
2017-09-06 9:44 ` Yegor Yefremov
2017-09-06 11:09 ` Ricardo Martincoski
2017-09-06 11:27 ` Thomas Petazzoni
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170906100656.5f5de18d@windsurf.lan \
--to=thomas.petazzoni@free-electrons.com \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox