Skip to content

Commit 565b003

Browse files
committed
add users testing
1 parent 1f18229 commit 565b003

File tree

3 files changed

+26
-19
lines changed

3 files changed

+26
-19
lines changed

apps/users/models.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
from django.db import models
55

66

7-
class User(AbstractUser, TimestempedAbstractModel):
7+
class User(TimestempedAbstractModel, AbstractUser):
88
email = models.EmailField(_("email address"), unique=True)
99

1010
@property

apps/users/tests.py

Lines changed: 17 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,23 @@
11
from django.test import TestCase
2+
from django.urls import reverse
23

34
from .models import User, UserProfile
45

56

67
class TestUsers(TestCase):
7-
def setUp(self) -> None:
8-
User.objects.create(
9-
username="User",
10-
email="user@example.com",
11-
password="password"
12-
)
13-
return super().setUp()
14-
15-
8+
9+
def test_user_register_page(self):
10+
response = self.client.get(reverse("users:register"))
11+
12+
self.assertEqual(response.status_code, 200)
13+
self.assertTemplateUsed(response, "auth/register.html")
14+
self.assertContains(response, "Register")
15+
# self.assertRedirects(response, reverse("users:login"))
16+
17+
def test_user_login_page(self):
18+
response = self.client.get(reverse("users:login"))
19+
20+
self.assertEqual(response.status_code, 200)
21+
self.assertTemplateUsed(response, "auth/login.html")
22+
self.assertContains(response, "Login")
23+
# self.assertRedirects(response, reverse("blog:home"))

apps/users/views.py

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
from django.shortcuts import render, redirect, get_object_or_404
2-
from django.urls import reverse
2+
from django.urls import reverse, reverse_lazy
33
from django.contrib.auth.mixins import LoginRequiredMixin
44
from django.contrib import messages
55
from django.views import View
@@ -31,11 +31,11 @@ def post(self, request):
3131
if form.is_valid():
3232
form.save()
3333
messages.success(request, "User succesfully registered")
34-
return redirect(reverse("users:login"))
34+
return redirect(reverse_lazy("users:login"))
3535

3636
messages.warning(request, "Invalid registration fields!")
37-
return render(request, "auth/register.html", {"form": form})
38-
37+
return redirect(reverse_lazy("auth:register"))
38+
3939

4040
class LoginPageView(CustomHtmxMixin, View):
4141
template_name = "auth/login.html"
@@ -59,19 +59,18 @@ def post(self, request):
5959

6060
if user is not None:
6161

62-
response = redirect(reverse("blog:home"))
62+
response = redirect(reverse_lazy("blog:home"))
6363

6464
# Login for jwt
6565
response = get_jwt_login_response(response, user)
6666

6767
messages.success(request, f"You are logged in as { username }")
6868
return response
6969

70-
else:
71-
messages.error(request, "Invalid username or password.")
72-
return redirect(reverse("users:login"))
70+
messages.error(request, "Invalid username or password.")
7371

74-
return render(request, "auth/login.html", {"form": form})
72+
return redirect(reverse_lazy("users:login"))
73+
7574

7675

7776
class LogoutPageView(CustomHtmxMixin, LoginRequiredMixin, View):

0 commit comments

Comments
 (0)