Rename 'next' to 'anext' on Response

Author: florimondmanca

docs/api.md

@@ -68,7 +68,7 @@
 * `def .aiter_text()` - **async text iterator**
 * `def .aiter_lines()` - **async text iterator**
 * `def .close()` - **None**
-* `def .next()` - **Response**
+* `def .anext()` - **Response**
 
 ## `Request`
 

httpx/client.py

@@ -476,7 +476,12 @@ async def send_handling_redirects(
 
             if not allow_redirects:
                 response.call_next = functools.partial(
-                    self.send_handling_redirects,
+                    # NOTE: not using 'self.send_handling_redirects' here, because
+                    # 'call_next' must reference a function (instead
+                    # of a method). This ensures that 'inspect.iscoroutinefunction()'
+                    # checks behave properly.
+                    Client.send_handling_redirects,
+                    self,
                     request=request,
                     auth=auth,
                     verify=verify,

httpx/models.py

@@ -1,6 +1,7 @@
 import cgi
 import datetime
 import email.message
+import inspect
 import json as jsonlib
 import typing
 import urllib.request
@@ -922,13 +923,14 @@ async def aiter_raw(self) -> typing.AsyncIterator[bytes]:
                 yield part
             await self.close()
 
-    async def next(self) -> "Response":
+    async def anext(self) -> "Response":
         """
         Get the next response from a redirect response.
         """
         if not self.is_redirect:
             raise NotRedirectResponse()
         assert self.call_next is not None
+        assert inspect.iscoroutinefunction(self.call_next)
         return await self.call_next()
 
     async def close(self) -> None:

tests/client/test_redirects.py

@@ -110,7 +110,7 @@ async def test_no_redirect():
     response = await client.get(url)
     assert response.status_code == 200
     with pytest.raises(NotRedirectResponse):
-        await response.next()
+        await response.anext()
 
 
 @pytest.mark.usefixtures("async_environment")
@@ -151,7 +151,7 @@ async def test_disallow_redirects():
     assert response.is_redirect is True
     assert len(response.history) == 0
 
-    response = await response.next()
+    response = await response.anext()
     assert response.status_code == codes.OK
     assert response.url == URL("https://example.org/")
     assert response.is_redirect is False
@@ -216,7 +216,7 @@ async def test_too_many_redirects_calling_next():
     response = await client.get(url, allow_redirects=False)
     with pytest.raises(TooManyRedirects):
         while response.is_redirect:
-            response = await response.next()
+            response = await response.anext()
 
 
 @pytest.mark.usefixtures("async_environment")
@@ -233,7 +233,7 @@ async def test_redirect_loop_calling_next():
     response = await client.get(url, allow_redirects=False)
     with pytest.raises(RedirectLoop):
         while response.is_redirect:
-            response = await response.next()
+            response = await response.anext()
 
 
 @pytest.mark.usefixtures("async_environment")