Adding feature to remove reactions

[attzonko]

Per enhancment request #100 this adds a method to the Message class
called remove_reaction() which takes in the emoji_name to be removed as
a parameter.

[knedlsepp]

I couldn't quite get this to work. I tried:

@listen_to('lunch\?$', re.IGNORECASE)
def lunch_reply(message):
    import time
    timeout_seconds = 10
    emojis = [
        "plate_with_cutlery", "b", "pizza", "curry", "bento",
        "stuffed_flatbread", "shell"
    ]
    for emoji in emojis:
        message.react(emoji)

    time.sleep(timeout_seconds)

    for emoji in emojis:
        message.remove_reaction(emoji)

but the reactions seem to stay there.

[seLain]

hi @attzonko , I tested it. The DELETE call at mattermost.py :

def delete_reaction(self, user_id, post_id, emoji_name):
        return self.delete(
            '/{0}/posts/{1}/reactions/{2}'.format(
                user_id, post_id, emoji_name))

returned 404

Was there a missing /users/ url prefix ?

def delete_reaction(self, user_id, post_id, emoji_name):
        return self.delete(
            '/users/{0}/posts/{1}/reactions/{2}'.format(
                user_id, post_id, emoji_name))

I got the DELETE call success by adding the /users/ prefix.

[knedlsepp]

@seLain Adding /users resolves the issue.

[attzonko]

Thanks @seLain for finding my typo 😃 and thank you @knedlsepp for testing this out. i want to figure out why our Travis CI is failing, so I'll keep this pull request open for now so I have something to test with. When I manually run pytest I see it passing so I need to investigate further.