Revert "Remove reactions to post message (scrapinghub#171)"

This reverts commit f43f44f.

README.md

@@ -124,17 +124,6 @@ def hi(message):
     # react with thumb up emoji
     message.react('+1')
 
-@respond_to('wave')
-def wave(message):
-    # bot adds reaction of :wave: emoji to message
-    message.react('wave')
-
-   from time import sleep
-   sleep(1)
-
-   # bot removes it's reaction of :wave: emoji on message
-   message.unreact('wave')
-
 @respond_to('I love you')
 def love(message):
     message.reply('I love you too!')

README_ja.md

@@ -123,18 +123,6 @@ def hi(message):
     # react with thumb up emoji
     message.react('+1')
 
-@respond_to('wave')
-def wave(message):
-    # bot adds reaction of :wave: emoji to message
-    message.react('wave')
-
-   from time import sleep
-   sleep(1)
-
-   # bot removes it's reaction of :wave: emoji on message
-   message.unreact('wave')
-
-
 @respond_to('I love you')
 def love(message):
     message.reply('I love you too!')

slackbot/dispatcher.py

@@ -282,15 +282,6 @@ def react(self, emojiname):
             channel=self._body['channel'],
             timestamp=self._body['ts'])
 
-    def unreact(self, emojiname):
-        """
-           Un react to a message using the web api
-        """
-        self._client.unreact_to_message(
-            emojiname=emojiname,
-            channel=self._body['channel'],
-            timestamp=self._body['ts'])
-
     @property
     def channel(self):
         return self._client.get_channel(self._body['channel'])

slackbot/slackclient.py

@@ -173,12 +173,6 @@ def react_to_message(self, emojiname, channel, timestamp):
             channel=channel,
             timestamp=timestamp)
 
-    def unreact_to_message(self, emojiname, channel, timestamp):
-        self.webapi.reactions.remove(
-            name=emojiname,
-            channel=channel,
-            timestamp=timestamp)
-
 
 class SlackConnectionError(Exception):
     pass

tests/functional/driver.py

@@ -131,25 +131,14 @@ def wait_for_file_uploaded(self, name, maxwait=60):
         else:
             raise AssertionError('expected to get file "{}", but got nothing'.format(name))
 
-    def ensure_reaction_adding_posted(self, emojiname, maxwait=5):
+    def ensure_reaction_posted(self, emojiname, maxwait=5):
         for _ in range(maxwait):
             time.sleep(1)
             if self._has_reacted(emojiname):
                 break
         else:
             raise AssertionError('expected to get reaction "{}", but got nothing'.format(emojiname))
 
-    def ensure_reaction_removing_posted(self, emojiname, maxwait=5):
-        if not self._has_reacted(emojiname):
-            raise RuntimeError('Have you the reaction "{}" fortesting?'.format(emojiname))
-
-        for _ in range(maxwait):
-            time.sleep(1)
-            if not self._has_reacted(emojiname):
-                break
-        else:
-            raise AssertionError('expected to get reaction "{}", but got nothing'.format(emojiname))
-
     def _send_message_to_bot(self, channel, msg):
         self.clear_events()
         self._start_ts = time.time()

tests/functional/test_functional.py

@@ -159,13 +159,7 @@ def test_bot_listen_to_channel_message(driver):
 
 def test_bot_react_to_channel_message(driver):
     driver.send_channel_message('hey!', tobot=False)
-    driver.ensure_reaction_adding_posted('eggplant')
-
-
-def test_bot_unreact_to_channel_message(driver):
-    driver.send_channel_message('hey!', tobot=False)
-    driver.ensure_reaction_adding_posted('eggplant')
-    driver.ensure_reaction_removing_posted('eggplant')
+    driver.ensure_reaction_posted('eggplant')
 
 
 def test_bot_reply_to_private_channel_message(driver):