Skip to content

[3.7] bpo-32663 Make SMTPUTF8SimTests run (GH-5314)#8471

Merged
bitdancer merged 1 commit intopython:3.7from
miss-islington:backport-48ed88a-3.7
Jul 26, 2018
Merged

[3.7] bpo-32663 Make SMTPUTF8SimTests run (GH-5314)#8471
bitdancer merged 1 commit intopython:3.7from
miss-islington:backport-48ed88a-3.7

Conversation

@miss-islington
Copy link
Copy Markdown
Contributor

@miss-islington miss-islington commented Jul 25, 2018

Enable and fix SMTPUTF8SimTests in test_smtplib.

The tests for SMTPUTF8SimTests in test_smtplib.py were not actually
being run because test_smtplib was still using the 'test_main' pattern,
and the class was never added to test_main.

Additionally, one of the tests needed to be moved to the non-UTF8 server
class because it relies on the server not being UTF-8 compatible (and it
had a bug in in).
(cherry picked from commit 48ed88a)

Co-authored-by: chason chason@gmail.com

https://bugs.python.org/issue32663

Enable and fix SMTPUTF8SimTests in test_smtplib.

The tests for SMTPUTF8SimTests in test_smtplib.py were not actually
being run because test_smtplib was still using the 'test_main' pattern,
and the class was never added to test_main.

Additionally, one of the tests needed to be moved to the non-UTF8 server
class because it relies on the server not being UTF-8 compatible (and it
had a bug in in).
(cherry picked from commit 48ed88a)

Co-authored-by: chason <chason@gmail.com>
@miss-islington
Copy link
Copy Markdown
Contributor Author

@chason and @bitdancer: Backport status check is done, and it's a failure ❌ .

@bitdancer bitdancer merged commit 89352b0 into python:3.7 Jul 26, 2018
@bedevere-bot
Copy link
Copy Markdown

@bitdancer: Please replace # with GH- in the commit message next time. Thanks!

@miss-islington miss-islington deleted the backport-48ed88a-3.7 branch July 26, 2018 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants