changeset:   104953:a02915e4c165
branch:      3.6
parent:      104950:6811df9e9797
parent:      104952:0669dcf1eb36
user:        Yury Selivanov <yury@magic.io>
date:        Mon Nov 07 16:07:30 2016 -0500
files:       Lib/asyncio/base_futures.py Lib/asyncio/futures.py Lib/test/test_asyncio/test_futures.py Misc/NEWS
description:
Merge 3.5 (issue #28634)


diff -r 6811df9e9797 -r a02915e4c165 Lib/asyncio/base_futures.py
--- a/Lib/asyncio/base_futures.py	Mon Nov 07 15:35:45 2016 -0500
+++ b/Lib/asyncio/base_futures.py	Mon Nov 07 16:07:30 2016 -0500
@@ -27,7 +27,8 @@
     itself as duck-type compatible by setting _asyncio_future_blocking.
     See comment in Future for more details.
     """
-    return getattr(obj, '_asyncio_future_blocking', None) is not None
+    return (hasattr(obj.__class__, '_asyncio_future_blocking') and
+            obj._asyncio_future_blocking is not None)
 
 
 def _format_callbacks(cb):
diff -r 6811df9e9797 -r a02915e4c165 Lib/test/test_asyncio/test_futures.py
--- a/Lib/test/test_asyncio/test_futures.py	Mon Nov 07 15:35:45 2016 -0500
+++ b/Lib/test/test_asyncio/test_futures.py	Mon Nov 07 16:07:30 2016 -0500
@@ -105,6 +105,29 @@
         self.loop = self.new_test_loop()
         self.addCleanup(self.loop.close)
 
+    def test_isfuture(self):
+        class MyFuture:
+            _asyncio_future_blocking = None
+
+            def __init__(self):
+                self._asyncio_future_blocking = False
+
+        self.assertFalse(asyncio.isfuture(MyFuture))
+        self.assertTrue(asyncio.isfuture(MyFuture()))
+        self.assertFalse(asyncio.isfuture(1))
+
+        # As `isinstance(Mock(), Future)` returns `False`
+        self.assertFalse(asyncio.isfuture(mock.Mock()))
+
+        f = self._new_future(loop=self.loop)
+        self.assertTrue(asyncio.isfuture(f))
+        self.assertFalse(asyncio.isfuture(type(f)))
+
+        # As `isinstance(Mock(Future), Future)` returns `True`
+        self.assertTrue(asyncio.isfuture(mock.Mock(type(f))))
+
+        f.cancel()
+
     def test_initial_state(self):
         f = self._new_future(loop=self.loop)
         self.assertFalse(f.cancelled())
diff -r 6811df9e9797 -r a02915e4c165 Misc/NEWS
--- a/Misc/NEWS	Mon Nov 07 15:35:45 2016 -0500
+++ b/Misc/NEWS	Mon Nov 07 16:07:30 2016 -0500
@@ -25,6 +25,8 @@
 - Issue #28613: Fix get_event_loop() return the current loop if
   called from coroutines/callbacks.
 
+- Issue #28634: Fix asyncio.isfuture() to support unittest.Mock.
+
 Documentation
 -------------