summaryrefslogtreecommitdiff
path: root/src/mailman/runners
diff options
context:
space:
mode:
Diffstat (limited to 'src/mailman/runners')
-rw-r--r--src/mailman/runners/archive.py4
-rw-r--r--src/mailman/runners/bounce.py5
-rw-r--r--src/mailman/runners/command.py8
-rw-r--r--src/mailman/runners/digest.py3
-rw-r--r--src/mailman/runners/incoming.py6
-rw-r--r--src/mailman/runners/lmtp.py6
-rw-r--r--src/mailman/runners/nntp.py3
-rw-r--r--src/mailman/runners/outgoing.py10
-rw-r--r--src/mailman/runners/pipeline.py5
-rw-r--r--src/mailman/runners/rest.py3
-rw-r--r--src/mailman/runners/retry.py3
-rw-r--r--src/mailman/runners/tests/test_archiver.py9
-rw-r--r--src/mailman/runners/tests/test_bounce.py12
-rw-r--r--src/mailman/runners/tests/test_confirm.py9
-rw-r--r--src/mailman/runners/tests/test_digest.py3
-rw-r--r--src/mailman/runners/tests/test_incoming.py6
-rw-r--r--src/mailman/runners/tests/test_join.py6
-rw-r--r--src/mailman/runners/tests/test_lmtp.py3
-rw-r--r--src/mailman/runners/tests/test_nntp.py8
-rw-r--r--src/mailman/runners/tests/test_outgoing.py14
-rw-r--r--src/mailman/runners/tests/test_owner.py12
-rw-r--r--src/mailman/runners/tests/test_pipeline.py9
-rw-r--r--src/mailman/runners/tests/test_rest.py3
-rw-r--r--src/mailman/runners/tests/test_retry.py6
-rw-r--r--src/mailman/runners/virgin.py5
25 files changed, 45 insertions, 116 deletions
diff --git a/src/mailman/runners/archive.py b/src/mailman/runners/archive.py
index b49f5c265..f81f9ee3e 100644
--- a/src/mailman/runners/archive.py
+++ b/src/mailman/runners/archive.py
@@ -17,9 +17,6 @@
"""Archive runner."""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'ArchiveRunner',
]
@@ -31,7 +28,6 @@ import logging
from email.utils import parsedate_tz, mktime_tz
from datetime import datetime
from lazr.config import as_timedelta
-
from mailman.config import config
from mailman.core.runner import Runner
from mailman.interfaces.archiver import ClobberDate
diff --git a/src/mailman/runners/bounce.py b/src/mailman/runners/bounce.py
index 9312a9158..3a85006fe 100644
--- a/src/mailman/runners/bounce.py
+++ b/src/mailman/runners/bounce.py
@@ -20,11 +20,10 @@
import logging
from flufl.bounce import all_failures, scan_message
-from zope.component import getUtility
-
from mailman.app.bounces import ProbeVERP, StandardVERP, maybe_forward
from mailman.core.runner import Runner
from mailman.interfaces.bounce import BounceContext, IBounceProcessor
+from zope.component import getUtility
COMMASPACE = ', '
@@ -33,7 +32,7 @@ log = logging.getLogger('mailman.bounce')
elog = logging.getLogger('mailman.error')
-
+
class BounceRunner(Runner):
"""The bounce runner."""
diff --git a/src/mailman/runners/command.py b/src/mailman/runners/command.py
index 7f8c7f470..b0775c4f4 100644
--- a/src/mailman/runners/command.py
+++ b/src/mailman/runners/command.py
@@ -17,9 +17,6 @@
"""-request robot command runner."""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'CommandRunner',
'Results',
@@ -37,15 +34,14 @@ import logging
from email.errors import HeaderParseError
from email.header import decode_header, make_header
from email.iterators import typed_subpart_iterator
-from zope.component import getUtility
-from zope.interface import implementer
-
from mailman.config import config
from mailman.core.i18n import _
from mailman.core.runner import Runner
from mailman.email.message import UserNotification
from mailman.interfaces.command import ContinueProcessing, IEmailResults
from mailman.interfaces.languages import ILanguageManager
+from zope.component import getUtility
+from zope.interface import implementer
NL = '\n'
diff --git a/src/mailman/runners/digest.py b/src/mailman/runners/digest.py
index 0a13a3c49..52bfb8859 100644
--- a/src/mailman/runners/digest.py
+++ b/src/mailman/runners/digest.py
@@ -17,9 +17,6 @@
"""Digest runner."""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'DigestRunner',
]
diff --git a/src/mailman/runners/incoming.py b/src/mailman/runners/incoming.py
index d75469a5e..a5d8fbea3 100644
--- a/src/mailman/runners/incoming.py
+++ b/src/mailman/runners/incoming.py
@@ -26,21 +26,17 @@ prepared for delivery. Rejections, discards, and holds are processed
immediately.
"""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'IncomingRunner',
]
-from zope.component import getUtility
-
from mailman.core.chains import process
from mailman.core.runner import Runner
from mailman.database.transaction import transaction
from mailman.interfaces.address import ExistingAddressError
from mailman.interfaces.usermanager import IUserManager
+from zope.component import getUtility
diff --git a/src/mailman/runners/lmtp.py b/src/mailman/runners/lmtp.py
index 2a685f18b..85730bb7d 100644
--- a/src/mailman/runners/lmtp.py
+++ b/src/mailman/runners/lmtp.py
@@ -34,9 +34,6 @@ so that the peer mail server can provide better diagnostics.
http://www.faqs.org/rfcs/rfc2033.html
"""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'LMTPRunner',
]
@@ -48,8 +45,6 @@ import logging
import asyncore
from email.utils import parseaddr
-from zope.component import getUtility
-
from mailman.config import config
from mailman.core.runner import Runner
from mailman.database.transaction import transactional
@@ -57,6 +52,7 @@ from mailman.email.message import Message
from mailman.interfaces.listmanager import IListManager
from mailman.utilities.datetime import now
from mailman.utilities.email import add_message_hash
+from zope.component import getUtility
elog = logging.getLogger('mailman.error')
diff --git a/src/mailman/runners/nntp.py b/src/mailman/runners/nntp.py
index d26001a57..7fb16f1b2 100644
--- a/src/mailman/runners/nntp.py
+++ b/src/mailman/runners/nntp.py
@@ -17,9 +17,6 @@
"""NNTP runner."""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'NNTPRunner',
]
diff --git a/src/mailman/runners/outgoing.py b/src/mailman/runners/outgoing.py
index db0d847c4..9af4e7c11 100644
--- a/src/mailman/runners/outgoing.py
+++ b/src/mailman/runners/outgoing.py
@@ -17,14 +17,16 @@
"""Outgoing runner."""
+__all__ = [
+ 'OutgoingRunner',
+ ]
+
+
import socket
import logging
from datetime import datetime
from lazr.config import as_boolean, as_timedelta
-from uuid import UUID
-from zope.component import getUtility
-
from mailman.config import config
from mailman.core.runner import Runner
from mailman.interfaces.bounce import BounceContext, IBounceProcessor
@@ -34,6 +36,8 @@ from mailman.interfaces.pending import IPendings
from mailman.interfaces.subscriptions import ISubscriptionService
from mailman.utilities.datetime import now
from mailman.utilities.modules import find_name
+from uuid import UUID
+from zope.component import getUtility
# This controls how often _do_periodic() will try to deal with deferred
diff --git a/src/mailman/runners/pipeline.py b/src/mailman/runners/pipeline.py
index 13226c6fc..357863d2e 100644
--- a/src/mailman/runners/pipeline.py
+++ b/src/mailman/runners/pipeline.py
@@ -22,6 +22,11 @@ through the 'preparation pipeline'. This pipeline adds, deletes and modifies
headers, calculates message recipients, and more.
"""
+__all__ = [
+ 'PipelineRunner',
+ ]
+
+
from mailman.core.pipelines import process
from mailman.core.runner import Runner
diff --git a/src/mailman/runners/rest.py b/src/mailman/runners/rest.py
index 5980e6263..d39a8a6ff 100644
--- a/src/mailman/runners/rest.py
+++ b/src/mailman/runners/rest.py
@@ -17,9 +17,6 @@
"""Start the administrative HTTP server."""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'RESTRunner',
]
diff --git a/src/mailman/runners/retry.py b/src/mailman/runners/retry.py
index b4148ee3a..f4705ba75 100644
--- a/src/mailman/runners/retry.py
+++ b/src/mailman/runners/retry.py
@@ -17,9 +17,6 @@
"""Retry delivery."""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'RetryRunner',
]
diff --git a/src/mailman/runners/tests/test_archiver.py b/src/mailman/runners/tests/test_archiver.py
index 12bdb0edd..9e3d9626c 100644
--- a/src/mailman/runners/tests/test_archiver.py
+++ b/src/mailman/runners/tests/test_archiver.py
@@ -17,9 +17,6 @@
"""Test the archive runner."""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'TestArchiveRunner',
]
@@ -29,19 +26,17 @@ import os
import unittest
from email import message_from_file
-from zope.interface import implementer
-
from mailman.app.lifecycle import create_list
from mailman.config import config
from mailman.interfaces.archiver import IArchiver
from mailman.interfaces.mailinglist import IListArchiverSet
from mailman.runners.archive import ArchiveRunner
from mailman.testing.helpers import (
- configuration,
- make_testable_runner,
+ configuration, make_testable_runner,
specialized_message_from_string as mfs)
from mailman.testing.layers import ConfigLayer
from mailman.utilities.datetime import RFC822_DATE_FMT, factory, now
+from zope.interface import implementer
diff --git a/src/mailman/runners/tests/test_bounce.py b/src/mailman/runners/tests/test_bounce.py
index b296f4476..875437dc2 100644
--- a/src/mailman/runners/tests/test_bounce.py
+++ b/src/mailman/runners/tests/test_bounce.py
@@ -17,9 +17,6 @@
"""Test the bounce runner."""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'TestBounceRunner',
'TestBounceRunnerBug876774',
@@ -29,9 +26,6 @@ __all__ = [
import unittest
-from zope.component import getUtility
-from zope.interface import implementer
-
from mailman.app.bounces import send_probe
from mailman.app.lifecycle import create_list
from mailman.config import config
@@ -42,11 +36,11 @@ from mailman.interfaces.styles import IStyle, IStyleManager
from mailman.interfaces.usermanager import IUserManager
from mailman.runners.bounce import BounceRunner
from mailman.testing.helpers import (
- LogFileMark,
- get_queue_messages,
- make_testable_runner,
+ LogFileMark, get_queue_messages, make_testable_runner,
specialized_message_from_string as message_from_string)
from mailman.testing.layers import ConfigLayer
+from zope.component import getUtility
+from zope.interface import implementer
diff --git a/src/mailman/runners/tests/test_confirm.py b/src/mailman/runners/tests/test_confirm.py
index d387fcfe6..11514044a 100644
--- a/src/mailman/runners/tests/test_confirm.py
+++ b/src/mailman/runners/tests/test_confirm.py
@@ -17,9 +17,6 @@
"""Test the `confirm` command."""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'TestConfirm',
]
@@ -29,8 +26,6 @@ import unittest
from datetime import datetime
from email.iterators import body_line_iterator
-from zope.component import getUtility
-
from mailman.app.lifecycle import create_list
from mailman.config import config
from mailman.database.transaction import transaction
@@ -38,10 +33,10 @@ from mailman.interfaces.registrar import IRegistrar
from mailman.interfaces.usermanager import IUserManager
from mailman.runners.command import CommandRunner
from mailman.testing.helpers import (
- get_queue_messages,
- make_testable_runner,
+ get_queue_messages, make_testable_runner,
specialized_message_from_string as mfs)
from mailman.testing.layers import ConfigLayer
+from zope.component import getUtility
diff --git a/src/mailman/runners/tests/test_digest.py b/src/mailman/runners/tests/test_digest.py
index 8b19188df..83156f04e 100644
--- a/src/mailman/runners/tests/test_digest.py
+++ b/src/mailman/runners/tests/test_digest.py
@@ -17,9 +17,6 @@
"""Test the digest runner."""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'TestDigest',
'TestI18nDigest',
diff --git a/src/mailman/runners/tests/test_incoming.py b/src/mailman/runners/tests/test_incoming.py
index 2d49ae550..77fe2da02 100644
--- a/src/mailman/runners/tests/test_incoming.py
+++ b/src/mailman/runners/tests/test_incoming.py
@@ -17,9 +17,6 @@
"""Test the incoming queue runner."""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'TestIncoming',
]
@@ -32,8 +29,7 @@ from mailman.chains.base import TerminalChainBase
from mailman.config import config
from mailman.runners.incoming import IncomingRunner
from mailman.testing.helpers import (
- get_queue_messages,
- make_testable_runner,
+ get_queue_messages, make_testable_runner,
specialized_message_from_string as mfs)
from mailman.testing.layers import ConfigLayer
diff --git a/src/mailman/runners/tests/test_join.py b/src/mailman/runners/tests/test_join.py
index 2aa361254..df24bc06b 100644
--- a/src/mailman/runners/tests/test_join.py
+++ b/src/mailman/runners/tests/test_join.py
@@ -17,9 +17,6 @@
"""Test mailing list joins."""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'TestJoin',
'TestJoinWithDigests',
@@ -29,8 +26,6 @@ __all__ = [
import unittest
from email.iterators import body_line_iterator
-from zope.component import getUtility
-
from mailman.app.lifecycle import create_list
from mailman.config import config
from mailman.interfaces.member import DeliveryMode
@@ -42,6 +37,7 @@ from mailman.testing.helpers import (
get_queue_messages, make_testable_runner, reset_the_world,
specialized_message_from_string as mfs)
from mailman.testing.layers import ConfigLayer
+from zope.component import getUtility
diff --git a/src/mailman/runners/tests/test_lmtp.py b/src/mailman/runners/tests/test_lmtp.py
index 41a4f93e0..44b6a0889 100644
--- a/src/mailman/runners/tests/test_lmtp.py
+++ b/src/mailman/runners/tests/test_lmtp.py
@@ -17,9 +17,6 @@
"""Tests for the LMTP server."""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'TestLMTP',
]
diff --git a/src/mailman/runners/tests/test_nntp.py b/src/mailman/runners/tests/test_nntp.py
index df3ea8c4b..e3218af33 100644
--- a/src/mailman/runners/tests/test_nntp.py
+++ b/src/mailman/runners/tests/test_nntp.py
@@ -17,9 +17,6 @@
"""Test the NNTP runner and related utilities."""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'TestPrepareMessage',
'TestNNTPRunner',
@@ -36,10 +33,7 @@ from mailman.config import config
from mailman.interfaces.nntp import NewsgroupModeration
from mailman.runners import nntp
from mailman.testing.helpers import (
- LogFileMark,
- configuration,
- get_queue_messages,
- make_testable_runner,
+ LogFileMark, configuration, get_queue_messages, make_testable_runner,
specialized_message_from_string as mfs)
from mailman.testing.layers import ConfigLayer
diff --git a/src/mailman/runners/tests/test_outgoing.py b/src/mailman/runners/tests/test_outgoing.py
index 68fb75fc3..8f51c4ce2 100644
--- a/src/mailman/runners/tests/test_outgoing.py
+++ b/src/mailman/runners/tests/test_outgoing.py
@@ -17,10 +17,11 @@
"""Test the outgoing runner."""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
+ 'TestOnce',
+ 'TestSocketError',
+ 'TestSomeRecipientsFailed',
+ 'TestVERPSettings',
]
@@ -32,8 +33,6 @@ import unittest
from contextlib import contextmanager
from datetime import datetime, timedelta
from lazr.config import as_timedelta
-from zope.component import getUtility
-
from mailman.app.bounces import send_probe
from mailman.app.lifecycle import create_list
from mailman.config import config
@@ -45,12 +44,11 @@ from mailman.interfaces.pending import IPendings
from mailman.interfaces.usermanager import IUserManager
from mailman.runners.outgoing import OutgoingRunner
from mailman.testing.helpers import (
- LogFileMark,
- get_queue_messages,
- make_testable_runner,
+ LogFileMark, get_queue_messages, make_testable_runner,
specialized_message_from_string as message_from_string)
from mailman.testing.layers import ConfigLayer, SMTPLayer
from mailman.utilities.datetime import factory, now
+from zope.component import getUtility
diff --git a/src/mailman/runners/tests/test_owner.py b/src/mailman/runners/tests/test_owner.py
index 503f1e18d..15ca07c2e 100644
--- a/src/mailman/runners/tests/test_owner.py
+++ b/src/mailman/runners/tests/test_owner.py
@@ -22,9 +22,6 @@
# tests. They're not exactly integration tests, but they do touch lots of
# parts of the system.
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'TestEmailToOwner',
]
@@ -32,22 +29,19 @@ __all__ = [
import unittest
-from operator import itemgetter
-from zope.component import getUtility
-
from mailman.app.lifecycle import create_list
from mailman.config import config
from mailman.database.transaction import transaction
from mailman.interfaces.member import MemberRole
from mailman.interfaces.usermanager import IUserManager
from mailman.testing.helpers import (
- TestableMaster,
- get_lmtp_client,
- make_testable_runner)
+ TestableMaster, get_lmtp_client, make_testable_runner)
from mailman.runners.incoming import IncomingRunner
from mailman.runners.outgoing import OutgoingRunner
from mailman.runners.pipeline import PipelineRunner
from mailman.testing.layers import SMTPLayer
+from operator import itemgetter
+from zope.component import getUtility
diff --git a/src/mailman/runners/tests/test_pipeline.py b/src/mailman/runners/tests/test_pipeline.py
index 1eba5cfbf..347bde16b 100644
--- a/src/mailman/runners/tests/test_pipeline.py
+++ b/src/mailman/runners/tests/test_pipeline.py
@@ -17,9 +17,6 @@
"""Test the pipeline runner."""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'TestPipelineRunner',
]
@@ -27,17 +24,15 @@ __all__ = [
import unittest
-from zope.interface import implementer
-
from mailman.app.lifecycle import create_list
from mailman.config import config
from mailman.interfaces.handler import IHandler
from mailman.interfaces.pipeline import IPipeline
from mailman.runners.pipeline import PipelineRunner
from mailman.testing.helpers import (
- make_testable_runner,
- specialized_message_from_string as mfs)
+ make_testable_runner, specialized_message_from_string as mfs)
from mailman.testing.layers import ConfigLayer
+from zope.interface import implementer
diff --git a/src/mailman/runners/tests/test_rest.py b/src/mailman/runners/tests/test_rest.py
index bbe026ad6..96ca19089 100644
--- a/src/mailman/runners/tests/test_rest.py
+++ b/src/mailman/runners/tests/test_rest.py
@@ -17,9 +17,6 @@
"""Test the REST runner."""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'TestRESTRunner',
]
diff --git a/src/mailman/runners/tests/test_retry.py b/src/mailman/runners/tests/test_retry.py
index 15775e5d8..0a0929991 100644
--- a/src/mailman/runners/tests/test_retry.py
+++ b/src/mailman/runners/tests/test_retry.py
@@ -17,9 +17,6 @@
"""Test the retry runner."""
-from __future__ import absolute_import, print_function, unicode_literals
-
-__metaclass__ = type
__all__ = [
'TestRetryRunner',
]
@@ -31,8 +28,7 @@ from mailman.app.lifecycle import create_list
from mailman.config import config
from mailman.runners.retry import RetryRunner
from mailman.testing.helpers import (
- get_queue_messages,
- make_testable_runner,
+ get_queue_messages, make_testable_runner,
specialized_message_from_string as message_from_string)
from mailman.testing.layers import ConfigLayer
diff --git a/src/mailman/runners/virgin.py b/src/mailman/runners/virgin.py
index 0f91d61af..8ff45e86e 100644
--- a/src/mailman/runners/virgin.py
+++ b/src/mailman/runners/virgin.py
@@ -23,6 +23,11 @@ to go through some minimal processing before they can be sent out to the
recipient.
"""
+__all__ = [
+ 'VirginRunner',
+ ]
+
+
from mailman.core.pipelines import process
from mailman.core.runner import Runner