Opened 15 years ago

Last modified 22 months ago

#342 defect closed fixed (fixed)

intermittent test_mail process failure[PATCH]

Reported by: warner Owned by:
Priority: high Milestone:
Component: Keywords:
Cc: spiv, warner Branch:


Change History (8)

comment:1 Changed 15 years ago by warner

I've seen this happen twice in the full-2.3 builder.
I suspect it is related to #337 (test_process error).

ERROR: testProcessAlias (twisted.test.test_mail.ProcessAliasTestCase)
"/home/buildbot/BuildBot/slave/full2.3/Twisted/twisted/test/", line
1029, in testProcessAlias
  File "/home/buildbot/BuildBot/slave/full2.3/Twisted/twisted/trial/",
line 72, in deferredResult
    raise result
Failure: [Failure instance: Traceback: twisted.internet.defer.TimeoutError,
Callback timed out

comment:2 Changed 15 years ago by spiv

Here's another failure for the same test, from the freebsd slave:

FAILURE: testCyclicAlias (twisted.test.test_mail.ProcessAliasTestCase)
Traceback (most recent call last):
  File "/home/slave/twisted/bsd-full2.2/Twisted/twisted/trial/", line
246, in _runPhase
    stage(*args, **kwargs)
  File "/home/slave/twisted/bsd-full2.2/Twisted/twisted/trial/", line 43,
in reactorCleanUp
    raise unittest.FailTest, msg
pendingTimedCalls still pending:
 <DelayedCall 145373580 [-0.00216996669769s] called=0 cancelled=0 <code object
reapAllProcesses at 0x83b06a0, file
"/home/slave/twisted/bsd-full2.2/Twisted/twisted/internet/", line 64>()>


comment:3 Changed 15 years ago by Moshe Zadka

My guess is that the reactor doesn't have time to clean up after itself.
Here's a patch which should fix the problem:

Index: twisted/test/
RCS file: /cvs/Twisted/twisted/test/,v
retrieving revision 1.33
diff -u -r1.33
--- twisted/test/   9 Oct 2003 17:32:28 -0000       1.33
+++ twisted/test/   21 Oct 2003 06:29:43 -0000
@@ -1099,6 +1099,7 @@
             mail.alias.MessageWrapper(p, 'echo')
         self.assertEquals(r, expected)
+        reactor.iterate(1)
 if not components.implements(reactor, interfaces.IReactorProcess):
     ProcessAliasTestCase = "IReactorProcess not supported"

comment:4 Changed 15 years ago by itamarst

I tried the solution, lets see if it helps.

comment:5 Changed 15 years ago by Moshe Zadka

It seems this problem (and 337) were fixed.
Let's give it a couple more tests, and close it.

comment:6 Changed 15 years ago by Moshe Zadka

This bug, and 337, seem gone.
Please open new issues for intermittent failures if they show
up again.

comment:7 Changed 7 years ago by <automation>

comment:8 Changed 22 months ago by GitHub <noreply@…>

In dae42dc:

Merge pull request #342 from twisted/8577-rodrigc-win-trial-2

Author: rodrigc
Reviewer: markrwilliams
Fixes: #8577

Remove "python" before invoking tests via "tox -e py35-tests"

Note: See TracTickets for help on using tickets.