Commit b842d2bc6e5f2b4474b044ecc07d5a3d4d8015f4
1 parent
225595d4
Exists in
master
and in
3 other branches
Adding login tests
Showing
2 changed files
with
57 additions
and
4 deletions
Show diff stats
users/tests/test_views.py
| @@ -0,0 +1,53 @@ | @@ -0,0 +1,53 @@ | ||
| 1 | +from django.test import TestCase, RequestFactory | ||
| 2 | +from django.core.urlresolvers import reverse_lazy | ||
| 3 | +from django.contrib.auth.models import AnonymousUser | ||
| 4 | +from ..models import User | ||
| 5 | +from .. import views | ||
| 6 | + | ||
| 7 | +class User_Test(TestCase): | ||
| 8 | + | ||
| 9 | + def setUp(self): | ||
| 10 | + self.factory = RequestFactory() | ||
| 11 | + self.user = User.objects.create(username = 'erik', email = 'egz@cin.ufpe.br', password = 'amadeus') | ||
| 12 | + self.admin = User.objects.create_superuser('admin', email = 'admin@amadeus.com', password = 'teste') | ||
| 13 | + | ||
| 14 | + def test_login_get_uauth(self): | ||
| 15 | + request = self.factory.get(reverse_lazy('users:login')) | ||
| 16 | + | ||
| 17 | + request.user = AnonymousUser() | ||
| 18 | + | ||
| 19 | + response = views.login(request) | ||
| 20 | + | ||
| 21 | + self.assertEquals(response.status_code, 200) | ||
| 22 | + | ||
| 23 | + def test_login_get_auth(self): | ||
| 24 | + request = self.factory.get(reverse_lazy('users:login')) | ||
| 25 | + | ||
| 26 | + request.user = self.user | ||
| 27 | + | ||
| 28 | + response = views.login(request) | ||
| 29 | + | ||
| 30 | + self.assertEquals(response.status_code, 302) | ||
| 31 | + | ||
| 32 | + def test_login_post_ok(self): | ||
| 33 | + data = { | ||
| 34 | + 'email': 'admin@amadeus.com', | ||
| 35 | + 'password': 'teste' | ||
| 36 | + } | ||
| 37 | + | ||
| 38 | + response = self.client.post(reverse_lazy('users:login'), data) | ||
| 39 | + | ||
| 40 | + self.assertEquals(response.status_code, 302) | ||
| 41 | + | ||
| 42 | + def test_login_post_invalid(self): | ||
| 43 | + data = { | ||
| 44 | + 'email': 'test@amadeus.com.br', | ||
| 45 | + 'password': 'anything' | ||
| 46 | + } | ||
| 47 | + | ||
| 48 | + response = self.client.post(reverse_lazy('users:login'), data) | ||
| 49 | + | ||
| 50 | + messages = response.context['messages'] | ||
| 51 | + | ||
| 52 | + self.assertEquals(response.status_code, 200) | ||
| 53 | + self.assertIsNotNone(messages) #checking if message was sent | ||
| 0 | \ No newline at end of file | 54 | \ No newline at end of file |
users/views.py
| @@ -424,14 +424,14 @@ class PasswordResetConfirmView(generic.FormView): | @@ -424,14 +424,14 @@ class PasswordResetConfirmView(generic.FormView): | ||
| 424 | user.set_password(new_password) | 424 | user.set_password(new_password) |
| 425 | user.save() | 425 | user.save() |
| 426 | 426 | ||
| 427 | - messages.success(request, 'Password reset successfully.') | 427 | + messages.success(request, _('Password reset successfully.')) |
| 428 | 428 | ||
| 429 | return self.form_valid(form) | 429 | return self.form_valid(form) |
| 430 | else: | 430 | else: |
| 431 | - messages.error(request, 'We were not able to reset your password.') | 431 | + messages.error(request, _('We were not able to reset your password.')) |
| 432 | return self.form_invalid(form) | 432 | return self.form_invalid(form) |
| 433 | else: | 433 | else: |
| 434 | - messages.error(request,'The reset password link is no longer valid.') | 434 | + messages.error(request, _('The reset password link is no longer valid.')) |
| 435 | return self.form_invalid(form) | 435 | return self.form_invalid(form) |
| 436 | 436 | ||
| 437 | 437 | ||
| @@ -447,7 +447,7 @@ def login(request): | @@ -447,7 +447,7 @@ def login(request): | ||
| 447 | login_user(request, user) | 447 | login_user(request, user) |
| 448 | return redirect(reverse("home")) | 448 | return redirect(reverse("home")) |
| 449 | else: | 449 | else: |
| 450 | - messages.add_message(request, messages.ERROR, _('E-mail or password are incorrect.')) | 450 | + messages.error(request, _('E-mail or password are incorrect.')) |
| 451 | context["username"] = username | 451 | context["username"] = username |
| 452 | elif request.user.is_authenticated: | 452 | elif request.user.is_authenticated: |
| 453 | return redirect(reverse('home')) | 453 | return redirect(reverse('home')) |