Hi,

On Fri, 31 May 2024 08:51:58 +0200 Paul Gevers <elb...@debian.org> wrote:
The autopkgtest failure is not due to the test failing itself, but due to the test writing to stderr, which by default is considered a test failure. You can ignore output to stderr by marking the test with the allow-stderr restriction. But please, if you do that, this test doesn't actually do anything anymore on Debian infrastructure so be sure to also mark it as superficial or mark the test as skippable and exit with exit code 77 in case all tests are skipped.

I prepared an NMU fixing this issue and have uploaded it.

I used dgit so the changes can be fetched as git commits, but I also attached them.

Paul
From 580c3256e181a18b9bdb40bfd109c2c43ba9afb3 Mon Sep 17 00:00:00 2001
From: Paul Gevers <elb...@debian.org>
Date: Sun, 28 Jul 2024 17:14:03 +0200
Subject: [PATCH 1/2] tests: don't fail on output to stderr and mark it
 superficial

The test currently doesn't work on ci.debian.net infrastructure because
$(hciconfig) doesn't seem to find any device and prints
"""
Can't open HCI socket.: Address family not supported by protocol
"""

to stderr. Autopkgtest by default fails when a test writes to stderr, so let's
not fail the test on that. However, as all tests are now skipped, in the
current for it can only be superficial, i.e. a passing test has no guarantees
about the functioning of the package.

Closes: #1071147
---
 debian/tests/control | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/debian/tests/control b/debian/tests/control
index c710150..4168957 100644
--- a/debian/tests/control
+++ b/debian/tests/control
@@ -1,3 +1,3 @@
 Tests: bluez-response
 Depends: python3-dbus:native, bluez, bluez-test-scripts
-Restrictions: needs-root, isolation-container
+Restrictions: needs-root, isolation-container, allow-stderr, superficial
-- 
2.43.0

From 9dd109e11eb01bdd8bccb56c8e4d7d7f5db771c0 Mon Sep 17 00:00:00 2001
From: Paul Gevers <elb...@debian.org>
Date: Sun, 28 Jul 2024 17:38:44 +0200
Subject: [PATCH 2/2] Update d/changelog for release 5.73-1.1

---
 debian/changelog | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/debian/changelog b/debian/changelog
index bc017b9..bd15608 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+bluez (5.73-1.1) unstable; urgency=medium
+
+  * tests: don't fail on output to stderr (Closes: #1071147)
+  * tests: mark the test as superficial as it doesn't test anything now
+
+ -- Paul Gevers <elb...@debian.org>  Sun, 28 Jul 2024 17:14:24 +0200
+
 bluez (5.73-1) unstable; urgency=medium
 
   * Update to 5.73. (Closes: #1060795, #1060242)
-- 
2.43.0

Attachment: OpenPGP_signature.asc
Description: OpenPGP digital signature

Reply via email to