2012-08-31 20:11:15 +02:00
|
|
|
from django.contrib.auth.models import User
|
|
|
|
from django.test import TestCase
|
2012-11-14 20:50:47 +01:00
|
|
|
from django.test.simple import DjangoTestSuiteRunner
|
2012-11-08 00:48:43 +01:00
|
|
|
from django.utils.timezone import now
|
2012-08-31 20:11:15 +02:00
|
|
|
from django.db.models import Q
|
2012-08-28 18:44:51 +02:00
|
|
|
|
2012-10-10 22:53:24 +02:00
|
|
|
from zephyr.models import Message, UserProfile, Stream, Recipient, Subscription, \
|
2013-01-10 22:01:33 +01:00
|
|
|
filter_by_subscriptions, get_display_recipient, Realm, Client
|
2013-01-08 17:44:22 +01:00
|
|
|
from zephyr.tornadoviews import json_get_updates, api_get_messages
|
2012-12-19 20:19:46 +01:00
|
|
|
from zephyr.decorator import RespondAsynchronously, RequestVariableConversionError
|
2012-11-15 19:42:17 +01:00
|
|
|
from zephyr.lib.initial_password import initial_password, initial_api_key
|
2013-01-28 23:06:52 +01:00
|
|
|
from zephyr.lib.actions import do_send_message, gather_subscriptions
|
2013-01-24 20:20:21 +01:00
|
|
|
from zephyr.lib.bugdown import convert
|
2012-08-28 18:44:51 +02:00
|
|
|
|
2012-09-06 20:53:13 +02:00
|
|
|
import simplejson
|
2012-09-04 22:31:56 +02:00
|
|
|
import subprocess
|
2012-11-14 21:22:08 +01:00
|
|
|
import optparse
|
2012-09-27 19:58:42 +02:00
|
|
|
from django.conf import settings
|
2012-10-02 17:47:18 +02:00
|
|
|
import re
|
2013-01-10 19:41:49 +01:00
|
|
|
import sys
|
2013-01-29 19:47:28 +01:00
|
|
|
import random
|
2013-01-10 19:41:49 +01:00
|
|
|
|
|
|
|
try:
|
|
|
|
settings.TEST_SUITE
|
|
|
|
except:
|
|
|
|
print
|
|
|
|
print "ERROR: Test suite only runs correctly with --settings=humbug.test_settings"
|
|
|
|
print
|
|
|
|
sys.exit(1)
|
2012-09-27 19:58:42 +02:00
|
|
|
|
2012-10-02 17:47:18 +02:00
|
|
|
def find_key_by_email(address):
|
|
|
|
from django.core.mail import outbox
|
|
|
|
key_regex = re.compile("accounts/do_confirm/([a-f0-9]{40})>")
|
|
|
|
for message in reversed(outbox):
|
|
|
|
if address in message.to:
|
|
|
|
return key_regex.search(message.body).groups()[0]
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2013-01-02 21:43:49 +01:00
|
|
|
def message_ids(result):
|
|
|
|
return set(message['id'] for message in result['messages'])
|
|
|
|
|
2012-08-31 20:11:15 +02:00
|
|
|
class AuthedTestCase(TestCase):
|
2012-10-12 17:49:22 +02:00
|
|
|
def login(self, email, password=None):
|
|
|
|
if password is None:
|
|
|
|
password = initial_password(email)
|
2012-08-31 20:11:15 +02:00
|
|
|
return self.client.post('/accounts/login/',
|
2012-10-12 17:49:22 +02:00
|
|
|
{'username':email, 'password':password})
|
2012-08-31 20:11:15 +02:00
|
|
|
|
|
|
|
def register(self, username, password):
|
2012-10-02 17:47:18 +02:00
|
|
|
self.client.post('/accounts/home/',
|
|
|
|
{'email': username + '@humbughq.com'})
|
2013-01-04 19:06:34 +01:00
|
|
|
return self.submit_reg_form_for_user(username, password)
|
|
|
|
|
|
|
|
def submit_reg_form_for_user(self, username, password):
|
|
|
|
"""
|
|
|
|
Stage two of the two-step registration process.
|
|
|
|
|
|
|
|
If things are working correctly the account should be fully
|
|
|
|
registered after this call.
|
|
|
|
"""
|
2012-08-31 20:11:15 +02:00
|
|
|
return self.client.post('/accounts/register/',
|
2012-11-02 21:13:35 +01:00
|
|
|
{'full_name': username, 'password': password,
|
2012-10-02 17:47:18 +02:00
|
|
|
'key': find_key_by_email(username + '@humbughq.com'),
|
2012-11-02 21:13:35 +01:00
|
|
|
'terms': True})
|
2013-01-04 19:06:34 +01:00
|
|
|
|
2012-11-15 19:42:17 +01:00
|
|
|
def get_api_key(self, email):
|
|
|
|
return initial_api_key(email)
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-10-22 20:15:25 +02:00
|
|
|
def get_user_profile(self, email):
|
2012-09-06 20:43:41 +02:00
|
|
|
"""
|
2012-09-21 00:26:59 +02:00
|
|
|
Given an email address, return the UserProfile object for the
|
|
|
|
User that has that email.
|
2012-09-06 20:43:41 +02:00
|
|
|
"""
|
|
|
|
# Usernames are unique, even across Realms.
|
2012-10-11 20:05:53 +02:00
|
|
|
return UserProfile.objects.get(user__email=email)
|
2012-09-06 20:43:41 +02:00
|
|
|
|
2012-10-03 21:16:34 +02:00
|
|
|
def send_message(self, sender_name, recipient_name, message_type):
|
2012-10-22 20:15:25 +02:00
|
|
|
sender = self.get_user_profile(sender_name)
|
2012-10-03 21:16:34 +02:00
|
|
|
if message_type == Recipient.PERSONAL:
|
2012-10-22 20:15:25 +02:00
|
|
|
recipient = self.get_user_profile(recipient_name)
|
2012-08-31 20:11:15 +02:00
|
|
|
else:
|
2012-10-10 22:53:24 +02:00
|
|
|
recipient = Stream.objects.get(name=recipient_name, realm=sender.realm)
|
2012-10-03 21:16:34 +02:00
|
|
|
recipient = Recipient.objects.get(type_id=recipient.id, type=message_type)
|
2012-11-08 00:48:43 +01:00
|
|
|
pub_date = now()
|
2012-10-22 19:23:11 +02:00
|
|
|
(sending_client, _) = Client.objects.get_or_create(name="test suite")
|
2012-10-19 21:37:37 +02:00
|
|
|
do_send_message(Message(sender=sender, recipient=recipient, subject="test",
|
|
|
|
pub_date=pub_date, sending_client=sending_client))
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-10-10 23:13:04 +02:00
|
|
|
def users_subscribed_to_stream(self, stream_name, realm_domain):
|
2012-09-05 22:30:50 +02:00
|
|
|
realm = Realm.objects.get(domain=realm_domain)
|
2012-10-10 23:13:04 +02:00
|
|
|
stream = Stream.objects.get(name=stream_name, realm=realm)
|
2012-10-10 22:57:21 +02:00
|
|
|
recipient = Recipient.objects.get(type_id=stream.id, type=Recipient.STREAM)
|
2012-09-05 21:55:40 +02:00
|
|
|
subscriptions = Subscription.objects.filter(recipient=recipient)
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-10-22 20:15:25 +02:00
|
|
|
return [subscription.user_profile.user for subscription in subscriptions]
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-10-03 21:16:34 +02:00
|
|
|
def message_stream(self, user):
|
2012-10-03 21:05:48 +02:00
|
|
|
return filter_by_subscriptions(Message.objects.all(), user)
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-09-06 21:34:38 +02:00
|
|
|
def assert_json_success(self, result):
|
|
|
|
"""
|
|
|
|
Successful POSTs return a 200 and JSON of the form {"result": "success",
|
|
|
|
"msg": ""}.
|
|
|
|
"""
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(result.status_code, 200)
|
2012-09-06 21:34:38 +02:00
|
|
|
json = simplejson.loads(result.content)
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(json.get("result"), "success")
|
2012-09-06 21:34:38 +02:00
|
|
|
# We have a msg key for consistency with errors, but it typically has an
|
|
|
|
# empty value.
|
2013-01-02 22:07:09 +01:00
|
|
|
self.assertIn("msg", json)
|
2012-09-06 21:34:38 +02:00
|
|
|
|
2012-12-20 23:57:26 +01:00
|
|
|
def get_json_error(self, result):
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(result.status_code, 400)
|
2012-12-20 23:57:26 +01:00
|
|
|
json = simplejson.loads(result.content)
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(json.get("result"), "error")
|
2012-12-20 23:57:26 +01:00
|
|
|
return json['msg']
|
|
|
|
|
2012-09-06 21:34:38 +02:00
|
|
|
def assert_json_error(self, result, msg):
|
|
|
|
"""
|
|
|
|
Invalid POSTs return a 400 and JSON of the form {"result": "error",
|
|
|
|
"msg": "reason"}.
|
|
|
|
"""
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(self.get_json_error(result), msg)
|
2012-12-20 23:57:26 +01:00
|
|
|
|
|
|
|
def assert_json_error_contains(self, result, msg_substring):
|
|
|
|
self.assertIn(msg_substring, self.get_json_error(result))
|
2012-08-31 20:11:15 +02:00
|
|
|
|
|
|
|
class PublicURLTest(TestCase):
|
|
|
|
"""
|
|
|
|
Account creation URLs are accessible even when not logged in. Authenticated
|
|
|
|
URLs redirect to a page.
|
|
|
|
"""
|
2013-01-23 17:58:40 +01:00
|
|
|
fixtures = ['messages.json']
|
|
|
|
|
|
|
|
def fetch(self, method, urls, expected_status):
|
2012-08-31 20:11:15 +02:00
|
|
|
for url in urls:
|
2013-01-23 17:58:40 +01:00
|
|
|
if method == "get":
|
|
|
|
response = self.client.get(url)
|
|
|
|
else:
|
|
|
|
response = self.client.post(url)
|
2012-08-31 20:11:15 +02:00
|
|
|
self.assertEqual(response.status_code, expected_status,
|
2013-01-23 17:58:40 +01:00
|
|
|
msg="Expected %d, received %d for %s to %s" % (
|
|
|
|
expected_status, response.status_code, method, url))
|
2012-08-31 20:11:15 +02:00
|
|
|
|
|
|
|
def test_public_urls(self):
|
|
|
|
"""
|
2012-11-01 19:59:23 +01:00
|
|
|
Test which views are accessible when not logged in.
|
2012-08-31 20:11:15 +02:00
|
|
|
"""
|
2012-10-16 23:24:58 +02:00
|
|
|
# FIXME: We should also test the Tornado URLs -- this codepath
|
|
|
|
# can't do so because this Django test mechanism doesn't go
|
|
|
|
# through Tornado.
|
2013-01-23 17:58:40 +01:00
|
|
|
get_urls = {200: ["/accounts/home/", "/accounts/login/"],
|
|
|
|
302: ["/"],
|
2012-08-31 20:11:15 +02:00
|
|
|
}
|
2013-01-23 17:58:40 +01:00
|
|
|
post_urls = {200: ["/accounts/login/"],
|
|
|
|
302: ["/accounts/logout/"],
|
|
|
|
401: ["/json/get_public_streams",
|
|
|
|
"/json/get_old_messages",
|
|
|
|
"/json/update_pointer",
|
|
|
|
"/json/send_message",
|
|
|
|
"/json/invite_users",
|
|
|
|
"/json/settings/change",
|
|
|
|
"/json/subscriptions/list",
|
|
|
|
"/json/subscriptions/remove",
|
|
|
|
"/json/subscriptions/exists",
|
|
|
|
"/json/subscriptions/add",
|
|
|
|
"/json/subscriptions/property",
|
|
|
|
"/json/get_subscribers",
|
|
|
|
"/json/fetch_api_key",
|
|
|
|
],
|
|
|
|
400: ["/api/v1/get_profile",
|
|
|
|
"/api/v1/get_old_messages",
|
|
|
|
"/api/v1/get_public_streams",
|
|
|
|
"/api/v1/subscriptions/list",
|
|
|
|
"/api/v1/subscriptions/add",
|
|
|
|
"/api/v1/subscriptions/remove",
|
|
|
|
"/api/v1/get_subscribers",
|
|
|
|
"/api/v1/send_message",
|
|
|
|
"/api/v1/update_pointer",
|
|
|
|
"/api/v1/external/github",
|
|
|
|
"/api/v1/fetch_api_key",
|
|
|
|
],
|
|
|
|
}
|
|
|
|
for status_code, url_set in get_urls.iteritems():
|
|
|
|
self.fetch("get", url_set, status_code)
|
|
|
|
for status_code, url_set in post_urls.iteritems():
|
|
|
|
self.fetch("post", url_set, status_code)
|
2012-08-31 20:11:15 +02:00
|
|
|
|
|
|
|
class LoginTest(AuthedTestCase):
|
|
|
|
"""
|
|
|
|
Logging in, registration, and logging out.
|
|
|
|
"""
|
2012-10-03 21:16:34 +02:00
|
|
|
fixtures = ['messages.json']
|
2012-08-31 20:11:15 +02:00
|
|
|
|
|
|
|
def test_login(self):
|
2012-10-12 17:49:22 +02:00
|
|
|
self.login("hamlet@humbughq.com")
|
2012-09-21 00:26:59 +02:00
|
|
|
user = User.objects.get(email='hamlet@humbughq.com')
|
2012-09-10 19:43:11 +02:00
|
|
|
self.assertEqual(self.client.session['_auth_user_id'], user.id)
|
2012-08-31 20:11:15 +02:00
|
|
|
|
|
|
|
def test_login_bad_password(self):
|
2012-09-21 16:10:36 +02:00
|
|
|
self.login("hamlet@humbughq.com", "wrongpassword")
|
2012-08-31 20:11:15 +02:00
|
|
|
self.assertIsNone(self.client.session.get('_auth_user_id', None))
|
|
|
|
|
|
|
|
def test_register(self):
|
|
|
|
self.register("test", "test")
|
2012-09-21 00:26:59 +02:00
|
|
|
user = User.objects.get(email='test@humbughq.com')
|
2012-09-10 19:43:11 +02:00
|
|
|
self.assertEqual(self.client.session['_auth_user_id'], user.id)
|
2012-08-31 20:11:15 +02:00
|
|
|
|
|
|
|
def test_logout(self):
|
2012-10-12 17:49:22 +02:00
|
|
|
self.login("hamlet@humbughq.com")
|
2012-08-31 20:11:15 +02:00
|
|
|
self.client.post('/accounts/logout/')
|
|
|
|
self.assertIsNone(self.client.session.get('_auth_user_id', None))
|
|
|
|
|
|
|
|
|
2012-10-03 21:16:34 +02:00
|
|
|
class PersonalMessagesTest(AuthedTestCase):
|
|
|
|
fixtures = ['messages.json']
|
2012-08-31 20:11:15 +02:00
|
|
|
|
|
|
|
def test_auto_subbed_to_personals(self):
|
|
|
|
"""
|
|
|
|
Newly created users are auto-subbed to the ability to receive
|
|
|
|
personals.
|
|
|
|
"""
|
|
|
|
self.register("test", "test")
|
2012-09-21 00:26:59 +02:00
|
|
|
user = User.objects.get(email='test@humbughq.com')
|
2012-10-03 21:16:34 +02:00
|
|
|
old_messages = self.message_stream(user)
|
|
|
|
self.send_message("test@humbughq.com", "test@humbughq.com", Recipient.PERSONAL)
|
|
|
|
new_messages = self.message_stream(user)
|
|
|
|
self.assertEqual(len(new_messages) - len(old_messages), 1)
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-09-10 19:43:11 +02:00
|
|
|
recipient = Recipient.objects.get(type_id=user.id, type=Recipient.PERSONAL)
|
2012-10-03 21:16:34 +02:00
|
|
|
self.assertEqual(new_messages[-1].recipient, recipient)
|
2012-08-31 20:11:15 +02:00
|
|
|
|
|
|
|
def test_personal_to_self(self):
|
|
|
|
"""
|
|
|
|
If you send a personal to yourself, only you see it.
|
|
|
|
"""
|
|
|
|
old_users = list(User.objects.all())
|
|
|
|
self.register("test1", "test1")
|
|
|
|
|
2012-10-03 21:16:34 +02:00
|
|
|
old_messages = []
|
2012-08-31 20:11:15 +02:00
|
|
|
for user in old_users:
|
2012-10-03 21:16:34 +02:00
|
|
|
old_messages.append(len(self.message_stream(user)))
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-10-03 21:16:34 +02:00
|
|
|
self.send_message("test1@humbughq.com", "test1@humbughq.com", Recipient.PERSONAL)
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-10-03 21:16:34 +02:00
|
|
|
new_messages = []
|
2012-08-31 20:11:15 +02:00
|
|
|
for user in old_users:
|
2012-10-03 21:16:34 +02:00
|
|
|
new_messages.append(len(self.message_stream(user)))
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-10-03 21:16:34 +02:00
|
|
|
self.assertEqual(old_messages, new_messages)
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-09-21 00:26:59 +02:00
|
|
|
user = User.objects.get(email="test1@humbughq.com")
|
2012-09-10 19:43:11 +02:00
|
|
|
recipient = Recipient.objects.get(type_id=user.id, type=Recipient.PERSONAL)
|
2012-10-03 21:16:34 +02:00
|
|
|
self.assertEqual(self.message_stream(user)[-1].recipient, recipient)
|
2012-08-31 20:11:15 +02:00
|
|
|
|
|
|
|
def test_personal(self):
|
|
|
|
"""
|
|
|
|
If you send a personal, only you and the recipient see it.
|
|
|
|
"""
|
2012-10-12 17:49:22 +02:00
|
|
|
self.login("hamlet@humbughq.com")
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-09-21 00:26:59 +02:00
|
|
|
old_sender = User.objects.filter(email="hamlet@humbughq.com")
|
2012-10-03 21:16:34 +02:00
|
|
|
old_sender_messages = len(self.message_stream(old_sender))
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-09-21 00:26:59 +02:00
|
|
|
old_recipient = User.objects.filter(email="othello@humbughq.com")
|
2012-10-03 21:16:34 +02:00
|
|
|
old_recipient_messages = len(self.message_stream(old_recipient))
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-09-21 00:26:59 +02:00
|
|
|
other_users = User.objects.filter(~Q(email="hamlet@humbughq.com") & ~Q(email="othello@humbughq.com"))
|
2012-10-03 21:16:34 +02:00
|
|
|
old_other_messages = []
|
2012-08-31 20:11:15 +02:00
|
|
|
for user in other_users:
|
2012-10-03 21:16:34 +02:00
|
|
|
old_other_messages.append(len(self.message_stream(user)))
|
2012-08-28 18:44:51 +02:00
|
|
|
|
2012-10-03 21:16:34 +02:00
|
|
|
self.send_message("hamlet@humbughq.com", "othello@humbughq.com", Recipient.PERSONAL)
|
2012-08-28 18:44:51 +02:00
|
|
|
|
2012-10-03 21:16:34 +02:00
|
|
|
# Users outside the conversation don't get the message.
|
|
|
|
new_other_messages = []
|
2012-08-31 20:11:15 +02:00
|
|
|
for user in other_users:
|
2012-10-03 21:16:34 +02:00
|
|
|
new_other_messages.append(len(self.message_stream(user)))
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-10-03 21:16:34 +02:00
|
|
|
self.assertEqual(old_other_messages, new_other_messages)
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-10-03 21:16:34 +02:00
|
|
|
# The personal message is in the streams of both the sender and receiver.
|
|
|
|
self.assertEqual(len(self.message_stream(old_sender)),
|
|
|
|
old_sender_messages + 1)
|
|
|
|
self.assertEqual(len(self.message_stream(old_recipient)),
|
|
|
|
old_recipient_messages + 1)
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-09-21 00:26:59 +02:00
|
|
|
sender = User.objects.get(email="hamlet@humbughq.com")
|
|
|
|
receiver = User.objects.get(email="othello@humbughq.com")
|
2012-09-10 19:43:11 +02:00
|
|
|
recipient = Recipient.objects.get(type_id=receiver.id, type=Recipient.PERSONAL)
|
2012-10-03 21:16:34 +02:00
|
|
|
self.assertEqual(self.message_stream(sender)[-1].recipient, recipient)
|
|
|
|
self.assertEqual(self.message_stream(receiver)[-1].recipient, recipient)
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-10-10 23:13:04 +02:00
|
|
|
class StreamMessagesTest(AuthedTestCase):
|
2012-10-03 21:16:34 +02:00
|
|
|
fixtures = ['messages.json']
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-10-10 23:13:04 +02:00
|
|
|
def test_message_to_stream(self):
|
2012-08-31 20:11:15 +02:00
|
|
|
"""
|
2012-10-10 23:13:04 +02:00
|
|
|
If you send a message to a stream, everyone subscribed to the stream
|
2012-10-03 21:16:34 +02:00
|
|
|
receives the messages.
|
2012-08-31 20:11:15 +02:00
|
|
|
"""
|
2012-10-10 23:13:04 +02:00
|
|
|
subscribers = self.users_subscribed_to_stream("Scotland", "humbughq.com")
|
2012-10-03 21:16:34 +02:00
|
|
|
old_subscriber_messages = []
|
2012-08-31 20:11:15 +02:00
|
|
|
for subscriber in subscribers:
|
2012-10-03 21:16:34 +02:00
|
|
|
old_subscriber_messages.append(len(self.message_stream(subscriber)))
|
2012-08-31 20:11:15 +02:00
|
|
|
|
|
|
|
non_subscribers = [user for user in User.objects.all() if user not in subscribers]
|
2012-10-03 21:16:34 +02:00
|
|
|
old_non_subscriber_messages = []
|
2012-08-31 20:11:15 +02:00
|
|
|
for non_subscriber in non_subscribers:
|
2012-10-03 21:16:34 +02:00
|
|
|
old_non_subscriber_messages.append(len(self.message_stream(non_subscriber)))
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-09-21 00:26:59 +02:00
|
|
|
a_subscriber_email = subscribers[0].email
|
2012-10-12 17:49:22 +02:00
|
|
|
self.login(a_subscriber_email)
|
2012-10-10 22:57:21 +02:00
|
|
|
self.send_message(a_subscriber_email, "Scotland", Recipient.STREAM)
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-10-03 21:16:34 +02:00
|
|
|
new_subscriber_messages = []
|
2012-08-31 20:11:15 +02:00
|
|
|
for subscriber in subscribers:
|
2012-10-03 21:16:34 +02:00
|
|
|
new_subscriber_messages.append(len(self.message_stream(subscriber)))
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-10-03 21:16:34 +02:00
|
|
|
new_non_subscriber_messages = []
|
2012-08-31 20:11:15 +02:00
|
|
|
for non_subscriber in non_subscribers:
|
2012-10-03 21:16:34 +02:00
|
|
|
new_non_subscriber_messages.append(len(self.message_stream(non_subscriber)))
|
2012-08-31 20:11:15 +02:00
|
|
|
|
2012-10-03 21:16:34 +02:00
|
|
|
self.assertEqual(old_non_subscriber_messages, new_non_subscriber_messages)
|
|
|
|
self.assertEqual(new_subscriber_messages, [elt + 1 for elt in old_subscriber_messages])
|
2012-09-04 22:31:56 +02:00
|
|
|
|
2012-09-06 20:53:13 +02:00
|
|
|
class PointerTest(AuthedTestCase):
|
2012-10-03 21:16:34 +02:00
|
|
|
fixtures = ['messages.json']
|
2012-09-06 20:53:13 +02:00
|
|
|
|
|
|
|
def test_update_pointer(self):
|
|
|
|
"""
|
|
|
|
Posting a pointer to /update (in the form {"pointer": pointer}) changes
|
|
|
|
the pointer we store for your UserProfile.
|
|
|
|
"""
|
2012-10-12 17:49:22 +02:00
|
|
|
self.login("hamlet@humbughq.com")
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(self.get_user_profile("hamlet@humbughq.com").pointer, -1)
|
2012-10-16 21:42:40 +02:00
|
|
|
result = self.client.post("/json/update_pointer", {"pointer": 1})
|
2012-09-06 21:34:38 +02:00
|
|
|
self.assert_json_success(result)
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(self.get_user_profile("hamlet@humbughq.com").pointer, 1)
|
2012-09-06 20:53:13 +02:00
|
|
|
|
2012-11-15 19:42:17 +01:00
|
|
|
def test_api_update_pointer(self):
|
|
|
|
"""
|
|
|
|
Same as above, but for the API view
|
|
|
|
"""
|
|
|
|
email = "hamlet@humbughq.com"
|
|
|
|
api_key = self.get_api_key(email)
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(self.get_user_profile(email).pointer, -1)
|
2012-11-15 19:42:17 +01:00
|
|
|
result = self.client.post("/api/v1/update_pointer", {"email": email,
|
|
|
|
"api-key": api_key,
|
|
|
|
"client_id": "blah",
|
|
|
|
"pointer": 1})
|
|
|
|
self.assert_json_success(result)
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(self.get_user_profile(email).pointer, 1)
|
2012-11-15 19:42:17 +01:00
|
|
|
|
2012-09-06 20:53:13 +02:00
|
|
|
def test_missing_pointer(self):
|
|
|
|
"""
|
2012-10-16 21:42:40 +02:00
|
|
|
Posting json to /json/update_pointer which does not contain a pointer key/value pair
|
2012-09-06 20:53:13 +02:00
|
|
|
returns a 400 and error message.
|
|
|
|
"""
|
2012-10-12 17:49:22 +02:00
|
|
|
self.login("hamlet@humbughq.com")
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(self.get_user_profile("hamlet@humbughq.com").pointer, -1)
|
2012-10-16 21:42:40 +02:00
|
|
|
result = self.client.post("/json/update_pointer", {"foo": 1})
|
2012-11-09 18:45:22 +01:00
|
|
|
self.assert_json_error(result, "Missing 'pointer' argument")
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(self.get_user_profile("hamlet@humbughq.com").pointer, -1)
|
2012-09-06 20:53:13 +02:00
|
|
|
|
|
|
|
def test_invalid_pointer(self):
|
|
|
|
"""
|
2012-10-16 21:42:40 +02:00
|
|
|
Posting json to /json/update_pointer with an invalid pointer returns a 400 and error
|
2012-09-06 20:53:13 +02:00
|
|
|
message.
|
|
|
|
"""
|
2012-10-12 17:49:22 +02:00
|
|
|
self.login("hamlet@humbughq.com")
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(self.get_user_profile("hamlet@humbughq.com").pointer, -1)
|
2012-10-16 21:42:40 +02:00
|
|
|
result = self.client.post("/json/update_pointer", {"pointer": "foo"})
|
2012-11-09 18:45:22 +01:00
|
|
|
self.assert_json_error(result, "Bad value for 'pointer': foo")
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(self.get_user_profile("hamlet@humbughq.com").pointer, -1)
|
2012-09-06 20:53:13 +02:00
|
|
|
|
|
|
|
def test_pointer_out_of_range(self):
|
|
|
|
"""
|
2012-10-16 21:42:40 +02:00
|
|
|
Posting json to /json/update_pointer with an out of range (< 0) pointer returns a 400
|
2012-09-06 20:53:13 +02:00
|
|
|
and error message.
|
|
|
|
"""
|
2012-10-12 17:49:22 +02:00
|
|
|
self.login("hamlet@humbughq.com")
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(self.get_user_profile("hamlet@humbughq.com").pointer, -1)
|
2012-10-16 21:42:40 +02:00
|
|
|
result = self.client.post("/json/update_pointer", {"pointer": -2})
|
2012-12-19 20:26:48 +01:00
|
|
|
self.assert_json_error(result, "Bad value for 'pointer': -2")
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(self.get_user_profile("hamlet@humbughq.com").pointer, -1)
|
2012-09-06 21:46:03 +02:00
|
|
|
|
2012-10-03 21:16:34 +02:00
|
|
|
class MessagePOSTTest(AuthedTestCase):
|
|
|
|
fixtures = ['messages.json']
|
2012-09-06 21:46:03 +02:00
|
|
|
|
2012-10-03 21:16:34 +02:00
|
|
|
def test_message_to_self(self):
|
2012-09-06 21:46:03 +02:00
|
|
|
"""
|
2012-10-10 23:13:04 +02:00
|
|
|
Sending a message to a stream to which you are subscribed is
|
2012-10-03 21:16:34 +02:00
|
|
|
successful.
|
2012-09-06 21:46:03 +02:00
|
|
|
"""
|
2012-10-12 17:49:22 +02:00
|
|
|
self.login("hamlet@humbughq.com")
|
2012-11-09 18:42:03 +01:00
|
|
|
result = self.client.post("/json/send_message", {"type": "stream",
|
2012-11-14 23:21:46 +01:00
|
|
|
"to": "Verona",
|
2012-11-09 18:42:03 +01:00
|
|
|
"client": "test suite",
|
|
|
|
"content": "Test message",
|
|
|
|
"subject": "Test subject"})
|
2012-09-06 21:46:03 +02:00
|
|
|
self.assert_json_success(result)
|
|
|
|
|
2012-11-15 19:42:17 +01:00
|
|
|
def test_api_message_to_self(self):
|
|
|
|
"""
|
|
|
|
Same as above, but for the API view
|
|
|
|
"""
|
|
|
|
email = "hamlet@humbughq.com"
|
|
|
|
api_key = self.get_api_key(email)
|
|
|
|
result = self.client.post("/api/v1/send_message", {"type": "stream",
|
2012-11-14 23:21:46 +01:00
|
|
|
"to": "Verona",
|
2012-11-15 19:42:17 +01:00
|
|
|
"client": "test suite",
|
|
|
|
"content": "Test message",
|
|
|
|
"subject": "Test subject",
|
|
|
|
"email": email,
|
|
|
|
"api-key": api_key})
|
|
|
|
self.assert_json_success(result)
|
|
|
|
|
2012-10-10 23:13:04 +02:00
|
|
|
def test_message_to_nonexistent_stream(self):
|
2012-09-06 21:46:03 +02:00
|
|
|
"""
|
2012-11-01 16:09:24 +01:00
|
|
|
Sending a message to a nonexistent stream fails.
|
2012-09-06 21:46:03 +02:00
|
|
|
"""
|
2012-10-12 17:49:22 +02:00
|
|
|
self.login("hamlet@humbughq.com")
|
2012-10-10 23:13:04 +02:00
|
|
|
self.assertFalse(Stream.objects.filter(name="nonexistent_stream"))
|
2012-11-09 18:42:03 +01:00
|
|
|
result = self.client.post("/json/send_message", {"type": "stream",
|
2012-11-14 23:21:46 +01:00
|
|
|
"to": "nonexistent_stream",
|
2012-11-09 18:42:03 +01:00
|
|
|
"client": "test suite",
|
|
|
|
"content": "Test message",
|
|
|
|
"subject": "Test subject"})
|
2012-10-30 21:50:58 +01:00
|
|
|
self.assert_json_error(result, "Stream does not exist")
|
2012-09-06 21:46:03 +02:00
|
|
|
|
2012-10-03 21:16:34 +02:00
|
|
|
def test_personal_message(self):
|
2012-09-06 21:46:03 +02:00
|
|
|
"""
|
2012-10-03 21:16:34 +02:00
|
|
|
Sending a personal message to a valid username is successful.
|
2012-09-06 21:46:03 +02:00
|
|
|
"""
|
2012-10-12 17:49:22 +02:00
|
|
|
self.login("hamlet@humbughq.com")
|
2012-11-08 00:38:21 +01:00
|
|
|
result = self.client.post("/json/send_message", {"type": "private",
|
2012-11-09 18:42:03 +01:00
|
|
|
"content": "Test message",
|
|
|
|
"client": "test suite",
|
2012-11-14 23:21:46 +01:00
|
|
|
"to": "othello@humbughq.com"})
|
2012-09-06 21:46:03 +02:00
|
|
|
self.assert_json_success(result)
|
|
|
|
|
2012-10-03 21:16:34 +02:00
|
|
|
def test_personal_message_to_nonexistent_user(self):
|
2012-09-06 21:46:03 +02:00
|
|
|
"""
|
2012-10-03 21:16:34 +02:00
|
|
|
Sending a personal message to an invalid email returns error JSON.
|
2012-09-06 21:46:03 +02:00
|
|
|
"""
|
2012-10-12 17:49:22 +02:00
|
|
|
self.login("hamlet@humbughq.com")
|
2012-11-08 00:38:21 +01:00
|
|
|
result = self.client.post("/json/send_message", {"type": "private",
|
2012-11-09 18:42:03 +01:00
|
|
|
"content": "Test message",
|
|
|
|
"client": "test suite",
|
2012-11-14 23:21:46 +01:00
|
|
|
"to": "nonexistent"})
|
2012-10-12 20:29:23 +02:00
|
|
|
self.assert_json_error(result, "Invalid email 'nonexistent'")
|
2012-09-06 21:53:26 +02:00
|
|
|
|
|
|
|
def test_invalid_type(self):
|
|
|
|
"""
|
2012-10-03 21:16:34 +02:00
|
|
|
Sending a message of unknown type returns error JSON.
|
2012-09-06 21:53:26 +02:00
|
|
|
"""
|
2012-10-12 17:49:22 +02:00
|
|
|
self.login("hamlet@humbughq.com")
|
2012-11-09 18:42:03 +01:00
|
|
|
result = self.client.post("/json/send_message", {"type": "invalid type",
|
|
|
|
"content": "Test message",
|
|
|
|
"client": "test suite",
|
2012-11-14 23:21:46 +01:00
|
|
|
"to": "othello@humbughq.com"})
|
2012-10-03 00:10:55 +02:00
|
|
|
self.assert_json_error(result, "Invalid message type")
|
2012-09-06 23:55:04 +02:00
|
|
|
|
2013-01-07 18:02:55 +01:00
|
|
|
def test_mirrored_huddle(self):
|
|
|
|
"""
|
|
|
|
Sending a mirrored huddle message works
|
|
|
|
"""
|
|
|
|
self.login("starnine@mit.edu")
|
|
|
|
result = self.client.post("/json/send_message", {"type": "private",
|
|
|
|
"sender": "sipbtest@mit.edu",
|
|
|
|
"content": "Test message",
|
|
|
|
"client": "zephyr_mirror",
|
|
|
|
"to": simplejson.dumps(["starnine@mit.edu",
|
|
|
|
"espuser@mit.edu"])})
|
|
|
|
self.assert_json_success(result)
|
|
|
|
|
|
|
|
def test_mirrored_personal(self):
|
|
|
|
"""
|
|
|
|
Sending a mirrored personal message works
|
|
|
|
"""
|
|
|
|
self.login("starnine@mit.edu")
|
|
|
|
result = self.client.post("/json/send_message", {"type": "private",
|
|
|
|
"sender": "sipbtest@mit.edu",
|
|
|
|
"content": "Test message",
|
|
|
|
"client": "zephyr_mirror",
|
|
|
|
"to": "starnine@mit.edu"})
|
|
|
|
self.assert_json_success(result)
|
|
|
|
|
2012-12-03 02:02:53 +01:00
|
|
|
class SubscriptionPropertiesTest(AuthedTestCase):
|
|
|
|
fixtures = ['messages.json']
|
|
|
|
|
|
|
|
def test_get_stream_colors(self):
|
|
|
|
"""
|
|
|
|
A GET request to
|
|
|
|
/json/subscriptions/property?property=stream_colors returns a
|
|
|
|
list of (stream, color) pairs, both of which are strings.
|
|
|
|
"""
|
|
|
|
test_email = "hamlet@humbughq.com"
|
|
|
|
self.login(test_email)
|
|
|
|
result = self.client.get("/json/subscriptions/property",
|
|
|
|
{"property": "stream_colors"})
|
|
|
|
|
|
|
|
self.assert_json_success(result)
|
|
|
|
json = simplejson.loads(result.content)
|
|
|
|
self.assertIn("stream_colors", json)
|
|
|
|
|
|
|
|
subs = gather_subscriptions(self.get_user_profile(test_email))
|
|
|
|
for stream, color in json["stream_colors"]:
|
2013-01-02 22:07:09 +01:00
|
|
|
self.assertIsInstance(color, str)
|
|
|
|
self.assertIsInstance(stream, str)
|
2013-02-05 19:47:43 +01:00
|
|
|
self.assertIn({'name': stream, 'in_home_view': True, 'color': color, 'invite_only': False}, subs)
|
|
|
|
subs.remove({'name': stream, 'in_home_view': True, 'color': color, 'invite_only': False})
|
2012-12-03 02:02:53 +01:00
|
|
|
self.assertFalse(subs)
|
|
|
|
|
|
|
|
def test_set_stream_color(self):
|
|
|
|
"""
|
|
|
|
A POST request to /json/subscriptions/property with stream_name and
|
|
|
|
color data sets the stream color, and for that stream only.
|
|
|
|
"""
|
|
|
|
test_email = "hamlet@humbughq.com"
|
|
|
|
self.login(test_email)
|
|
|
|
|
|
|
|
old_subs = gather_subscriptions(self.get_user_profile(test_email))
|
2013-01-29 21:40:16 +01:00
|
|
|
sub = old_subs[0]
|
|
|
|
stream_name = sub['name']
|
|
|
|
old_color = sub['color']
|
2013-02-05 19:47:43 +01:00
|
|
|
invite_only = sub['invite_only']
|
2012-12-03 02:02:53 +01:00
|
|
|
new_color = "#ffffff" # TODO: ensure that this is different from old_color
|
|
|
|
result = self.client.post("/json/subscriptions/property",
|
|
|
|
{"property": "stream_colors",
|
|
|
|
"stream_name": stream_name,
|
|
|
|
"color": "#ffffff"})
|
|
|
|
|
|
|
|
self.assert_json_success(result)
|
|
|
|
|
|
|
|
new_subs = gather_subscriptions(self.get_user_profile(test_email))
|
2013-02-05 19:47:43 +01:00
|
|
|
self.assertIn({'name': stream_name, 'in_home_view': True, 'color': new_color, 'invite_only': invite_only}, new_subs)
|
2012-12-03 02:02:53 +01:00
|
|
|
|
2013-02-05 19:47:43 +01:00
|
|
|
old_subs.remove({'name': stream_name, 'in_home_view': True, 'color': old_color, 'invite_only': invite_only})
|
|
|
|
new_subs.remove({'name': stream_name, 'in_home_view': True, 'color': new_color, 'invite_only': invite_only})
|
2012-12-03 02:02:53 +01:00
|
|
|
self.assertEqual(old_subs, new_subs)
|
|
|
|
|
|
|
|
def test_set_color_missing_stream_name(self):
|
|
|
|
"""
|
|
|
|
Updating the stream_colors property requires a stream_name.
|
|
|
|
"""
|
|
|
|
test_email = "hamlet@humbughq.com"
|
|
|
|
self.login(test_email)
|
|
|
|
result = self.client.post("/json/subscriptions/property",
|
|
|
|
{"property": "stream_colors",
|
|
|
|
"color": "#ffffff"})
|
|
|
|
|
2013-01-18 18:20:58 +01:00
|
|
|
self.assert_json_error(result, "Missing 'stream_name' argument")
|
2012-12-03 02:02:53 +01:00
|
|
|
|
|
|
|
def test_set_color_missing_color(self):
|
|
|
|
"""
|
|
|
|
Updating the stream_colors property requires a color.
|
|
|
|
"""
|
|
|
|
test_email = "hamlet@humbughq.com"
|
|
|
|
self.login(test_email)
|
|
|
|
result = self.client.post("/json/subscriptions/property",
|
|
|
|
{"property": "stream_colors",
|
|
|
|
"stream_name": "test"})
|
|
|
|
|
2013-01-18 18:20:58 +01:00
|
|
|
self.assert_json_error(result, "Missing 'color' argument")
|
2012-12-03 02:02:53 +01:00
|
|
|
|
|
|
|
def test_set_invalid_property(self):
|
|
|
|
"""
|
|
|
|
Trying to set an invalid property returns a JSON error.
|
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
|
|
|
result = self.client.post("/json/subscriptions/property",
|
|
|
|
{"property": "bad"})
|
|
|
|
|
|
|
|
self.assert_json_error(result,
|
|
|
|
"Unknown property or invalid verb for bad")
|
|
|
|
|
2013-01-29 19:47:28 +01:00
|
|
|
class SubscriptionAPITest(AuthedTestCase):
|
|
|
|
fixtures = ['messages.json']
|
|
|
|
|
|
|
|
def setUp(self):
|
|
|
|
"""
|
|
|
|
All tests will be logged in as hamlet. Also save various useful values
|
|
|
|
as attributes that tests can access.
|
|
|
|
"""
|
|
|
|
self.test_email = "hamlet@humbughq.com"
|
|
|
|
self.login(self.test_email)
|
|
|
|
self.user_profile = self.get_user_profile(self.test_email)
|
|
|
|
self.realm = self.user_profile.realm
|
|
|
|
self.streams = self.get_streams(self.test_email)
|
|
|
|
|
|
|
|
def get_streams(self, email):
|
|
|
|
"""
|
|
|
|
Helper function to get the stream names for a user
|
|
|
|
"""
|
|
|
|
user_profile = self.get_user_profile(email)
|
|
|
|
subs = Subscription.objects.filter(
|
|
|
|
user_profile = user_profile,
|
|
|
|
active = True,
|
|
|
|
recipient__type = Recipient.STREAM)
|
|
|
|
return [get_display_recipient(sub.recipient) for sub in subs]
|
|
|
|
|
|
|
|
def make_random_stream_names(self, existing_stream_names, names_to_avoid):
|
|
|
|
"""
|
|
|
|
Helper function to make up random stream names. It takes
|
|
|
|
existing_stream_names and randomly appends a digit to the end of each,
|
|
|
|
but avoids names that appear in the list names_to_avoid.
|
|
|
|
"""
|
|
|
|
random_streams = []
|
|
|
|
for stream in existing_stream_names:
|
|
|
|
random_stream = stream + str(random.randint(0, 9))
|
|
|
|
if not random_stream in names_to_avoid:
|
|
|
|
random_streams.append(random_stream)
|
|
|
|
return random_streams
|
|
|
|
|
|
|
|
def test_successful_subscriptions_list(self):
|
|
|
|
"""
|
|
|
|
Calling /json/subscriptions/list should successfully return your subscriptions.
|
|
|
|
"""
|
|
|
|
result = self.client.post("/json/subscriptions/list", {})
|
|
|
|
self.assert_json_success(result)
|
|
|
|
json = simplejson.loads(result.content)
|
|
|
|
self.assertIn("subscriptions", json)
|
2013-01-29 21:40:16 +01:00
|
|
|
for stream in json["subscriptions"]:
|
|
|
|
self.assertIsInstance(stream['name'], str)
|
|
|
|
self.assertIsInstance(stream['color'], str)
|
2013-02-05 19:47:43 +01:00
|
|
|
self.assertIsInstance(stream['invite_only'], bool)
|
2013-01-29 19:47:28 +01:00
|
|
|
# check that the stream name corresponds to an actual stream
|
|
|
|
try:
|
2013-01-29 21:40:16 +01:00
|
|
|
Stream.objects.get(name__iexact=stream['name'], realm=self.realm)
|
2013-01-29 19:47:28 +01:00
|
|
|
except Stream.DoesNotExist:
|
|
|
|
self.fail("stream does not exist")
|
2013-01-29 21:40:16 +01:00
|
|
|
list_streams = [stream['name'] for stream in json["subscriptions"]]
|
2013-01-29 19:47:28 +01:00
|
|
|
# also check that this matches the list of your subscriptions
|
|
|
|
self.assertItemsEqual(list_streams, self.streams)
|
|
|
|
|
2013-01-31 20:58:58 +01:00
|
|
|
def helper_check_subs_before_and_after_add(self, url, subscriptions, other_params,
|
|
|
|
json_dict, email, new_subs):
|
2013-01-29 19:47:28 +01:00
|
|
|
"""
|
2013-01-31 20:58:58 +01:00
|
|
|
Check result of adding subscriptions.
|
|
|
|
|
|
|
|
You can add subscriptions for yourself or possibly many
|
|
|
|
principals, which is why e-mails map to subscriptions in the
|
|
|
|
result.
|
|
|
|
|
|
|
|
The result json is of the form
|
|
|
|
|
|
|
|
{"msg": "",
|
|
|
|
"result": "success",
|
|
|
|
"already_subscribed": {"iago@humbughq.com": ["Venice", "Verona"]},
|
|
|
|
"subscribed": {"iago@humbughq.com": ["Venice8"]}}
|
2013-01-29 19:47:28 +01:00
|
|
|
"""
|
|
|
|
data = {"subscriptions": simplejson.dumps(subscriptions)}
|
|
|
|
data.update(other_params)
|
|
|
|
result = self.client.post(url, data)
|
|
|
|
self.assert_json_success(result)
|
|
|
|
json = simplejson.loads(result.content)
|
2013-01-31 20:58:58 +01:00
|
|
|
for subscription_status, val in json_dict.iteritems():
|
|
|
|
# keys are subscribed, already_subscribed.
|
|
|
|
# vals are a dict mapping e-mails to streams.
|
|
|
|
self.assertIn(subscription_status, json)
|
|
|
|
for email, streams in val.iteritems():
|
|
|
|
self.assertItemsEqual(streams, json[subscription_status][email])
|
2013-01-29 19:47:28 +01:00
|
|
|
new_streams = self.get_streams(email)
|
|
|
|
self.assertItemsEqual(new_streams, new_subs)
|
|
|
|
|
|
|
|
def test_successful_subscriptions_add(self):
|
|
|
|
"""
|
|
|
|
Calling /json/subscriptions/add should successfully add streams, and
|
|
|
|
should determine which are new subscriptions vs which were already
|
|
|
|
subscribed. We randomly generate stream names to add, because it
|
|
|
|
doesn't matter whether the stream already exists.
|
|
|
|
"""
|
|
|
|
self.assertNotEqual(len(self.streams), 0) # necessary for full test coverage
|
|
|
|
add_streams = self.make_random_stream_names(self.streams, self.streams)
|
|
|
|
self.assertNotEqual(len(add_streams), 0) # necessary for full test coverage
|
2013-01-31 20:58:58 +01:00
|
|
|
self.helper_check_subs_before_and_after_add(
|
|
|
|
"/json/subscriptions/add", self.streams + add_streams, {},
|
|
|
|
{"subscribed": {self.test_email: add_streams},
|
|
|
|
"already_subscribed": {self.test_email: self.streams}},
|
|
|
|
self.test_email, self.streams + add_streams)
|
2013-01-29 19:47:28 +01:00
|
|
|
|
|
|
|
def test_subscriptions_add_too_long(self):
|
|
|
|
"""
|
|
|
|
Calling /json/subscriptions/add on a stream whose name is >30
|
|
|
|
characters should return a JSON error.
|
|
|
|
"""
|
|
|
|
# character limit is 30 characters
|
|
|
|
long_stream_name = "a" * 31
|
|
|
|
result = self.client.post("/json/subscriptions/add",
|
|
|
|
{"subscriptions": simplejson.dumps([long_stream_name])})
|
|
|
|
self.assert_json_error(result,
|
|
|
|
"Stream name (%s) too long." % (long_stream_name,))
|
|
|
|
|
|
|
|
def test_subscriptions_add_invalid_stream(self):
|
|
|
|
"""
|
|
|
|
Calling /json/subscriptions/add on a stream whose name is invalid (as
|
|
|
|
defined by valid_stream_name in zephyr/views.py) should return a JSON
|
|
|
|
error.
|
|
|
|
"""
|
|
|
|
# currently, the only invalid name is the empty string
|
|
|
|
invalid_stream_name = ""
|
|
|
|
result = self.client.post("/json/subscriptions/add",
|
|
|
|
{"subscriptions": simplejson.dumps([invalid_stream_name])})
|
|
|
|
self.assert_json_error(result,
|
|
|
|
"Invalid stream name (%s)." % (invalid_stream_name,))
|
|
|
|
|
|
|
|
def test_subscriptions_add_for_principal(self):
|
|
|
|
"""
|
|
|
|
Calling /json/subscriptions/add on behalf of another principal (for
|
|
|
|
whom you have permission to add subscriptions) should successfully add
|
|
|
|
those subscriptions and send a message to the subscribee notifying
|
|
|
|
them.
|
|
|
|
"""
|
|
|
|
other_email = "iago@humbughq.com"
|
|
|
|
other_profile = UserProfile.objects.get(user__email=other_email)
|
|
|
|
self.assertIsInstance(other_profile, UserProfile)
|
|
|
|
current_streams = self.get_streams(other_email)
|
|
|
|
self.assertNotEqual(len(current_streams), 0) # necessary for full test coverage
|
|
|
|
add_streams = self.make_random_stream_names(current_streams, current_streams)
|
|
|
|
self.assertNotEqual(len(add_streams), 0) # necessary for full test coverage
|
|
|
|
streams_to_sub = add_streams[:1] # just add one, to make the message easier to check
|
|
|
|
streams_to_sub.extend(current_streams)
|
2013-01-31 20:58:58 +01:00
|
|
|
self.helper_check_subs_before_and_after_add(
|
|
|
|
"/json/subscriptions/add", streams_to_sub,
|
|
|
|
{"principals": simplejson.dumps([other_email])},
|
|
|
|
{"subscribed": {other_email: add_streams[:1]},
|
|
|
|
"already_subscribed": {other_email: current_streams}},
|
|
|
|
other_email, streams_to_sub)
|
2013-01-29 19:47:28 +01:00
|
|
|
# verify that the user was sent a message informing them about the subscription
|
|
|
|
msg = Message.objects.latest('id')
|
|
|
|
self.assertEqual(msg.recipient.type, msg.recipient.PERSONAL)
|
2013-01-31 20:58:58 +01:00
|
|
|
self.assertEqual(msg.sender_id, UserProfile.objects.get(
|
|
|
|
user__email="humbug+notifications@humbughq.com").id)
|
2013-01-29 19:47:28 +01:00
|
|
|
expected_msg = ("Hi there! We thought you'd like to know that %s just "
|
2013-01-31 20:58:58 +01:00
|
|
|
"subscribed you to the stream '%s'"
|
2013-01-29 19:47:28 +01:00
|
|
|
% (self.user_profile.full_name, add_streams[0]))
|
|
|
|
self.assertEqual(msg.content, expected_msg)
|
|
|
|
recipients = get_display_recipient(msg.recipient)
|
|
|
|
self.assertEqual(len(recipients), 1)
|
|
|
|
self.assertEqual(recipients[0]['email'], other_email)
|
|
|
|
|
|
|
|
def test_subscription_add_invalid_principal(self):
|
|
|
|
"""
|
|
|
|
Calling /json/subscriptions/add on behalf of a principal that does not
|
|
|
|
exist should return a JSON error.
|
|
|
|
"""
|
|
|
|
invalid_principal = "rosencrantz-and-guildenstern@humbughq.com"
|
|
|
|
# verify that invalid_principal actually doesn't exist
|
|
|
|
with self.assertRaises(UserProfile.DoesNotExist):
|
|
|
|
UserProfile.objects.get(user__email=invalid_principal)
|
|
|
|
result = self.client.post("/json/subscriptions/add",
|
|
|
|
{"subscriptions": simplejson.dumps(self.streams),
|
2013-01-31 20:58:58 +01:00
|
|
|
"principals": simplejson.dumps([invalid_principal])})
|
2013-01-29 19:47:28 +01:00
|
|
|
self.assert_json_error(result, "User not authorized to execute queries on behalf of '%s'"
|
|
|
|
% (invalid_principal,))
|
|
|
|
|
|
|
|
def test_subscription_add_principal_other_realm(self):
|
|
|
|
"""
|
|
|
|
Calling /json/subscriptions/add on behalf of a principal in another
|
|
|
|
realm should return a JSON error.
|
|
|
|
"""
|
|
|
|
principal = "starnine@mit.edu"
|
|
|
|
profile = UserProfile.objects.get(user__email=principal)
|
|
|
|
# verify that principal exists (thus, the reason for the error is the cross-realming)
|
|
|
|
self.assertIsInstance(profile, UserProfile)
|
|
|
|
result = self.client.post("/json/subscriptions/add",
|
|
|
|
{"subscriptions": simplejson.dumps(self.streams),
|
2013-01-31 20:58:58 +01:00
|
|
|
"principals": simplejson.dumps([principal])})
|
2013-01-29 19:47:28 +01:00
|
|
|
self.assert_json_error(result, "User not authorized to execute queries on behalf of '%s'"
|
|
|
|
% (principal,))
|
|
|
|
|
2013-01-31 20:58:58 +01:00
|
|
|
def helper_check_subs_before_and_after_remove(self, url, subscriptions, other_params,
|
|
|
|
json_dict, email, new_subs):
|
|
|
|
"""
|
|
|
|
Check result of removing subscriptions.
|
|
|
|
|
|
|
|
Unlike adding subscriptions, you can only remove subscriptions
|
|
|
|
for yourself, so the result format is different.
|
|
|
|
|
|
|
|
{"msg": "",
|
|
|
|
"removed": ["Denmark", "Scotland", "Verona"],
|
|
|
|
"not_subscribed": ["Rome"], "result": "success"}
|
|
|
|
"""
|
|
|
|
data = {"subscriptions": simplejson.dumps(subscriptions)}
|
|
|
|
data.update(other_params)
|
|
|
|
result = self.client.post(url, data)
|
|
|
|
self.assert_json_success(result)
|
|
|
|
json = simplejson.loads(result.content)
|
|
|
|
for key, val in json_dict.iteritems():
|
|
|
|
self.assertItemsEqual(val, json[key]) # we don't care about the order of the items
|
|
|
|
new_streams = self.get_streams(email)
|
|
|
|
self.assertItemsEqual(new_streams, new_subs)
|
|
|
|
|
2013-01-29 19:47:28 +01:00
|
|
|
def test_successful_subscriptions_remove(self):
|
|
|
|
"""
|
|
|
|
Calling /json/subscriptions/remove should successfully remove streams,
|
|
|
|
and should determine which were removed vs which weren't subscribed to.
|
|
|
|
We cannot randomly generate stream names because the remove code
|
|
|
|
verifies whether streams exist.
|
|
|
|
"""
|
|
|
|
if len(self.streams) < 2:
|
|
|
|
self.fail() # necesssary for full test coverage
|
|
|
|
streams_to_remove = self.streams[1:]
|
|
|
|
not_subbed = []
|
|
|
|
for stream in Stream.objects.all():
|
|
|
|
if not stream.name in self.streams:
|
|
|
|
not_subbed.append(stream.name)
|
|
|
|
random.shuffle(not_subbed)
|
|
|
|
self.assertNotEqual(len(not_subbed), 0) # necessary for full test coverage
|
|
|
|
try_to_remove = not_subbed[:3] # attempt to remove up to 3 streams not already subbed to
|
|
|
|
streams_to_remove.extend(try_to_remove)
|
2013-01-31 20:58:58 +01:00
|
|
|
self.helper_check_subs_before_and_after_remove(
|
|
|
|
"/json/subscriptions/remove", streams_to_remove, {},
|
|
|
|
{"removed": self.streams[1:], "not_subscribed": try_to_remove},
|
|
|
|
self.test_email, [self.streams[0]])
|
2013-01-29 19:47:28 +01:00
|
|
|
|
|
|
|
def test_subscriptions_remove_fake_stream(self):
|
|
|
|
"""
|
|
|
|
Calling /json/subscriptions/remove on a stream that doesn't exist
|
|
|
|
should return a JSON error.
|
|
|
|
"""
|
|
|
|
all_stream_names = [stream.name for stream in Stream.objects.filter(realm=self.realm)]
|
|
|
|
random_streams = self.make_random_stream_names(self.streams, all_stream_names)
|
|
|
|
self.assertNotEqual(len(random_streams), 0) # necessary for full test coverage
|
|
|
|
streams_to_remove = random_streams[:1] # pick only one fake stream, to make checking the error message easy
|
|
|
|
result = self.client.post("/json/subscriptions/remove",
|
|
|
|
{"subscriptions": simplejson.dumps(streams_to_remove)})
|
2013-01-30 22:40:00 +01:00
|
|
|
self.assert_json_error(result, "Stream(s) (%s) do not exist" % (random_streams[0],))
|
2013-01-29 19:47:28 +01:00
|
|
|
|
|
|
|
def helper_subscriptions_exists(self, stream, exists, subscribed):
|
|
|
|
"""
|
|
|
|
A helper function that calls /json/subscriptions/exists on a stream and
|
|
|
|
verifies that the returned JSON dictionary has the exists and
|
|
|
|
subscribed values passed in as parameters. (If subscribed should not be
|
|
|
|
present, pass in None.)
|
|
|
|
"""
|
|
|
|
result = self.client.post("/json/subscriptions/exists",
|
|
|
|
{"stream": stream})
|
|
|
|
self.assert_json_success(result)
|
|
|
|
json = simplejson.loads(result.content)
|
|
|
|
self.assertIn("exists", json)
|
|
|
|
self.assertEqual(json["exists"], exists)
|
|
|
|
if not subscribed is None:
|
|
|
|
self.assertIn("subscribed", json)
|
|
|
|
self.assertEqual(json["subscribed"], subscribed)
|
|
|
|
|
|
|
|
def test_successful_subscriptions_exists_subbed(self):
|
|
|
|
"""
|
|
|
|
Calling /json/subscriptions/exist on a stream to which you are subbed
|
|
|
|
should return that it exists and that you are subbed.
|
|
|
|
"""
|
|
|
|
self.assertNotEqual(len(self.streams), 0) # necessary for full test coverage
|
|
|
|
self.helper_subscriptions_exists(self.streams[0], True, True)
|
|
|
|
|
|
|
|
def test_successful_subscriptions_exists_not_subbed(self):
|
|
|
|
"""
|
|
|
|
Calling /json/subscriptions/exist on a stream to which you are not
|
|
|
|
subbed should return that it exists and that you are not subbed.
|
|
|
|
"""
|
|
|
|
all_stream_names = [stream.name for stream in Stream.objects.filter(realm=self.realm)]
|
|
|
|
streams_not_subbed = list(set(all_stream_names) - set(self.streams))
|
|
|
|
self.assertNotEqual(len(streams_not_subbed), 0) # necessary for full test coverage
|
|
|
|
self.helper_subscriptions_exists(streams_not_subbed[0], True, False)
|
|
|
|
|
|
|
|
def test_subscriptions_does_not_exist(self):
|
|
|
|
"""
|
|
|
|
Calling /json/subscriptions/exist on a stream that doesn't exist should
|
|
|
|
return that it doesn't exist.
|
|
|
|
"""
|
|
|
|
all_stream_names = [stream.name for stream in Stream.objects.filter(realm=self.realm)]
|
|
|
|
random_streams = self.make_random_stream_names(self.streams, all_stream_names)
|
|
|
|
self.assertNotEqual(len(random_streams), 0) # necessary for full test coverage
|
|
|
|
self.helper_subscriptions_exists(random_streams[0], False, None)
|
|
|
|
|
|
|
|
def test_subscriptions_exist_invalid_name(self):
|
|
|
|
"""
|
|
|
|
Calling /json/subscriptions/exist on a stream whose name is invalid (as
|
|
|
|
defined by valid_stream_name in zephyr/views.py) should return a JSON
|
|
|
|
error.
|
|
|
|
"""
|
|
|
|
# currently, the only invalid stream name is the empty string
|
|
|
|
invalid_stream_name = ""
|
|
|
|
result = self.client.post("/json/subscriptions/exists",
|
|
|
|
{"stream": invalid_stream_name})
|
|
|
|
self.assert_json_error(result, "Invalid characters in stream name")
|
|
|
|
|
2012-12-08 18:01:17 +01:00
|
|
|
class GetOldMessagesTest(AuthedTestCase):
|
|
|
|
fixtures = ['messages.json']
|
|
|
|
|
|
|
|
def post_with_params(self, modified_params):
|
2012-12-19 23:58:02 +01:00
|
|
|
post_params = {"anchor": 1, "num_before": 1, "num_after": 1}
|
2012-12-08 18:01:17 +01:00
|
|
|
post_params.update(modified_params)
|
2013-01-02 21:56:00 +01:00
|
|
|
result = self.client.post("/json/get_old_messages", dict(post_params))
|
|
|
|
self.assert_json_success(result)
|
|
|
|
return simplejson.loads(result.content)
|
2012-12-08 18:01:17 +01:00
|
|
|
|
|
|
|
def check_well_formed_messages_response(self, result):
|
|
|
|
self.assertIn("messages", result)
|
2013-01-02 22:07:09 +01:00
|
|
|
self.assertIsInstance(result["messages"], list)
|
2012-12-08 18:01:17 +01:00
|
|
|
for message in result["messages"]:
|
|
|
|
for field in ("content", "content_type", "display_recipient",
|
|
|
|
"gravatar_hash", "recipient_id", "sender_full_name",
|
|
|
|
"sender_short_name", "timestamp"):
|
|
|
|
self.assertIn(field, message)
|
|
|
|
|
2013-01-02 21:56:00 +01:00
|
|
|
def test_successful_get_old_messages(self):
|
2012-12-08 18:01:17 +01:00
|
|
|
"""
|
|
|
|
A call to /json/get_old_messages with valid parameters returns a list of
|
|
|
|
messages.
|
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
2013-01-03 19:46:29 +01:00
|
|
|
self.check_well_formed_messages_response(self.post_with_params({}))
|
2012-12-08 18:01:17 +01:00
|
|
|
|
2012-12-19 23:58:02 +01:00
|
|
|
def test_get_old_messages_with_narrow_pm_with(self):
|
2012-12-08 18:01:17 +01:00
|
|
|
"""
|
2012-12-19 23:58:02 +01:00
|
|
|
A request for old messages with a narrow by pm-with only returns
|
|
|
|
conversations with that user.
|
2012-12-08 18:01:17 +01:00
|
|
|
"""
|
2012-12-19 23:58:02 +01:00
|
|
|
me = 'hamlet@humbughq.com'
|
|
|
|
def dr_emails(dr):
|
|
|
|
return ','.join(sorted(set([r['email'] for r in dr] + [me])))
|
|
|
|
|
|
|
|
personals = [m for m in self.message_stream(User.objects.get(email=me))
|
|
|
|
if m.recipient.type == Recipient.PERSONAL
|
|
|
|
or m.recipient.type == Recipient.HUDDLE]
|
|
|
|
if not personals:
|
|
|
|
# FIXME: This is bad. We should use test data that is guaranteed
|
|
|
|
# to contain some personals for every user. See #617.
|
|
|
|
return
|
|
|
|
emails = dr_emails(get_display_recipient(personals[0].recipient))
|
2012-12-08 18:01:17 +01:00
|
|
|
|
2012-12-19 23:58:02 +01:00
|
|
|
self.login(me)
|
2013-01-02 21:56:00 +01:00
|
|
|
result = self.post_with_params({"narrow": simplejson.dumps(
|
2012-12-19 23:58:02 +01:00
|
|
|
[['pm-with', emails]])})
|
2012-12-08 18:01:17 +01:00
|
|
|
self.check_well_formed_messages_response(result)
|
|
|
|
|
|
|
|
for message in result["messages"]:
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(dr_emails(message['display_recipient']), emails)
|
2012-12-08 18:01:17 +01:00
|
|
|
|
2013-01-02 21:56:00 +01:00
|
|
|
def test_get_old_messages_with_narrow_stream(self):
|
2012-12-08 18:01:17 +01:00
|
|
|
"""
|
2012-12-19 23:58:02 +01:00
|
|
|
A request for old messages with a narrow by stream only returns
|
|
|
|
messages for that stream.
|
2012-12-08 18:01:17 +01:00
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
2013-01-08 21:59:52 +01:00
|
|
|
# We need to send a message here to ensure that we actually
|
|
|
|
# have a stream message in this narrow view.
|
|
|
|
self.send_message("hamlet@humbughq.com", "Scotland", Recipient.STREAM)
|
2012-12-08 18:01:17 +01:00
|
|
|
messages = self.message_stream(User.objects.get(email="hamlet@humbughq.com"))
|
|
|
|
stream_messages = filter(lambda msg: msg.recipient.type == Recipient.STREAM,
|
|
|
|
messages)
|
|
|
|
stream_name = get_display_recipient(stream_messages[0].recipient)
|
|
|
|
stream_id = stream_messages[0].recipient.id
|
|
|
|
|
2013-01-02 21:56:00 +01:00
|
|
|
result = self.post_with_params({"narrow": simplejson.dumps(
|
2012-12-19 23:58:02 +01:00
|
|
|
[['stream', stream_name]])})
|
2012-12-08 18:01:17 +01:00
|
|
|
self.check_well_formed_messages_response(result)
|
|
|
|
|
|
|
|
for message in result["messages"]:
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(message["type"], "stream")
|
|
|
|
self.assertEqual(message["recipient_id"], stream_id)
|
2012-12-08 18:01:17 +01:00
|
|
|
|
|
|
|
def test_missing_params(self):
|
|
|
|
"""
|
2012-12-19 23:58:02 +01:00
|
|
|
anchor, num_before, and num_after are all required
|
2012-12-08 18:01:17 +01:00
|
|
|
POST parameters for get_old_messages.
|
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
|
|
|
|
2012-12-19 23:58:02 +01:00
|
|
|
required_args = (("anchor", 1), ("num_before", 1), ("num_after", 1))
|
2012-12-08 18:01:17 +01:00
|
|
|
|
|
|
|
for i in range(len(required_args)):
|
2013-01-03 20:32:53 +01:00
|
|
|
post_params = dict(required_args[:i] + required_args[i + 1:])
|
2012-12-08 18:01:17 +01:00
|
|
|
result = self.client.post("/json/get_old_messages", post_params)
|
|
|
|
self.assert_json_error(result,
|
|
|
|
"Missing '%s' argument" % (required_args[i][0],))
|
|
|
|
|
|
|
|
def test_bad_int_params(self):
|
|
|
|
"""
|
|
|
|
anchor, num_before, num_after, and narrow must all be non-negative
|
|
|
|
integers or strings that can be converted to non-negative integers.
|
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
|
|
|
|
|
|
|
other_params = [("narrow", {})]
|
|
|
|
int_params = ["anchor", "num_before", "num_after"]
|
|
|
|
|
|
|
|
bad_types = (False, "", "-1", -1)
|
|
|
|
for idx, param in enumerate(int_params):
|
|
|
|
for type in bad_types:
|
|
|
|
# Rotate through every bad type for every integer
|
|
|
|
# parameter, one at a time.
|
|
|
|
post_params = dict(other_params + [(param, type)] + \
|
|
|
|
[(other_param, 0) for other_param in \
|
|
|
|
int_params[:idx] + int_params[idx + 1:]]
|
|
|
|
)
|
|
|
|
result = self.client.post("/json/get_old_messages", post_params)
|
|
|
|
self.assert_json_error(result,
|
|
|
|
"Bad value for '%s': %s" % (param, type))
|
|
|
|
|
|
|
|
def test_bad_narrow_type(self):
|
|
|
|
"""
|
2012-12-19 23:58:02 +01:00
|
|
|
narrow must be a list of string pairs.
|
2012-12-08 18:01:17 +01:00
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
|
|
|
|
|
|
|
other_params = [("anchor", 0), ("num_before", 0), ("num_after", 0)]
|
|
|
|
|
2012-12-19 23:58:02 +01:00
|
|
|
bad_types = (False, 0, '', '{malformed json,',
|
2013-01-02 21:43:49 +01:00
|
|
|
'{foo: 3}', '[1,2]', '[["x","y","z"]]')
|
2012-12-08 18:01:17 +01:00
|
|
|
for type in bad_types:
|
|
|
|
post_params = dict(other_params + [("narrow", type)])
|
|
|
|
result = self.client.post("/json/get_old_messages", post_params)
|
|
|
|
self.assert_json_error(result,
|
|
|
|
"Bad value for 'narrow': %s" % (type,))
|
|
|
|
|
2013-01-02 21:43:49 +01:00
|
|
|
def test_old_empty_narrow(self):
|
|
|
|
"""
|
|
|
|
'{}' is accepted to mean 'no narrow', for use by old mobile clients.
|
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
|
|
|
all_result = self.post_with_params({})
|
|
|
|
narrow_result = self.post_with_params({'narrow': '{}'})
|
|
|
|
|
|
|
|
for r in (all_result, narrow_result):
|
|
|
|
self.check_well_formed_messages_response(r)
|
|
|
|
|
|
|
|
self.assertEqual(message_ids(all_result), message_ids(narrow_result))
|
|
|
|
|
2012-12-19 23:58:02 +01:00
|
|
|
def test_bad_narrow_operator(self):
|
|
|
|
"""
|
|
|
|
Unrecognized narrow operators are rejected.
|
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
|
|
|
for operator in ['', 'foo', 'stream:verona', '__init__']:
|
|
|
|
params = dict(anchor=0, num_before=0, num_after=0,
|
|
|
|
narrow=simplejson.dumps([[operator, '']]))
|
|
|
|
result = self.client.post("/json/get_old_messages", params)
|
|
|
|
self.assert_json_error_contains(result,
|
|
|
|
"Invalid narrow operator: unknown operator")
|
|
|
|
|
|
|
|
def exercise_bad_narrow_operand(self, operator, operands, error_msg):
|
2012-12-08 18:01:17 +01:00
|
|
|
other_params = [("anchor", 0), ("num_before", 0), ("num_after", 0)]
|
2012-12-19 23:58:02 +01:00
|
|
|
for operand in operands:
|
|
|
|
post_params = dict(other_params + [
|
|
|
|
("narrow", simplejson.dumps([[operator, operand]]))])
|
2012-12-08 18:01:17 +01:00
|
|
|
result = self.client.post("/json/get_old_messages", post_params)
|
2012-12-19 23:58:02 +01:00
|
|
|
self.assert_json_error_contains(result, error_msg)
|
2012-12-08 18:01:17 +01:00
|
|
|
|
|
|
|
def test_bad_narrow_stream_content(self):
|
|
|
|
"""
|
|
|
|
If an invalid stream name is requested in get_old_messages, an error is
|
|
|
|
returned.
|
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
2012-12-19 23:58:02 +01:00
|
|
|
bad_stream_content = (0, [], ["x", "y"])
|
|
|
|
self.exercise_bad_narrow_operand("stream", bad_stream_content,
|
|
|
|
"Bad value for 'narrow'")
|
2012-12-08 18:01:17 +01:00
|
|
|
|
|
|
|
def test_bad_narrow_one_on_one_email_content(self):
|
|
|
|
"""
|
2012-12-19 23:58:02 +01:00
|
|
|
If an invalid 'pm-with' is requested in get_old_messages, an
|
2012-12-08 18:01:17 +01:00
|
|
|
error is returned.
|
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
2012-12-19 23:58:02 +01:00
|
|
|
bad_stream_content = (0, [], ["x","y"])
|
|
|
|
self.exercise_bad_narrow_operand("pm-with", bad_stream_content,
|
|
|
|
"Bad value for 'narrow'")
|
|
|
|
|
|
|
|
def test_bad_narrow_nonexistent_stream(self):
|
|
|
|
self.login("hamlet@humbughq.com")
|
|
|
|
self.exercise_bad_narrow_operand("stream", ['non-existent stream'],
|
|
|
|
"Invalid narrow operator: unknown stream")
|
|
|
|
|
|
|
|
def test_bad_narrow_nonexistent_email(self):
|
|
|
|
self.login("hamlet@humbughq.com")
|
|
|
|
self.exercise_bad_narrow_operand("pm-with", ['non-existent-user@humbughq.com'],
|
|
|
|
"Invalid narrow operator: unknown user")
|
|
|
|
|
2012-12-08 18:01:17 +01:00
|
|
|
|
2013-01-04 19:06:34 +01:00
|
|
|
class InviteUserTest(AuthedTestCase):
|
|
|
|
fixtures = ['messages.json']
|
|
|
|
|
|
|
|
def invite(self, users, streams):
|
|
|
|
"""
|
|
|
|
Invites the specified users to Humbug with the specified streams.
|
|
|
|
|
|
|
|
users should be a string containing the users to invite, comma or
|
|
|
|
newline separated.
|
|
|
|
|
|
|
|
streams should be a list of strings.
|
|
|
|
"""
|
|
|
|
|
|
|
|
return self.client.post("/json/invite_users",
|
|
|
|
{"invitee_emails": users,
|
|
|
|
"stream": streams})
|
|
|
|
|
|
|
|
def test_successful_invite_user(self):
|
|
|
|
"""
|
|
|
|
A call to /json/invite_users with valid parameters causes an invitation
|
|
|
|
email to be sent.
|
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
2013-01-14 21:58:45 +01:00
|
|
|
self.assert_json_success(self.invite("alice-test@humbughq.com", ["Denmark"]))
|
2013-01-04 19:06:34 +01:00
|
|
|
self.assertTrue(find_key_by_email("alice-test@humbughq.com"))
|
|
|
|
|
|
|
|
def test_multi_user_invite(self):
|
|
|
|
"""
|
|
|
|
Invites multiple users with a variety of delimiters.
|
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
|
|
|
# Intentionally use a weird string.
|
|
|
|
self.assert_json_success(self.invite(
|
|
|
|
"""bob-test@humbughq.com, carol-test@humbughq.com,
|
|
|
|
dave-test@humbughq.com
|
|
|
|
|
|
|
|
|
2013-01-14 21:58:45 +01:00
|
|
|
earl-test@humbughq.com""", ["Denmark"]))
|
2013-01-04 19:06:34 +01:00
|
|
|
for user in ("bob", "carol", "dave", "earl"):
|
|
|
|
self.assertTrue(find_key_by_email("%s-test@humbughq.com" % user))
|
|
|
|
|
|
|
|
def test_missing_params(self):
|
|
|
|
"""
|
|
|
|
Tests inviting with various invalid parameters.
|
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
|
|
|
self.assert_json_error(
|
|
|
|
self.client.post("/json/invite_users", {"invitee_emails": "foo@humbughq.com"}),
|
|
|
|
"You must specify at least one stream for invitees to join.")
|
|
|
|
|
|
|
|
for address in ("noatsign.com", "outsideyourdomain@example.net"):
|
|
|
|
self.assert_json_error(
|
|
|
|
self.invite(address, ["Denmark"]),
|
|
|
|
"Some emails did not validate. No invites have been sent.")
|
|
|
|
|
|
|
|
def test_invalid_stream(self):
|
|
|
|
"""
|
|
|
|
Tests inviting to a non-existent stream.
|
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
2013-01-14 21:57:54 +01:00
|
|
|
self.assert_json_error(self.invite("iago-test@humbughq.com", ["NotARealStream"]),
|
2013-01-04 19:06:34 +01:00
|
|
|
"Stream does not exist: NotARealStream. No invites were sent.")
|
2012-12-21 01:06:53 +01:00
|
|
|
|
|
|
|
class ChangeSettingsTest(AuthedTestCase):
|
|
|
|
fixtures = ['messages.json']
|
|
|
|
|
|
|
|
def post_with_params(self, modified_params):
|
|
|
|
post_params = {"full_name": "Foo Bar",
|
|
|
|
"old_password": initial_password("hamlet@humbughq.com"),
|
|
|
|
"new_password": "foobar1", "confirm_password": "foobar1",
|
|
|
|
"enable_desktop_notifications": ""}
|
|
|
|
post_params.update(modified_params)
|
|
|
|
return self.client.post("/json/settings/change", dict(post_params))
|
|
|
|
|
|
|
|
def check_well_formed_change_settings_response(self, result):
|
|
|
|
self.assertIn("full_name", result)
|
|
|
|
self.assertIn("enable_desktop_notifications", result)
|
|
|
|
|
2012-12-21 16:59:08 +01:00
|
|
|
def test_successful_change_settings(self):
|
2012-12-21 01:06:53 +01:00
|
|
|
"""
|
|
|
|
A call to /json/settings/change with valid parameters changes the user's
|
|
|
|
settings correctly and returns correct values.
|
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
|
|
|
json_result = self.post_with_params({})
|
|
|
|
self.assert_json_success(json_result)
|
2012-12-21 16:59:08 +01:00
|
|
|
result = simplejson.loads(json_result.content)
|
2012-12-21 01:06:53 +01:00
|
|
|
self.check_well_formed_change_settings_response(result)
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(self.get_user_profile("hamlet@humbughq.com").
|
2012-12-21 01:06:53 +01:00
|
|
|
full_name, "Foo Bar")
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(self.get_user_profile("hamlet@humbughq.com").
|
2012-12-21 01:06:53 +01:00
|
|
|
enable_desktop_notifications, False)
|
|
|
|
self.client.post('/accounts/logout/')
|
|
|
|
self.login("hamlet@humbughq.com", "foobar1")
|
|
|
|
user = User.objects.get(email='hamlet@humbughq.com')
|
|
|
|
self.assertEqual(self.client.session['_auth_user_id'], user.id)
|
|
|
|
|
2012-12-21 16:59:08 +01:00
|
|
|
def test_missing_params(self):
|
|
|
|
"""
|
|
|
|
full_name, old_password, and new_password are all required POST
|
|
|
|
parameters for json_change_settings. (enable_desktop_notifications is
|
|
|
|
false by default)
|
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
|
|
|
required_params = (("full_name", "Foo Bar"),
|
|
|
|
("old_password", initial_password("hamlet@humbughq.com")),
|
|
|
|
("new_password", initial_password("hamlet@humbughq.com")),
|
|
|
|
("confirm_password", initial_password("hamlet@humbughq.com")))
|
|
|
|
for i in range(len(required_params)):
|
2013-01-03 20:32:53 +01:00
|
|
|
post_params = dict(required_params[:i] + required_params[i + 1:])
|
2012-12-21 16:59:08 +01:00
|
|
|
result = self.client.post("/json/settings/change", post_params)
|
|
|
|
self.assert_json_error(result,
|
|
|
|
"Missing '%s' argument" % (required_params[i][0],))
|
|
|
|
|
2012-12-21 01:06:53 +01:00
|
|
|
def test_mismatching_passwords(self):
|
|
|
|
"""
|
|
|
|
new_password and confirm_password must match
|
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
|
|
|
result = self.post_with_params({"new_password": "mismatched_password"})
|
|
|
|
self.assert_json_error(result,
|
|
|
|
"New password must match confirmation password!")
|
|
|
|
|
|
|
|
def test_wrong_old_password(self):
|
|
|
|
"""
|
|
|
|
new_password and confirm_password must match
|
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
|
|
|
result = self.post_with_params({"old_password": "bad_password"})
|
|
|
|
self.assert_json_error(result, "Wrong password!")
|
|
|
|
|
|
|
|
|
2012-09-06 23:55:04 +02:00
|
|
|
class DummyHandler(object):
|
2012-11-28 07:59:57 +01:00
|
|
|
def __init__(self, assert_callback):
|
|
|
|
self.assert_callback = assert_callback
|
2012-09-06 23:55:04 +02:00
|
|
|
|
2012-11-28 07:59:57 +01:00
|
|
|
# Mocks RequestHandler.async_callback, which wraps a callback to
|
|
|
|
# handle exceptions. We return the callback as-is.
|
|
|
|
def async_callback(self, cb):
|
|
|
|
return cb
|
2012-09-06 23:55:04 +02:00
|
|
|
|
2012-11-28 07:59:57 +01:00
|
|
|
def write(self, response):
|
|
|
|
raise NotImplemented
|
|
|
|
|
|
|
|
def finish(self, response):
|
|
|
|
if self.assert_callback:
|
|
|
|
self.assert_callback(response)
|
2012-10-01 22:57:01 +02:00
|
|
|
|
2012-10-29 19:53:56 +01:00
|
|
|
class DummySession(object):
|
|
|
|
session_key = "0"
|
|
|
|
|
2012-09-06 23:55:04 +02:00
|
|
|
class POSTRequestMock(object):
|
|
|
|
method = "POST"
|
|
|
|
|
2012-11-28 07:59:57 +01:00
|
|
|
def __init__(self, post_data, user, assert_callback=None):
|
2012-09-06 23:55:04 +02:00
|
|
|
self.POST = post_data
|
|
|
|
self.user = user
|
|
|
|
self._tornado_handler = DummyHandler(assert_callback)
|
2012-10-29 19:53:56 +01:00
|
|
|
self.session = DummySession()
|
2012-11-09 18:22:13 +01:00
|
|
|
self.META = {'PATH_INFO': 'test'}
|
2012-09-06 23:55:04 +02:00
|
|
|
|
2012-09-27 21:44:54 +02:00
|
|
|
class GetUpdatesTest(AuthedTestCase):
|
2012-10-03 21:16:34 +02:00
|
|
|
fixtures = ['messages.json']
|
2012-09-06 23:55:04 +02:00
|
|
|
|
2012-11-15 19:42:17 +01:00
|
|
|
def common_test_get_updates(self, view_func, extra_post_data = {}):
|
2012-09-21 00:26:59 +02:00
|
|
|
user = User.objects.get(email="hamlet@humbughq.com")
|
2012-09-06 23:55:04 +02:00
|
|
|
|
2012-11-28 07:59:57 +01:00
|
|
|
def callback(response):
|
|
|
|
correct_message_ids = [m.id for m in
|
|
|
|
filter_by_subscriptions(Message.objects.all(), user)]
|
|
|
|
for message in response['messages']:
|
|
|
|
self.assertGreater(message['id'], 1)
|
|
|
|
self.assertIn(message['id'], correct_message_ids)
|
2012-09-06 23:55:04 +02:00
|
|
|
|
2013-01-17 21:03:43 +01:00
|
|
|
post_data = {}
|
2012-11-15 19:42:17 +01:00
|
|
|
post_data.update(extra_post_data)
|
|
|
|
request = POSTRequestMock(post_data, user, callback)
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(view_func(request), RespondAsynchronously)
|
2012-11-15 19:42:17 +01:00
|
|
|
|
|
|
|
def test_json_get_updates(self):
|
|
|
|
"""
|
|
|
|
json_get_updates returns messages with IDs greater than the
|
|
|
|
last_received ID.
|
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
|
|
|
self.common_test_get_updates(json_get_updates)
|
|
|
|
|
|
|
|
def test_api_get_messages(self):
|
|
|
|
"""
|
|
|
|
Same as above, but for the API view
|
|
|
|
"""
|
|
|
|
email = "hamlet@humbughq.com"
|
|
|
|
api_key = self.get_api_key(email)
|
|
|
|
self.common_test_get_updates(api_get_messages, {'email': email, 'api-key': api_key})
|
2012-09-06 23:55:04 +02:00
|
|
|
|
|
|
|
def test_missing_last_received(self):
|
|
|
|
"""
|
2012-10-30 21:54:30 +01:00
|
|
|
Calling json_get_updates without any arguments should work
|
2012-09-06 23:55:04 +02:00
|
|
|
"""
|
2012-10-12 17:49:22 +02:00
|
|
|
self.login("hamlet@humbughq.com")
|
2012-09-21 00:26:59 +02:00
|
|
|
user = User.objects.get(email="hamlet@humbughq.com")
|
2012-09-06 23:55:04 +02:00
|
|
|
|
2012-11-28 07:59:57 +01:00
|
|
|
request = POSTRequestMock({}, user)
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(json_get_updates(request), RespondAsynchronously)
|
2012-10-30 21:54:30 +01:00
|
|
|
|
2012-12-19 20:19:46 +01:00
|
|
|
def test_bad_input(self):
|
|
|
|
"""
|
|
|
|
Specifying a bad value for 'pointer' should return an error
|
|
|
|
"""
|
|
|
|
self.login("hamlet@humbughq.com")
|
|
|
|
user = User.objects.get(email="hamlet@humbughq.com")
|
|
|
|
|
|
|
|
request = POSTRequestMock({'pointer': 'foo'}, user)
|
|
|
|
self.assertRaises(RequestVariableConversionError, json_get_updates, request)
|
|
|
|
|
2013-01-22 20:07:51 +01:00
|
|
|
class GetProfileTest(AuthedTestCase):
|
|
|
|
fixtures = ['messages.json']
|
|
|
|
|
|
|
|
def common_update_pointer(self, email, pointer):
|
|
|
|
self.login(email)
|
|
|
|
result = self.client.post("/json/update_pointer", {"pointer": 1})
|
|
|
|
self.assert_json_success(result)
|
|
|
|
|
|
|
|
def common_get_profile(self, email):
|
|
|
|
user = User.objects.get(email=email)
|
|
|
|
|
|
|
|
api_key = self.get_api_key(email)
|
2013-01-28 22:59:25 +01:00
|
|
|
result = self.client.post("/api/v1/get_profile", {'email': email, 'api-key': api_key})
|
2013-01-22 20:07:51 +01:00
|
|
|
|
|
|
|
stream = self.message_stream(user)
|
|
|
|
max_id = -1
|
|
|
|
if len(stream) > 0:
|
|
|
|
max_id = stream[-1].id
|
|
|
|
|
|
|
|
self.assert_json_success(result)
|
|
|
|
json = simplejson.loads(result.content)
|
|
|
|
|
|
|
|
self.assertIn("client_id", json)
|
|
|
|
self.assertIn("max_message_id", json)
|
|
|
|
self.assertIn("pointer", json)
|
|
|
|
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(json["max_message_id"], max_id)
|
2013-01-22 20:07:51 +01:00
|
|
|
return json
|
|
|
|
|
|
|
|
def test_api_get_empty_profile(self):
|
|
|
|
"""
|
|
|
|
Ensure get_profile returns a max message id and returns successfully
|
|
|
|
"""
|
|
|
|
json = self.common_get_profile("othello@humbughq.com")
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(json["pointer"], -1)
|
2013-01-22 20:07:51 +01:00
|
|
|
|
|
|
|
def test_profile_with_pointer(self):
|
|
|
|
"""
|
|
|
|
Ensure get_profile returns a proper pointer id after the pointer is updated
|
|
|
|
"""
|
|
|
|
json = self.common_get_profile("hamlet@humbughq.com")
|
|
|
|
|
|
|
|
self.common_update_pointer("hamlet@humbughq.com", 1)
|
|
|
|
json = self.common_get_profile("hamlet@humbughq.com")
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(json["pointer"], 1)
|
2013-01-22 20:07:51 +01:00
|
|
|
|
|
|
|
self.common_update_pointer("hamlet@humbughq.com", 0)
|
|
|
|
json = self.common_get_profile("hamlet@humbughq.com")
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(json["pointer"], 1)
|
2012-12-19 20:19:46 +01:00
|
|
|
|
2013-01-23 16:18:08 +01:00
|
|
|
class GetPublicStreamsTest(AuthedTestCase):
|
|
|
|
fixtures = ['messages.json']
|
|
|
|
|
|
|
|
def test_public_streams(self):
|
|
|
|
"""
|
|
|
|
Ensure that get_public_streams successfully returns a list of streams
|
|
|
|
"""
|
|
|
|
email = 'hamlet@humbughq.com'
|
2013-01-28 22:59:25 +01:00
|
|
|
self.login(email)
|
2013-01-23 16:18:08 +01:00
|
|
|
|
|
|
|
api_key = self.get_api_key(email)
|
2013-01-28 22:59:25 +01:00
|
|
|
result = self.client.post("/json/get_public_streams", {'email': email, 'api-key': api_key})
|
2013-01-23 16:18:08 +01:00
|
|
|
|
|
|
|
self.assert_json_success(result)
|
|
|
|
json = simplejson.loads(result.content)
|
|
|
|
|
|
|
|
self.assertIn("streams", json)
|
|
|
|
self.assertIsInstance(json["streams"], list)
|
|
|
|
|
2013-01-23 20:39:35 +01:00
|
|
|
class InviteOnlyStreamTest(AuthedTestCase):
|
|
|
|
fixtures = ['messages.json']
|
|
|
|
|
|
|
|
def common_subscribe_to_stream(self, email, streams, extra_post_data = {}, invite_only=False):
|
|
|
|
api_key = self.get_api_key(email)
|
|
|
|
|
|
|
|
post_data = {'email': email,
|
|
|
|
'api-key': api_key,
|
|
|
|
'subscriptions': streams,
|
2013-01-31 20:58:58 +01:00
|
|
|
'invite_only': simplejson.dumps(invite_only)}
|
2013-01-23 20:39:35 +01:00
|
|
|
post_data.update(extra_post_data)
|
2013-01-28 22:59:25 +01:00
|
|
|
|
2013-01-30 22:40:00 +01:00
|
|
|
result = self.client.post("/api/v1/subscriptions/add", post_data)
|
2013-01-23 20:39:35 +01:00
|
|
|
return result
|
|
|
|
|
2013-02-05 20:06:35 +01:00
|
|
|
def test_list_respects_invite_only_bit(self):
|
|
|
|
"""
|
|
|
|
Make sure that /json/subscriptions/list properly returns
|
|
|
|
the invite-only bit for streams that are invite-only
|
|
|
|
"""
|
|
|
|
email = 'hamlet@humbughq.com'
|
|
|
|
self.login(email)
|
|
|
|
|
|
|
|
result1 = self.common_subscribe_to_stream(email, '["Saxony"]', invite_only=True)
|
|
|
|
self.assert_json_success(result1)
|
|
|
|
result2 = self.common_subscribe_to_stream(email, '["Normandy"]', invite_only=False)
|
|
|
|
self.assert_json_success(result2)
|
|
|
|
result = self.client.post("/json/subscriptions/list", {})
|
|
|
|
self.assert_json_success(result)
|
|
|
|
json = simplejson.loads(result.content)
|
|
|
|
self.assertIn("subscriptions", json)
|
|
|
|
for sub in json["subscriptions"]:
|
|
|
|
if sub['name'] == "Normandy":
|
|
|
|
self.assertEqual(sub['invite_only'], False, "Normandy was mistakenly marked invite-only")
|
|
|
|
if sub['name'] == "Saxony":
|
|
|
|
self.assertEqual(sub['invite_only'], True, "Saxony was not properly marked invite-only")
|
|
|
|
|
2013-01-23 20:39:35 +01:00
|
|
|
def test_inviteonly(self):
|
|
|
|
# Creating an invite-only stream is allowed
|
|
|
|
email = 'hamlet@humbughq.com'
|
2013-01-28 22:59:25 +01:00
|
|
|
|
2013-01-23 20:39:35 +01:00
|
|
|
result = self.common_subscribe_to_stream(email, '["Saxony"]', invite_only=True)
|
|
|
|
self.assert_json_success(result)
|
|
|
|
|
|
|
|
json = simplejson.loads(result.content)
|
2013-01-31 20:58:58 +01:00
|
|
|
self.assertEqual(json["subscribed"], {email: ['Saxony']})
|
|
|
|
self.assertEqual(json["already_subscribed"], {})
|
2013-01-23 20:39:35 +01:00
|
|
|
|
|
|
|
# Subscribing oneself to an invite-only stream is not allowed
|
|
|
|
email = "othello@humbughq.com"
|
2013-02-04 22:32:18 +01:00
|
|
|
self.login(email)
|
2013-01-23 20:39:35 +01:00
|
|
|
result = self.common_subscribe_to_stream(email, '["Saxony"]')
|
2013-01-30 22:40:00 +01:00
|
|
|
self.assert_json_error(result, 'Unable to access invite-only stream (Saxony).')
|
2013-01-23 20:39:35 +01:00
|
|
|
|
|
|
|
# Inviting another user to an invite-only stream is allowed
|
|
|
|
email = 'hamlet@humbughq.com'
|
2013-02-04 22:32:18 +01:00
|
|
|
self.login(email)
|
2013-01-31 20:58:58 +01:00
|
|
|
result = self.common_subscribe_to_stream(
|
|
|
|
email, '["Saxony"]',
|
|
|
|
extra_post_data={'principals': simplejson.dumps(["othello@humbughq.com"])})
|
|
|
|
json = simplejson.loads(result.content)
|
|
|
|
self.assertEqual(json["subscribed"], {"othello@humbughq.com": ['Saxony']})
|
|
|
|
self.assertEqual(json["already_subscribed"], {})
|
2013-01-23 20:39:35 +01:00
|
|
|
|
|
|
|
# Make sure both users are subscribed to this stream
|
2013-01-30 22:40:00 +01:00
|
|
|
result = self.client.post("/api/v1/get_subscribers", {'email':email,
|
2013-01-28 22:59:25 +01:00
|
|
|
'api-key': self.get_api_key(email),
|
|
|
|
'stream': 'Saxony'})
|
2013-01-23 20:39:35 +01:00
|
|
|
self.assert_json_success(result)
|
|
|
|
json = simplejson.loads(result.content)
|
|
|
|
|
|
|
|
self.assertTrue('othello@humbughq.com' in json['subscribers'])
|
|
|
|
self.assertTrue('hamlet@humbughq.com' in json['subscribers'])
|
|
|
|
|
2013-02-05 04:24:16 +01:00
|
|
|
class GetSubscribersTest(AuthedTestCase):
|
|
|
|
fixtures = ['messages.json']
|
|
|
|
|
|
|
|
def setUp(self):
|
|
|
|
self.email = "hamlet@humbughq.com"
|
|
|
|
self.api_key = self.get_api_key(self.email)
|
|
|
|
self.user_profile = self.get_user_profile(self.email)
|
|
|
|
self.login(self.email)
|
|
|
|
|
|
|
|
def check_well_formed_result(self, result, stream_name, domain):
|
|
|
|
"""
|
|
|
|
A successful call to get_subscribers returns the list of subscribers in
|
|
|
|
the form:
|
|
|
|
|
|
|
|
{"msg": "",
|
|
|
|
"result": "success",
|
|
|
|
"subscribers": ["hamlet@humbughq.com", "prospero@humbughq.com"]}
|
|
|
|
"""
|
|
|
|
self.assertIn("subscribers", result)
|
|
|
|
self.assertIsInstance(result["subscribers"], list)
|
|
|
|
true_subscribers = [user.email for user in self.users_subscribed_to_stream(
|
|
|
|
stream_name, domain)]
|
|
|
|
self.assertItemsEqual(result["subscribers"], true_subscribers)
|
|
|
|
|
|
|
|
def make_subscriber_request(self, stream_name):
|
|
|
|
return self.client.post("/json/get_subscribers",
|
|
|
|
{'email': self.email, 'api-key': self.api_key,
|
|
|
|
'stream': stream_name})
|
|
|
|
|
|
|
|
def make_successful_subscriber_request(self, stream_name):
|
|
|
|
result = self.make_subscriber_request(stream_name)
|
|
|
|
self.assert_json_success(result)
|
|
|
|
self.check_well_formed_result(simplejson.loads(result.content),
|
|
|
|
stream_name, self.user_profile.realm.domain)
|
|
|
|
|
|
|
|
def test_subscriber(self):
|
|
|
|
"""
|
|
|
|
get_subscribers returns the list of subscribers.
|
|
|
|
"""
|
2013-01-29 21:40:16 +01:00
|
|
|
stream_name = gather_subscriptions(self.user_profile)[0]['name']
|
2013-02-05 04:24:16 +01:00
|
|
|
self.make_successful_subscriber_request(stream_name)
|
|
|
|
|
|
|
|
def test_nonsubscriber(self):
|
|
|
|
"""
|
|
|
|
Even a non-subscriber to a public stream can query a stream's membership
|
|
|
|
with get_subscribers.
|
|
|
|
"""
|
|
|
|
# Create a stream for which Hamlet is the only subscriber.
|
|
|
|
stream_name = "Saxony"
|
|
|
|
self.client.post("/json/subscriptions/add",
|
|
|
|
{"subscriptions": simplejson.dumps([stream_name])})
|
|
|
|
other_email = "othello@humbughq.com"
|
|
|
|
|
|
|
|
# Fetch the subscriber list as a non-member.
|
|
|
|
self.login(other_email)
|
|
|
|
self.make_successful_subscriber_request(stream_name)
|
|
|
|
|
|
|
|
def test_subscriber_private_stream(self):
|
|
|
|
"""
|
|
|
|
A subscriber to a private stream can query that stream's membership.
|
|
|
|
"""
|
|
|
|
stream_name = "Saxony"
|
|
|
|
self.client.post("/json/subscriptions/add",
|
|
|
|
{"subscriptions": simplejson.dumps([stream_name]),
|
|
|
|
"invite_only": simplejson.dumps(True)})
|
|
|
|
self.make_successful_subscriber_request(stream_name)
|
|
|
|
|
|
|
|
def test_nonsubscriber_private_stream(self):
|
|
|
|
"""
|
|
|
|
A non-subscriber to a private stream can't query that stream's membership.
|
|
|
|
"""
|
|
|
|
# Create a private stream for which Hamlet is the only subscriber.
|
|
|
|
stream_name = "Saxony"
|
|
|
|
self.client.post("/json/subscriptions/add",
|
|
|
|
{"subscriptions": simplejson.dumps([stream_name]),
|
|
|
|
"invite_only": simplejson.dumps(True)})
|
|
|
|
other_email = "othello@humbughq.com"
|
|
|
|
|
|
|
|
# Try to fetch the subscriber list as a non-member.
|
|
|
|
self.login(other_email)
|
|
|
|
result = self.make_subscriber_request(stream_name)
|
|
|
|
self.assert_json_error(result,
|
|
|
|
"Unable to retrieve subscribers for invite-only stream")
|
|
|
|
|
2013-01-24 20:20:21 +01:00
|
|
|
class BugdownTest(TestCase):
|
|
|
|
|
|
|
|
def common_bugdown_test(self, text, expected):
|
|
|
|
converted = convert(text)
|
2013-01-17 18:12:02 +01:00
|
|
|
self.assertEqual(converted, expected)
|
2013-01-24 20:20:21 +01:00
|
|
|
|
|
|
|
def test_codeblock_hilite(self):
|
|
|
|
fenced_code = \
|
|
|
|
"""Hamlet said:
|
|
|
|
~~~~.python
|
|
|
|
def speak(self):
|
|
|
|
x = 1
|
|
|
|
~~~~"""
|
|
|
|
|
|
|
|
expected_convert = \
|
|
|
|
"""<p>Hamlet said:</p>
|
|
|
|
<div class="codehilite"><pre><span class="k">def</span> <span class="nf">\
|
|
|
|
speak</span><span class="p">(</span><span class="bp">self</span><span class="p">):</span>
|
|
|
|
<span class="n">x</span> <span class="o">=</span> <span class="mi">1</span>
|
|
|
|
</pre></div>"""
|
|
|
|
|
|
|
|
self.common_bugdown_test(fenced_code, expected_convert)
|
|
|
|
|
|
|
|
def test_codeblock_multiline(self):
|
|
|
|
fenced_code = \
|
|
|
|
"""Hamlet once said
|
2013-01-25 23:02:35 +01:00
|
|
|
~~~~
|
2013-01-24 20:20:21 +01:00
|
|
|
def func():
|
|
|
|
x = 1
|
|
|
|
|
|
|
|
|
|
|
|
y = 2
|
|
|
|
|
|
|
|
z = 3
|
2013-01-25 23:02:35 +01:00
|
|
|
~~~~
|
2013-01-24 20:20:21 +01:00
|
|
|
And all was good."""
|
|
|
|
|
|
|
|
expected_convert = \
|
|
|
|
"""<p>Hamlet once said</p>
|
|
|
|
<div class="codehilite"><pre>def func():
|
|
|
|
x = 1
|
|
|
|
|
|
|
|
y = 2
|
|
|
|
|
|
|
|
z = 3
|
|
|
|
</pre></div>
|
|
|
|
|
|
|
|
|
|
|
|
<p>And all was good.</p>"""
|
|
|
|
|
|
|
|
self.common_bugdown_test(fenced_code, expected_convert)
|
|
|
|
|
|
|
|
|
|
|
|
def test_hanging_multi_codeblock(self):
|
|
|
|
fenced_code = \
|
|
|
|
"""Hamlet said:
|
|
|
|
~~~~
|
|
|
|
def speak(self):
|
|
|
|
x = 1
|
|
|
|
~~~~
|
|
|
|
|
|
|
|
Then he mentioned ````y = 4 + x**2```` and
|
|
|
|
~~~~
|
|
|
|
def foobar(self):
|
|
|
|
return self.baz()"""
|
|
|
|
|
|
|
|
expected_convert = \
|
|
|
|
"""<p>Hamlet said:</p>
|
|
|
|
<div class="codehilite"><pre>def speak(self):
|
|
|
|
x = 1
|
|
|
|
</pre></div>
|
|
|
|
|
|
|
|
|
|
|
|
<p>Then he mentioned <code>y = 4 + x**2</code> and</p>
|
|
|
|
<div class="codehilite"><pre>def foobar(self):
|
|
|
|
return self.baz()
|
|
|
|
</pre></div>"""
|
|
|
|
self.common_bugdown_test(fenced_code, expected_convert)
|
|
|
|
|
|
|
|
def test_dangerous_block(self):
|
|
|
|
fenced_code = u'xxxxxx xxxxx xxxxxxxx xxxx. x xxxx xxxxxxxxxx:\n\n```\
|
|
|
|
"xxxx xxxx\\xxxxx\\xxxxxx"```\n\nxxx xxxx xxxxx:```xx.xxxxxxx(x\'^xxxx$\'\
|
|
|
|
, xx.xxxxxxxxx)```\n\nxxxxxxx\'x xxxx xxxxxxxxxx ```\'xxxx\'```, xxxxx \
|
|
|
|
xxxxxxxxx xxxxx ^ xxx $ xxxxxx xxxxx xxxxxxxxxxxx xxx xxxx xx x xxxx xx xxxx xx xxx xxxxx xxxxxx?'
|
|
|
|
|
|
|
|
expected = """<p>xxxxxx xxxxx xxxxxxxx xxxx. x xxxx xxxxxxxxxx:</p>\n\
|
|
|
|
<p><code>"xxxx xxxx\\xxxxx\\xxxxxx"</code></p>\n<p>xxx xxxx xxxxx:<code>xx.xxxxxxx\
|
|
|
|
(x\'^xxxx$\', xx.xxxxxxxxx)</code></p>\n<p>xxxxxxx\'x xxxx xxxxxxxxxx <code>\'xxxx\'\
|
|
|
|
</code>, xxxxx xxxxxxxxx xxxxx ^ xxx $ xxxxxx xxxxx xxxxxxxxxxxx xxx xxxx xx x \
|
|
|
|
xxxx xx xxxx xx xxx xxxxx xxxxxx?</p>"""
|
|
|
|
|
|
|
|
self.common_bugdown_test(fenced_code, expected)
|
|
|
|
|
|
|
|
fenced_code = """``` one ```
|
|
|
|
|
|
|
|
``` two ```
|
|
|
|
|
|
|
|
~~~~
|
|
|
|
x = 1"""
|
|
|
|
expected_convert = '<p><code>one</code></p>\n<p><code>two</code></p>\n<div class="codehilite"><pre>x = 1\n</pre></div>'
|
|
|
|
self.common_bugdown_test(fenced_code, expected_convert)
|
|
|
|
|
2013-01-24 19:35:28 +01:00
|
|
|
def test_ulist_standard(self):
|
|
|
|
ulisted = """Some text with a list:
|
|
|
|
|
|
|
|
* One item
|
|
|
|
* Two items
|
|
|
|
* Three items"""
|
|
|
|
|
|
|
|
expected = """<p>Some text with a list:</p>
|
|
|
|
<ul>
|
|
|
|
<li>One item</li>
|
|
|
|
<li>Two items</li>
|
|
|
|
<li>Three items</li>
|
|
|
|
</ul>"""
|
|
|
|
self.common_bugdown_test(ulisted, expected)
|
|
|
|
|
|
|
|
def test_ulist_hanging(self):
|
|
|
|
ulisted = """Some text with a hanging list:
|
|
|
|
* One item
|
|
|
|
* Two items
|
|
|
|
* Three items"""
|
|
|
|
|
|
|
|
expected = """<p>Some text with a hanging list:</p>
|
|
|
|
<ul>
|
|
|
|
<li>One item</li>
|
|
|
|
<li>Two items</li>
|
|
|
|
<li>Three items</li>
|
|
|
|
</ul>"""
|
|
|
|
self.common_bugdown_test(ulisted, expected)
|
|
|
|
|
|
|
|
def test_ulist_hanging_mixed(self):
|
|
|
|
ulisted = """Plain list
|
|
|
|
|
|
|
|
* Alpha
|
|
|
|
|
|
|
|
* Beta
|
|
|
|
|
|
|
|
Then hang it off:
|
|
|
|
* Ypsilon
|
|
|
|
* Zeta"""
|
|
|
|
|
|
|
|
expected = """<p>Plain list</p>
|
|
|
|
<ul>
|
|
|
|
<li>
|
|
|
|
<p>Alpha</p>
|
|
|
|
</li>
|
|
|
|
<li>
|
|
|
|
<p>Beta</p>
|
|
|
|
</li>
|
|
|
|
</ul>
|
|
|
|
<p>Then hang it off:</p>
|
|
|
|
<ul>
|
|
|
|
<li>Ypsilon</li>
|
|
|
|
<li>Zeta</li>
|
|
|
|
</ul>"""
|
|
|
|
self.common_bugdown_test(ulisted, expected)
|
|
|
|
|
|
|
|
def test_hanging_multi(self):
|
|
|
|
ulisted = """Plain list
|
|
|
|
* Alpha
|
|
|
|
* Beta
|
|
|
|
|
|
|
|
And Again:
|
|
|
|
* A
|
|
|
|
* B
|
|
|
|
* C
|
|
|
|
|
|
|
|
Once more for feeling:
|
|
|
|
* Q
|
|
|
|
* E
|
|
|
|
* D"""
|
|
|
|
|
|
|
|
expected = '<p>Plain list</p>\n<ul>\n<li>Alpha</li>\n<li>Beta\
|
|
|
|
</li>\n</ul>\n<p>And Again:</p>\n<ul>\n<li>A</li>\n<li>B</li>\n<li>C\
|
|
|
|
</li>\n</ul>\n<p>Once more for feeling:</p>\n<ul>\n<li>Q</li>\n<li>E\
|
|
|
|
</li>\n<li>D</li>\n</ul>'
|
|
|
|
self.common_bugdown_test(ulisted, expected)
|
|
|
|
|
|
|
|
def test_ulist_codeblock(self):
|
|
|
|
ulisted_code = """~~~
|
|
|
|
int x = 3
|
|
|
|
* 4;
|
|
|
|
~~~"""
|
|
|
|
|
|
|
|
expected = '<div class="codehilite"><pre>int x = 3\n* 4;\n</pre></div>'
|
|
|
|
self.common_bugdown_test(ulisted_code, expected)
|
|
|
|
|
2013-01-29 16:15:25 +01:00
|
|
|
def test_malformed_fence(self):
|
|
|
|
bad = "~~~~~~~~xxxxxxxxx: xxxxxxxxxxxx xxxxx x xxxxxxxx~~~~~~"
|
|
|
|
good = "<p>~~~~~~~~xxxxxxxxx: xxxxxxxxxxxx xxxxx x xxxxxxxx~~~~~~</p>"
|
|
|
|
self.common_bugdown_test(bad, good)
|
|
|
|
|
2013-01-31 21:17:43 +01:00
|
|
|
def test_italic_bold(self):
|
|
|
|
'''Italics (*foo*, _foo_) and bold syntax __foo__ are disabled.
|
|
|
|
Bold **foo** still works.'''
|
|
|
|
self.common_bugdown_test('_foo_', '<p>_foo_</p>')
|
|
|
|
self.common_bugdown_test('*foo*', '<p>*foo*</p>')
|
|
|
|
self.common_bugdown_test('__foo__', '<p>__foo__</p>')
|
|
|
|
self.common_bugdown_test('**foo**', '<p><strong>foo</strong></p>')
|
|
|
|
|
2013-02-01 20:05:14 +01:00
|
|
|
def test_linkify(self):
|
2013-02-01 23:15:22 +01:00
|
|
|
def replaced(payload, url, phrase=''):
|
|
|
|
if url is None: # xss type
|
|
|
|
href = ''
|
|
|
|
url = phrase
|
|
|
|
elif url[:4] == 'http':
|
|
|
|
href = url
|
|
|
|
elif '@' in url:
|
|
|
|
href = 'mailto:' + url
|
|
|
|
else:
|
|
|
|
href = 'http://' + url
|
|
|
|
return payload % ("<a href=\"%s\" target=\"_blank\" title=\"%s\">%s</a>" % (href, href, url),)
|
2013-02-01 20:05:14 +01:00
|
|
|
|
|
|
|
conversions = \
|
2013-02-01 23:15:22 +01:00
|
|
|
[
|
|
|
|
# General linkification tests
|
|
|
|
('http://www.google.com', "<p>%s</p>", 'http://www.google.com'),
|
2013-02-01 20:05:14 +01:00
|
|
|
('https://www.google.com', "<p>%s</p>", 'https://www.google.com'),
|
2013-02-05 19:04:45 +01:00
|
|
|
('http://www.theregister.co.uk/foo/bar', "<p>%s</p>", 'http://www.theregister.co.uk/foo/bar'),
|
2013-02-01 20:05:14 +01:00
|
|
|
(' some text https://www.google.com/', "<p>some text %s</p>", 'https://www.google.com/'),
|
|
|
|
('with short example.com url', "<p>with short %s url</p>", 'example.com'),
|
|
|
|
('t.co', "<p>%s</p>", 't.co'),
|
|
|
|
('go to views.org please', "<p>go to %s please</p>", 'views.org'),
|
|
|
|
('http://foo.com/blah_blah/', "<p>%s</p>", 'http://foo.com/blah_blah/'),
|
|
|
|
('python class views.py is', "<p>python class views.py is</p>", ''),
|
|
|
|
('with www www.humbughq.com/foo ok?', "<p>with www %s ok?</p>", 'www.humbughq.com/foo'),
|
|
|
|
('allow questions like foo.com?', "<p>allow questions like %s?</p>", 'foo.com'),
|
|
|
|
('"is.gd/foo/ "', "<p>\"%s \"</p>", 'is.gd/foo/'),
|
|
|
|
('end of sentence https://t.co.', "<p>end of sentence %s.</p>", 'https://t.co'),
|
|
|
|
('(Something like http://foo.com/blah_blah)', "<p>(Something like %s)</p>", 'http://foo.com/blah_blah'),
|
|
|
|
('"is.gd/foo/"', "<p>\"%s\"</p>", 'is.gd/foo/'),
|
|
|
|
('end with a quote www.google.com"', "<p>end with a quote %s\"</p>", 'www.google.com'),
|
|
|
|
|
2013-02-01 23:15:22 +01:00
|
|
|
# XSS Sanitization
|
|
|
|
('javascript:alert(\'hi\');.com', "<p>%s</p>", None), # None marks xss
|
|
|
|
('javascript:foo.com', "<p>%s</p>", None),
|
|
|
|
('about:blank.com', "<p>%s</p>", None),
|
|
|
|
|
|
|
|
# Emails
|
|
|
|
('Sent to othello@humbughq.com', "<p>Sent to %s</p>", 'othello@humbughq.com'),
|
|
|
|
('http://leo@foo.com/my/file', "<p>%s</p>", None),
|
|
|
|
|
2013-02-01 20:05:14 +01:00
|
|
|
('http://example.com/something?with,commas,in,url, but not at end',
|
|
|
|
"<p>%s but not at end</p>", 'http://example.com/something?with,commas,in,url,'),
|
|
|
|
(' some text https://www.google.com/baz_(match)?with=foo&bar=baz with extras',
|
|
|
|
"<p>some text %s with extras</p>", 'https://www.google.com/baz_(match)?with=foo&bar=baz'),
|
|
|
|
('hash it http://foo.com/blah_(wikipedia)_blah#cite-1',
|
|
|
|
"<p>hash it %s</p>", 'http://foo.com/blah_(wikipedia)_blah#cite-1'),]
|
|
|
|
|
|
|
|
for inline_url, reference, url in conversions:
|
|
|
|
try:
|
2013-02-01 23:15:22 +01:00
|
|
|
match = replaced(reference, url, phrase=inline_url)
|
2013-02-01 20:05:14 +01:00
|
|
|
except TypeError:
|
|
|
|
match = reference
|
|
|
|
converted = convert(inline_url)
|
|
|
|
self.assertEqual(match, converted)
|
|
|
|
|
2012-11-14 20:50:47 +01:00
|
|
|
class Runner(DjangoTestSuiteRunner):
|
2012-11-14 21:22:08 +01:00
|
|
|
option_list = (
|
|
|
|
optparse.make_option('--skip-generate',
|
|
|
|
dest='generate', default=True, action='store_false',
|
|
|
|
help='Skip generating test fixtures')
|
|
|
|
,)
|
|
|
|
|
|
|
|
def __init__(self, generate, *args, **kwargs):
|
|
|
|
if generate:
|
|
|
|
subprocess.check_call("zephyr/tests/generate-fixtures");
|
2012-11-14 20:50:47 +01:00
|
|
|
DjangoTestSuiteRunner.__init__(self, *args, **kwargs)
|