* Refactor add_member() so that it uses a RequestRecord namedtuple.

* RequestRecord contains no password key so these are not part of the held
  requests database any more.
* Pending record contains `email` now instead of `address`.

1 files changed, 5 insertions, 3 deletions

diff --git a/src/mailman/mta/tests/test_delivery.py b/src/mailman/mta/tests/test_delivery.py
index 77d31d3a3..18eef8845 100644
--- a/src/mailman/mta/tests/test_delivery.py
+++ b/src/mailman/mta/tests/test_delivery.py
@@ -32,6 +32,7 @@ from mailman.app.membership import add_member
 from mailman.config import config
 from mailman.interfaces.mailinglist import Personalization
 from mailman.interfaces.member import DeliveryMode
+from mailman.interfaces.subscriptions import RequestRecord
 from mailman.mta.deliver import Deliver
 from mailman.testing.helpers import (
     specialized_message_from_string as mfs)
@@ -63,9 +64,10 @@ class TestIndividualDelivery(unittest.TestCase):
         self._mlist = create_list('test@example.com')
         self._mlist.personalize = Personalization.individual
         # Make Anne a member of this mailing list.
-        self._anne = add_member(self._mlist,
-                                'anne@example.org', 'Anne Person',
-                                'xyz', DeliveryMode.regular, 'en')
+        self._anne = add_member(
+            self._mlist,
+            RequestRecord('anne@example.org', 'Anne Person',
+                          DeliveryMode.regular, 'en'))
         # Clear out any results from the previous test.
         del _deliveries[:]
         self._msg = mfs("""\