Skip to content

bpo-38242: Revert new asyncio streaming API #16455

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.

Sign up for GitHub

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

Closed
aeros wants to merge 22 commits into from
Closed

bpo-38242: Revert new asyncio streaming API #16455

Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
ddbd854
Revert "bpo-36889: Merge asyncio streams (GH-13251)"
aeros Sep 28, 2019
4fb43ff
Revert "bpo-37105: Add deprecated-remove information on stream doc (#…
aeros Sep 28, 2019
cd886e3
Revert "bpo-36889: Document asyncio Stream and StreamServer (GH-14203)"
aeros Sep 28, 2019
aaf003d
Revert "bpo-36889: Document Stream class and add docstrings (GH-14488)"
aeros Sep 28, 2019
f390d31
Fix broken method names
aeros Sep 28, 2019
5e2e75a
Remove explicit passing of loop arg
aeros Sep 28, 2019
557974e
Replace asyncio.Task with create_task
aeros Sep 28, 2019
917b47a
Fix create_task
aeros Sep 28, 2019
01ce34b
Replace asyncio.Task with loop.create_task in test_streams
aeros Sep 28, 2019
db3a53a
Merge branch 'revert-asyncio-streaming' of https://github.com/aeros16…
aeros Sep 28, 2019
2d0324b
Patchcheck and fix missing paren
aeros Sep 28, 2019
110c65b
Restore deprecation warnings for explicit loop arg
aeros Sep 28, 2019
dc7e868
Restore deprecation warnings in test_streams
aeros Sep 28, 2019
e1c42ce
Restore test__all__
aeros Sep 28, 2019
9b733e1
Restore test_sslproto
aeros Sep 28, 2019
4abc7fe
Restore test_base_events
aeros Sep 28, 2019
226187e
Restore test_server
aeros Sep 28, 2019
20406d3
Remove _asyncio_internal from StreamReader and StreamReaderProtocol
aeros Sep 28, 2019
bc82fd3
Remove unneeded _asyncio_internal in test_subprocess
aeros Sep 28, 2019
2d02e8a
Remove _asyncio_internal from Process
aeros Sep 28, 2019
e49359c
Remove DeprecationWarning tests from new API
aeros Sep 28, 2019
ec1c0b7
Fix test name
aeros Sep 29, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Restore test__all__
  • Loading branch information
@aeros
aeros committed Sep 28, 2019
commit e1c42ce829ec68f8e4da4871a1a435daa8f4ebed
36 changes: 21 additions & 15 deletions Lib/test/test___all__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,21 +30,27 @@ def check_all(self, modname):
raise NoAll(modname)
names = {}
with self.subTest(module=modname):
try:
exec("from %s import *" % modname, names)
except Exception as e:
# Include the module name in the exception string
self.fail("__all__ failure in {}: {}: {}".format(
modname, e.__class__.__name__, e))
if "__builtins__" in names:
del names["__builtins__"]
if '__annotations__' in names:
del names['__annotations__']
keys = set(names)
all_list = sys.modules[modname].__all__
all_set = set(all_list)
self.assertCountEqual(all_set, all_list, "in module {}".format(modname))
self.assertEqual(keys, all_set, "in module {}".format(modname))
with support.check_warnings(
("", DeprecationWarning),
("", ResourceWarning),
quiet=True):
try:
exec("from %s import *" % modname, names)
except Exception as e:
# Include the module name in the exception string
self.fail("__all__ failure in {}: {}: {}".format(
modname, e.__class__.__name__, e))
if "__builtins__" in names:
del names["__builtins__"]
if '__annotations__' in names:
del names['__annotations__']
if "__warningregistry__" in names:
del names["__warningregistry__"]
keys = set(names)
all_list = sys.modules[modname].__all__
all_set = set(all_list)
self.assertCountEqual(all_set, all_list, "in module {}".format(modname))
self.assertEqual(keys, all_set, "in module {}".format(modname))

def walk_modules(self, basedir, modpath):
for fn in sorted(os.listdir(basedir)):
Expand Down