[Pkg-mailman-hackers] Bug#1142825: mailman3: flufl.bounce 5.x will regress several tests in TestCLISyncMembers

[Colin Watson]

Source: mailman3
Version: 3.3.10-2
Severity: important

I'd like to upload flufl.bounce 5.x to unstable, because it will fix 
https://bugs.debian.org/1083397.  However, in 
https://debusine.debian.net/debian/developers/work-request/936438/ I 
found that this makes mailman3's autopkgtests worse.  It wouldn't 
technically prevent testing migration since mailman3's autopkgtests are 
already failing in testing (#XXX), but I'd rather get it fixed anyway.  
The new test failures are:

======================================================================
FAIL: test_subscription_pending (mailman.commands.tests.test_cli_syncmembers.TestCLISyncMembers.test_subscription_pending)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/autopkgtest.F76XuB/build.UOg/src/src/mailman/commands/tests/test_cli_syncmembers.py", line 130, in test_subscription_pending
    self.assertEqual(result.output,
    ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^
                     '[ADD] Anne Person'
                     ^^^^^^^^^^^^^^^^^^^
    ...<3 lines>...
                     '[ADD] Bart Person'
                     ^^^^^^^^^^^^^^^^^^^
                     ' <bperson at example.com>\n')
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: '[ADD[32 chars]om>\n[ADD] Bart Person <bperson at example.com>\n[72 chars]m>\n' != '[ADD[32 chars]om>\nSubscription already pending (skipping): [72 chars]m>\n'
  [ADD] Anne Person <aperson at example.com>
+ Subscription already pending (skipping): Anne Person <aperson at example.com>
  [ADD] Bart Person <bperson at example.com>
- Subscription already pending (skipping): Anne Person <aperson at example.com>


======================================================================
FAIL: test_sync_empty_tuple (mailman.commands.tests.test_cli_syncmembers.TestCLISyncMembers.test_sync_empty_tuple)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/autopkgtest.F76XuB/build.UOg/src/src/mailman/commands/tests/test_cli_syncmembers.py", line 99, in test_sync_empty_tuple
    self.assertEqual(result.output, "Cannot parse as valid email "
    ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
                                    "address (skipping): \"\"\n"
                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
                                    "[DEL] Bart Person "
                                    ^^^^^^^^^^^^^^^^^^^^
                                    "<bperson at example.com>\n")
                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: '[DEL] Bart Person <bperson at example.com>\n[48 chars]""\n' != 'Cannot parse as valid email address (skip[48 chars]m>\n'
+ Cannot parse as valid email address (skipping): ""
  [DEL] Bart Person <bperson at example.com>
- Cannot parse as valid email address (skipping): ""


======================================================================
FAIL: test_sync_invalid_email (mailman.commands.tests.test_cli_syncmembers.TestCLISyncMembers.test_sync_invalid_email)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/autopkgtest.F76XuB/build.UOg/src/src/mailman/commands/tests/test_cli_syncmembers.py", line 72, in test_sync_invalid_email
    self.assertEqual(
    ~~~~~~~~~~~~~~~~^
       result.output,
       ^^^^^^^^^^^^^^
    ...<3 lines>...
       ' (skipping): not-a-valid at email\n'
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
       'Nothing to do\n')
       ^^^^^^^^^^^^^^^^^^
AssertionError: 'Nothing to do\nCannot parse as valid emai[121 chars]il\n' != 'Cannot parse as valid email address (skip[121 chars]do\n'
- Nothing to do
  Cannot parse as valid email address (skipping): Dont Subscribe <not-a-valid-email>
  Cannot parse as valid email address (skipping): not-a-valid at email
+ Nothing to do

Thanks,

-- 
Colin Watson (he/him)                              [cjwatson at debian.org]