On Mon, 25 Jul 2022 at 09:26:20 +0100, Julian Gilbey wrote:
> Source package header
> ---------------------
>
> To allow test execution environments to discover packages which provide
> tests, their source packages can have a ``Testsuite:`` header
> containing ``autopkgtest`` (or a value like ``autopkgtest-pkg-perl``,
> see below). Multiple values are comma separated, as usual in control
> files, and this results in multiple testsuites being run. If the
> source package has a debian/tests/control file, then ``autopkgtest``
> is automatically added to the list of testsuites; in most cases you
> can therefore leave this header field out.
I like this text, but there is a practical problem with it: multiple values
for Testsuite *should*, but don't, result in multiple testsuites being run,
because autodep8 doesn't implement that (#1042717, #1061620, #1077645).
For now I adjusted this to "this is intended to result in multiple
testsuites being run" and added a note about this not working as intended
due to autodep8 limitations.
> I would also add an extra sentence to the end of the following section
> ("Automatic test control file for known package types"), lengthening
> the second paragraph to say:
>
> Those packages do not have to provide ``debian/tests/``, but they should
> still include an appropriate source package header
> (``Testsuite: autopkgtest-pkg-perl`` or similar) so that they can be
> discovered in the archive. (Note that adding ``Testsuite:
> autopkgtest-pkg-perl`` to a package which has a
> ``debian/tests/control`` file will cause ``autodep8`` to be run in
> addition to the tests defined in that file.)
Applied, thanks. I removed the parentheses, I don't think they add much.
smcv
