Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tests still rely on the removed stdlib modules asyncore/asynchat #93243

Closed
2 of 12 tasks
arhadthedev opened this issue May 26, 2022 · 5 comments
Closed
2 of 12 tasks

tests still rely on the removed stdlib modules asyncore/asynchat #93243

arhadthedev opened this issue May 26, 2022 · 5 comments
Labels
tests Tests in the Lib/test dir type-feature A feature request or enhancement

Comments

@arhadthedev
Copy link
Member

arhadthedev commented May 26, 2022

Reference: PEP 594 – Removing dead batteries from the standard library.

This issue tracks removal of the following modules:

Related issues:

@arhadthedev arhadthedev added the type-feature A feature request or enhancement label May 26, 2022
@JelleZijlstra
Copy link
Member

This requires finishing #1864 (which I need to get back to).

@arhadthedev
Copy link
Member Author

Asyncore-based test servers from test_ftp and test_logging cannot be ported to asyncio, the tested clients are blocking. However, instead of writing a server thread class from scratch, it would be nice to reuse test.support.threading_helper.Server from gh-92475.

miss-islington pushed a commit that referenced this issue Aug 6, 2022
gh-93243

This PR is required to reduce diffs of the following porting (no need to either maintain documentation and tests consistent with each porting step, or try to port everything and remove smtpd in a single PR).

Automerge-Triggered-By: GH:warsaw
iritkatriel pushed a commit to iritkatriel/cpython that referenced this issue Aug 11, 2022
…-93246)

pythongh-93243

This PR is required to reduce diffs of the following porting (no need to either maintain documentation and tests consistent with each porting step, or try to port everything and remove smtpd in a single PR).

Automerge-Triggered-By: GH:warsaw
@iritkatriel iritkatriel added the tests Tests in the Lib/test dir label Nov 29, 2023
@iritkatriel iritkatriel changed the title PEP 594: Process "to be removed in 3.12" part of the plan tests still rely on the removed stdlib modules asyncore/asynchat Nov 29, 2023
@iritkatriel
Copy link
Member

Changing the title to reflect what still needs to be done.

@vstinner
Copy link
Member

vstinner commented Feb 27, 2025

PEP 594 has been fully implemented: asyncore, asynchat and smtpd have been removed from the stdlib.

Some tests still use asyncore, asynchat and smtpd: there is now a copy in Lib/test/support/ only for tests. While it would be nice to rewrite these tests using asyncio or threads, it seems like no one is interested to work on that.

I suggest to close the issue as "not planned". It's ok to leave tests are they are, since tests are now reliable and stable.

@iritkatriel
Copy link
Member

I agree.

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 type-feature A feature request or enhancement
Projects
None yet
Development

No branches or pull requests

4 participants