]> git.ipfire.org Git - thirdparty/tornado.git/commitdiff
testing: Fix incorrect docs for gen_test 2084/head
authorBen Darnell <ben@bendarnell.com>
Sun, 11 Jun 2017 16:10:19 +0000 (12:10 -0400)
committerBen Darnell <ben@bendarnell.com>
Sun, 11 Jun 2017 17:22:10 +0000 (13:22 -0400)
Pointed out in #1154

tornado/testing.py

index b5ff364193dd7b08519d35af1c4af4aaa561608d..b79ba59283711082b52c75e4bd41c57e379ea43d 100644 (file)
@@ -458,7 +458,7 @@ def gen_test(func=None, timeout=None):
         class MyTest(AsyncHTTPTestCase):
             @gen_test
             def test_something(self):
-                response = yield gen.Task(self.fetch('/'))
+                response = yield self.http_client.fetch(self.get_url('/'))
 
     By default, ``@gen_test`` times out after 5 seconds. The timeout may be
     overridden globally with the ``ASYNC_TEST_TIMEOUT`` environment variable,
@@ -467,7 +467,11 @@ def gen_test(func=None, timeout=None):
         class MyTest(AsyncHTTPTestCase):
             @gen_test(timeout=10)
             def test_something_slow(self):
-                response = yield gen.Task(self.fetch('/'))
+                response = yield self.http_client.fetch(self.get_url('/'))
+
+    Note that ``@gen_test`` is incompatible with `AsyncTestCase.stop`,
+    `AsyncTestCase.wait`, and `AsyncHTTPTestCase.fetch`. Use ``yield
+    self.http_client.fetch(self.get_url())`` as shown above instead.
 
     .. versionadded:: 3.1
        The ``timeout`` argument and ``ASYNC_TEST_TIMEOUT`` environment
@@ -476,6 +480,7 @@ def gen_test(func=None, timeout=None):
     .. versionchanged:: 4.0
        The wrapper now passes along ``*args, **kwargs`` so it can be used
        on functions with arguments.
+
     """
     if timeout is None:
         timeout = get_async_test_timeout()