Skip to content

bpo-29679: Implement @contextlib.asynccontextmanager #360

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

Merged
1st1 merged 18 commits into from
May 1, 2017
+343 −6
Merged

bpo-29679: Implement @contextlib.asynccontextmanager #360

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
904e8a8
implement @asynccontextmanager
JelleZijlstra Jan 19, 2017
b3d59f1
add it to __all__
JelleZijlstra Feb 28, 2017
a1d5b3f
add tests (duplicating the @contextmanager ones)
JelleZijlstra Feb 28, 2017
c5b8b43
add docs
JelleZijlstra Mar 1, 2017
ca77cd2
back out ContextDecorator for asynccontextmanager (it doesn't work); …
JelleZijlstra Mar 1, 2017
689f4a5
move asynccontextmanager tests into their own file
JelleZijlstra Mar 1, 2017
299d968
fix when tests are run after test_asyncio
JelleZijlstra Mar 2, 2017
e974d48
add a few more tests
JelleZijlstra Mar 2, 2017
5808a4c
combine duplicate tests
JelleZijlstra Mar 2, 2017
9caa243
additional test for RuntimeError wrapping
JelleZijlstra Mar 2, 2017
64e6908
address 1st1's comments
JelleZijlstra Mar 2, 2017
178433b
clean up "except:" and explain why we can't do that for @contextmanager
JelleZijlstra Mar 3, 2017
6d0dddb
old-style classes, not strings
JelleZijlstra Mar 3, 2017
ad65b4d
add to whatsnew (and alphabetize modules)
JelleZijlstra Mar 3, 2017
737fd0f
add wraps decorator, add to docstring
JelleZijlstra Mar 3, 2017
3fc20a7
unalphabetize whatsnew
JelleZijlstra Mar 3, 2017
06697a8
Merge branch 'master' into asynccontextmanager
1st1 Mar 8, 2017
bb8de0d
Merge branch 'master' into asynccontextmanager
JelleZijlstra Apr 30, 2017
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
add a few more tests
  • Loading branch information
@JelleZijlstra
JelleZijlstra committed Mar 2, 2017
commit e974d480aafb3bfe770911cd89dcbcfe6d8857b0
37 changes: 36 additions & 1 deletion Lib/test/test_contextlib_async.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ async def whee():
yield
ctx = whee()
await ctx.__aenter__()
# Calling __exit__ should not result in an exception
# Calling __aexit__ should not result in an exception
self.assertFalse(await ctx.__aexit__(TypeError, TypeError("foo"), None))

@_async_test
Expand All @@ -70,6 +70,41 @@ async def whoo():
with self.assertRaises(RuntimeError):
await ctx.__aexit__(TypeError, TypeError('foo'), None)

@_async_test
async def test_contextmanager_trap_no_yield(self):
@asynccontextmanager
async def whoo():
if False:
yield
ctx = whoo()
with self.assertRaises(RuntimeError):
await ctx.__aenter__()

@_async_test
async def test_contextmanager_trap_second_yield(self):
@asynccontextmanager
async def whoo():
yield
yield
ctx = whoo()
await ctx.__aenter__()
with self.assertRaises(RuntimeError):
await ctx.__aexit__(None, None, None)

@_async_test
async def test_contextmanager_non_normalised(self):
@asynccontextmanager
async def whoo():
try:
yield
except RuntimeError:
raise SyntaxError

ctx = whoo()
await ctx.__aenter__()
with self.assertRaises(SyntaxError):
await ctx.__aexit__(RuntimeError, None, None)

@_async_test
async def test_contextmanager_except(self):
state = []
Expand Down