Merge pull request #38 from nvllsvm/link

Change HTTPResponse.links to return empty list when Link header is not present

docs/history.rst

@@ -1,6 +1,10 @@
 Version History
 ===============
 
+Next
+----
+- Change ``HTTPResponse.links`` to return empty list when ``Link`` header is not present
+
 `2.4.1`_ Nov 30 2020
 --------------------
 - Make request retry timeout configurable

sprockets/mixins/http/__init__.py

@@ -152,8 +152,8 @@ class HTTPResponse:
         """
         if not self._responses:
             return None
-        if 'Link' in self._responses[-1].headers:
         links = []
+        if 'Link' in self._responses[-1].headers:
             for l in headers.parse_link(self._responses[-1].headers['Link']):
                 link = {'target': l.target}
                 link.update({k: v for (k, v) in l.parameters})

tests.py

@@ -258,7 +258,7 @@ class MixinTestCase(testing.AsyncHTTPTestCase):
             'foo': ['bar'],
             'status_code': ['200']
         })
-        self.assertIsNone(response.links)
+        self.assertEqual(response.links, [])
 
     @testing.gen_test
     def test_post(self):