2017-05-25 19:48:36 +02:00
|
|
|
# -*- coding: utf-8 -*-
|
|
|
|
|
2017-10-20 17:24:09 +02:00
|
|
|
import json
|
2017-05-25 19:48:36 +02:00
|
|
|
import mock
|
|
|
|
from typing import Any, Union, Mapping, Callable
|
|
|
|
|
2017-09-26 23:55:15 +02:00
|
|
|
from zerver.lib.actions import (
|
|
|
|
do_create_user,
|
|
|
|
get_service_bot_events,
|
|
|
|
)
|
2017-10-16 18:38:46 +02:00
|
|
|
from zerver.lib.bot_lib import StateHandler, StateHandlerError
|
2017-05-25 19:48:36 +02:00
|
|
|
from zerver.lib.test_classes import ZulipTestCase
|
|
|
|
from zerver.models import (
|
2017-08-25 05:20:26 +02:00
|
|
|
get_realm,
|
2017-10-16 18:38:46 +02:00
|
|
|
BotUserStateData,
|
2017-05-25 19:48:36 +02:00
|
|
|
UserProfile,
|
|
|
|
Recipient,
|
|
|
|
)
|
|
|
|
|
2017-05-25 20:01:31 +02:00
|
|
|
BOT_TYPE_TO_QUEUE_NAME = {
|
|
|
|
UserProfile.OUTGOING_WEBHOOK_BOT: 'outgoing_webhooks',
|
|
|
|
UserProfile.EMBEDDED_BOT: 'embedded_bots',
|
|
|
|
}
|
|
|
|
|
2017-09-26 23:55:15 +02:00
|
|
|
class TestServiceBotBasics(ZulipTestCase):
|
|
|
|
def _get_outgoing_bot(self):
|
|
|
|
# type: () -> UserProfile
|
|
|
|
outgoing_bot = do_create_user(
|
|
|
|
email="bar-bot@zulip.com",
|
|
|
|
password="test",
|
|
|
|
realm=get_realm("zulip"),
|
|
|
|
full_name="BarBot",
|
|
|
|
short_name='bb',
|
|
|
|
bot_type=UserProfile.OUTGOING_WEBHOOK_BOT,
|
|
|
|
bot_owner=self.example_user('cordelia'),
|
|
|
|
)
|
|
|
|
|
|
|
|
return outgoing_bot
|
|
|
|
|
|
|
|
def test_service_events_for_pms(self):
|
|
|
|
# type: () -> None
|
|
|
|
sender = self.example_user('hamlet')
|
|
|
|
assert(not sender.is_bot)
|
|
|
|
|
|
|
|
outgoing_bot = self._get_outgoing_bot()
|
|
|
|
|
|
|
|
event_dict = get_service_bot_events(
|
|
|
|
sender=sender,
|
|
|
|
service_bot_tuples=[
|
|
|
|
(outgoing_bot.id, outgoing_bot.bot_type),
|
|
|
|
],
|
|
|
|
mentioned_user_ids=set(),
|
|
|
|
recipient_type=Recipient.PERSONAL,
|
|
|
|
)
|
|
|
|
|
|
|
|
expected = dict(
|
|
|
|
outgoing_webhooks=[
|
|
|
|
dict(trigger='private_message', user_profile_id=outgoing_bot.id),
|
|
|
|
],
|
|
|
|
)
|
|
|
|
|
|
|
|
self.assertEqual(event_dict, expected)
|
|
|
|
|
|
|
|
def test_service_events_for_stream_mentions(self):
|
|
|
|
# type: () -> None
|
|
|
|
sender = self.example_user('hamlet')
|
|
|
|
assert(not sender.is_bot)
|
|
|
|
|
|
|
|
outgoing_bot = self._get_outgoing_bot()
|
|
|
|
|
|
|
|
event_dict = get_service_bot_events(
|
|
|
|
sender=sender,
|
|
|
|
service_bot_tuples=[
|
|
|
|
(outgoing_bot.id, outgoing_bot.bot_type),
|
|
|
|
],
|
|
|
|
mentioned_user_ids={outgoing_bot.id},
|
|
|
|
recipient_type=Recipient.STREAM,
|
|
|
|
)
|
|
|
|
|
|
|
|
expected = dict(
|
|
|
|
outgoing_webhooks=[
|
|
|
|
dict(trigger='mention', user_profile_id=outgoing_bot.id),
|
|
|
|
],
|
|
|
|
)
|
|
|
|
|
|
|
|
self.assertEqual(event_dict, expected)
|
|
|
|
|
2017-09-27 14:01:12 +02:00
|
|
|
def test_service_events_for_unsubscribed_stream_mentions(self):
|
|
|
|
# type: () -> None
|
|
|
|
sender = self.example_user('hamlet')
|
|
|
|
assert(not sender.is_bot)
|
|
|
|
|
|
|
|
outgoing_bot = self._get_outgoing_bot()
|
|
|
|
|
|
|
|
'''
|
|
|
|
If an outgoing bot is mentioned on a stream message, we will
|
|
|
|
create an event for it even if it is not subscribed to the
|
|
|
|
stream and not part of our original `service_bot_tuples`.
|
|
|
|
|
|
|
|
Note that we add Cordelia as a red herring value that the
|
|
|
|
code should ignore, since she is not a bot.
|
|
|
|
'''
|
|
|
|
|
|
|
|
cordelia = self.example_user('cordelia')
|
|
|
|
|
|
|
|
event_dict = get_service_bot_events(
|
|
|
|
sender=sender,
|
|
|
|
service_bot_tuples=[],
|
|
|
|
mentioned_user_ids={
|
|
|
|
outgoing_bot.id,
|
|
|
|
cordelia.id, # should be excluded, not a service bot
|
|
|
|
},
|
|
|
|
recipient_type=Recipient.STREAM,
|
|
|
|
)
|
|
|
|
|
|
|
|
expected = dict(
|
|
|
|
outgoing_webhooks=[
|
|
|
|
dict(trigger='mention', user_profile_id=outgoing_bot.id),
|
|
|
|
],
|
|
|
|
)
|
|
|
|
|
|
|
|
self.assertEqual(event_dict, expected)
|
|
|
|
|
2017-10-16 18:38:46 +02:00
|
|
|
class TestServiceBotStateHandler(ZulipTestCase):
|
|
|
|
def setUp(self):
|
|
|
|
# type: () -> None
|
|
|
|
self.user_profile = self.example_user("othello")
|
|
|
|
self.bot_profile = do_create_user(email="embedded-bot-1@zulip.com",
|
|
|
|
password="test",
|
|
|
|
realm=get_realm("zulip"),
|
|
|
|
full_name="EmbeddedBo1",
|
|
|
|
short_name="embedded-bot-1",
|
|
|
|
bot_type=UserProfile.EMBEDDED_BOT,
|
|
|
|
bot_owner=self.user_profile)
|
|
|
|
self.second_bot_profile = do_create_user(email="embedded-bot-2@zulip.com",
|
|
|
|
password="test",
|
|
|
|
realm=get_realm("zulip"),
|
|
|
|
full_name="EmbeddedBot2",
|
|
|
|
short_name="embedded-bot-2",
|
|
|
|
bot_type=UserProfile.EMBEDDED_BOT,
|
|
|
|
bot_owner=self.user_profile)
|
|
|
|
|
|
|
|
def test_basic_storage_and_retrieval(self):
|
|
|
|
# type: () -> None
|
2017-10-20 17:42:57 +02:00
|
|
|
storage = StateHandler(self.bot_profile)
|
2017-10-20 18:47:06 +02:00
|
|
|
storage.put('some key', 'some value')
|
|
|
|
storage.put('some other key', 'some other value')
|
|
|
|
self.assertEqual(storage.get('some key'), 'some value')
|
|
|
|
self.assertEqual(storage.get('some other key'), 'some other value')
|
|
|
|
self.assertTrue(storage.contains('some key'))
|
|
|
|
self.assertFalse(storage.contains('nonexistent key'))
|
|
|
|
self.assertRaises(BotUserStateData.DoesNotExist, lambda: storage.get('nonexistent key'))
|
2017-10-20 17:42:57 +02:00
|
|
|
|
|
|
|
second_storage = StateHandler(self.second_bot_profile)
|
2017-10-20 18:47:06 +02:00
|
|
|
self.assertRaises(BotUserStateData.DoesNotExist, lambda: second_storage.get('some key'))
|
|
|
|
second_storage.put('some key', 'yet another value')
|
|
|
|
self.assertEqual(storage.get('some key'), 'some value')
|
|
|
|
self.assertEqual(second_storage.get('some key'), 'yet another value')
|
2017-10-16 18:38:46 +02:00
|
|
|
|
2017-10-20 17:24:09 +02:00
|
|
|
def test_marshaling(self):
|
|
|
|
# type: () -> None
|
2017-10-20 17:42:57 +02:00
|
|
|
storage = StateHandler(self.bot_profile)
|
2017-10-20 17:24:09 +02:00
|
|
|
serializable_obj = {'foo': 'bar', 'baz': [42, 'cux']}
|
2017-10-20 18:47:06 +02:00
|
|
|
storage.put('some key', serializable_obj) # type: ignore # Ignore for testing.
|
|
|
|
self.assertEqual(storage.get('some key'), serializable_obj)
|
2017-10-20 17:24:09 +02:00
|
|
|
|
|
|
|
def test_invalid_calls(self):
|
|
|
|
# type: () -> None
|
2017-10-20 17:42:57 +02:00
|
|
|
storage = StateHandler(self.bot_profile)
|
|
|
|
storage.marshal = lambda obj: obj
|
|
|
|
storage.demarshal = lambda obj: obj
|
2017-10-20 17:24:09 +02:00
|
|
|
serializable_obj = {'foo': 'bar', 'baz': [42, 'cux']}
|
|
|
|
with self.assertRaisesMessage(StateHandlerError, "Cannot set state. The value type is "
|
|
|
|
"<class 'dict'>, but it should be str."):
|
2017-10-20 18:47:06 +02:00
|
|
|
storage.put('some key', serializable_obj) # type: ignore # We intend to test an invalid type.
|
2017-10-20 17:24:09 +02:00
|
|
|
with self.assertRaisesMessage(StateHandlerError, "Cannot set state. The key type is "
|
|
|
|
"<class 'dict'>, but it should be str."):
|
2017-10-20 18:47:06 +02:00
|
|
|
storage.put(serializable_obj, 'some value') # type: ignore # We intend to test an invalid type.
|
2017-10-20 17:24:09 +02:00
|
|
|
|
2017-10-16 18:38:46 +02:00
|
|
|
def test_storage_limit(self):
|
|
|
|
# type: () -> None
|
|
|
|
# Reduce maximal state size for faster test string construction.
|
|
|
|
StateHandler.state_size_limit = 100
|
2017-10-20 17:42:57 +02:00
|
|
|
storage = StateHandler(self.bot_profile)
|
2017-10-16 18:38:46 +02:00
|
|
|
key = 'capacity-filling entry'
|
2017-10-20 18:47:06 +02:00
|
|
|
storage.put(key, 'x' * (StateHandler.state_size_limit - len(key)))
|
2017-10-16 18:38:46 +02:00
|
|
|
|
2017-10-20 17:24:09 +02:00
|
|
|
with self.assertRaisesMessage(StateHandlerError, "Cannot set state. Request would require 134 bytes storage. "
|
2017-10-16 18:38:46 +02:00
|
|
|
"The current storage limit is 100."):
|
2017-10-20 18:47:06 +02:00
|
|
|
storage.put('too much data', 'a few bits too long')
|
2017-10-16 18:38:46 +02:00
|
|
|
|
2017-10-20 17:42:57 +02:00
|
|
|
second_storage = StateHandler(self.second_bot_profile)
|
2017-10-20 18:47:06 +02:00
|
|
|
second_storage.put('another big entry', 'x' * (StateHandler.state_size_limit - 40))
|
|
|
|
second_storage.put('normal entry', 'abcd')
|
2017-10-16 18:38:46 +02:00
|
|
|
|
2017-05-25 19:54:06 +02:00
|
|
|
class TestServiceBotEventTriggers(ZulipTestCase):
|
2017-05-25 19:48:36 +02:00
|
|
|
|
|
|
|
def setUp(self):
|
|
|
|
# type: () -> None
|
2017-05-25 19:54:06 +02:00
|
|
|
self.user_profile = self.example_user("othello")
|
|
|
|
self.bot_profile = do_create_user(email="foo-bot@zulip.com",
|
|
|
|
password="test",
|
2017-08-25 05:20:26 +02:00
|
|
|
realm=get_realm("zulip"),
|
2017-05-25 19:54:06 +02:00
|
|
|
full_name="FooBot",
|
|
|
|
short_name="foo-bot",
|
|
|
|
bot_type=UserProfile.OUTGOING_WEBHOOK_BOT,
|
|
|
|
bot_owner=self.user_profile)
|
2017-05-25 20:01:31 +02:00
|
|
|
self.second_bot_profile = do_create_user(email="bar-bot@zulip.com",
|
|
|
|
password="test",
|
2017-08-25 05:20:26 +02:00
|
|
|
realm=get_realm("zulip"),
|
2017-05-25 20:01:31 +02:00
|
|
|
full_name="BarBot",
|
|
|
|
short_name="bar-bot",
|
|
|
|
bot_type=UserProfile.OUTGOING_WEBHOOK_BOT,
|
|
|
|
bot_owner=self.user_profile)
|
2017-05-25 19:54:06 +02:00
|
|
|
|
|
|
|
# TODO: In future versions this won't be required
|
2017-08-25 06:01:29 +02:00
|
|
|
self.subscribe(self.bot_profile, 'Denmark')
|
2017-05-25 19:54:06 +02:00
|
|
|
|
2017-05-25 20:01:31 +02:00
|
|
|
@mock.patch('zerver.lib.actions.queue_json_publish')
|
|
|
|
def test_trigger_on_stream_mention_from_user(self, mock_queue_json_publish):
|
|
|
|
# type: (mock.Mock) -> None
|
|
|
|
for bot_type, expected_queue_name in BOT_TYPE_TO_QUEUE_NAME.items():
|
|
|
|
self.bot_profile.bot_type = bot_type
|
|
|
|
self.bot_profile.save()
|
|
|
|
|
|
|
|
content = u'@**FooBot** foo bar!!!'
|
|
|
|
recipient = 'Denmark'
|
|
|
|
trigger = 'mention'
|
|
|
|
message_type = Recipient._type_names[Recipient.STREAM]
|
|
|
|
|
2017-10-18 19:51:27 +02:00
|
|
|
def check_values_passed(queue_name, trigger_event, x, call_consume_in_tests):
|
|
|
|
# type: (Any, Union[Mapping[Any, Any], Any], Callable[[Any], None], bool) -> None
|
2017-05-25 20:01:31 +02:00
|
|
|
self.assertEqual(queue_name, expected_queue_name)
|
|
|
|
self.assertEqual(trigger_event["failed_tries"], 0)
|
|
|
|
self.assertEqual(trigger_event["message"]["content"], content)
|
|
|
|
self.assertEqual(trigger_event["message"]["display_recipient"], recipient)
|
|
|
|
self.assertEqual(trigger_event["message"]["sender_email"], self.user_profile.email)
|
|
|
|
self.assertEqual(trigger_event["message"]["type"], message_type)
|
|
|
|
self.assertEqual(trigger_event['trigger'], trigger)
|
|
|
|
self.assertEqual(trigger_event['user_profile_id'], self.bot_profile.id)
|
|
|
|
mock_queue_json_publish.side_effect = check_values_passed
|
|
|
|
|
|
|
|
self.send_message(
|
|
|
|
self.user_profile.email,
|
|
|
|
'Denmark',
|
|
|
|
Recipient.STREAM,
|
|
|
|
content)
|
|
|
|
self.assertTrue(mock_queue_json_publish.called)
|
|
|
|
|
|
|
|
@mock.patch('zerver.lib.actions.queue_json_publish')
|
|
|
|
def test_no_trigger_on_stream_message_without_mention(self, mock_queue_json_publish):
|
|
|
|
# type: (mock.Mock) -> None
|
|
|
|
sender_email = self.user_profile.email
|
2017-05-25 19:48:36 +02:00
|
|
|
recipients = "Denmark"
|
|
|
|
message_type = Recipient.STREAM
|
2017-05-25 20:01:31 +02:00
|
|
|
self.send_message(sender_email, recipients, message_type)
|
|
|
|
self.assertFalse(mock_queue_json_publish.called)
|
2017-05-25 19:48:36 +02:00
|
|
|
|
2017-05-25 20:01:31 +02:00
|
|
|
@mock.patch('zerver.lib.actions.queue_json_publish')
|
|
|
|
def test_no_trigger_on_stream_mention_from_bot(self, mock_queue_json_publish):
|
|
|
|
# type: (mock.Mock) -> None
|
|
|
|
for bot_type in BOT_TYPE_TO_QUEUE_NAME:
|
|
|
|
self.bot_profile.bot_type = bot_type
|
|
|
|
self.bot_profile.save()
|
2017-05-25 19:48:36 +02:00
|
|
|
|
2017-05-25 20:01:31 +02:00
|
|
|
self.send_message(
|
|
|
|
self.second_bot_profile.email,
|
|
|
|
'Denmark',
|
|
|
|
Recipient.STREAM,
|
|
|
|
u'@**FooBot** foo bar!!!')
|
|
|
|
self.assertFalse(mock_queue_json_publish.called)
|
2017-05-25 19:48:36 +02:00
|
|
|
|
|
|
|
@mock.patch('zerver.lib.actions.queue_json_publish')
|
2017-05-25 19:54:06 +02:00
|
|
|
def test_trigger_on_personal_message_from_user(self, mock_queue_json_publish):
|
2017-05-25 19:48:36 +02:00
|
|
|
# type: (mock.Mock) -> None
|
2017-05-25 20:01:31 +02:00
|
|
|
for bot_type, expected_queue_name in BOT_TYPE_TO_QUEUE_NAME.items():
|
|
|
|
self.bot_profile.bot_type = bot_type
|
|
|
|
self.bot_profile.save()
|
|
|
|
|
|
|
|
sender_email = self.user_profile.email
|
|
|
|
recipient_email = self.bot_profile.email
|
|
|
|
message_type = Recipient.PERSONAL
|
|
|
|
|
2017-10-18 19:51:27 +02:00
|
|
|
def check_values_passed(queue_name, trigger_event, x, call_consume_in_tests):
|
|
|
|
# type: (Any, Union[Mapping[Any, Any], Any], Callable[[Any], None], bool) -> None
|
2017-05-25 20:01:31 +02:00
|
|
|
self.assertEqual(queue_name, expected_queue_name)
|
|
|
|
self.assertEqual(trigger_event["user_profile_id"], self.bot_profile.id)
|
|
|
|
self.assertEqual(trigger_event["trigger"], "private_message")
|
|
|
|
self.assertEqual(trigger_event["failed_tries"], 0)
|
|
|
|
self.assertEqual(trigger_event["message"]["sender_email"], sender_email)
|
|
|
|
display_recipients = [
|
|
|
|
trigger_event["message"]["display_recipient"][0]["email"],
|
|
|
|
trigger_event["message"]["display_recipient"][1]["email"],
|
|
|
|
]
|
|
|
|
self.assertTrue(sender_email in display_recipients)
|
|
|
|
self.assertTrue(recipient_email in display_recipients)
|
|
|
|
mock_queue_json_publish.side_effect = check_values_passed
|
|
|
|
|
|
|
|
self.send_message(sender_email, recipient_email, message_type, subject='', content='test')
|
|
|
|
self.assertTrue(mock_queue_json_publish.called)
|
2017-05-25 19:48:36 +02:00
|
|
|
|
2017-05-25 20:02:13 +02:00
|
|
|
@mock.patch('zerver.lib.actions.queue_json_publish')
|
|
|
|
def test_no_trigger_on_personal_message_from_bot(self, mock_queue_json_publish):
|
|
|
|
# type: (mock.Mock) -> None
|
|
|
|
for bot_type in BOT_TYPE_TO_QUEUE_NAME:
|
|
|
|
self.bot_profile.bot_type = bot_type
|
|
|
|
self.bot_profile.save()
|
|
|
|
|
|
|
|
sender_email = self.second_bot_profile.email
|
|
|
|
recipient_email = self.bot_profile.email
|
|
|
|
message_type = Recipient.PERSONAL
|
|
|
|
self.send_message(sender_email, recipient_email, message_type)
|
|
|
|
self.assertFalse(mock_queue_json_publish.called)
|
|
|
|
|
2017-05-25 19:48:36 +02:00
|
|
|
@mock.patch('zerver.lib.actions.queue_json_publish')
|
2017-05-25 19:54:06 +02:00
|
|
|
def test_trigger_on_huddle_message_from_user(self, mock_queue_json_publish):
|
2017-05-25 19:48:36 +02:00
|
|
|
# type: (mock.Mock) -> None
|
2017-05-25 20:01:31 +02:00
|
|
|
for bot_type, expected_queue_name in BOT_TYPE_TO_QUEUE_NAME.items():
|
|
|
|
self.bot_profile.bot_type = bot_type
|
|
|
|
self.bot_profile.save()
|
|
|
|
|
|
|
|
self.second_bot_profile.bot_type = bot_type
|
|
|
|
self.second_bot_profile.save()
|
|
|
|
|
|
|
|
sender_email = self.user_profile.email
|
|
|
|
recipient_emails = [self.bot_profile.email, self.second_bot_profile.email]
|
|
|
|
message_type = Recipient.HUDDLE
|
|
|
|
profile_ids = [self.bot_profile.id, self.second_bot_profile.id]
|
|
|
|
|
2017-10-18 19:51:27 +02:00
|
|
|
def check_values_passed(queue_name, trigger_event, x, call_consume_in_tests):
|
|
|
|
# type: (Any, Union[Mapping[Any, Any], Any], Callable[[Any], None], bool) -> None
|
2017-05-25 20:01:31 +02:00
|
|
|
self.assertEqual(queue_name, expected_queue_name)
|
|
|
|
self.assertIn(trigger_event["user_profile_id"], profile_ids)
|
|
|
|
profile_ids.remove(trigger_event["user_profile_id"])
|
|
|
|
self.assertEqual(trigger_event["trigger"], "private_message")
|
|
|
|
self.assertEqual(trigger_event["failed_tries"], 0)
|
|
|
|
self.assertEqual(trigger_event["message"]["sender_email"], sender_email)
|
|
|
|
self.assertEqual(trigger_event["message"]["type"], u'private')
|
|
|
|
mock_queue_json_publish.side_effect = check_values_passed
|
|
|
|
|
|
|
|
self.send_message(sender_email, recipient_emails, message_type, subject='', content='test')
|
|
|
|
self.assertEqual(mock_queue_json_publish.call_count, 2)
|
|
|
|
mock_queue_json_publish.reset_mock()
|
2017-05-25 20:02:13 +02:00
|
|
|
|
|
|
|
@mock.patch('zerver.lib.actions.queue_json_publish')
|
|
|
|
def test_no_trigger_on_huddle_message_from_bot(self, mock_queue_json_publish):
|
|
|
|
# type: (mock.Mock) -> None
|
|
|
|
for bot_type in BOT_TYPE_TO_QUEUE_NAME:
|
|
|
|
self.bot_profile.bot_type = bot_type
|
|
|
|
self.bot_profile.save()
|
|
|
|
|
|
|
|
sender_email = self.second_bot_profile.email
|
|
|
|
recipient_emails = [self.user_profile.email, self.bot_profile.email]
|
|
|
|
message_type = Recipient.HUDDLE
|
|
|
|
self.send_message(sender_email, recipient_emails, message_type)
|
|
|
|
self.assertFalse(mock_queue_json_publish.called)
|