From 28cbe77676e17045c762167bd7a5aa4d2f916dc4 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Andr=C3=A9s=20=C3=81lvarez?= <andresalvarez353@gmail.com>
Date: Thu, 3 Dec 2020 15:06:42 -0600
Subject: [PATCH] Add repr to Cookies for displaying available cookies (#1411)

* Add repr to Cookies for displaying available cookies

* Add unit test

* Simplify repr

* Remove file
---
 httpx/_models.py             | 10 ++++++++++
 tests/models/test_cookies.py | 11 +++++++++++
 2 files changed, 21 insertions(+)

diff --git a/httpx/_models.py b/httpx/_models.py
index 3310ff51..a77f7a57 100644
--- a/httpx/_models.py
+++ b/httpx/_models.py
@@ -1491,6 +1491,16 @@ class Cookies(MutableMapping):
             return True
         return False
 
+    def __repr__(self) -> str:
+        cookies_repr = ", ".join(
+            [
+                f"<Cookie {cookie.name}={cookie.value} for {cookie.domain} />"
+                for cookie in self.jar
+            ]
+        )
+
+        return f"<Cookies[{cookies_repr}]>"
+
     class _CookieCompatRequest(urllib.request.Request):
         """
         Wraps a `Request` instance up in a compatibility interface suitable
diff --git a/tests/models/test_cookies.py b/tests/models/test_cookies.py
index 98027812..dbe1bfb9 100644
--- a/tests/models/test_cookies.py
+++ b/tests/models/test_cookies.py
@@ -85,3 +85,14 @@ def test_cookies_can_be_a_list_of_tuples():
     assert len(cookies.items()) == 2
     for k, v in cookies_val:
         assert cookies[k] == v
+
+
+def test_cookies_repr():
+    cookies = httpx.Cookies()
+    cookies.set(name="foo", value="bar", domain="http://blah.com")
+    cookies.set(name="fizz", value="buzz", domain="http://hello.com")
+
+    assert (
+        repr(cookies)
+        == "<Cookies[<Cookie foo=bar for http://blah.com />, <Cookie fizz=buzz for http://hello.com />]>"
+    )
-- 
2.47.3