Missing dep8 tests

Bug #1677889 reported by Joshua Powers
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
libapache2-reload-perl (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

As of March 29, 2017, this source package did not contain dep8 tests in
the current development release of Ubuntu, named Zesty. This was
determined by running `pull-lp-source libapache2-reload-perl zesty` and then
checking for the existence of 'debian/tests/' and
'debian/tests/control'.

Test automation is essential to higher levels of quality and confidence
in updates to packages. dep8 tests [1] specify how automatic testing can
be integrated into packages and then run by package maintainers before
new uploads.

This defect is to report the absence of these tests and to report the
opportunity as a potential item for development by both new and
experienced contributors.

[1] http://packaging.ubuntu.com/html/auto-pkg-test.html

 affects ubuntu/libapache2-reload-perl
 status new
 importance wishlist
 tag needs-dep8

- ---
Joshua Powers
Ubuntu Server
Canonical Ltd

-----BEGIN PGP SIGNATURE-----

iQIcBAEBCAAGBQJY3YJOAAoJEIP8BxPaZgwl8skQAIHeFt3NYIo/SeRLbERYWnNV
71LCJ8Y5gvNWEaOds9Wb9eAEbAuWjooctTuo6iYYZOYDo2qVfmKgM29nXK2get7H
zh96CCwEZRYRleK9pf1P4KWgMUDhgvoHaIpKCXkshOmhzcwiTif0u/DogFEZoAQG
Q5lfOczSG1CFkxKmVgmzaTfgwsURjtCj+A+rCUvBj/LRp/W9RO4V9QvUzUouypNC
K++sln4/AgVPIHiP9O6drZbuJj3pEjXH1GswvgS8bAqg95oe3RqNl2Mh0JcsBzs+
dXOyxxsH9Dn3QnRGQtvzPxASbqwi/yRlQVOxFMCPPHqDlKUyarX/vyALHokpAy3N
5iy0ZEqana9qUR3bWPxqHJS+z3rdFdOVnLVK8kS/6/5XjKO4jBpIT9SzP+F1pS2M
cOKDNjxvn42cMocak0ky7L75rDkEQDuwaJayVH8tmiDv9ugg/UH+D0trvFt8oPly
W6YjwGx6YtYKxO8Jdyr9dAi7pLj+O5ymqJqviaJoJRxW1ZO5iFcnVUKVftAvFN1s
nEL7xrqZnnq4wugxcoC1wPtriHu93TQzKz2ifeKkRFLEuLge5sRMSmC9hCJutEGL
HC1ohFSvNEjZtbL1tkRGveo8UNcjuGC4WdHaEG6yLsmr8TFROoewamYHcLx6VoK5
Vgh4Dn5sPZ1RjykoYTJu
=jXg+
-----END PGP SIGNATURE-----

Tags: needs-dep8
Revision history for this message
gregor herrmann (gregoa) wrote :

This check is incomplete for perl packages, the usually just have
"Testsuite: autopkgtest-pkg-perl"
in the source stanza of debian/control.
Cf. https://pkg-perl.alioth.debian.org/autopkgtest.html

Besides that, most perl packages can be autotested even without this field in debian/control, cf. autodep8, and https://anonscm.debian.org/cgit/collab-maint/debian-ci-config.git/tree/cookbooks/debci/files/default/whitelist-perl.txt

This is not the case for libapache2-reload-perl which needs some manual work.
I've now pushed a commit enabling (parts of) the autopkgtest-pkg-perl tests:

https://anonscm.debian.org/cgit/pkg-perl/packages/libapache2-reload-perl.git/commit/?id=2bb507c2cffbd328189bd84be67c41c10cc1ad27

Cheers,
gregor

Revision history for this message
Joshua Powers (powersj) wrote :

gregor - thank you for the hint about perl testsuite! I will add a check for that. I plan to re-run today/Monday and try to go through and update a few of defects.

And thank you for enabling some of the tests!

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package libapache2-reload-perl - 0.13-2

---------------
libapache2-reload-perl (0.13-2) unstable; urgency=medium

  * Team upload

  [ Salvatore Bonaccorso ]
  * debian/control: Use HTTPS transport protocol for Vcs-Git URI

  [ gregor herrmann ]
  * debian/copyright: change Copyright-Format 1.0 URL to HTTPS.
  * Mark package as autopkgtest-able. (LP: #1677889)
  * Declare compliance with Debian Policy 3.9.8.

  [ Salvatore Bonaccorso ]
  * Update Vcs-* headers for switch to salsa.debian.org

  [ Laurent Baillet ]
  * fix lintian file-contains-trailing-whitespace warning

  [ Xavier Guimard ]
  * Add upstream/metadata
  * Declare compliance with policy 4.2.1
  * debian/copyright: use https link
  * Bump debhelper compat to 10
  * Add spelling.patch (reported)
  * Improve autopkgtest using custom test

 -- Xavier Guimard <email address hidden> Fri, 31 Aug 2018 09:39:27 +0200

Changed in libapache2-reload-perl (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.