Browse Source

fix #641: test for validy of user's avatar url

Rafał Pitoń 8 years ago
parent
commit
cd4e2f98c3
2 changed files with 7 additions and 2 deletions
  1. 6 1
      misago/core/tests/test_mailer.py
  2. 1 1
      misago/users/models/user.py

+ 6 - 1
misago/core/tests/test_mailer.py

@@ -9,13 +9,18 @@ class MisagoMailerTests(TestCase):
     def test_mail_user(self):
         """mail_user sets message in backend"""
         User = get_user_model()
-        User.objects.create_user('Bob', 'bob@bob.com', 'pass123')
+        user = User.objects.create_user('Bob', 'bob@bob.com', 'pass123')
 
         response = self.client.get(reverse('test-mail-user'))
         self.assertEqual(response.status_code, 200)
 
         self.assertEqual(mail.outbox[0].subject, "Misago Test Mail")
 
+        # assert that url to user's avatar is valid
+        html_body = mail.outbox[0].alternatives[0][0]
+        user_avatar_url = 'http://testserver%s' % user.avatars[0]['url']
+        self.assertIn(user_avatar_url, html_body)
+
     def test_mail_users(self):
         """mail_users sets messages in backend"""
         User = get_user_model()

+ 1 - 1
misago/users/models/user.py

@@ -132,7 +132,7 @@ class UserManager(BaseUserManager):
                                        settings.default_gravatar_fallback)
         else:
             # just for test purposes
-            user.avatars = [{'size': 400, 'url': '//placekitten.com/400/400'}]
+            user.avatars = [{'size': 400, 'url': '/placekitten.com/400/400'}]
 
         authenticated_role = Role.objects.get(special_role='authenticated')
         if authenticated_role not in user.roles.all():