core::dbus::Fixture leaves stale dbus-daemon processes around if test suite terminates

Bug #1326200 reported by James Henstridge
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
dbus-cpp
Fix Released
Medium
Thomas Voß
dbus-cpp (Ubuntu)
Fix Released
Undecided
Thomas Voß

Bug Description

If a test suite terminates without core::dbus::Fixture's destructor running, two stale dbus-daemon subprocesses are left running. Among other things, this can lead to apparent test suite hangs when running under cmake/ctest: ctest doesn't consider a test to have completed until stdout/stderr are closed, and the dbus-daemon subprocesses hold those file descriptors open. The default timeout for tests seems to be 2 hours, so this can introduce a long delay.

It is clear that cases where this happens are errors, but as this class is intended to be used in test suites, it would be nice if it was a bit more robust about making sure the subprocesses are cleaned up.

Related branches

description: updated
Changed in dbus-cpp:
importance: Undecided → Medium
status: New → Confirmed
Changed in dbus-cpp:
assignee: nobody → Thomas Voß (thomas-voss)
status: Confirmed → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package dbus-cpp - 4.0.0+14.10.20140730-0ubuntu1

---------------
dbus-cpp (4.0.0+14.10.20140730-0ubuntu1) utopic; urgency=low

  [ Ubuntu daily release ]
  * debian/libdbus-cpp4.symbols.32bit: auto-update to released version
  * debian/libdbus-cpp4.symbols.64bit: auto-update to released version

  [ thomas-voss ]
  * Make sure that dbus daemon instances fired up for testing are torn
    down after a configurable timeout. (LP: #1326200)
  * Make sure that core::dbus::Service::Service(...) interprets return
    values of dbus_bus_request_name correctly. (LP: #1329211)
 -- Ubuntu daily release <email address hidden> Wed, 30 Jul 2014 16:12:30 +0000

Changed in dbus-cpp (Ubuntu):
status: New → Fix Released
Changed in dbus-cpp:
status: In Progress → Fix Released
Changed in dbus-cpp (Ubuntu):
assignee: nobody → Thomas Voß (thomas-voss)
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.