Merge pull request #706 from seratch/issue-705-no-param-pagination

Fix #705 by modifying SlackResponse to consider None value in dict

slack/web/slack_response.py

@@ -131,9 +131,11 @@ def __next__(self):
         if self._iteration == 1:
             return self
         if self._next_cursor_is_present(self.data):
-            self.req_args.get("params", {}).update(
-                {"cursor": self.data["response_metadata"]["next_cursor"]}
-            )
+            params = self.req_args.get("params", {})
+            if params is None:
+                params = {}
+            params.update({"cursor": self.data["response_metadata"]["next_cursor"]})
+            self.req_args.update({"params": params})
 
             if self._use_sync_aiohttp:
                 # We no longer recommend going with this way

tests/web/test_web_client.py

@@ -177,3 +177,10 @@ async def test_issue_690_oauth_access_async(self):
         self.assertIsNone(resp["error"])
         with self.assertRaises(err.SlackApiError):
             await self.async_client.oauth_access(client_id="999.999", client_secret="secret", code="codeeeeeeeeee")
+
+    def test_issue_705_no_param_request_pagination(self):
+        self.client.token = "xoxb-users_list_pagination"
+        users = []
+        for page in self.client.users_list():
+            users = users + page["members"]
+        self.assertTrue(len(users) == 4)