Merge lp:~sinzui/launchpad/mailing-list-email-0 into lp:launchpad

[Curtis Hovey]

Fix the footer in mailman emails.

    Launchpad bug: https://bugs.launchpad.net/bugs/689431
    Pre-implementation: no one
    Test command: ./bin/test -vv -t mailman.tests.test_

Bug #689431 [change the mailing lists default signature]
    The mailman email signature is not interpreted by the mail clients like a
    real signature and appear in the quote when you write a reply.

--------------------------------------------------------------------

RULES

Bug #689431 [change the mailing lists default signature]
    * Used '-- ' like all other Lp emails
    * Replace the decorations test with a unittest
    * If time permits, add tests for lpstanding and lphandler

QA

Bug #689431 [change the mailing lists default signature]
    * Send an email to a list.
    * verify the footer is separated from the body by '-- '

LINT

    lib/lp/services/mailman/doc/basic-integration.txt
    lib/lp/services/mailman/doc/contact-address.txt
    lib/lp/services/mailman/doc/postings.txt
    lib/lp/services/mailman/monkeypatches/lphandler.py
    lib/lp/services/mailman/monkeypatches/lpstanding.py
    lib/lp/services/mailman/monkeypatches/mm_cfg.py.in
    lib/lp/services/mailman/testing/__init__.py
    lib/lp/services/mailman/tests/test_lphandler.py
    lib/lp/services/mailman/tests/test_lpheaders.py
    lib/lp/services/mailman/tests/test_lpstanding.py

^ There is lint in the old doctests. I can clean up the white space and
line length issues before I land my branch.

IMPLEMENTATION

Changed the mailman footer, made it like other footers from Launchpad. Added
test_lpheaders to replace decorations.txt. The `mlist.use_dollar_strings`
change was required to test the actual formatting because lp lists are
automatically set to use $substitutions.
    lib/lp/services/mailman/doc/basic-integration.txt
    lib/lp/services/mailman/doc/contact-address.txt
    lib/lp/services/mailman/doc/postings.txt
    lib/lp/services/mailman/monkeypatches/mm_cfg.py.in
    lib/lp/services/mailman/testing/__init__.p
    lib/lp/services/mailman/tests/test_lpheaders.py

Added replacement tests for lpstanding and lphandler. Both the the modules
were needed updating to get the xmlrpc proxy using the proper method that
can also be tested. I deleted the standing/first post moderation section of
the posting.txt since it was not directly testing our standing code.
    lib/lp/services/mailman/monkeypatches/lphandler.py
    lib/lp/services/mailman/monkeypatches/lpstanding.py
    lib/lp/services/mailman/tests/test_lphandler.py
    lib/lp/services/mailman/tests/test_lpstanding.py

[Māris Fogels (mars)]

Hi Curtis,

This change looks good to me, r=mars. Great job updating the test suite, too.

Maris

review: Approve

=== modified file 'lib/lp/registry/tests/test_mailinglistapi.py'
--- lib/lp/registry/tests/test_mailinglistapi.py	2010-10-26 15:47:24 +0000
+++ lib/lp/registry/tests/test_mailinglistapi.py	2010-12-15 16:00:37 +0000
@@ -9,6 +9,7 @@
 
 import transaction
 
+from canonical.config import config
 from canonical.launchpad.interfaces.emailaddress import EmailAddressStatus
 from canonical.testing.layers import DatabaseFunctionalLayer
 from lp.registry.tests.mailinglists_helper import new_team
@@ -75,3 +76,10 @@
     def test_isRegisteredInLaunchpad_email_without_person(self):
         self.factory.makeAccount('Me', email='[e-mail]')
         self.assertFalse(self.api.isRegisteredInLaunchpad('[e-mail]'))
+
+    def test_isRegisteredInLaunchpad_archive_address_is_false(self):
+        # The Mailman archive address can never be owned by an Lp user
+        # because such a user would have acces to all lists.
+        email = config.mailman.archive_address
+        self.factory.makePerson(email=email)
+        self.assertFalse(self.api.isRegisteredInLaunchpad(email))

=== modified file 'lib/lp/services/mailman/doc/basic-integration.txt'
--- lib/lp/services/mailman/doc/basic-integration.txt	2010-09-17 20:46:58 +0000
+++ lib/lp/services/mailman/doc/basic-integration.txt	2010-12-15 16:00:37 +0000
@@ -113,8 +113,4 @@
     X-RcptTo: [e-mail]
     <BLANKLINE>
     This is a test message.
-    _______________________________________________
-    Mailing list: http://launchpad.dev/~alpha
-    Post to     : [e-mail]
-    Unsubscribe : http://launchpad.dev/~alpha
-    More help   : http://help.launchpad.dev/ListHelp
+    ...

=== removed file 'lib/lp/services/mailman/doc/bounces.txt'
--- lib/lp/services/mailman/doc/bounces.txt	2010-10-18 22:24:59 +0000
+++ lib/lp/services/mailman/doc/bounces.txt	1970-01-01 00:00:00 +0000
@@ -1,54 +0,0 @@
-=================
-Bounce processing
-=================
-
-Currently, we do no bounce processing for Launchpad mailing lists.  We still
-run the bounce processor occasionally though in order to clear any accumulated
-bounce messages.
-
-    >>> from lp.services.mailman.testing import helpers
-    >>> list_one = helpers.create_list('itest-one')
-    >>> helpers.subscribe('Anne', 'itest-one')
-
-A (faked) DSN bounce message is received from Anne's mail server.
-
-    >>> from Mailman import mm_cfg
-    >>> from Mailman.Post import inject
-
-    >>> inject('itest-one', """\
-    ... From: <[e-mail]>
-    ... Message-ID: <aardvark>
-    ... MIME-Version: 1.0
-    ... Content-Type: multipart/report; report-type=delivery-status;
-    ...  boundary="AAA"
-    ...
-    ... --AAA
-    ... ignore
-    ...
-    ... --AAA
-    ... Content-Type: message/delivery-status
-    ...
-    ... Original-Recipient: rfc822; [e-mail]
-    ... Action: failed
-    ...
-    ... --AAA--
-    ... """, qdir=mm_cfg.BOUNCEQUEUE_DIR)
-
-    # Because bounce processing is disabled, and because the bounce runner
-    # provides no feedback when it discards bounce messages, there really is
-    # no good way to prove that nothing is processed for the right reasons.
-    # This is the best we can do.  If bounce processing were not disabled,
-    # this wait would not time out and the getBounceInfo() call below would
-    # return an object.  Take my word for it.
-    >>> bounce_watcher.expecting_timeout = True
-    >>> print bounce_watcher.wait_for_processing()
-    Timed out
-
-Even though the bounce message was processed, Anne has no bounce score because
-no bounces are being registered.
-
-    >>> from Mailman.MailList import MailList
-    >>> mailing_list = MailList('itest-one', lock=False)
-    >>> info = mailing_list.getBounceInfo('[e-mail]')
-    >>> print info
-    None

=== removed file 'lib/lp/services/mailman/doc/contact-address.txt'
--- lib/lp/services/mailman/doc/contact-address.txt	2010-10-19 01:42:48 +0000
+++ lib/lp/services/mailman/doc/contact-address.txt	1970-01-01 00:00:00 +0000
@@ -1,379 +0,0 @@
-================================================
-Using a mailing list as a team's contact address
-================================================
-
-The contact address for any team can be set to the team's mailing list, in
-which case all notifications sent by Launchpad to the team will be sent to the
-mailing list's address.
-
-Create a mailing list for the contact address of our new team.
-
-    >>> from lp.services.mailman.testing import helpers
-    >>> list_one = helpers.create_list('itest-one')
-
-    # Ignore the new list notification message.
-    >>> smtpd.reset()
-
-Anne subscribes to the team and its mailing list.
-
-    >>> helpers.subscribe('Anne', 'itest-one')
-
-The team is set as the project's answer contact.
-
-    >>> from canonical.launchpad.testing.pages import strip_label
-
-    >>> from canonical.testing.layers import BaseLayer
-    >>> root_url = BaseLayer.appserver_root_url()
-    >>> browser = Browser('[e-mail]:test')
-    >>> browser.open('%s/~itest-one/+contactaddress' % root_url)
-    >>> browser.getControl('The Launchpad mailing list').selected = True
-    >>> browser.getControl('Change').click()
-
-    >>> browser.getLink('Change details').click()
-    >>> browser.getLink('Set contact address').click()
-    >>> control = browser.getControl(name='field.contact_method')
-    >>> [strip_label(label) for label in control.displayValue]
-    ['The Launchpad mailing list for this team...]
-
-    >>> answers_url = BaseLayer.appserver_root_url('answers')
-    >>> browser.open(
-    ...     '%s/firefox/+answer-contact' % answers_url)
-    >>> browser.getControl(name='field.answer_contact_teams').displayValue = [
-    ...     'Itest One']
-    >>> browser.getControl('Continue').click()
-
-Now that the contact address for the team is its mailing list, a new questions
-will be delivered to the mailing list.
-
-    >>> browser.open('%s/firefox/+addquestion' % answers_url)
-    >>> browser.getControl('Summary').value = 'A new question'
-    >>> browser.getControl('Continue').click()
-    >>> browser.getControl('Description').value = 'More detail.'
-    >>> browser.getControl('Post Question').click()
-
-There are now three messages in the mailbox.  One message goes to
[e-mail] because he added the question, one goes to the mailing
-list's archive, and one goes to Anne because she is the mailing list's only
-member.
-
-    # The three messages addressed to the mailing list seen on the smtp server
-    # are 1) the original posted message on the +addquestion, 2) the message
-    # as sent to Anne, 3) the message as sent to the archiver.
-    >>> smtpd_watcher.wait_for_list_traffic('itest-one')
-    >>> smtpd_watcher.wait_for_list_traffic('itest-one', personal=True)
-    >>> smtpd_watcher.wait_for_list_traffic('itest-one', personal=True)
-
-    >>> messages = list(smtpd)
-    >>> len(messages)
-    3
-
-Here is the message to the person who added the question.
-
-    >>> print messages[0].as_string()
-    Content-Type: text/plain; charset="utf-8"
-    ...
-    X-Launchpad-Question: product=firefox; status=Open; assignee=None;
-      priority=Normal; language=en
-    Reply-To: [e-mail]
-    X-Launchpad-Message-Rationale: Subscriber
-    To: [e-mail]
-    From: No Privileges Person <[e-mail]>
-    Subject: [Question #...]: A new question
-    ...
-    Sender: [e-mail]
-    Errors-To: [e-mail]
-    Return-Path: [e-mail]
-    Precedence: bulk
-    ...
-    X-MailFrom: [e-mail]
-    X-RcptTo: [e-mail]
-    <BLANKLINE>
-    New question #... on Mozilla Firefox:
-    http://answers.launchpad.dev:.../firefox/+question/...
-    <BLANKLINE>
-    More detail.
-    <BLANKLINE>
-    -- =
-    <BLANKLINE>
-    You received this question notification because you are a direct
-    subscriber of the question.
-
-Here is the message to Anne, the mailing list member.
-
-    >>> print messages[1].as_string()
-    MIME-Version: 1.0
-    X-Launchpad-Question: product=firefox; status=Open; assignee=None;
-      priority=Normal; language=en
-    X-Launchpad-Message-Rationale: Answer Contact (firefox) @itest-one
-    To: [e-mail]
-    From: No Privileges Person <[e-mail]>
-    ...
-    Subject: [Itest-one] [Question #...]: A new question
-    X-BeenThere: [e-mail]
-    X-Mailman-Version: ...
-    Reply-To: [e-mail]
-    List-Id: <itest-one.lists.launchpad.dev>
-    List-Help: <http://help.launchpad.dev/ListHelp>
-    List-Subscribe: <http://launchpad.dev/~itest-one>
-    List-Unsubscribe: <http://launchpad.dev/~itest-one>
-    List-Post: <mailto:[e-mail]>
-    List-Archive: <http://lists.launchpad.dev/itest-one>
-    List-Owner: <http://launchpad.dev/~itest-one>
-    Content-Type: text/plain; charset="us-ascii"
-    Content-Transfer-Encoding: 7bit
-    Sender: itest-one-bounces+anne.person=[e-mail]
-    Errors-To: itest-one-bounces+anne.person=[e-mail]
-    ...
-    X-MailFrom: itest-one-bounces+anne.person=[e-mail]
-    X-RcptTo: [e-mail]
-    <BLANKLINE>
-    New question #... on Mozilla Firefox:
-    http://answers.launchpad.dev:.../firefox/+question/...
-    <BLANKLINE>
-    More detail.
-    <BLANKLINE>
-    --
-    You received this question notification because you are a member of
-    Itest One, which is an answer contact for Mozilla Firefox.
-    _______________________________________________
-    Mailing list: http://launchpad.dev/~itest-one
-    Post to     : [e-mail]
-    Unsubscribe : http://launchpad.dev/~itest-one
-    More help   : http://help.launchpad.dev/ListHelp
-
-Here is the message to the archive submission address.
-
-    # This is mostly the same as the message to Anne, so we'll be succinct.
-    >>> print messages[2].as_string()
-    MIME-Version: 1.0
-    ...
-    Sender: itest-one-bounces+archive=[e-mail]
-    Errors-To: itest-one-bounces+archive=[e-mail]
-    ...
-    <BLANKLINE>
-    <BLANKLINE>
-
-Similarly, when the team is subscribed to a blueprint, a notification will be
-sent to Anne and to the archive, via the mailing list.
-
-    >>> browser.open(
-    ...     '%s/firefox/+spec/canvas/+addsubscriber' % root_url)
-    >>> browser.getControl('Subscriber').value = 'itest-one'
-    >>> browser.getControl('Continue').click()
-
-Launchpad sends a message to the team's contact address.  This message is
-delivered directly to Mailman and is not available for viewing in the smtp
-server.
-
-    >>> smtpd_watcher.wait_for_list_traffic('itest-one')
-
-Mailman then delivers two messages to the team mailing list's recipients.
-One is sent to Anne and the other is sent to the archiver.
-
-    >>> smtpd_watcher.wait_for_list_traffic('itest-one', personal=True)
-    >>> smtpd_watcher.wait_for_list_traffic('itest-one', personal=True)
-    >>> messages = list(smtpd)
-    >>> len(messages)
-    2
-    >>> from operator import itemgetter
-    >>> messages.sort(key=itemgetter('sender'))
-
-This one gets sent to Anne...
-
-    >>> print messages[0].as_string()
-    MIME-Version: 1.0
-    To: [e-mail]
-    From: No Privileges Person <[e-mail]>
-    ...
-    Subject: [Itest-one] [Blueprint canvas] Support <canvas> Objects
-    X-BeenThere: [e-mail]
-    ...
-    Sender: itest-one-bounces+anne.person=[e-mail]
-    Errors-To: itest-one-bounces+anne.person=[e-mail]
-    ...
-    X-MailFrom: itest-one-bounces+anne.person=[e-mail]
-    X-RcptTo: [e-mail]
-    <BLANKLINE>
-    You are now subscribed to the blueprint canvas - Support <canvas>
-    Objects.
-    <BLANKLINE>
-    --
-      http://blueprints.launchpad.dev:.../firefox/+spec/canvas
-    _______________________________________________
-    Mailing list: http://launchpad.dev/~itest-one
-    Post to     : [e-mail]
-    Unsubscribe : http://launchpad.dev/~itest-one
-    More help   : http://help.launchpad.dev/ListHelp
-
-...and this one gets sent to the archives.
-
-    >>> print messages[1].as_string()
-    MIME-Version: 1.0
-    To: [e-mail]
-    From: No Privileges Person <[e-mail]>
-    ...
-    Subject: [Itest-one] [Blueprint canvas] Support <canvas> Objects
-    X-BeenThere: [e-mail]
-    ...
-    Sender: itest-one-bounces+archive=[e-mail]
-    Errors-To: itest-one-bounces+archive=[e-mail]
-    ...
-    X-MailFrom: itest-one-bounces+archive=[e-mail]
-    X-RcptTo: [e-mail]
-    <BLANKLINE>
-    You are now subscribed to the blueprint canvas - Support <canvas>
-    Objects.
-    <BLANKLINE>
-    --
-      http://blueprints.launchpad.dev:.../firefox/+spec/canvas
-    _______________________________________________
-    Mailing list: http://launchpad.dev/~itest-one
-    Post to     : [e-mail]
-    Unsubscribe : http://launchpad.dev/~itest-one
-    More help   : http://help.launchpad.dev/ListHelp
-
-The team's contact address is set to each team member individually.
-Notifications will no longer be sent to the mailing list.
-
-    >>> browser.open('%s/~itest-one/+contactaddress' % root_url)
-    >>> browser.getControl('Each member individually').selected = True
-    >>> browser.getControl('Change').click()
-
-    >>> browser.getLink('Change details').click()
-    >>> browser.getLink('Set contact address').click()
-    >>> control = browser.getControl(name='field.contact_method')
-    >>> [strip_label(label) for label in control.displayValue]
-    ['Each member individually']
-
-    >>> browser.open('%s/firefox/+addquestion' % answers_url)
-    >>> browser.getControl('Summary').value = 'Another question'
-    >>> browser.getControl('Continue').click()
-    >>> browser.getControl('Description').value = 'More detail.'
-    >>> browser.getControl('Post Question').click()
-
-    >>> smtpd_watcher.expecting_timeout = True
-    >>> smtpd_watcher.wait_for_list_traffic('itest-one')
-    'Timed out'
-
-There are two messages in the mailbox.  One message was sent to
[e-mail] because he added the question.  The other goes to Anne
-as a member of the team, since the team's contact address is not set.  Nothing
-is sent to the archive because the mailing list was not used.
-
-    >>> messages = list(smtpd)
-    >>> len(messages)
-    2
-
-    # Because this doesn't go through the mailing list, the Sender header will
-    # be the same for both, so sort on the To field instead.
-    >>> messages.sort(key=itemgetter('to'), reverse=True)
-
-This message goes to the submitter of the question.
-
-    >>> print messages[0].as_string()
-    Content-Type: text/plain; charset="utf-8"
-    ...
-    To: [e-mail]
-    From: No Privileges Person <[e-mail]>
-    Subject: [Question #...]: Another question
-    ...
-    <BLANKLINE>
-    New question #... on Mozilla Firefox:
-    http://answers.launchpad.dev:8085/firefox/+question/...
-    <BLANKLINE>
-    More detail.
-    <BLANKLINE>
-    -- =
-    <BLANKLINE>
-    You received this question notification because you are a direct
-    subscriber of the question.
-    <BLANKLINE>
-    <BLANKLINE>
-
-This message goes to the team members individually, and Anne is the only team
-member.
-
-    >>> print messages[1].as_string()
-    Content-Type: text/plain; charset="utf-8"
-    ...
-    Reply-To: [e-mail]
-    X-Launchpad-Message-Rationale: Answer Contact (firefox) @itest-one
-    To: [e-mail]
-    From: No Privileges Person <[e-mail]>
-    Subject: [Question #...]: Another question
-    ...
-    <BLANKLINE>
-    New question #... on Mozilla Firefox:
-    http://answers.launchpad.dev:8085/firefox/+question/...
-    <BLANKLINE>
-    More detail.
-    <BLANKLINE>
-    -- =
-    <BLANKLINE>
-    You received this question notification because you are a member of
-    Itest One, which is an answer contact for Mozilla Firefox.
-    <BLANKLINE>
-    <BLANKLINE>
-
-When emails are sent directly to the mailing list address they're still
-delivered since the mailing list is still active.
-
-    >>> from Mailman.Post import inject
-    >>> inject('itest-one', """\
-    ... From: [e-mail]
-    ... To: [e-mail]
-    ... Subject: A member post
-    ... Message-ID: <first-injection>
-    ...
-    ... Hi, I am a member of Launchpad.
-    ... """)
-    >>> smtpd_watcher.wait_for_mbox_delivery('first-injection')
-    >>> smtpd_watcher.wait_for_mbox_delivery('first-injection')
-    >>> messages = list(smtpd)
-
-Anne (as the only member of the mailing list) and the archive submission
-address both get copies of the message.
-
-    >>> len(messages)
-    2
-    >>> messages.sort(key=itemgetter('sender'))
-
-    >>> print messages[0].as_string()
-    From: [e-mail]
-    To: [e-mail]
-    Message-ID: <first-injection>
-    Subject: [Itest-one] A member post
-    X-BeenThere: [e-mail]
-    ...
-    Sender: itest-one-bounces+anne.person=[e-mail]
-    Errors-To: itest-one-bounces+anne.person=[e-mail]
-    ...
-    X-MailFrom: itest-one-bounces+anne.person=[e-mail]
-    X-RcptTo: [e-mail]
-    <BLANKLINE>
-    Hi, I am a member of Launchpad.
-    _______________________________________________
-    Mailing list: http://launchpad.dev/~itest-one
-    Post to     : [e-mail]
-    Unsubscribe : http://launchpad.dev/~itest-one
-    More help   : http://help.launchpad.dev/ListHelp
-
-    >>> print messages[1].as_string()
-    From: [e-mail]
-    To: [e-mail]
-    Message-ID: <first-injection>
-    Subject: [Itest-one] A member post
-    X-BeenThere: [e-mail]
-    ...
-    Sender: itest-one-bounces+archive=[e-mail]
-    Errors-To: itest-one-bounces+archive=[e-mail]
-    ...
-    X-MailFrom: itest-one-bounces+archive=[e-mail]
-    X-RcptTo: [e-mail]
-    <BLANKLINE>
-    Hi, I am a member of Launchpad.
-    _______________________________________________
-    Mailing list: http://launchpad.dev/~itest-one
-    Post to     : [e-mail]
-    Unsubscribe : http://launchpad.dev/~itest-one
-    More help   : http://help.launchpad.dev/ListHelp

=== removed file 'lib/lp/services/mailman/doc/decorations.txt'
--- lib/lp/services/mailman/doc/decorations.txt	2010-10-19 01:42:48 +0000
+++ lib/lp/services/mailman/doc/decorations.txt	1970-01-01 00:00:00 +0000
@@ -1,169 +0,0 @@
-===========
-Decorations
-===========
-
-Messages sent by Mailman to team mailing list recipients have a number of
-decorations that help users interact better with the list and its archive.
-These decorations live in the message headers, and in the text footer that
-Mailman adds to the bottom of very message.
-
-A mailing list is created, and Anne subscribes to it.
-
-    >>> from lp.services.mailman.testing import helpers
-    >>> list_one = helpers.create_list('itest-one')
-    >>> helpers.subscribe('Anne', 'itest-one')
-
-Anne sends a message to the list.
-
-    >>> smtpd.reset()
-    >>> from Mailman.Post import inject
-    >>> inject('itest-one', """\
-    ... From: [e-mail]
-    ... To: [e-mail]
-    ... Subject: A member post
-    ... Message-ID: <first-injection>
-    ...
-    ... Hi, I am a member of this team's list.
-    ... """)
-
-    # Wait for two deliveries, one to Anne and the other to the archiver.
-    >>> smtpd_watcher.wait_for_mbox_delivery('first-injection')
-    >>> smtpd_watcher.wait_for_mbox_delivery('first-injection')
-
-
-VERP headers
-============
-
-VERP stands for Variable Envelope Return Path and it is a technique used to
-track users for bounce processing.  The idea is that you encode the actual
-subscribed address in the Sender and Errors-To headers so that they can be
-extracted if a bounce is ever received.  This works because remote MTAs must
-bounce to this address, regardless of how many intervening hops, forwards, or
-rewrites there are.
-
-(Technically, Mailman doesn't do VERP because that's defined as happening in
-the MTA, but it's close enough in intent and exactly the same in
-implementation so we use the same terminology in Mailman.)
-
-    >>> from operator import itemgetter
-    >>> messages = sorted(smtpd, key=itemgetter('sender'))
-    >>> for message in messages:
-    ...     print message['sender']
-    ...     print message['errors-to']
-    ...     print '---'
-    itest-one-bounces+anne.person=[e-mail]
-    itest-one-bounces+anne.person=[e-mail]
-    ---
-    itest-one-bounces+archive=[e-mail]
-    itest-one-bounces+archive=[e-mail]
-    ---
-
-For the following discussion, it doesn't matter which message we look at in
-detail, so the last one in the sorted sequence is just fine.
-
-
-RFC 2369 headers
-================
-
-RFC 2369 defines a set of headers for mailing lists, often called the List-*
-headers due to the common prefix of these headers.
-
-
-List-Help
----------
-
-The List-Help header points to "an instructive website" to provide help to
-users.
-
-    >>> print message['list-help']
-    <http://help.launchpad.dev/ListHelp>
-
-
-List-Id
--------
-
-The List-Id header uniquely identifies the mailing list.  Mailman crafts the
-List-Id header using the team name and the host name on which the lists
-reside.
-
-    >>> print message['list-id']
-    <itest-one.lists.launchpad.dev>
-
-
-List-Subscribe and List-Unsubscribe
------------------------------------
-
-The List-Unsubscribe header points to the contact address of the mechanism the
-user can access to unsubscribe from the mailing list.  Because we have not yet
-enabled the email command robot for Launchpad, the only way to unsubscribe is
-for a user to hit her email settings page.  The List-Subscribe header is
-similar.
-
-    >>> print message['list-subscribe']
-    <http://launchpad.dev/~itest-one>
-    >>> print message['list-unsubscribe']
-    <http://launchpad.dev/~itest-one>
-
-
-List-Archive
-------------
-
-The List-Archive header describes how to access the archives for this mailing
-list.  It is a list-wide url, not a url-specific to the individual message
-(see draft RFC 5064 for that).
-
-This url should exactly match the link given on the team's overview page.
-
-    >>> from canonical.testing.layers import BaseLayer
-    >>> root_url = BaseLayer.appserver_root_url()
-    >>> browser = Browser('[e-mail]:test')
-    >>> browser.open('%s/~itest-one' % root_url)
-    >>> browser.getLink(id='mailing-list-archive')
-    <Link text='View archive' url='http://lists.launchpad.dev/itest-one'>
-    >>> print message['list-archive']
-    <http://lists.launchpad.dev/itest-one>
-
-
-List-Post
----------
-
-The List-Post header is a mailto url that tells people where they can post new
-messages for this mailing list.
-
-    >>> print message['list-post']
-    <mailto:[e-mail]>
-
-
-List-Owner
-----------
-
-The List-Owner contains a url pointing to the human contact for the mailing
-list.  This header may be omitted if it is the same as the postmaster, but in
-our case, we point this at the team owning the mailing list.
-
-    >>> print message['list-owner']
-    <http://launchpad.dev/~itest-one>
-
-
-Message footer
-==============
-
-Mailman appends a footer of helpful text on each message.  This footer
-contains a link to this message's archive (which should be the same as the RFC
-5064 link), the message's posting address, the unsubscribe address, etc.
-The easiest way to see the footer is to just scan the message body for the
-separator.
-
-    >>> body_lines = message.get_payload().splitlines(True)
-    >>> for line_number, line in enumerate(body_lines):
-    ...     if line.startswith('_____'):
-    ...         break
-    ... else:
-    ...     raise AssertionError('No footer found')
-    >>> print ''.join(body_lines[line_number + 1:])
-    Mailing list: http://launchpad.dev/~itest-one
-    Post to     : [e-mail]
-    Unsubscribe : http://launchpad.dev/~itest-one
-    More help   : http://help.launchpad.dev/ListHelp
-    <BLANKLINE>
-    <BLANKLINE>

=== removed file 'lib/lp/services/mailman/doc/messages.txt'
--- lib/lp/services/mailman/doc/messages.txt	2009-09-24 18:51:29 +0000
+++ lib/lp/services/mailman/doc/messages.txt	1970-01-01 00:00:00 +0000
@@ -1,33 +0,0 @@
-========
-Messages
-========
-
-Launchpad uses a custom set of message templates that it installs
-in mailman/templates/site. The installation conforms to the standard
-way of defining templates that apply to all lists managed by the
-instance of mailman. The site template directory contains only the
-templates that are different from the default mailman templates.
-
-The postheld message text does not include a link to cancel the posting,
-which launchpad does not support.
-
-    >>> from Mailman import Utils
-
-    >>> postheld_dict = {
-    ...     'listname': 'fake-list',
-    ...     'hostname': 'lists.launchpad.net',
-    ...     'reason': 'The attachment is too large to post to the list.',
-    ...     'sender': '[e-mail]',
-    ...     'subject' : "testing post held for large attachment",
-    ...     'admindb_url': 'http://lists.launchpad.net/fake/admin',
-    ...     }
-    >>> text, filename = Utils.findtext('postheld.txt', dict=postheld_dict)
-    >>> filename
-    '.../lib/mailman/templates/site/en/postheld.txt'
-    >>> print text
-    Your mail to 'fake-list' with the subject
-      testing post held for large attachment
-    Is being held until the list moderator can review it for approval.
-    ...
-    Either the message will get posted to the list, or you will receive
-    notification of the moderator's decision.

=== removed file 'lib/lp/services/mailman/doc/postings.txt'
--- lib/lp/services/mailman/doc/postings.txt	2010-12-13 19:58:54 +0000
+++ lib/lp/services/mailman/doc/postings.txt	1970-01-01 00:00:00 +0000
@@ -1,812 +0,0 @@
-=======
-Posting
-=======
-
-Only Launchpad members are allowed to post to team mailing lists.  A message
-by a sender who is not a Launchpad member will be summarily discarded.
-
-A team mailing list is created.
-
-    >>> from lp.services.mailman.testing import helpers
-    >>> list_one = helpers.create_list('itest-one')
-
-    # Ignore the list creation notification message.
-    >>> smtpd.reset()
-
-
-Non-member postings
-===================
-
-Postings from addresses that are not registered and verified with Launchpad
-are summarily discarded.
-
-A non-Launchpad member sends a message to the mailing list.  This message is
-discarded immediately.
-
-    >>> from Mailman.Post import inject
-    >>> inject('itest-one', """\
-    ... From: [e-mail]
-    ... To: [e-mail]
-    ... Subject: A non-member post
-    ... Message-ID: <aardvark>
-    ...
-    ... Hi, I am not a member of Launchpad (yet).
-    ... """)
-    >>> vette_watcher.wait_for_discard('aardvark')
-    >>> len(list(smtpd))
-    0
-
-
-Non-validated member postings
-=============================
-
-Similarly, an address that has been registered with Launchpad but not verified
-will also be summarily discarded.  Anne registers with Launchpad but does not
-validate her alternative address.
-
-    >>> from lp.registry.tests.mailinglists_helper import (
-    ...     get_alternative_email)
-    >>> from canonical.launchpad.ftests import login, logout
-    >>> from canonical.launchpad.interfaces.emailaddress import EmailAddressStatus
-
-    >>> login('[e-mail]')
-    >>> anne = factory.makePersonByName('Anne')
-    >>> alternative_email = get_alternative_email(anne)
-
-    # Simulate Anne not having validated her alternative address.
-    >>> alternative_email.status = EmailAddressStatus.NEW
-    >>> transaction.commit()
-    >>> print alternative_email.email, alternative_email.status.title
-    [e-mail] New Email Address
-
-Anne sends a message from her non-validated alternative address.  This message
-gets discarded.
-
-    >>> inject('itest-one', """\
-    ... From: [e-mail]
-    ... To: [e-mail]
-    ... Subject: A non-member post
-    ... Message-ID: <beaver>
-    ...
-    ... Hi, I am not a verified member of Launchpad (yet).
-    ... """)
-    >>> vette_watcher.wait_for_discard('beaver')
-    >>> len(list(smtpd))
-    0
-
-
-Unsubscribed Launchpad members
-==============================
-
-Anne's preferred email address is still validated, and she posts a message
-from that address.  However, because she has not yet joined the team or
-subscribed to its mailing list, her messages gets held for moderation by the
-team's administrator.
-
-    >>> print anne.preferredemail.email, anne.preferredemail.status.title
-    [e-mail] Preferred Email Address
-    >>> smtpd.reset()
-    >>> len(list(smtpd))
-    0
-    >>> inject('itest-one', """\
-    ... From: [e-mail]
-    ... To: [e-mail]
-    ... Subject: An unsubscribed post
-    ... Message-ID: <caribou>
-    ...
-    ... Hi, I am not a member of this team's list (yet).
-    ... """)
-    >>> vette_watcher.wait_for_hold('itest-one', 'caribou')
-
-The team administrator will receive a notification of the held message.
-
-    >>> smtpd_watcher.wait_for_personal_traffic('[e-mail]')
-    >>> messages = list(smtpd)
-    >>> len(messages)
-    1
-    >>> print messages[0].as_string()
-    Content-Type: text/plain; charset="utf-8"
-    ...
-    From: Itest One <[e-mail]>
-    Subject: New mailing list message requiring approval for Itest One
-    ...
-    <BLANKLINE>
-    Hello No Privileges Person,
-    <BLANKLINE>
-    Itest One has a new message requiring your approval.
-    <BLANKLINE>
-        Subject: An unsubscribed post
-        Author name: Anne Person
-        Author url: http://launchpad.dev:.../~anne
-        Date: ...
-        Message-ID: <caribou>
-    <BLANKLINE>
-    A message has been posted to the mailing list for your team, but this
-    message requires your approval before it will be sent to the list
-    members.  After reviewing the message, you may approve, discard or
-    reject it.
-    <BLANKLINE>
-    To review all messages pending approval, visit:
-    <BLANKLINE>
-        http://launchpad.dev:.../~itest-one/+mailinglist-moderate
-    <BLANKLINE>
-    Regards,
-    The Launchpad team
-    <BLANKLINE>
-    <BLANKLINE>
-
-The message is held for moderation by the team administrator.
-
-    >>> holds = helpers.pending_hold_ids('itest-one')
-    >>> len(holds)
-    1
-    >>> helpers.print_mailman_hold('itest-one', holds[0])
-    From: [e-mail]
-    To: [e-mail]
-    Subject: An unsubscribed post
-    Message-ID: <caribou>
-    <BLANKLINE>
-    Hi, I am not a member of this team's list (yet).
-    <BLANKLINE>
-
-The team administrator can accept, decline or discard this posting.  By
-declining the message, Launchpad forgets about it, while Mailman sends a
-rejection notice to Anne.
-
-    >>> from canonical.testing.layers import BaseLayer
-    >>> root_url = BaseLayer.appserver_root_url()
-    >>> browser = Browser('[e-mail]:test')
-    >>> browser.open('%s/~itest-one/+mailinglist-moderate' % root_url)
-    >>> browser.getControl(name='field.%3Ccaribou%3E').value = [
-    ...     'reject']
-    >>> browser.getControl('Moderate').click()
-    >>> xmlrpc_watcher.wait_for_reject('caribou')
-
-    >>> holds = helpers.pending_hold_ids('itest-one')
-    >>> len(holds)
-    0
-
-    >>> smtpd_watcher.wait_for_personal_traffic('[e-mail]')
-    >>> messages = list(smtpd)
-    >>> len(messages)
-    1
-    >>> print messages[0].as_string()
-    MIME-Version: 1.0
-    ...
-    Subject: Request to mailing list Itest-one rejected
-    From: [e-mail]
-    To: [e-mail]
-    ...
-
-Anne posts another message to the mailing list, but she is still not
-subscribed to it.  The team administrator deems this message to be spam and
-discards it.  No rejection notice is sent to Anne.
-
-    >>> inject('itest-one', """\
-    ... From: [e-mail]
-    ... To: [e-mail]
-    ... Subject: Wanna buy something?
-    ... Message-ID: <donkey>
-    ...
-    ... Hi, do you want to buy something?
-    ... """)
-
-    >>> vette_watcher.wait_for_hold('itest-one', 'donkey')
-    >>> messages = list(smtpd)
-    >>> len(messages)
-    1
-    >>> print messages[0].as_string()
-    Content-Type: text/plain; charset="utf-8"
-    ...
-    From: Itest One <[e-mail]>
-    Subject: New mailing list message requiring approval for Itest One
-    ...
-    <BLANKLINE>
-    Hello No Privileges Person,
-    <BLANKLINE>
-    Itest One has a new message requiring your approval.
-    <BLANKLINE>
-        Subject: Wanna buy something?
-        Author name: Anne Person
-        Author url: http://launchpad.dev:.../~anne
-        Date: ...
-        Message-ID: <donkey>
-    ...
-
-    >>> browser.open('%s/~itest-one/+mailinglist-moderate' % root_url)
-    >>> browser.getControl(name='field.%3Cdonkey%3E').value = [
-    ...     'discard']
-    >>> browser.getControl('Moderate').click()
-    >>> xmlrpc_watcher.wait_for_discard('donkey')
-
-    >>> holds = helpers.pending_hold_ids('itest-one')
-    >>> len(holds)
-    0
-    >>> len(list(smtpd))
-    0
-
-Now Anne posts another message to the mailing list she is not subscribed to.
-This time, her message is on topic and is accepted.
-
-    >>> inject('itest-one', """\
-    ... From: [e-mail]
-    ... To: [e-mail]
-    ... Subject: Another unsubscribed post
-    ... Message-ID: <elephant>
-    ...
-    ... Hi, I am still not yet a member of this team's list.
-    ... """)
-
-    >>> vette_watcher.wait_for_hold('itest-one', 'elephant')
-    >>> messages = list(smtpd)
-    >>> len(messages)
-    1
-    >>> print messages[0].as_string()
-    Content-Type: text/plain; charset="utf-8"
-    ...
-    From: Itest One <[e-mail]>
-    Subject: New mailing list message requiring approval for Itest One
-    ...
-    <BLANKLINE>
-    Hello No Privileges Person,
-    <BLANKLINE>
-    Itest One has a new message requiring your approval.
-    <BLANKLINE>
-        Subject: Another unsubscribed post
-        Author name: Anne Person
-        Author url: http://launchpad.dev:.../~anne
-        Date: ...
-        Message-ID: <elephant>
-    ...
-
-    >>> holds = helpers.pending_hold_ids('itest-one')
-    >>> len(holds)
-    1
-    >>> helpers.print_mailman_hold('itest-one', holds[0])
-    From: [e-mail]
-    To: [e-mail]
-    Subject: Another unsubscribed post
-    Message-ID: <elephant>
-    <BLANKLINE>
-    Hi, I am still not yet a member of this team's list.
-    <BLANKLINE>
-
-    >>> browser.open('%s/~itest-one/+mailinglist-moderate' % root_url)
-    >>> browser.getControl(name='field.%3Celephant%3E').value = [
-    ...     'approve']
-    >>> browser.getControl('Moderate').click()
-    >>> xmlrpc_watcher.wait_for_approval('elephant')
-
-    >>> holds = helpers.pending_hold_ids('itest-one')
-    >>> len(holds)
-    0
-
-This message is delivered to the archive.
-
-    >>> smtpd_watcher.wait_for_list_traffic('itest-one', personal=True)
-    >>> messages = list(smtpd)
-    >>> len(messages)
-    1
-    >>> print messages[0].as_string()
-    From: [e-mail]
-    To: [e-mail]
-    Message-ID: <elephant>
-    X-Mailman-Approved-At: ...
-    Subject: [Itest-one] Another unsubscribed post
-    ...
-    List-Id: <itest-one.lists.launchpad.dev>
-    List-Help: <http://help.launchpad.dev/ListHelp>
-    List-Subscribe: <http://launchpad.dev/~itest-one>
-    List-Unsubscribe: <http://launchpad.dev/~itest-one>
-    List-Post: <mailto:[e-mail]>
-    List-Archive: <http://lists.launchpad.dev/itest-one>
-    List-Owner: <http://launchpad.dev/~itest-one>
-    ...
-    Sender: itest-one-bounces+archive=[e-mail]
-    Errors-To: itest-one-bounces+archive=[e-mail]
-    ...
-    X-MailFrom: itest-one-bounces+archive=[e-mail]
-    X-RcptTo: [e-mail]
-    <BLANKLINE>
-    Hi, I am still not yet a member of this team's list.
-    _______________________________________________
-    Mailing list: http://launchpad.dev/~itest-one
-    Post to     : [e-mail]
-    Unsubscribe : http://launchpad.dev/~itest-one
-    More help   : http://help.launchpad.dev/ListHelp
-
-
-Unsubscribed team member
-========================
-
-Once Anne joins the team, she may post to it, though until she subscribes to
-the mailing list, she will not get a copy of the message.
-
-    # Reset Anne's MessageApproval so that she'll no longer be pre-approved to
-    # post to this mailing list.
-    >>> from canonical.database.sqlbase import sqlvalues
-    >>> from lp.registry.model.mailinglist import MessageApproval
-    >>> approvals = MessageApproval.select("""
-    ...     MessageApproval.posted_by = %s
-    ...     """ % sqlvalues(anne))
-    >>> for approval in approvals:
-    ...     approval.destroySelf()
-    >>> transaction.commit()
-
-    >>> helpers.ensure_nonmembership('itest-one', anne)
-
-    >>> from zope.component import getUtility
-    >>> from lp.registry.interfaces.person import IPersonSet
-    >>> login('[e-mail]')
-    >>> team_one = getUtility(IPersonSet).getByName('itest-one')
-    >>> anne.join(team_one)
-    >>> transaction.commit()
-
-    >>> helpers.ensure_membership('itest-one', anne)
-    >>> helpers.ensure_addresses_are_disabled(
-    ...     'itest-one', '[e-mail]')
-    >>> logout()
-
-    >>> inject('itest-one', """\
-    ... From: [e-mail]
-    ... To: [e-mail]
-    ... Subject: A member post
-    ... Message-ID: <emu>
-    ...
-    ... Hi, I have just joined this team.
-    ... """)
-    >>> smtpd_watcher.wait_for_mbox_delivery('emu')
-
-Only the archiver gets the message delivery.
-
-    >>> messages = list(smtpd)
-    >>> len(messages)
-    1
-    >>> print messages[0].as_string()
-    From: [e-mail]
-    To: [e-mail]
-    Message-ID: <emu>
-    Subject: [Itest-one] A member post
-    ...
-    Sender: itest-one-bounces+archive=[e-mail]
-    Errors-To: itest-one-bounces+archive=[e-mail]
-    ...
-    Hi, I have just joined this team.
-    _______________________________________________
-    Mailing list: http://launchpad.dev/~itest-one
-    Post to     : [e-mail]
-    Unsubscribe : http://launchpad.dev/~itest-one
-    More help   : http://help.launchpad.dev/ListHelp
-    <BLANKLINE>
-    <BLANKLINE>
-
-In addition, the in-house MHonArc archiver has archived this message.
-
-    >>> mhonarc_watcher.wait_for_message_number(2)
-    >>> helpers.collect_archive_message_ids('itest-one')
-    ['elephant', 'emu']
-
-
-Verified and registered member postings
-=======================================
-
-Anne now subscribes to the team's mailing list.  She gets a copy of each
-message posted to the mailing list.
-
-    >>> helpers.subscribe('Anne', 'itest-one')
-    >>> helpers.ensure_addresses_are_enabled(
-    ...     'itest-one', '[e-mail]')
-
-    >>> inject('itest-one', """\
-    ... From: [e-mail]
-    ... To: [e-mail]
-    ... Subject: A member post
-    ... Message-ID: <falcon>
-    ...
-    ... Hi, I am now a member of this team's list!
-    ... """)
-
-Anne's message got posted to the mailing list.  Two copies were sent; one to
-Anne herself as a member of the mailing list, and another to the
-mail-archive.com archiving recipient.  These copies are very similar,
-differing only in the headers used to do VERP bounce detection.
-
-    >>> smtpd_watcher.wait_for_mbox_delivery('falcon')
-    >>> smtpd_watcher.wait_for_mbox_delivery('falcon')
-    >>> messages = list(smtpd)
-    >>> len(messages)
-    2
-    >>> from operator import itemgetter
-    >>> messages.sort(key=itemgetter('sender'))
-
-Here's the copy sent to Anne...
-
-    >>> print messages[0].as_string()
-    From: [e-mail]
-    To: [e-mail]
-    Message-ID: <falcon>
-    Subject: [Itest-one] A member post
-    ...
-    Sender: itest-one-bounces+anne.person=[e-mail]
-    Errors-To: itest-one-bounces+anne.person=[e-mail]
-    ...
-    X-MailFrom: itest-one-bounces+anne.person=[e-mail]
-    X-RcptTo: [e-mail]
-    <BLANKLINE>
-    Hi, I am now a member of this team's list!
-    _______________________________________________
-    Mailing list: http://launchpad.dev/~itest-one
-    Post to     : [e-mail]
-    Unsubscribe : http://launchpad.dev/~itest-one
-    More help   : http://help.launchpad.dev/ListHelp
-
-...and the copy sent to the archives.
-
-    >>> print messages[1].as_string()
-    From: [e-mail]
-    To: [e-mail]
-    Message-ID: <falcon>
-    Subject: [Itest-one] A member post
-    ...
-    Sender: itest-one-bounces+archive=[e-mail]
-    Errors-To: itest-one-bounces+archive=[e-mail]
-    ...
-    X-MailFrom: itest-one-bounces+archive=[e-mail]
-    X-RcptTo: [e-mail]
-    <BLANKLINE>
-    Hi, I am now a member of this team's list!
-    _______________________________________________
-    Mailing list: http://launchpad.dev/~itest-one
-    Post to     : [e-mail]
-    Unsubscribe : http://launchpad.dev/~itest-one
-    More help   : http://help.launchpad.dev/ListHelp
-
-In addition, the in-house MHonArc archiver has archived this message.
-
-    >>> mhonarc_watcher.wait_for_message_number(3)
-    >>> helpers.collect_archive_message_ids('itest-one')
-    ['elephant', 'emu', 'falcon']
-
-Anne cannot post from her alternative email address because it has not been
-validated yet.
-
-    >>> inject('itest-one', """\
-    ... From: [e-mail]
-    ... To: [e-mail]
-    ... Subject: A non-member post
-    ... Message-ID: <giraffe>
-    ...
-    ... Hi, I am not a verified member of this team (yet).
-    ... """)
-    >>> vette_watcher.wait_for_discard('giraffe')
-    >>> len(list(smtpd))
-    0
-
-However, once she validates this address, she can post from it.
-
-    >>> login('[e-mail]')
-    >>> alternative_email.status = EmailAddressStatus.VALIDATED
-    >>> logout()
-    >>> transaction.commit()
-    >>> helpers.ensure_membership('itest-one', anne)
-
-    >>> inject('itest-one', """\
-    ... From: [e-mail]
-    ... To: [e-mail]
-    ... Subject: A member post
-    ... Message-ID: <gerbil>
-    ...
-    ... Hi, I am now a verified member of this team!
-    ... """)
-
-    >>> smtpd_watcher.wait_for_mbox_delivery('gerbil')
-    >>> smtpd_watcher.wait_for_mbox_delivery('gerbil')
-    >>> messages = list(smtpd)
-    >>> len(messages)
-    2
-    >>> messages.sort(key=itemgetter('sender'))
-
-    >>> print messages[0].as_string()
-    From: [e-mail]
-    To: [e-mail]
-    Message-ID: <gerbil>
-    Subject: [Itest-one] A member post
-    ...
-    Sender: itest-one-bounces+anne.person=[e-mail]
-    Errors-To: itest-one-bounces+anne.person=[e-mail]
-    ...
-    X-MailFrom: itest-one-bounces+anne.person=[e-mail]
-    X-RcptTo: [e-mail]
-    <BLANKLINE>
-    Hi, I am now a verified member of this team!
-    _______________________________________________
-    Mailing list: http://launchpad.dev/~itest-one
-    Post to     : [e-mail]
-    Unsubscribe : http://launchpad.dev/~itest-one
-    More help   : http://help.launchpad.dev/ListHelp
-
-    >>> print messages[1].as_string()
-    From: [e-mail]
-    To: [e-mail]
-    Message-ID: <gerbil>
-    Subject: [Itest-one] A member post
-    ...
-    Sender: itest-one-bounces+archive=[e-mail]
-    Errors-To: itest-one-bounces+archive=[e-mail]
-    ...
-    X-MailFrom: itest-one-bounces+archive=[e-mail]
-    X-RcptTo: [e-mail]
-    <BLANKLINE>
-    Hi, I am now a verified member of this team!
-    _______________________________________________
-    Mailing list: http://launchpad.dev/~itest-one
-    Post to     : [e-mail]
-    Unsubscribe : http://launchpad.dev/~itest-one
-    More help   : http://help.launchpad.dev/ListHelp
-
-There should also be no held requests pending.
-
-    >>> len(helpers.pending_hold_ids('itest-one'))
-    0
-
-
-Leaving and re-joining the team
-===============================
-
-Anne now leaves the team, but she does not unsubscribe from the mailing list.
-
-    >>> login('[e-mail]')
-    >>> team_one = getUtility(IPersonSet).getByName('itest-one')
-    >>> anne.leave(team_one)
-    >>> logout()
-    >>> transaction.commit()
-    >>> helpers.ensure_nonmembership('itest-one', anne)
-
-Because she is not a team member, her message is held for moderation.
-
-    >>> inject('itest-one', """\
-    ... From: [e-mail]
-    ... To: [e-mail]
-    ... Subject: A non-team member post
-    ... Message-ID: <horse>
-    ...
-    ... Hi, I am not a team member.
-    ... """)
-    >>> vette_watcher.wait_for_hold('itest-one', 'horse')
-    >>> messages = list(smtpd)
-    >>> len(messages)
-    1
-    >>> print messages[0].as_string()
-    Content-Type: text/plain; charset="utf-8"
-    ...
-    Subject: New mailing list message requiring approval for Itest One
-    ...
-        Subject: A non-team member post
-        Author name: Anne Person
-        Author url: http://launchpad.dev:.../~anne
-        Date: ...
-        Message-ID: <horse>
-    ...
-
-The team administrator discards the message.
-
-    >>> browser.open('%s/~itest-one/+mailinglist-moderate' % root_url)
-    >>> browser.getControl(name='field.%3Chorse%3E').value = [
-    ...     'discard']
-    >>> browser.getControl('Moderate').click()
-    >>> xmlrpc_watcher.wait_for_discard('horse')
-
-Anne re-joins the team and once again can post to the mailing list.  She does
-not need to re-subscribe to the mailing list.
-
-    >>> login('[e-mail]')
-    >>> anne.join(team_one)
-    >>> logout()
-    >>> transaction.commit()
-    >>> helpers.ensure_membership('itest-one', anne)
-
-    >>> inject('itest-one', """\
-    ... From: [e-mail]
-    ... To: [e-mail]
-    ... Subject: A posting from a member once again
-    ... Message-ID: <iguana>
-    ...
-    ... Hi, I am a team member again!
-    ... """)
-
-    # Wait for mailing list deliveries to both recipients.
-    >>> smtpd_watcher.wait_for_mbox_delivery('iguana')
-    >>> smtpd_watcher.wait_for_mbox_delivery('iguana')
-    >>> messages = list(smtpd)
-    >>> len(messages)
-    2
-    >>> messages.sort(key=itemgetter('sender'))
-    >>> for message in sorted(messages, key=itemgetter('sender')):
-    ...     print message['sender']
-    ...     print message['subject']
-    ...     print message['message-id']
-    itest-one-bounces+anne.person=[e-mail]
-    [Itest-one] A posting from a member once again
-    <iguana>
-    itest-one-bounces+archive=[e-mail]
-    [Itest-one] A posting from a member once again
-    <iguana>
-
-
-First post moderation
-=====================
-
-Normally, Launchpad members who are not subscribed to the mailing list will
-have their posts held for moderation.
-
-    >>> login('[e-mail]')
-    >>> bart = factory.makePersonByName('Bart')
-    >>> transaction.commit()
-    >>> bart.personal_standing
-    <DBItem PersonalStanding.UNKNOWN...
-
-    >>> def print_message_summaries():
-    ...     messages = sorted(smtpd, key=itemgetter('sender'))
-    ...     print 'Number of messages:', len(messages)
-    ...     for message in messages:
-    ...         print message['sender']
-    ...         print '   ', message['message-id']
-    ...         print '   ', message['from']
-    ...         print '   ', message['subject']
-
-    >>> inject('itest-one', """\
-    ... From: [e-mail]
-    ... To: [e-mail]
-    ... Subject: A non-member post
-    ... Message-ID: <jackal>
-    ...
-    ... Hi, I am not a member of the mailing list.
-    ... """)
-    >>> vette_watcher.wait_for_hold('itest-one', 'jackal')
-    >>> print_message_summaries()
-    Number of messages: 1
-    [e-mail]
-        ...
-        Itest One <[e-mail]>
-        New mailing list message requiring approval for Itest One
-
-However, a Launchpad member in good standing is allowed to post to any mailing
-list.
-
-    >>> from lp.registry.interfaces.person import PersonalStanding
-    >>> login('[e-mail]')
-    >>> bart.personal_standing = PersonalStanding.GOOD
-    >>> transaction.commit()
-
-    >>> inject('itest-one', """\
-    ... From: [e-mail]
-    ... To: [e-mail]
-    ... Subject: A member in good standing
-    ... Message-ID: <kestrel>
-    ...
-    ... Hi, I am a Launchpad member in good standing!
-    ... """)
-
-    # Wait for mailing list deliveries to both recipients.
-    >>> smtpd_watcher.wait_for_mbox_delivery('kestrel')
-    >>> smtpd_watcher.wait_for_mbox_delivery('kestrel')
-    >>> print_message_summaries()
-    Number of messages: 2
-    itest-one-bounces+anne.person=[e-mail]
-        <kestrel>
-        [e-mail]
-        [Itest-one] A member in good standing
-    itest-one-bounces+archive=[e-mail]
-        <kestrel>
-        [e-mail]
-        [Itest-one] A member in good standing
-
-A member in excellent standing can of course also post to the list.
-
-    >>> bart.personal_standing = PersonalStanding.EXCELLENT
-    >>> transaction.commit()
-
-    >>> inject('itest-one', """\
-    ... From: [e-mail]
-    ... To: [e-mail]
-    ... Subject: A member in excellent standing
-    ... Message-ID: <llama>
-    ...
-    ... Hi, I am a Launchpad member in excellent standing!
-    ... """)
-
-    # Wait for mailing list deliveries to both recipients.
-    >>> smtpd_watcher.wait_for_mbox_delivery('llama')
-    >>> smtpd_watcher.wait_for_mbox_delivery('llama')
-    >>> print_message_summaries()
-    Number of messages: 2
-    itest-one-bounces+anne.person=[e-mail]
-        <llama>
-        [e-mail]
-        [Itest-one] A member in excellent standing
-    itest-one-bounces+archive=[e-mail]
-        <llama>
-        [e-mail]
-        [Itest-one] A member in excellent standing
-
-But a person in poor standing will have their messages held for approval.
-
-    >>> bart.personal_standing = PersonalStanding.POOR
-    >>> transaction.commit()
-
-    >>> inject('itest-one', """\
-    ... From: [e-mail]
-    ... To: [e-mail]
-    ... Subject: A non-member post
-    ... Message-ID: <mongoose>
-    ...
-    ... Hi, I am not a member of the mailing list.
-    ... """)
-    >>> vette_watcher.wait_for_hold('itest-one', 'mongoose')
-    >>> print_message_summaries()
-    Number of messages: 1
-    [e-mail]
-        ...
-        Itest One <[e-mail]>
-        New mailing list message requiring approval for Itest One
-
-Should a non-team member's held post be approved, they are then allowed to
-post to just that mailing list without further approval required.
-
-    >>> browser.open('%s/~itest-one/+mailinglist-moderate' % root_url)
-    >>> browser.getControl(name='field.%3Cmongoose%3E').value = ['approve']
-    >>> browser.getControl('Moderate').click()
-    >>> smtpd_watcher.wait_for_mbox_delivery('mongoose')
-    >>> smtpd_watcher.wait_for_mbox_delivery('mongoose')
-    >>> helpers.ensure_addresses_are_enabled(
-    ...     'itest-one', '[e-mail]')
-
-    >>> smtpd.reset()
-    >>> inject('itest-one', """\
-    ... From: [e-mail]
-    ... To: [e-mail]
-    ... Subject: A non-member post
-    ... Message-ID: <otter>
-    ...
-    ... Hi, I am not a member of the mailing list.
-    ... """)
-    >>> smtpd_watcher.wait_for_mbox_delivery('otter')
-    >>> smtpd_watcher.wait_for_mbox_delivery('otter')
-    >>> messages = sorted(smtpd, key=itemgetter('sender'))
-    >>> len(messages)
-    2
-    >>> for message in messages:
-    ...     print message['sender']
-    ...     print message['subject']
-    ...     print message['message-id']
-    itest-one-bounces+anne.person=[e-mail]
-    [Itest-one] A non-member post
-    <otter>
-    itest-one-bounces+archive=[e-mail]
-    [Itest-one] A non-member post
-    <otter>
-
-
-Preventing archiver forgeries
-=============================
-
-We archive messages by sending them to a special address owned by the Mail
-Archive <http://www.mail-archive.com>.  This address becomes a recipient of
-the mailing list, but we also prevent this address from being used to post
-messages to the list.
-
-    >>> from canonical.config import config
-    >>> config.mailman.archive_address
-    '[e-mail]'
-    >>> inject('itest-one', """\
-    ... From: %s
-    ... To: [e-mail]
-    ... Subject: A forgery
-    ... Message-ID: <narwhale>
-    ...
-    ... Hi, I am forging me some spam!
-    ... """ % config.mailman.archive_address)
-
-    >>> vette_watcher.wait_for_discard('narwhale')
-    >>> len(list(smtpd))
-    0

=== modified file 'lib/lp/services/mailman/monkeypatches/__init__.py'
--- lib/lp/services/mailman/monkeypatches/__init__.py	2010-09-17 20:46:58 +0000
+++ lib/lp/services/mailman/monkeypatches/__init__.py	2010-12-15 16:00:37 +0000
@@ -29,7 +29,7 @@
     # provide everything Mailman needs.
     #
     # Remember, don't rely on Launchpad's config object in the mm_cfg.py file
-    # or in the canonical.mailman.monkeypatches.defaults module because
+    # or in the lp.services.mailman.monkeypatches.defaults module because
     # Mailman will not be able to initialize Launchpad's configuration system.
     # Instead, anything that's needed from config should be written to the
     # mm_cfg.py file now.

=== modified file 'lib/lp/services/mailman/monkeypatches/lphandler.py'
--- lib/lp/services/mailman/monkeypatches/lphandler.py	2010-08-20 20:31:18 +0000
+++ lib/lp/services/mailman/monkeypatches/lphandler.py	2010-12-15 16:00:37 +0000
@@ -5,7 +5,7 @@
 
 
 import hashlib
-import xmlrpclib
+from Mailman.Queue import XMLRPCRunner
 
 from Mailman import (
     Errors,
@@ -42,7 +42,7 @@
     # can't talk to Launchpad, I believe it's better to let the message get
     # posted to the list than to discard or hold it.
     is_member = True
-    proxy = xmlrpclib.ServerProxy(mm_cfg.XMLRPC_URL)
+    proxy = proxy = XMLRPCRunner.get_mailing_list_api_proxy()
     # This will fail if we can't talk to Launchpad.  That's okay though
     # because Mailman's IncomingRunner will re-queue the message and re-start
     # processing at this handler.

=== modified file 'lib/lp/services/mailman/monkeypatches/lpstanding.py'
--- lib/lp/services/mailman/monkeypatches/lpstanding.py	2009-06-25 05:30:52 +0000
+++ lib/lp/services/mailman/monkeypatches/lpstanding.py	2010-12-15 16:00:37 +0000
@@ -7,9 +7,7 @@
 whether list non-members are allowed to post to a mailing list.
 """
 
-import xmlrpclib
-
-from Mailman import mm_cfg
+from Mailman.Queue import XMLRPCRunner
 
 
 def process(mlist, msg, msgdata):
@@ -26,7 +24,7 @@
     sender = msg.get_sender()
     # Ask Launchpad about the standing of this member.
     in_good_standing = False
-    proxy = xmlrpclib.ServerProxy(mm_cfg.XMLRPC_URL)
+    proxy = XMLRPCRunner.get_mailing_list_api_proxy()
     # This will fail if we can't talk to Launchpad.  That's okay though
     # because Mailman's IncomingRunner will re-queue the message and re-start
     # processing at this handler.

=== modified file 'lib/lp/services/mailman/monkeypatches/mm_cfg.py.in'
--- lib/lp/services/mailman/monkeypatches/mm_cfg.py.in	2010-09-17 20:46:58 +0000
+++ lib/lp/services/mailman/monkeypatches/mm_cfg.py.in	2010-12-15 16:00:37 +0000
@@ -44,7 +44,7 @@
 
 SITE_LIST_OWNER = '%(site_list_owner)s'
 
-DEFAULT_MSG_FOOTER = '''_______________________________________________
+DEFAULT_MSG_FOOTER = '''-- 
 %(footer)s'''
 
 # Set up MHonArc archiving.

=== modified file 'lib/lp/services/mailman/testing/__init__.py'
--- lib/lp/services/mailman/testing/__init__.py	2010-12-14 19:42:51 +0000
+++ lib/lp/services/mailman/testing/__init__.py	2010-12-15 16:00:37 +0000
@@ -63,6 +63,9 @@
         mlist.Create(team.name, owner_email, 'password')
         mlist.host_name = 'lists.launchpad.dev'
         mlist.web_page_url = 'http://lists.launchpad.dev/mailman/'
+        mlist.personalize = 1
+        mlist.include_rfc2369_headers = False
+        mlist.use_dollar_strings = True
         mlist.Save()
         mlist.addNewMember(owner_email)
         return mlist

=== added file 'lib/lp/services/mailman/tests/test_lphandler.py'
--- lib/lp/services/mailman/tests/test_lphandler.py	1970-01-01 00:00:00 +0000
+++ lib/lp/services/mailman/tests/test_lphandler.py	2010-12-15 16:00:37 +0000
@@ -0,0 +1,83 @@
+# Copyright 20010 Canonical Ltd.  This software is licensed under the
+# GNU Affero General Public License version 3 (see the file LICENSE).
+"""Test the LaunchpadMember monekypatches"""
+
+__metaclass__ = type
+__all__ = []
+
+import hashlib
+
+from Mailman import (
+    Errors,
+    mm_cfg,
+    )
+from Mailman.Handlers import LaunchpadMember
+
+from canonical.testing.layers import DatabaseFunctionalLayer
+from lp.services.mailman.testing import MailmanTestCase
+
+
+class TestLaunchpadMemberTestCase(MailmanTestCase):
+    """Test lphandler.
+
+    Mailman process() methods quietly return. They may set msg_data key-values
+    or raise an error to end processing. This group of tests tests often check
+    for errors, but that does not mean there is an error condition, it only
+    means message processing has reached a final decision. Messages that do
+    not cause a final decision pass-through and the process() methods ends
+    without a return.
+    """
+
+    layer = DatabaseFunctionalLayer
+
+    def setUp(self):
+        super(TestLaunchpadMemberTestCase, self).setUp()
+        self.team, self.mailing_list = self.factory.makeTeamAndMailingList(
+            'team-1', 'team-1-owner')
+        self.mm_list = self.makeMailmanList(self.mailing_list)
+
+    def tearDown(self):
+        super(TestLaunchpadMemberTestCase, self).tearDown()
+        self.cleanMailmanList(self.mm_list)
+
+    def test_messages_from_unknown_senders_are_discarded(self):
+        # A massage from an unknown email address is discarded.
+        message = self.makeMailmanMessage(
+            self.mm_list, '[e-mail]', 'subject', 'any content.')
+        msg_data = {}
+        args = (self.mm_list, message, msg_data)
+        self.assertRaises(
+            Errors.DiscardMessage, LaunchpadMember.process, *args)
+
+    def test_preapproved_messages_are_always_accepted(self):
+        # An approved message is accepted even if the email address is
+        # unknown.
+        message = self.makeMailmanMessage(
+            self.mm_list, '[e-mail]', 'subject', 'any content.')
+        msg_data = dict(approved=True)
+        silence = LaunchpadMember.process(self.mm_list, message, msg_data)
+        self.assertEqual(None, silence)
+
+    def test_messages_from_launchpad_users_are_accepted(self):
+        # A message from a launchpad user is accepted.
+        lp_user_email = '[e-mail]'
+        lp_user = self.factory.makePerson(email=lp_user_email)
+        message = self.makeMailmanMessage(
+            self.mm_list, lp_user_email, 'subject', 'any content.')
+        msg_data = {}
+        silence = LaunchpadMember.process(self.mm_list, message, msg_data)
+        self.assertEqual(None, silence)
+
+    def test_messages_from_launchpad_itself_are_accepted(self):
+        # A message from launchpad itself is accepted. Launchpad will sent
+        # a secret.
+        message = self.makeMailmanMessage(
+            self.mm_list, '[e-mail]', 'subject', 'any content.')
+        message['message-id'] = 'hamster.hamster'
+        hash = hashlib.sha1(mm_cfg.LAUNCHPAD_SHARED_SECRET)
+        hash.update(message['message-id'])
+        message['x-launchpad-hash'] = hash.hexdigest()
+        msg_data = {}
+        silence = LaunchpadMember.process(self.mm_list, message, msg_data)
+        self.assertEqual(None, silence)
+        self.assertEqual(True, msg_data['approved'])

=== added file 'lib/lp/services/mailman/tests/test_lpheaders.py'
--- lib/lp/services/mailman/tests/test_lpheaders.py	1970-01-01 00:00:00 +0000
+++ lib/lp/services/mailman/tests/test_lpheaders.py	2010-12-15 16:00:37 +0000
@@ -0,0 +1,103 @@
+# Copyright 20010 Canonical Ltd.  This software is licensed under the
+# GNU Affero General Public License version 3 (see the file LICENSE).
+"""Test the lpheaders monekypatches"""
+
+__metaclass__ = type
+__all__ = []
+
+from Mailman.Handlers import (
+    Decorate,
+    LaunchpadHeaders,
+    )
+
+from canonical.testing.layers import DatabaseFunctionalLayer
+from lp.services.mailman.testing import MailmanTestCase
+
+
+class TestLaunchpadHeadersTestCase(MailmanTestCase):
+    """Test lpheaders.
+
+    Mailman process() methods quietly return. They may set msg_data key-values
+    or raise an error to end processing. This group of tests tests often check
+    for errors, but that does not mean there is an error condition, it only
+    means message processing has reached a final decision. Messages that do
+    not cause a final decision pass-through and the process() methods ends
+    without a return.
+    """
+
+    layer = DatabaseFunctionalLayer
+
+    def setUp(self):
+        super(TestLaunchpadHeadersTestCase, self).setUp()
+        self.team, self.mailing_list = self.factory.makeTeamAndMailingList(
+            'team-1', 'team-1-owner')
+        self.mm_list = self.makeMailmanList(self.mailing_list)
+        self.lp_user_email = '[e-mail]'
+        self.lp_user = self.factory.makePerson(
+            name='albatros', email=self.lp_user_email)
+
+    def tearDown(self):
+        super(TestLaunchpadHeadersTestCase, self).tearDown()
+        self.cleanMailmanList(self.mm_list)
+
+    def test_message_launchpad_headers(self):
+        # All messages get updated headers.
+        message = self.makeMailmanMessage(
+            self.mm_list, self.lp_user_email, 'subject', 'any content.')
+        msg_data = {}
+        silence = LaunchpadHeaders.process(self.mm_list, message, msg_data)
+        self.assertEqual(None, silence)
+        self.assertEqual(
+            '<team-1.lists.launchpad.dev>', message['List-Id'])
+        self.assertEqual(
+            '<http://help.launchpad.dev/ListHelp>', message['List-Help'])
+        self.assertEqual(
+            '<http://launchpad.dev/~team-1>', message['List-Subscribe'])
+        self.assertEqual(
+            '<http://launchpad.dev/~team-1>', message['List-Unsubscribe'])
+        self.assertEqual(
+            '<mailto:[e-mail]>', message['List-Post'])
+        self.assertEqual(
+            '<http://lists.launchpad.dev/team-1>', message['List-Archive'])
+        self.assertEqual(
+            '<http://launchpad.dev/~team-1>', message['List-Owner'])
+
+    def test_message_decoration_data(self):
+        # The lpheaders process method provides decoration-data.
+        message = self.makeMailmanMessage(
+            self.mm_list, self.lp_user_email, 'subject', 'any content.')
+        msg_data = {}
+        silence = LaunchpadHeaders.process(self.mm_list, message, msg_data)
+        self.assertEqual(None, silence)
+        self.assertTrue('decoration-data' in msg_data)
+        decoration_data = msg_data['decoration-data']
+        self.assertEqual(
+            'http://launchpad.dev/~team-1',
+            decoration_data['list_owner'])
+        self.assertEqual(
+            '[e-mail]',
+            decoration_data['list_post'])
+        self.assertEqual(
+            'http://launchpad.dev/~team-1',
+            decoration_data['list_unsubscribe'])
+        self.assertEqual(
+            'http://help.launchpad.dev/ListHelp',
+            decoration_data['list_help'])
+
+    def test_message_decorate_footer(self):
+        # The Decorate handler uses the lpheaders decoration-data.
+        message = self.makeMailmanMessage(
+            self.mm_list, self.lp_user_email, 'subject', 'any content.')
+        msg_data = {}
+        LaunchpadHeaders.process(self.mm_list, message, msg_data)
+        self.assertTrue('decoration-data' in msg_data)
+        silence = Decorate.process(self.mm_list, message, msg_data)
+        self.assertEqual(None, silence)
+        body, footer = message.get_payload()[1].get_payload().rsplit('-- ', 1)
+        expected = (
+            "\n"
+            "Mailing list: http://launchpad.dev/~team-1\n"
+            "Post to     : [e-mail]\n"
+            "Unsubscribe : http://launchpad.dev/~team-1\n"
+            "More help   : http://help.launchpad.dev/ListHelp\n")
+        self.assertEqual(expected, footer)

=== added file 'lib/lp/services/mailman/tests/test_lpstanding.py'
--- lib/lp/services/mailman/tests/test_lpstanding.py	1970-01-01 00:00:00 +0000
+++ lib/lp/services/mailman/tests/test_lpstanding.py	2010-12-15 16:00:37 +0000
@@ -0,0 +1,59 @@
+# Copyright 20010 Canonical Ltd.  This software is licensed under the
+# GNU Affero General Public License version 3 (see the file LICENSE).
+"""Test the lpstanding monekypatches"""
+
+__metaclass__ = type
+__all__ = []
+
+from Mailman.Handlers import LPStanding
+
+from canonical.testing.layers import DatabaseFunctionalLayer
+from lp.registry.interfaces.person import PersonalStanding
+from lp.services.mailman.testing import MailmanTestCase
+from lp.testing import celebrity_logged_in
+
+
+class TestLPStandingTestCase(MailmanTestCase):
+    """Test lpstanding.
+
+    Mailman process() methods quietly return. They may set msg_data key-values
+    or raise an error to end processing. This group of tests tests often check
+    for errors, but that does not mean there is an error condition, it only
+    means message processing has reached a final decision. Messages that do
+    not cause a final decision pass-through and the process() methods ends
+    without a return.
+    """
+
+    layer = DatabaseFunctionalLayer
+
+    def setUp(self):
+        super(TestLPStandingTestCase, self).setUp()
+        self.team, self.mailing_list = self.factory.makeTeamAndMailingList(
+            'team-1', 'team-1-owner')
+        self.mm_list = self.makeMailmanList(self.mailing_list)
+        self.lp_user_email = '[e-mail]'
+        self.lp_user = self.factory.makePerson(email=self.lp_user_email)
+
+    def tearDown(self):
+        super(TestLPStandingTestCase, self).tearDown()
+        self.cleanMailmanList(self.mm_list)
+
+    def test_non_subscriber_without_good_standing_is_not_approved(self):
+        # Non-subscribers without good standing are not approved to post.
+        message = self.makeMailmanMessage(
+            self.mm_list, self.lp_user_email, 'subject', 'any content.')
+        msg_data = {}
+        silence = LPStanding.process(self.mm_list, message, msg_data)
+        self.assertEqual(None, silence)
+        self.assertFalse('approved' in msg_data)
+
+    def test_non_subscriber_with_good_standing_is_approved(self):
+        # Non-subscribers with good standing are approved to post.
+        with celebrity_logged_in('admin'):
+            self.lp_user.personal_standing = PersonalStanding.GOOD
+        message = self.makeMailmanMessage(
+            self.mm_list, self.lp_user_email, 'subject', 'any content.')
+        msg_data = {}
+        silence = LPStanding.process(self.mm_list, message, msg_data)
+        self.assertEqual(None, silence)
+        self.assertTrue(msg_data['approved'])

=== added file 'lib/lp/services/mailman/tests/test_mm_cfg.py'
--- lib/lp/services/mailman/tests/test_mm_cfg.py	1970-01-01 00:00:00 +0000
+++ lib/lp/services/mailman/tests/test_mm_cfg.py	2010-12-15 16:00:37 +0000
@@ -0,0 +1,189 @@
+# Copyright 20010 Canonical Ltd.  This software is licensed under the
+# GNU Affero General Public License version 3 (see the file LICENSE).
+"""Test the Launchpad defaults monekypatch and mm_cfg."""
+
+__metaclass__ = type
+__all__ = []
+
+
+from Mailman import (
+    mm_cfg,
+    Utils,
+    )
+
+from canonical.config import config
+from canonical.testing.layers import FunctionalLayer
+from lp.services.mailman.config import configure_prefix
+from lp.services.mailman.monkeypatches import monkey_patch
+from lp.testing import TestCase
+
+
+class TestMMCfgDefaultsTestCase(TestCase):
+    """Test launchapd default overrides."""
+
+    layer = FunctionalLayer
+
+    def test_common_values(self):
+        # Launchpad's boolean and string parameters.
+        self.assertEqual('unused_mailman_site_list', mm_cfg.MAILMAN_SITE_LIST)
+        self.assertEqual(None, mm_cfg.MTA)
+        self.assertEqual(3, mm_cfg.DEFAULT_GENERIC_NONMEMBER_ACTION)
+        self.assertEqual(False, mm_cfg.DEFAULT_SEND_REMINDERS)
+        self.assertEqual(True, mm_cfg.DEFAULT_SEND_WELCOME_MSG)
+        self.assertEqual(False, mm_cfg.DEFAULT_SEND_GOODBYE_MSG)
+        self.assertEqual(False, mm_cfg.DEFAULT_DIGESTABLE)
+        self.assertEqual(False, mm_cfg.DEFAULT_BOUNCE_NOTIFY_OWNER_ON_DISABLE)
+        self.assertEqual(False, mm_cfg.DEFAULT_BOUNCE_NOTIFY_OWNER_ON_REMOVAL)
+        self.assertEqual(True, mm_cfg.VERP_PERSONALIZED_DELIVERIES)
+        self.assertEqual(False, mm_cfg.DEFAULT_FORWARD_AUTO_DISCARDS)
+        self.assertEqual(False, mm_cfg.DEFAULT_BOUNCE_PROCESSING)
+
+    def test_qrunners(self):
+        # The queue runners used by Launchpad.
+        runners = [pair[0] for pair in mm_cfg.QRUNNERS if pair[1] == 1]
+        expected = [
+            'ArchRunner', 'BounceRunner', 'IncomingRunner', 'OutgoingRunner',
+            'VirginRunner', 'RetryRunner', 'XMLRPCRunner']
+        self.assertEqual(expected, runners)
+
+    def test_global_pipeline(self):
+        # The ordered list of handlers used by Launchpad.
+        # NB. This is a very important list when debuggin were a message
+        # has been touched.
+        expected = [
+            'LaunchpadMember', 'SpamDetect', 'Approve', 'Replybot',
+            'LPStanding', 'LPModerate', 'LPSize',
+            'MimeDel', 'Scrubber', 'Emergency', 'Tagger', 'CalcRecips',
+            'AvoidDuplicates', 'Cleanse', 'CleanseDKIM', 'CookHeaders',
+            'LaunchpadHeaders', 'ToDigest', 'ToArchive', 'ToUsenet',
+            'AfterDelivery', 'Acknowledge', 'ToOutgoing']
+        self.assertEqual(expected, mm_cfg.GLOBAL_PIPELINE)
+
+
+class TestMMCfgLaunchpadConfigTestCase(TestCase):
+    """Test launchapd default overrides.
+
+    The mailman config is generated from the selected launchpad config.
+    The config will either be the test runner or the app server depending
+    on the what was previously used when mailman was run. The config must
+    be created in setup to ensure predicable values.
+    """
+
+    layer = FunctionalLayer
+
+    def setUp(self):
+        super(TestMMCfgLaunchpadConfigTestCase, self).setUp()
+        # Generate a mailman config using this environment's config.
+        mailman_path = configure_prefix(config.mailman.build_prefix)
+        monkey_patch(mailman_path, config)
+        reload(mm_cfg)
+
+    def test_mail_server(self):
+        # Launchpad's smtp config values.
+        host, port = config.mailman.smtp.split(':')
+        self.assertEqual(host, mm_cfg.SMTPHOST)
+        self.assertEqual(int(port), mm_cfg.SMTPPORT)
+
+    def test_xmlrpc_server(self):
+        # Launchpad's smtp config values.
+        self.assertEqual(
+            config.mailman.xmlrpc_url,
+            mm_cfg.XMLRPC_URL)
+        self.assertEqual(
+            config.mailman.xmlrpc_runner_sleep,
+            mm_cfg.XMLRPC_SLEEPTIME)
+        self.assertEqual(
+            config.mailman.subscription_batch_size,
+            mm_cfg.XMLRPC_SUBSCRIPTION_BATCH_SIZE)
+        self.assertEqual(
+            config.mailman.shared_secret,
+            mm_cfg.LAUNCHPAD_SHARED_SECRET)
+
+    def test_messge_footer(self):
+        # Launchpad's email footer.
+        self.assertEqual(
+            config.mailman.list_help_header,
+            mm_cfg.LIST_HELP_HEADER)
+        self.assertEqual(
+            config.mailman.list_owner_header_template,
+            mm_cfg.LIST_SUBSCRIPTION_HEADERS)
+        self.assertEqual(
+            config.mailman.archive_url_template,
+            mm_cfg.LIST_ARCHIVE_HEADER_TEMPLATE)
+        self.assertEqual(
+            config.mailman.list_owner_header_template,
+            mm_cfg.LIST_OWNER_HEADER_TEMPLATE)
+        self.assertEqual(
+            "-- \n"
+            "Mailing list: $list_owner\n"
+            "Post to     : $list_post\n"
+            "Unsubscribe : $list_unsubscribe\n"
+            "More help   : $list_help\n",
+            mm_cfg.DEFAULT_MSG_FOOTER)
+
+    def test_message_rules(self):
+        # Launchpad's rules for handling messages.
+        self.assertEqual(
+            config.mailman.soft_max_size,
+            mm_cfg.LAUNCHPAD_SOFT_MAX_SIZE)
+        self.assertEqual(
+            config.mailman.hard_max_size,
+            mm_cfg.LAUNCHPAD_HARD_MAX_SIZE)
+        self.assertEqual(
+            config.mailman.register_bounces_every,
+            mm_cfg.REGISTER_BOUNCES_EVERY)
+
+    def test_archive_setup(self):
+        # Launchpad's rules for setting up list archives.
+        self.assertTrue('-add' in mm_cfg.PUBLIC_EXTERNAL_ARCHIVER)
+        self.assertTrue('-spammode' in mm_cfg.PUBLIC_EXTERNAL_ARCHIVER)
+        self.assertTrue('-umask 022'in mm_cfg.PUBLIC_EXTERNAL_ARCHIVER)
+        self.assertTrue(
+            '-dbfile'
+            '/var/tmp/mailman/archives/private/%(listname)s.mbox/mhonarc.db',
+           mm_cfg.PUBLIC_EXTERNAL_ARCHIVER)
+        self.assertTrue(
+            '-outdit /var/tmp/mailman/mhonarc/%(listname)s',
+            mm_cfg.PUBLIC_EXTERNAL_ARCHIVER)
+        self.assertTrue(
+            '-definevar ML-NAME=%(listname)s',
+            mm_cfg.PUBLIC_EXTERNAL_ARCHIVER)
+        self.assertTrue(
+            '-rcfile var/tmp/mailman/data/lp-mhonarc-common.mrc',
+            mm_cfg.PUBLIC_EXTERNAL_ARCHIVER)
+        self.assertTrue(
+            '-stderr /var/tmp/mailman/logs/mhonarc',
+            mm_cfg.PUBLIC_EXTERNAL_ARCHIVER)
+        self.assertTrue(
+            '-stdout /var/tmp/mailman/logs/mhonarc',
+            mm_cfg.PUBLIC_EXTERNAL_ARCHIVER)
+        self.assertEqual(
+            mm_cfg.PRIVATE_EXTERNAL_ARCHIVER, mm_cfg.PUBLIC_EXTERNAL_ARCHIVER)
+
+
+class TestSiteTemplates(TestCase):
+    """Test launchapd site templates."""
+
+    layer = FunctionalLayer
+
+    def test_postheld(self):
+        postheld_dict = {
+            'listname': 'fake-list',
+            'hostname': 'lists.launchpad.net',
+            'reason': 'XXX',
+            'sender': '[e-mail]',
+            'subject': "YYY",
+            'admindb_url': 'http://lists.launchpad.net/fake/admin',
+            }
+        text, file_name = Utils.findtext('postheld.txt', dict=postheld_dict)
+        self.assertTrue(
+            file_name.endswith('/lib/mailman/templates/site/en/postheld.txt'))
+        self.assertEqual(
+            "Your mail to 'fake-list' with the subject\n\n"
+            "    YYY\n\n"
+            "Is being held until the list moderator can review it for "
+            "approval.\n\nThe reason it is being held:\n\n"
+            "    XXX\n\n"
+            "Either the message will get posted to the list, or you will "
+            "receive\nnotification of the moderator's decision.\n",
+            text)