aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZac Medico <zmedico@gentoo.org>2018-09-03 13:16:37 -0700
committerZac Medico <zmedico@gentoo.org>2018-09-03 13:20:40 -0700
commitb37256a524a0fbf88ffad20c9f01aaf37409ec66 (patch)
tree280194110590abf412a6879bfbb00952209cbc2c
parentUpdates for portage-2.3.49 release (diff)
downloadportage-b37256a5.tar.gz
portage-b37256a5.tar.bz2
portage-b37256a5.zip
compat_coroutine: CancelledError cancels coroutine's future
-rw-r--r--lib/portage/tests/util/futures/test_compat_coroutine.py8
-rw-r--r--lib/portage/util/futures/compat_coroutine.py2
2 files changed, 8 insertions, 2 deletions
diff --git a/lib/portage/tests/util/futures/test_compat_coroutine.py b/lib/portage/tests/util/futures/test_compat_coroutine.py
index cbc070869..b6f75b1a2 100644
--- a/lib/portage/tests/util/futures/test_compat_coroutine.py
+++ b/lib/portage/tests/util/futures/test_compat_coroutine.py
@@ -71,6 +71,10 @@ class CompatCoroutineTestCase(TestCase):
loop.run_until_complete, future)
def test_cancelled_future(self):
+ """
+ When a coroutine raises CancelledError, the coroutine's
+ future is cancelled.
+ """
@coroutine
def cancelled_future_coroutine(loop=None):
@@ -81,8 +85,8 @@ class CompatCoroutineTestCase(TestCase):
yield future
loop = asyncio.get_event_loop()
- self.assertRaises(asyncio.CancelledError,
- loop.run_until_complete, cancelled_future_coroutine(loop=loop))
+ future = loop.run_until_complete(asyncio.wait([cancelled_future_coroutine()]))[0].pop()
+ self.assertTrue(future.cancelled())
def test_yield_expression_result(self):
@coroutine
diff --git a/lib/portage/util/futures/compat_coroutine.py b/lib/portage/util/futures/compat_coroutine.py
index 59fdc31b6..3edfa6bee 100644
--- a/lib/portage/util/futures/compat_coroutine.py
+++ b/lib/portage/util/futures/compat_coroutine.py
@@ -102,6 +102,8 @@ class _GeneratorTask(object):
self._generator.throw(previous.exception())
future = next(self._generator)
+ except asyncio.CancelledError:
+ self._result.cancel()
except _CoroutineReturnValue as e:
if not self._result.cancelled():
self._result.set_result(e.result)