zope.app.twisted FTP tests should not leave pending calls and threads behind

Bug #406894 reported by Sidnei da Silva on 2009-07-30
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zope.app.twisted
Low
Unassigned

Bug Description

After updating an internal project to use Twisted 8.2, I had to change the ftp tests to get them to pass. One thing that changed was the '_Janitor' class, which doesn't have a 'do_cleanThreads' method anymore. Instead, I had to replace that by the following code:

+ janitor = util._Janitor(None, None)
+ janitor._cleanPending()
+ janitor._cleanThreads()
+ janitor._cleanReactor()

This code should not need to be called to get the tests to pass. Instead, the tests should cleanup after themselves.

Tres Seaver (tseaver) on 2010-04-15
Changed in zope3:
status: New → Triaged
importance: Medium → Low
affects: zope3 → zope.app.twisted
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers