Removing unnecessary log statements and more test cleanup

Rodney Urquhart · Rodney Urquhart · commit a9922fe1bc62 · 2019-04-22T03:55:37.000-07:00
diff --git a/slack/rtm/client.py b/slack/rtm/client.py
@@ -350,9 +350,6 @@ async def _read_messages(self):
             async for message in self._websocket:
                 if message.type == aiohttp.WSMsgType.TEXT:
                     payload = message.json()
-                    self._logger.debug(
-                        f"The Websocket received a new message. {payload}"
-                    )
                     event = payload.pop("type", "Unknown")
                     self._dispatch_event(event, data=payload)
                 elif message.type == aiohttp.WSMsgType.ERROR:
diff --git a/slack/web/base_client.py b/slack/web/base_client.py
@@ -143,12 +143,6 @@ def api_call(
             "ssl": self.ssl,
             "proxy": self.proxy,
         }
-        self._logger.debug(
-            "Sending a '%s' request to '%s' with the following data: %s",
-            http_verb,
-            api_url,
-            req_args,
-        )
 
         req_future = asyncio.ensure_future(
             self._send(http_verb=http_verb, api_url=api_url, req_args=req_args),
diff --git a/tests/rtm/test_rtm_client_functional.py b/tests/rtm/test_rtm_client_functional.py
@@ -51,7 +51,6 @@ async def on_shutdown(self, app):
     def setUp(self):
         self.loop = asyncio.new_event_loop()
         asyncio.set_event_loop(self.loop)
-        self.stop = self.loop.create_future()
         task = asyncio.ensure_future(self.mock_server(), loop=self.loop)
         self.loop.run_until_complete(asyncio.wait_for(task, 0.1))
         self.client = slack.RTMClient(loop=self.loop, auto_reconnect=False)
diff --git a/tests/web/test_web_client_functional.py b/tests/web/test_web_client_functional.py
@@ -32,6 +32,13 @@ async def handler(self, request):
         assert request.content_type == "application/json"
         return web.json_response({"ok": True})
 
-    def test_api_test(self):
+    def test_requests_with_use_session_turned_off(self):
+        self.client.use_session = False
+        resp = self.client.api_test()
+        assert resp["ok"]
+
+    def test_subsequent_requests_with_a_session_succeeds(self):
+        resp = self.client.api_test()
+        assert resp["ok"]
         resp = self.client.api_test()
         assert resp["ok"]
