Issue 25073: Document asyncio.test_utils - Python tracker (original) (raw)
This issue has been migrated to GitHub: https://github.com/python/cpython/issues/69260
classification
Title: | Document asyncio.test_utils | ||
---|---|---|---|
Type: | enhancement | Stage: | resolved |
Components: | asyncio, Documentation | Versions: | Python 3.6, Python 3.4, Python 3.5 |
process
Status: | closed | Resolution: | not a bug |
---|---|---|---|
Dependencies: | Superseder: | ||
Assigned To: | docs@python | Nosy List: | docs@python, gvanrossum, vstinner, yselivanov, zach.ware |
Priority: | normal | Keywords: |
Created on 2015-09-11 18:09 by zach.ware, last changed 2022-04-11 14:58 by admin. This issue is now closed.
Messages (5) | ||
---|---|---|
msg250482 - (view) | Author: Zachary Ware (zach.ware) * ![]() |
Date: 2015-09-11 18:09 |
asyncio.test_utils is not documented. I'm unsure as to whether that's because it's meant to be private, or if it just hasn't been done yet. | ||
msg250483 - (view) | Author: Yury Selivanov (yselivanov) * ![]() |
Date: 2015-09-11 18:15 |
It's meant to be private. | ||
msg250484 - (view) | Author: Guido van Rossum (gvanrossum) * ![]() |
Date: 2015-09-11 18:17 |
Agreed, but then why isn't it in the tests directory? There are some examples in the asyncio repo on GitHub that use test_utils.dummy_ssl_context(), and that's probably why; but that's a fairly bad practice plus it's really a one-liner. | ||
msg250492 - (view) | Author: Yury Selivanov (yselivanov) * ![]() |
Date: 2015-09-11 19:32 |
> Agreed, but then why isn't it in the tests directory? I think that the main reason is to make it possible for tests to import some common functionality from 'asyncio' package, when you run them from the cloned github asyncio repo. In fact, I think we should prefix the module with an underscore. | ||
msg250494 - (view) | Author: Guido van Rossum (gvanrossum) * ![]() |
Date: 2015-09-11 20:15 |
The _ is a good idea. Please do fix up the examples! |
History | |||
---|---|---|---|
Date | User | Action | Args |
2022-04-11 14:58:20 | admin | set | github: 69260 |
2015-09-11 20:15:18 | gvanrossum | set | messages: + |
2015-09-11 19:32:05 | yselivanov | set | messages: + |
2015-09-11 18:17:01 | gvanrossum | set | messages: + |
2015-09-11 18:15:43 | yselivanov | set | status: open -> closedresolution: not a bugmessages: + stage: needs patch -> resolved |
2015-09-11 18:09:38 | zach.ware | create |