#11062: Fix adding a message from file to Babyl mailbox

Lib/mailbox.py

@@ -1440,9 +1440,9 @@ class Babyl(_singlefileMailbox):
                     line = line[:-1] + b'\n'
                 self._file.write(line.replace(b'\n', linesep))
                 if line == b'\n' or not line:
-                    self._file.write(b'*** EOOH ***' + linesep)
                     if first_pass:
                         first_pass = False
+                        self._file.write(b'*** EOOH ***' + linesep)
                         message.seek(original_pos)
                     else:
                         break

Lib/test/test_mailbox.py

@@ -152,8 +152,6 @@ class TestMailbox(TestBase):
             f.write(_bytes_sample_message)
             f.seek(0)
             key = self._box.add(f)
-        # See issue 11062
-        if not isinstance(self._box, mailbox.Babyl):
         self.assertEqual(self._box.get_bytes(key).split(b'\n'),
             _bytes_sample_message.split(b'\n'))
 
@@ -162,8 +160,6 @@ class TestMailbox(TestBase):
             f.write(self._non_latin_bin_msg)
             f.seek(0)
             key = self._box.add(f)
-        # See issue 11062
-        if not isinstance(self._box, mailbox.Babyl):
         self.assertEqual(self._box.get_bytes(key).split(b'\n'),
             self._non_latin_bin_msg.split(b'\n'))
 
@@ -173,8 +169,6 @@ class TestMailbox(TestBase):
             f.seek(0)
             with self.assertWarns(DeprecationWarning):
                 key = self._box.add(f)
-        # See issue 11062
-        if not isinstance(self._box, mailbox.Babyl):
         self.assertEqual(self._box.get_bytes(key).split(b'\n'),
             _bytes_sample_message.split(b'\n'))
 

Misc/NEWS

@@ -13,6 +13,8 @@ Core and Builtins
 Library
 -------
 
+- Issue #11062: Fix adding a message from file to Babyl mailbox.
+
 - Issue #15646: Prevent equivalent of a fork bomb when using
   multiprocessing on Windows without the "if __name__ == '__main__'"
   idiom.