changeset:   105131:179e556a50ce
branch:      3.5
parent:      105128:da2ac103d326
user:        Yury Selivanov <yury@magic.io>
date:        Tue Nov 15 15:20:34 2016 -0500
files:       Lib/asyncio/coroutines.py Lib/test/test_asyncio/test_tasks.py Misc/NEWS
description:
Issue #28703: Fix asyncio.iscoroutinefunction to handle Mock objects.


diff -r da2ac103d326 -r 179e556a50ce Lib/asyncio/coroutines.py
--- a/Lib/asyncio/coroutines.py	Tue Nov 15 09:48:06 2016 -0800
+++ b/Lib/asyncio/coroutines.py	Tue Nov 15 15:20:34 2016 -0500
@@ -33,12 +33,16 @@
 
 try:
     _types_coroutine = types.coroutine
+    _types_CoroutineType = types.CoroutineType
 except AttributeError:
+    # Python 3.4
     _types_coroutine = None
+    _types_CoroutineType = None
 
 try:
     _inspect_iscoroutinefunction = inspect.iscoroutinefunction
 except AttributeError:
+    # Python 3.4
     _inspect_iscoroutinefunction = lambda func: False
 
 try:
@@ -238,19 +242,27 @@
             w.__qualname__ = getattr(func, '__qualname__', None)
             return w
 
-    wrapper._is_coroutine = True  # For iscoroutinefunction().
+    wrapper._is_coroutine = _is_coroutine  # For iscoroutinefunction().
     return wrapper
 
 
+# A marker for iscoroutinefunction.
+_is_coroutine = object()
+
+
 def iscoroutinefunction(func):
     """Return True if func is a decorated coroutine function."""
-    return (getattr(func, '_is_coroutine', False) or
+    return (getattr(func, '_is_coroutine', None) is _is_coroutine or
             _inspect_iscoroutinefunction(func))
 
 
 _COROUTINE_TYPES = (types.GeneratorType, CoroWrapper)
 if _CoroutineABC is not None:
     _COROUTINE_TYPES += (_CoroutineABC,)
+if _types_CoroutineType is not None:
+    # Prioritize native coroutine check to speed-up
+    # asyncio.iscoroutine.
+    _COROUTINE_TYPES = (_types_CoroutineType,) + _COROUTINE_TYPES
 
 
 def iscoroutine(obj):
diff -r da2ac103d326 -r 179e556a50ce Lib/test/test_asyncio/test_tasks.py
--- a/Lib/test/test_asyncio/test_tasks.py	Tue Nov 15 09:48:06 2016 -0800
+++ b/Lib/test/test_asyncio/test_tasks.py	Tue Nov 15 15:20:34 2016 -0500
@@ -1376,6 +1376,8 @@
             yield
         self.assertTrue(asyncio.iscoroutinefunction(fn2))
 
+        self.assertFalse(asyncio.iscoroutinefunction(mock.Mock()))
+
     def test_yield_vs_yield_from(self):
         fut = asyncio.Future(loop=self.loop)
 
diff -r da2ac103d326 -r 179e556a50ce Misc/NEWS
--- a/Misc/NEWS	Tue Nov 15 09:48:06 2016 -0800
+++ b/Misc/NEWS	Tue Nov 15 15:20:34 2016 -0500
@@ -465,6 +465,8 @@
 
 - Issue #28653: Fix a refleak in functools.lru_cache.
 
+- Issue #28703: Fix asyncio.iscoroutinefunction to handle Mock objects.
+
 IDLE
 ----