remove Deferred.setTimeout usage from twisted.mail

[exarkun]

This is started in process-alias-timeout-2729. I think the tests still need a bit of work, though.

[Peaker]

The branch needs to be forward-merged with trunk or restarted.

Can't a similar fix as in http://twistedmatrix.com/trac/attachment/ticket/2245/cdefer.3.patch be used? (The patch includes more things, take a look just at the alias.py fix).

[therve]

This not as simple, because your patch doesn't add tests.

[exarkun]

I merged this forward to process-alias-timeout-2729-2. One specific area I notice is short on test coverage is ProcessAliasProtocol.processEnded. If you run trial --coverage twisted.mail.test, you see that the entire else suite in that method is uncovered. The other area of the code I'm worried about is MessageWrapper._processEnded: the test suite only exercises it once - which means it only goes through the true case of the if self._timeoutCallID is not None: test, which means the conditional doesn't need to be there at all (as far as the test suite is concerned). A test which went through this method with that conditional evaluating to false would be nice - if it is possible to do that, otherwise the check should be dropped.

I'll get to this after I finish all the reviews, but if someone beats me to it that'd be great.

[exarkun]

Oh yea, this is blocking #2245 which I want to review, so I guess I'll end up working on this first thing tomorrow.

[exarkun]

Done in process-alias-timeout-2729-2

[therve]

• unused import error in alias.py
• StubProcess doesn't document its methods

Otherwise, that looks great, so please merge once done.

[exarkun]

(In [21622]) remove unused import

refs #2729

[exarkun]

(In [21624]) docstrings for StubProcess

refs #2729

[exarkun]

(In [21628]) Merge process-alias-timeout-2729-2

Author: exarkun Reviewer: therve Fixes #2729

Improve test coverage for the aliases(5) process support, change the implementation not to use Deferred.setTimeout, and fix some other cases related to process termination which were handled incorrectly.