[slack] Update resource endpoints

This commit updates the resource endpoints
to fetch the messages and information of a
channel. The endpoints changed are as follows:
* channels.history -> conversations.history
* channels.info -> conversations.info

This change is required since the `channels.*`
menthods are deprecated since Jan 7, 2020.
Slack suggests the usage of 'conversations.*'
methods as the alternative.
Ref: https://api.slack.com/changelog/2020-01-deprecating-antecedents-to-the-conversations-api

The tests have been updated accordingly.
The backend version is now 0.9.1

Signed-off-by: Animesh Kumar <[email protected]>

Author: animeshk08

perceval/backends/core/slack.py

@@ -61,7 +61,7 @@ class Slack(Backend):
     :param archive: archive to store/retrieve items
     :param ssl_verify: enable/disable SSL verification
     """
-    version = '0.9.0'
+    version = '0.9.1'
 
     CATEGORIES = [CATEGORY_MESSAGE]
     EXTRA_SEARCH_FIELDS = {
@@ -302,9 +302,9 @@ class SlackClient(HttpClient):
     URL = urijoin(SLACK_URL, 'api', '%(resource)s')
 
     AUTHORIZATION_HEADER = 'Authorization'
-    RCONVERSATION_INFO = 'conversations.members'
-    RCHANNEL_INFO = 'channels.info'
-    RCHANNEL_HISTORY = 'channels.history'
+    RCONVERSATION_MEMBERS = 'conversations.members'
+    RCONVERSATION_INFO = 'conversations.info'
+    RCONVERSATION_HISTORY = 'conversations.history'
     RUSER_INFO = 'users.info'
 
     PCHANNEL = 'channel'
@@ -327,7 +327,7 @@ def conversation_members(self, conversation):
         """
         members = 0
 
-        resource = self.RCONVERSATION_INFO
+        resource = self.RCONVERSATION_MEMBERS
 
         params = {
             self.PCHANNEL: conversation,
@@ -348,7 +348,7 @@ def conversation_members(self, conversation):
     def channel_info(self, channel):
         """Fetch information about a channel."""
 
-        resource = self.RCHANNEL_INFO
+        resource = self.RCONVERSATION_INFO
 
         params = {
             self.PCHANNEL: channel,
@@ -361,7 +361,7 @@ def channel_info(self, channel):
     def history(self, channel, oldest=None, latest=None):
         """Fetch the history of a channel."""
 
-        resource = self.RCHANNEL_HISTORY
+        resource = self.RCONVERSATION_HISTORY
 
         params = {
             self.PCHANNEL: channel,

tests/test_slack.py

@@ -44,11 +44,11 @@
 from base import TestCaseBackendArchive
 
 
-SLACK_API_URL = 'https://slack.com/api'
-SLACK_CHANNEL_INFO_URL = SLACK_API_URL + '/channels.info'
-SLACK_CHANNEL_HISTORY_URL = SLACK_API_URL + '/channels.history'
-SLACK_CONVERSATION_MEMBERS = SLACK_API_URL + '/conversations.members'
-SLACK_USER_INFO_URL = SLACK_API_URL + '/users.info'
+SLACK_API_URL = 'https://slack.com/api/'
+SLACK_CHANNEL_INFO_URL = SLACK_API_URL + SlackClient.RCONVERSATION_INFO
+SLACK_CHANNEL_HISTORY_URL = SLACK_API_URL + SlackClient.RCONVERSATION_HISTORY
+SLACK_CONVERSATION_MEMBERS = SLACK_API_URL + SlackClient.RCONVERSATION_MEMBERS
+SLACK_USER_INFO_URL = SLACK_API_URL + SlackClient.RUSER_INFO
 
 
 def read_file(filename, mode='r'):
@@ -703,7 +703,7 @@ def test_conversation_members(self):
 
         req = http_requests[0]
         self.assertEqual(req.method, 'GET')
-        self.assertRegex(req.path, '/conversations.members')
+        self.assertRegex(req.path, SlackClient.RCONVERSATION_MEMBERS)
 
         for i in range(len(expected)):
             self.assertIn((SlackClient.AUTHORIZATION_HEADER, 'Bearer aaaa'), http_requests[i].headers._headers)
@@ -727,7 +727,7 @@ def test_channel_info(self):
 
         req = http_requests[0]
         self.assertEqual(req.method, 'GET')
-        self.assertRegex(req.path, '/channels.info')
+        self.assertRegex(req.path, SlackClient.RCONVERSATION_INFO)
         self.assertDictEqual(req.querystring, expected)
         self.assertIn((SlackClient.AUTHORIZATION_HEADER, 'Bearer aaaa'), req.headers._headers)
 
@@ -754,7 +754,7 @@ def test_history(self):
 
         req = http_requests[0]
         self.assertEqual(req.method, 'GET')
-        self.assertRegex(req.path, '/channels.history')
+        self.assertRegex(req.path, SlackClient.RCONVERSATION_HISTORY)
         self.assertDictEqual(req.querystring, expected)
         self.assertIn((SlackClient.AUTHORIZATION_HEADER, 'Bearer aaaa'), req.headers._headers)
 
@@ -781,7 +781,7 @@ def test_history_format_latest(self):
 
         req = http_requests[0]
         self.assertEqual(req.method, 'GET')
-        self.assertRegex(req.path, '/channels.history')
+        self.assertRegex(req.path, SlackClient.RCONVERSATION_HISTORY)
         self.assertDictEqual(req.querystring, expected)
         self.assertIn((SlackClient.AUTHORIZATION_HEADER, 'Bearer aaaa'), req.headers._headers)
 
@@ -804,7 +804,7 @@ def test_user(self):
 
         req = http_requests[0]
         self.assertEqual(req.method, 'GET')
-        self.assertRegex(req.path, '/users.info')
+        self.assertRegex(req.path, SlackClient.RUSER_INFO)
         self.assertDictEqual(req.querystring, expected)
         self.assertIn((SlackClient.AUTHORIZATION_HEADER, 'Bearer aaaa'), req.headers._headers)