Opened 15 years ago

Last modified 15 years ago

#2006 defect new

trial can misreport leftover DelayedCalls

Reported by: Jean-Paul Calderone Owned by:
Priority: normal Milestone:
Component: trial Keywords:
Cc: Branch:


On a recent run of the full test suite, this output was observed:

twisted.mail.test.test_imap.AuthenticatorTestCase.testFailedCramMD5 ... 
/tmp/Twisted/twisted/trial/ twisted.trial.reporter.BrokenTestCaseWarning: REACTOR UNCLEAN! traceback(s) follow: 
Traceback (most recent call last):
  File "/tmp/Twisted/twisted/trial/", line 447, in _cleanUp
  File "/tmp/Twisted/twisted/trial/", line 64, in postCaseCleanup
    return self._dispatch('logErrCheck', 'cleanPending')
  File "/tmp/Twisted/twisted/trial/", line 72, in _dispatch
    getattr(self, "do_%s" % attr)()
  File "/tmp/Twisted/twisted/trial/", line 99, in do_cleanPending
    raise PendingTimedCallsError(s)
twisted.trial.util.PendingTimedCallsError: pendingTimedCalls still pending (consider setting twisted.internet.base.DelayedCall.debug = True): <DelayedCall 1100221964 [-0.000146865844727s] called=0 cancelled=0 _SimpleCallQueue.turn()>

However, _SimpleCallQueue is a twisted.pb feature which twisted.mail does not use.

Change History (5)

comment:1 Changed 15 years ago by Ralph Meijer

Resolution: fixed
Status: newclosed

(In [17910]) Make iq tracker clean up after disconnect or firing the associated deferred.

Author: ralphm Reviewer: exarkun Fixes #2006

comment:2 Changed 15 years ago by Ralph Meijer

Resolution: fixed
Status: closedreopened

Oops, wrong issue mentioned.

comment:3 Changed 15 years ago by Jonathan Lange


I wonder if we'll ever be rid of these.

comment:4 Changed 15 years ago by Jonathan Lange

Status: reopenednew

comment:5 Changed 11 years ago by <automation>

Owner: Jonathan Lange deleted
Note: See TracTickets for help on using tickets.