[3.6] Refactor the test to fail instead of hanging (#4225) (#4229)

(cherry picked from commit c7d841d)

Co-authored-by: Andrew Svetlov <andrew.svetlov@gmail.com>

tests/test_web_websocket_functional.py

@@ -246,18 +246,17 @@ async def handler(request):
 
 async def test_close_timeout(loop, aiohttp_client) -> None:
     aborted = loop.create_future()
+    elapsed = 1e10  # something big
 
     async def handler(request):
+        nonlocal elapsed
         ws = web.WebSocketResponse(timeout=0.1)
         await ws.prepare(request)
         assert 'request' == (await ws.receive_str())
         await ws.send_str('reply')
         begin = ws._loop.time()
         assert (await ws.close())
         elapsed = ws._loop.time() - begin
-        assert elapsed < 0.201, \
-            'close() should have returned before ' \
-            'at most 2x timeout.'
         assert ws.close_code == 1006
         assert isinstance(ws.exception(), asyncio.TimeoutError)
         aborted.set_result(1)
@@ -294,6 +293,10 @@ async def handler(request):
     await asyncio.sleep(0.08, loop=loop)
     assert (await aborted)
 
+    assert elapsed < 0.201, \
+        'close() should have returned before ' \
+        'at most 2x timeout.'
+
     await ws.close()