AutopilotTestCase._kill_pid and _kill_process need refactoring

Bug #1228121 reported by Thomi Richards
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Autopilot
Fix Released
Medium
Unassigned

Bug Description

These methods should be functions instead, and should use the psutils methods instead of subprocess. Additionally, we should refactor things such that as have a separate class that understands how to launch and kill applications, and how to get their exit code and stdout/stderr. This would make the code around launching processes under upstart / directly much cleaner.

Probably too late to get this in for saucy though.

Revision history for this message
Christopher Lee (veebers) wrote :

This was resolved in revno 402[1] so I'm closing this as Fix Released.

[1] http://bazaar.launchpad.net/~autopilot/autopilot/trunk/revision/402

Changed in autopilot:
status: Triaged → 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.