diff options
Diffstat (limited to 'src/mailman/model')
| -rw-r--r-- | src/mailman/model/digests.py | 2 | ||||
| -rw-r--r-- | src/mailman/model/listmanager.py | 3 | ||||
| -rw-r--r-- | src/mailman/model/tests/test_listmanager.py | 9 |
3 files changed, 12 insertions, 2 deletions
diff --git a/src/mailman/model/digests.py b/src/mailman/model/digests.py index e94bb073e..1b7140824 100644 --- a/src/mailman/model/digests.py +++ b/src/mailman/model/digests.py @@ -45,7 +45,7 @@ class OneLastDigest(Model): id = Column(Integer, primary_key=True) mailing_list_id = Column(Integer, ForeignKey('mailinglist.id')) - maling_list = relationship('MailingList') + mailing_list = relationship('MailingList') address_id = Column(Integer, ForeignKey('address.id')) address = relationship('Address') diff --git a/src/mailman/model/listmanager.py b/src/mailman/model/listmanager.py index 3806f9497..43a2b8f2a 100644 --- a/src/mailman/model/listmanager.py +++ b/src/mailman/model/listmanager.py @@ -110,7 +110,8 @@ class ListManager: """See `IListManager`.""" result_set = store.query(MailingList) for list_id in result_set.values(MailingList._list_id): - yield list_id + assert isinstance(list_id, tuple) and len(list_id) == 1 + yield list_id[0] @property @dbconnection diff --git a/src/mailman/model/tests/test_listmanager.py b/src/mailman/model/tests/test_listmanager.py index 287a4dba5..3951e8250 100644 --- a/src/mailman/model/tests/test_listmanager.py +++ b/src/mailman/model/tests/test_listmanager.py @@ -80,6 +80,15 @@ class TestListManager(unittest.TestCase): self.assertTrue(isinstance(self._events[1], ListDeletedEvent)) self.assertEqual(self._events[1].fqdn_listname, 'another@example.com') + def test_list_manager_list_ids(self): + # You can get all the list ids for all the existing mailing lists. + create_list('ant@example.com') + create_list('bee@example.com') + create_list('cat@example.com') + self.assertEqual( + sorted(getUtility(IListManager).list_ids), + ['ant.example.com', 'bee.example.com', 'cat.example.com']) + class TestListLifecycleEvents(unittest.TestCase): |
