Skip to content

Commit 36fe656

Browse files
committed
Added test to simulate an error I had where an imported reference failed and the python property swallowed the exception raised in my authenticator.
1 parent b3af4d9 commit 36fe656

File tree

1 file changed

+19
-0
lines changed

1 file changed

+19
-0
lines changed

tests/test_request.py

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -333,6 +333,25 @@ def test_user_can_login(self):
333333
login(self.request, self.user)
334334
self.assertEqual(self.request.user, self.user)
335335

336+
def test_calling_user_fails_when_exception_is_raised(self):
337+
class AuthRaisesError(object):
338+
def authenticate(self, request):
339+
raise AttributeError('We should see this error!')
340+
# import rest_framework
341+
# rest_framework.MISSPELLED_NAME_THAT_DOESNT_EXIST
342+
343+
self.request = Request(factory.get('/'), authenticators=(AuthRaisesError(),))
344+
SessionMiddleware().process_request(self.request)
345+
346+
login(self.request, self.user)
347+
error_seen = None
348+
try:
349+
self.request.user
350+
except AttributeError as error:
351+
error_seen = error
352+
353+
self.assertEqual('We should see this error!', error_seen.message)
354+
336355
def test_user_can_logout(self):
337356
self.request.user = self.user
338357
self.assertFalse(self.request.user.is_anonymous())

0 commit comments

Comments
 (0)
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy