Convert the rest of test_runners.py to doctests; even though incomplete, they

test everything the old unit tests tested.  There are XXX's left in the
doctests as reminders to flesh them out.

Change the NNTP_REWRITE_DUPLICATE_HEADERS to use proper capitalization.

Revert a change I made in the conversion of the Switchboard class:
Switchboard.files is no longer a generator.  The Runner implementation is
cleaner if this returns a concrete list, so that's what it does now.  Update
the tests to reflect that.

The Runner simplifies now too because it no longer needs _open_files() or the
_listcache WeakValueDictionary.  The standard list manager handles all this
now, so just use it directly.

Also change the way the Runner sets the language context in _onefile().  It
still tries to set it to the preferred language of the sender, if the sender
is a member of the list.  Otherwise it sets it to the list's preferred
language, not the system's preferred language.  Removed a conditional that
can't possibly happen.

1 files changed, 0 insertions, 163 deletions

diff --git a/Mailman/Queue/tests/test_runners.py b/Mailman/Queue/tests/test_runners.py
deleted file mode 100644
index 27269909f..000000000
--- a/Mailman/Queue/tests/test_runners.py
+++ /dev/null
@@ -1,163 +0,0 @@
-# Copyright (C) 2001-2007 by the Free Software Foundation, Inc.
-#
-# This program is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License
-# as published by the Free Software Foundation; either version 2
-# of the License, or (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301,
-# USA.
-
-"""Unit tests for the various Mailman qrunner modules."""
-
-import os
-import email
-import shutil
-import tempfile
-import unittest
-
-from Mailman.Message import Message
-from Mailman.Queue.NewsRunner import prepare_message
-from Mailman.Queue.Runner import Runner
-from Mailman.Queue.Switchboard import Switchboard
-from Mailman.testing.base import TestBase
-
-
-
-class TestPrepMessage(TestBase):
-    def test_remove_unacceptables(self):
-        eq = self.assertEqual
-        msg = email.message_from_string("""\
-From: aperson@dom.ain
-To: _xtest@dom.ain
-NNTP-Posting-Host: news.dom.ain
-NNTP-Posting-Date: today
-X-Trace: blah blah
-X-Complaints-To: abuse@dom.ain
-Xref: blah blah
-Xref: blah blah
-Date-Received: yesterday
-Posted: tomorrow
-Posting-Version: 99.99
-Relay-Version: 88.88
-Received: blah blah
-
-A message
-""")
-        msgdata = {}
-        prepare_message(self._mlist, msg, msgdata)
-        eq(msgdata.get('prepped'), 1)
-        eq(msg['from'], 'aperson@dom.ain')
-        eq(msg['to'], '_xtest@dom.ain')
-        eq(msg['nntp-posting-host'], None)
-        eq(msg['nntp-posting-date'], None)
-        eq(msg['x-trace'], None)
-        eq(msg['x-complaints-to'], None)
-        eq(msg['xref'], None)
-        eq(msg['date-received'], None)
-        eq(msg['posted'], None)
-        eq(msg['posting-version'], None)
-        eq(msg['relay-version'], None)
-        eq(msg['received'], None)
-
-    def test_munge_duplicates_no_duplicates(self):
-        eq = self.assertEqual
-        msg = email.message_from_string("""\
-From: aperson@dom.ain
-To: _xtest@dom.ain
-Cc: someother@dom.ain
-Content-Transfer-Encoding: yes
-
-A message
-""")
-        msgdata = {}
-        prepare_message(self._mlist, msg, msgdata)
-        eq(msgdata.get('prepped'), 1)
-        eq(msg['from'], 'aperson@dom.ain')
-        eq(msg['to'], '_xtest@dom.ain')
-        eq(msg['cc'], 'someother@dom.ain')
-        eq(msg['content-transfer-encoding'], 'yes')
-
-    def test_munge_duplicates(self):
-        eq = self.assertEqual
-        msg = email.message_from_string("""\
-From: aperson@dom.ain
-To: _xtest@dom.ain
-To: two@dom.ain
-Cc: three@dom.ain
-Cc: four@dom.ain
-Cc: five@dom.ain
-Content-Transfer-Encoding: yes
-Content-Transfer-Encoding: no
-Content-Transfer-Encoding: maybe
-
-A message
-""")
-        msgdata = {}
-        prepare_message(self._mlist, msg, msgdata)
-        eq(msgdata.get('prepped'), 1)
-        eq(msg.get_all('from'), ['aperson@dom.ain'])
-        eq(msg.get_all('to'), ['_xtest@dom.ain'])
-        eq(msg.get_all('cc'), ['three@dom.ain'])
-        eq(msg.get_all('content-transfer-encoding'), ['yes'])
-        eq(msg.get_all('x-original-to'), ['two@dom.ain'])
-        eq(msg.get_all('x-original-cc'), ['four@dom.ain', 'five@dom.ain'])
-        eq(msg.get_all('x-original-content-transfer-encoding'),
-           ['no', 'maybe'])
-
-
-
-class TestableRunner(Runner):
-    def _dispose(self, mlist, msg, msgdata):
-        self.msg = msg
-        self.data = msgdata
-        return False
-
-    def _doperiodic(self):
-        self.stop()
-
-    def _snooze(self, filecnt):
-        return
-
-
-class TestRunner(TestBase):
-    def setUp(self):
-        TestBase.setUp(self)
-        self._tmpdir = tempfile.mkdtemp()
-        self._msg = email.message_from_string("""\
-From: aperson@dom.ain
-To: _xtest@dom.ain
-
-A test message.
-""", Message)
-        class MyRunner(TestableRunner):
-            QDIR = self._tmpdir
-        self._runner = MyRunner()
-
-    def tearDown(self):
-        shutil.rmtree(self._tmpdir, True)
-        TestBase.tearDown(self)
-
-    def test_run_loop(self):
-        eq = self.assertEqual
-        sb = Switchboard(self._tmpdir)
-        sb.enqueue(self._msg, listname='_xtest@example.com', foo='yes')
-        self._runner.run()
-        eq(self._runner.msg['from'], self._msg['from'])
-        eq(self._runner.msg['to'], self._msg['to'])
-        eq(self._runner.data['foo'], 'yes')
-
-
-
-def test_suite():
-    suite = unittest.TestSuite()
-    suite.addTest(unittest.makeSuite(TestPrepMessage))
-    suite.addTest(unittest.makeSuite(TestRunner))
-    return suite