summaryrefslogtreecommitdiffstats
path: root/Lib/test/test_email
diff options
context:
space:
mode:
authorR David Murray <rdmurray@bitdance.com>2016-09-07 17:41:06 (GMT)
committerR David Murray <rdmurray@bitdance.com>2016-09-07 17:41:06 (GMT)
commit91afe7ec9fc82328bc17ab6d7312489dffac2223 (patch)
treef69054645f6be23b685d9c4b2e535d5bdd5625be /Lib/test/test_email
parent5223f08e78efcfadb7fe2288fd48bb985d6ca7a7 (diff)
parentc2e2473bc210ff83c2cbc73c4b916893f773b480 (diff)
downloadcpython-91afe7ec9fc82328bc17ab6d7312489dffac2223.zip
cpython-91afe7ec9fc82328bc17ab6d7312489dffac2223.tar.gz
cpython-91afe7ec9fc82328bc17ab6d7312489dffac2223.tar.bz2
Merge: 27988: Make sure iter_attachments does not mutate the payload list.
Diffstat (limited to 'Lib/test/test_email')
-rw-r--r--Lib/test/test_email/test_message.py10
1 files changed, 10 insertions, 0 deletions
diff --git a/Lib/test/test_email/test_message.py b/Lib/test/test_email/test_message.py
index d78049e..4345162 100644
--- a/Lib/test/test_email/test_message.py
+++ b/Lib/test/test_email/test_message.py
@@ -732,6 +732,16 @@ class TestEmailMessageBase:
m.set_param('filename', 'abc.png', 'Content-Disposition')
self.assertTrue(m.is_attachment())
+ def test_iter_attachments_mutation(self):
+ # We had a bug where iter_attachments was mutating the list.
+ m = self._make_message()
+ m.set_content('arbitrary text as main part')
+ m.add_related('more text as a related part')
+ m.add_related('yet more text as a second "attachment"')
+ orig = m.get_payload().copy()
+ self.assertEqual(len(list(m.iter_attachments())), 2)
+ self.assertEqual(m.get_payload(), orig)
+
class TestEmailMessage(TestEmailMessageBase, TestEmailBase):
message = EmailMessage