paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-03 21:43:21 +00:00
Code Issues Packages Projects Releases Wiki Activity

test_decorators: Use the default value of user.

Browse Source 
`HostRequestMock` has `user` default to `None`, which later gets
initialized as `AnonymousUser`. The separate initialization here is
unnecessary.

Signed-off-by: Zixuan James Li <p359101898@gmail.com>
This commit is contained in:
Zixuan James Li
2022-07-18 17:14:42 -04:00
committed by Tim Abbott
parent a3efc777fa
commit 2925615ee7
1 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
zerver/tests/test_decorators.py
View File

@@ -9,7 +9,6 @@ from unittest import mock, skipUnless
import orjson import orjson
from django.conf import settings from django.conf import settings
from django.contrib.auth.models import AnonymousUser
from django.core.exceptions import ValidationError from django.core.exceptions import ValidationError
from django.http import HttpRequest, HttpResponse from django.http import HttpRequest, HttpResponse
from django.utils.timezone import now as timezone_now from django.utils.timezone import now as timezone_now
@@ -1919,8 +1918,7 @@ class TestZulipLoginRequiredDecorator(ZulipTestCase):
return HttpResponse("Success") return HttpResponse("Success")
with self.settings(TWO_FACTOR_AUTHENTICATION_ENABLED=True): with self.settings(TWO_FACTOR_AUTHENTICATION_ENABLED=True):
user = AnonymousUser() request = HostRequestMock()
request = HostRequestMock(user_profile=user)
response = test_view(request) response = test_view(request)
self.assertEqual(response.content.decode(), "Success") self.assertEqual(response.content.decode(), "Success")