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

Bug #406894 reported by Sidnei da Silva
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zope.app.twisted
Triaged
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)
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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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